Bug 525 – New Signal()-Slot() syntax does not work for subclass of QMenu

PySide Bugzilla Closed for New Bugs

PySide is now a Qt Add-on and uses the Qt Project's JIRA Bug Tracker instead of this Bugzilla instance. This Bugzilla is left for reference purposes.

Bug 525 - New Signal()-Slot() syntax does not work for subclass of QMenu

Summary:

New Signal()-Slot() syntax does not work for subclass of QMenu

Status:	CLOSED FIXED

Product:	PySide
Classification:	Unclassified
Component:	PySide
Version:	1.0.0 beta1
Platform:	Macintosh Mac OS

Importance:	P2 normal
Assigned To:	renato filho

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2010-12-03 21:15 EET by Thomas Sturm
Modified:	2010-12-20 11:26 EET (History)
CC List:	8 users (show)

See Also:

Attachments
Produces the bug and contains a workaround in comments. (1.09 KB, text/x-python-script) 2010-12-03 21:15 EET, Thomas Sturm	Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Thomas Sturm 2010-12-03 21:15:54 EET

Created attachment 164 [details]
Produces the bug and contains a workaround in comments.

For a subclass of QMenu I cannot connect the signal aboutToShow using the new
syntax while the old style works. This problems occurs only since I upgraded
from an older binary distribution to pyside-1.0.0-beta1-qt47-py26apple.pkg + Qt
4.7.1 on the Mac.

I am connecting on my application many other signals in the new style without
problems.

I am attaching a small example demonstrating the problem. The error message I
get when running this example is the following:

Traceback (most recent call last):
  File "./bug.py", line 36, in <module>
    mainwindow = Window()
  File "./bug.py", line 19, in __init__
    self.m2.aboutToShow.connect(self.aboutToShowHandler)
AttributeError: 'PySide.QtCore.Signal' object has no attribute 'connect'

Comment 1 Matti Airas 2010-12-04 08:14:27 EET

If I understand correctly, this is a regression (used to work but does no
longer)? Prioritizing P2.

Comment 2 Thomas Sturm 2010-12-04 14:53:18 EET

Yes, it used to work but does not anymore.

Comment 3 renato filho 2010-12-17 17:45:50 EET

fixed on shiboken commit:

commit 5d07b706d29cb80aa10b34fad1f883a78266a9c2
Author: Renato Araujo Oliveira Filho <renato.filho@openbossa.org>
Date:   Wed Dec 15 17:55:51 2010 -0300

Comment 4 Hugo Parente Lima 2010-12-20 11:26:13 EET

Closign bugs after release of 1.0.0 beta2.