The Search and Replace filters are not being saved in a session. Other settings from the search form are being saved. If I look at the .katesession file it doesn't appear the "Filters" entry is ever modified. If I modify it manually it does load that value, so it appears to be a save issue. (Though I can't produce multiple filters correctly since the patterns are comma separated, but so must the filters) Reproducible: Always I have seen an old bug related to this, but that was resolved some versions ago and this bug is new. I updated my git version and tried again with the same results.
Probably related is that nothing in <plugin_search.cpp> ever appears to add items to the filterCombo object. That is, I don't see any possibility that there would actually be a list of items created, other than by loading them. Just for a test I add some items just before saving and they are indeed saved, so it'd appear that the loop on line 869 will just never get the user's changes. What is the intended behaviour? Should the filter be added to the list each time the user searches with a new filter? This wouldn't be hard to add.
Created attachment 76361 [details] adds filters to the combobox as used
Thanks. I noticed this a little while ago but forgot about it. Hitting "enter" in the filter combo will add it. /Kåre
No, I don't believe "enter" adds it anymore, as that is directly connected to "startSearch" now.
I just tested and it _does_ save the new filter. There combo-box is connected to the returnPressed() signal of the line-edit. But this _is_ a bug that has to be fixed ASAP. /Kåre
Git commit eceecef2eb736bdee17563f88814cb865bdd2024 by Kåre Särs. Committed on 10/01/2013 at 23:56. Pushed by sars into branch 'master'. Search plugin: Fix updating of comboboxes on search and replace Thanks for the patch I committed it wit slight changes :) M +9 -0 kate/plugins/search/plugin_search.cpp http://commits.kde.org/kate/eceecef2eb736bdee17563f88814cb865bdd2024
Git commit ff37da01472d78eb1f60c7af6bc0cf846eef02bf by Kåre Särs. Committed on 10/01/2013 at 23:56. Pushed by sars into branch 'KDE/4.10'. Search plugin: Fix updating of comboboxes on search and replace Thanks for the patch I committed it wit slight changes :) (cherry picked from commit eceecef2eb736bdee17563f88814cb865bdd2024) M +9 -0 kate/plugins/search/plugin_search.cpp http://commits.kde.org/kate/ff37da01472d78eb1f60c7af6bc0cf846eef02bf
Looks good, defintiely more useful now. One question though, should perhaps the filter not be a kind of Least-Recently-Used list? That is, should we not perhaps always move the recent item to the top of the list (since only the most recent 10 are saved)?