Add button to re-run search in files, so a user doesn't need to search again through the dialog.
Works fine, just as expected. All the tests were passed.
I think we need a more sophisticated approach. I suggest creating a new struct GrepJobSettings which holds all settings you can specify in a search dialog. Then, remove all the individual setters/getters (e.g. GrepDialog::patternString, GrepJob::setPatternString) in both GrepDialog/GrepJob and instead just use GrepJobSettings in both places.
To store the last used settings one could add another field to GrepPlugin, e.g. m_lastGrepJobSettings;. Makes sense?
This will use the pattern from the current selection instead of the last search pattern. This is not what we want unfortunately.
Some nitpicks. :)
Unneeded indentation change.
Probably it's better to use "const GrepJobSettings& settings" like in GrepJob.
Maybe some more readable name? Something like "template" or maybe "searchTemplate".
What's point of using comments here? It's not necessary since old version still be available on git in case it'll be needed.
Same there. Indentation issue.
Is m_templateString used somewhere now? If no, please remove it.