Review Request: Only show language context menu extensions if the menu is invoked in a context relevant to that language

Sven Brauch svenbrauch at gmx.de
Fri Mar 9 23:22:14 UTC 2012


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

Review request for KDevelop and Milian Wolff.


Description
-------

I tried to implement proper rename support for python, and during that task I noticed this behaviour: any loaded language plugin will add its context menu extensions to any context menu. The c++ plugin adds its rename dialog to php and python files. This patch fixes that behaviour by retrieving the URL of the file the action was invoked on, and then checks that the plugin -- if it is a language plugin -- fits that language.


Diffs
-----

  shell/plugincontroller.cpp 23408a6 

Diff: http://git.reviewboard.kde.org/r/104212/diff/


Testing
-------

The "rename" action no longer shows up in python, but seems to work as before in c++.


Thanks,

Sven Brauch

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20120309/bcb55081/attachment.html>


More information about the KDevelop-devel mailing list