After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 645762 - Fix Python Console plugin
Fix Python Console plugin
Status: RESOLVED FIXED
Product: totem
Classification: Core
Component: Plugins
2.91.x
Other Linux
: Normal blocker
: ---
Assigned To: General Totem maintainer(s)
General Totem maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2011-03-26 17:37 UTC by Philip Withnall
Modified: 2011-03-27 15:36 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
pythonconsole: Fix calls to Gtk.Action() (1.60 KB, patch)
2011-03-26 17:40 UTC, Philip Withnall
committed Details | Review
pythonconsole: Port fixes to key event handling from gedit's pythonconsole (3.50 KB, patch)
2011-03-26 17:41 UTC, Philip Withnall
committed Details | Review

Description Philip Withnall 2011-03-26 17:37:37 UTC
Patches coming up to fix some blocker issues which are stopping the pythonconsole plugin working at all.
Comment 1 Philip Withnall 2011-03-26 17:40:23 UTC
Created attachment 184306 [details] [review]
pythonconsole: Fix calls to Gtk.Action()

This patch fixes the pythonconsole plugin's use of Gtk.Action(). We were calling it without passing the stock_id parameter, which is missing an allow-none annotation. Not passing the parameter causes a Python error, meaning the plugin can't be activated.
Comment 2 Philip Withnall 2011-03-26 17:41:52 UTC
Created attachment 184307 [details] [review]
pythonconsole: Port fixes to key event handling from gedit's pythonconsole

Patch to fix the key event handling in the pythonconsole plugin. Changes to the GIR file for GDK since the plugin was last tested mean that without these changes, no commands are ever recognised by the plugin and it's effectively useless.

Based on the changes in the corresponding plugin in gedit:
 • http://git.gnome.org/browse/gedit/commit/plugins/pythonconsole/pythonconsole/console.py?id=da49b588a394717b5ea0a3f9e6a823a86e62feaehttp://git.gnome.org/browse/gedit/commit/plugins/pythonconsole/pythonconsole/console.py?id=ba74278347888e4a16f3a0fee2a359b7f2bcde8b
Comment 3 Bastien Nocera 2011-03-27 04:22:37 UTC
Comment on attachment 184307 [details] [review]
pythonconsole: Port fixes to key event handling from gedit's pythonconsole

Committed with a fixed up commit message.
Comment 4 Bastien Nocera 2011-03-27 04:23:59 UTC
I'm leaving this opened. Ctrl+D doesn't seem to close the console anymore, and completion with tab seems to be broken. I never used the console before. Are those expected (in which case I'll file a separate bug), or are they regressions?
Comment 5 Philip Withnall 2011-03-27 07:33:12 UTC
(In reply to comment #4)
> I'm leaving this opened. Ctrl+D doesn't seem to close the console anymore, and
> completion with tab seems to be broken. I never used the console before. Are
> those expected (in which case I'll file a separate bug), or are they
> regressions?

I don't think Ctrl+D ever closed the console — at least, the code in the “destroy” method has always been commented out. I don't think tab completion ever existed.

What would be more useful in the next cycle is to merge our pythonconsole plugin with the one in gedit, probably by moving them both to libpeas or something.
Comment 6 Bastien Nocera 2011-03-27 15:36:35 UTC
Fair enough, filed as Bug 645831.