Bug 553 – A warning against using QUILoader is needed in the documentation

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 553 - A warning against using QUILoader is needed in the documentation

Summary:

A warning against using QUILoader is needed in the documentation

Status:	CLOSED FIXED

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

Importance:	P2 normal
Assigned To:	Marcelo Lira

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2010-12-17 14:10 EET by Matti Airas
Modified:	2011-01-21 15:44 EET (History)
CC List:	8 users (show)

See Also:

Attachments
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-17 14:10:19 EET

Since use of QUILoader is not recommended until a complete rewrite, there needs
to be a prominent warning about the fact in the documentation.

Comment 1 Matti Airas 2010-12-17 14:10:56 EET

I'm giving this P2 since this should be rather trivial to do and potentially
saves developers a lot of headaches.

Comment 2 renato filho 2010-12-27 19:28:03 EET

Like We discuss before I do not think this is a bug. 
We need check if a code rewrite is really necessary or not, and if the current
QUiLoader has any problems. We can decide this on IRC tomorrow, and then try to
find a solution. 

This bug is related with bug #533.

Comment 3 Hugo Parente Lima 2011-01-05 14:55:46 EET

As discussed on IRC, QUiLoader will still valid, so I think we could mark this
bug as INVALID, Matti?

Comment 4 renato filho 2011-01-05 15:39:55 EET

Like discuss on the meeting this bug will still open until we create a
documentation about virtual methods override problems, and a complete
documentation to explain how to use custom widget with QUiLoder and PySide.

Comment 5 Matti Airas 2011-01-06 06:18:41 EET

Yes, Renato is right - the QtUILoader issues need to be documented. Since
QtUILoader is considered valid after all :-), we don't need a warning against
using it at all, but just need to document the related gotchas.

Comment 6 Marcelo Lira 2011-01-14 19:34:47 EET

"Thou shall not duck punch virtual methods!"
http://www.pyside.org/docs/shiboken/wordsofadvice.html#duck-punching-and-virtual-methods

How to properly use QUiLoader.registerCustomWidget(customWidgetType):
http://www.pyside.org/docs/pyside/PySide/QtUiTools/QUiLoader.html#PySide.QtUiTools.QUiLoader.registerCustomWidget

Comment 7 renato filho 2011-01-21 15:44:44 EET

release beta4