Review Request: krunner not running apps / crashing after hitting enter

[wilderkde at gmail.com]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/452/
-----------------------------------------------------------

Review request for Plasma.


Summary
-------

Calling KHistoryComboBox::addToHistory( item ),  in some cases (depending on the position of the item in history) causes KLineEdit to emit userTextChanged; this in turns triggers queryTextChanged("") in interface, which resets the context. When the runner is asked to run, the context query is "", so either the runner either doesn't do anything (#181453) or it crashes (#188186 #186036) [at least before today's fix in the libs by dfaure].
This behaviour is compatible with br 181453 i.e. if one repeats the same query two times in a row, the second time it's guaranteed to work. 

This patch moves addToHistory after the run, therefore the context can't possibly be reset before the match is run.
The patch goes together with a patch in kdelibs which prevents the signal to be emitted in some cases (the ones observed for this situation), but, after discussing with dfaure, we can't block the signals altogheter, so it might still happen that userTextChanged is emitted in some weird edge cases I can't think of.

In any case, the patch is quite harmless and it moves the call in the right place in case, later on, we might want to make sure that the run actually works before adding the query term to history (right now my history is polluted with typos)

Too bad I found this after tagging. We can't do anything about it, can we?


This addresses bugs 181453, 186036 and 188186.
    https://bugs.kde.org/show_bug.cgi?id=181453
    https://bugs.kde.org/show_bug.cgi?id=186036
    https://bugs.kde.org/show_bug.cgi?id=188186


Diffs
-----

  branches/KDE/4.2/kdebase/workspace/krunner/interfaces/default/interface.cpp 945379 

Diff: http://reviewboard.kde.org/r/452/diff


Testing
-------

The patch solves the problem. So far krunner always launched what it was asked to run without any crash


Thanks,

wilder