Find and replace with regex does not work for "replace all instances"

Miguel-LlamasLanza commented 5 months ago

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
- [x] Restarted Spyder
- [ ] Reset preferences with spyder --reset
- [ ] Reinstalled the latest version of Anaconda
- [ ] Tried the other applicable steps from the Troubleshooting Guide
[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description

Using find and replace, I normally can change from 4 spaces to tabs by using the regex expression '\t'. In the last versions of spyder, this works good for "replace next occurrence" and "replace occurrences in selection" but not for "replace all occurrences". With "replace all", the spaces are replaced by the characters \t and not by a tabulation. Occurs at least with 5.5.1 and 5.5.4. It used to work good with version 5.4.3 (to be verified the exact version)

What steps reproduce the problem?

Find and replace, with regex expression.
Spaces in the "find", and '\t' in the "replace"
Click on replace all occurrences.

Versions

Spyder version: 5.5.4 (conda)
Python version: 3.10.13 64-bit
Qt version: 5.15.2
PyQt5 version: 5.15.10
Operating System: Linux-6.5.0-26-generic-x86_64-with-glibc2.35

Dependencies

# Mandatory:
atomicwrites >=1.2.0                 :  1.4.1 (OK)
chardet >=2.0.0                      :  5.2.0 (OK)
cloudpickle >=0.5.0                  :  3.0.0 (OK)
cookiecutter >=1.6.0                 :  2.5.0 (OK)
diff_match_patch >=20181111          :  20230430 (OK)
intervaltree >=3.0.2                 :  3.1.0 (OK)
IPython >=8.13.0,<9.0.0,!=8.17.1     :  8.21.0 (OK)
jedi >=0.17.2,<0.20.0                :  0.19.1 (OK)
jellyfish >=0.7                      :  1.0.1 (OK)
jsonschema >=3.2.0                   :  4.21.1 (OK)
keyring >=17.0.0                     :  24.3.0 (OK)
nbconvert >=4.0                      :  7.16.0 (OK)
numpydoc >=0.6.0                     :  1.6.0 (OK)
parso >=0.7.0,<0.9.0                 :  0.8.3 (OK)
pexpect >=4.4.0                      :  4.9.0 (OK)
pickleshare >=0.4                    :  0.7.5 (OK)
psutil >=5.3                         :  5.9.0 (OK)
pygments >=2.0                       :  2.17.2 (OK)
pylint >=3.1,<4                      :  3.1.0 (OK)
pylint_venv >=3.0.2                  :  3.0.3 (OK)
pyls_spyder >=0.4.0                  :  0.4.0 (OK)
pylsp >=1.11.0,<1.12.0               :  1.11.0 (OK)
pylsp_black >=2.0.0,<3.0.0           :  2.0.0 (OK)
qdarkstyle >=3.2.0,<3.3.0            :  3.2.3 (OK)
qstylizer >=0.2.2                    :  0.2.2 (OK)
qtawesome >=1.3.1,<1.4.0             :  1.3.1 (OK)
qtconsole >=5.5.1,<5.6.0             :  5.5.1 (OK)
qtpy >=2.1.0                         :  2.4.1 (OK)
rtree >=0.9.7                        :  1.2.0 (OK)
setuptools >=49.6.0                  :  68.2.2 (OK)
sphinx >=0.6.6                       :  7.2.6 (OK)
spyder_kernels >=2.5.1,<2.6.0        :  2.5.1 (OK)
textdistance >=4.2.0                 :  4.5.0 (OK)
three_merge >=0.1.1                  :  0.1.1 (OK)
watchdog >=0.10.3                    :  4.0.0 (OK)
xdg >=0.26                           :  0.28 (OK)
zmq >=24.0.0                         :  25.1.2 (OK)

# Optional:
cython >=0.21                        :  None (NOK)
matplotlib >=3.0.0                   :  3.8.0 (OK)
numpy >=1.7                          :  1.26.3 (OK)
pandas >=1.1.1                       :  2.1.4 (OK)
scipy >=0.17.0                       :  1.11.4 (OK)
sympy >=0.7.3                        :  None (NOK)

# Spyder plugins:
spyder_notebook.notebookplugin 0.5.1 :  0.5.1 (OK)

ccordoba12 commented 5 months ago

Hey @Miguel-LlamasLanza, thanks for reporting. We'll take a look at this one after we release Spyder 6 in a couple of months (sorry, we're busy trying to wrap up that release).

Miguel-LlamasLanza commented 5 months ago

Thank you. For further information, I checked that it also worked properly with version 5.4.5. So the problem happened somewhere between this version and 5.5.1

ccordoba12 commented 5 months ago

Ok, if this was working before then it's a regression. So, I'll try to fix it in our next version (5.5.5), to be released shortly.

ccordoba12 commented 3 months ago

Sorry, I'm going to leave this for 6.0.1 because I don't have time to take a look at it right now and it's a minor regression.

spyder-ide / spyder