Bug 512 – QGridLayout::getItemPosition() is not available

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 512 - QGridLayout::getItemPosition() is not available

Summary:

QGridLayout::getItemPosition() is not available

Status:	CLOSED FIXED

Product:	PySide
Classification:	Unclassified
Component:	QtGui
Version:	HEAD
Platform:	All All

Importance:	P2 normal
Assigned To:	Hugo Parente Lima

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2010-12-01 09:30 EET by Matti Airas
Modified:	2010-12-20 11:26 EET (History)
CC List:	8 users (show)

See Also:

Attachments
Patch to PySide/QtGui/typesystem_gui_common.xml to expose QGridLayout::getItemPosition() (1.36 KB, patch) 2010-12-07 20:59 EET, Patrick Hartling	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 Matti Airas 2010-12-01 09:30:33 EET

From the mailing list by Patrick Hartling:

"""
I noticed that the method QGridLayout::getItemPosition() is not available in
QtGui.QGridLayout. Is there a technical limitation that prevents that method
from being available? If so, I am willing to help try to resolve it. I imagine
that anywhere that Qt has by-reference parameters for types that are immutable
in Python would currently be excluded in the same manner as
QGridLayout::getItemPosition().

 -Patrick
"""

Comment 1 Matti Airas 2010-12-01 09:31:20 EET

Confirming, setting priority P3. This will be fixed during the upcoming
sprints.

Comment 2 Patrick Hartling 2010-12-07 20:59:14 EET

Created attachment 171 [details]
Patch to PySide/QtGui/typesystem_gui_common.xml to expose
QGridLayout::getItemPosition()

This change to PySide/QtGui/typesystem_gui_common.xml exposes
QGridLayout::getItemPosition(). It seems to work for me, but there may have
been an easier way to handle this case.

Comment 3 Matti Airas 2010-12-08 05:22:05 EET

Thanks for the patch! I'm raising priority to P2 to not keep the patch
unnecessarily waiting.

Comment 4 Hugo Parente Lima 2010-12-08 14:05:42 EET

Nice patch, just a small comment about it:

You can use:

%CPPSELF.%FUNCTION_NAME(%1, &amp;a, &amp;b, &amp;c, &amp;d);

instead of:

%CPPSELF-&gt;%FUNCTION_NAME(%1, &amp;a, &amp;b, &amp;c, &amp;d);

The generator will take care if it needs to write "->" or ".".

Comment 5 Hugo Parente Lima 2010-12-14 15:31:20 EET

I changed your patch a bit, but the essence still the same.

fixed in commit:
pyside/e3796fa3e79b7c28ba1fb2142c09f98dc5abb085

Comment 6 Hugo Parente Lima 2010-12-20 11:26:10 EET

Closign bugs after release of 1.0.0 beta2.