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 635437 - override UIManager.insert_action_group to make pos optional
override UIManager.insert_action_group to make pos optional
Status: RESOLVED FIXED
Product: pygobject
Classification: Bindings
Component: introspection
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Nobody's working on this now (help wanted and appreciated)
Python bindings maintainers
Depends on:
Blocks:
 
 
Reported: 2010-11-21 14:32 UTC by Paolo Borelli
Modified: 2010-11-23 18:22 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch (2.39 KB, patch)
2010-11-21 14:37 UTC, Paolo Borelli 		none Details | Review
patch (2.41 KB, patch)
2010-11-21 15:26 UTC, Paolo Borelli 		committed Details | Review

Description Paolo Borelli 2010-11-21 14:32:48 UTC 
In PyGtk the position arg was optional (default to -1).

Add an override to make it do the same in pygi.

Note: the test currently crashed due to bug #635436
Comment 1 Paolo Borelli 2010-11-21 14:37:05 UTC 
Created attachment 174959 [details] [review]
patch

(I moved the whole test for UIManager after the one for ActionGroups since the former uses ActionGroups)
Comment 2 Paolo Borelli 2010-11-21 15:26:49 UTC 
Created attachment 174963 [details] [review]
patch

As specified in the other bug actions groups must have a name