Bug 676 – wrong QLocale.toFloat() description

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 676 - wrong QLocale.toFloat() description

Summary:

wrong QLocale.toFloat() description

Status:	CLOSED FIXED

Product:	PySide
Classification:	Unclassified
Component:	Documentation
Version:	1.0.0 beta5
Platform:	All All

Importance:	P4 minor
Assigned To:	renato filho

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2011-02-09 12:55 EET by Krasilnikov Sergey
Modified:	2011-03-04 15:10 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 Krasilnikov Sergey 2011-02-09 12:55:49 EET

from online documentation
QLocale.toFloat(s)
Parameter:    s – unicode
Return type:    float

but function returning [float, bool]
----------------------------------
curve.setAmplitude(QtCore.QLocale().toFloat(value))
TypeError: 'PySide.QtCore.QEasingCurve.setAmplitude' called with wrong argument
types:
  PySide.QtCore.QEasingCurve.setAmplitude(float, bool)
Supported signatures:
  PySide.QtCore.QEasingCurve.setAmplitude(float)

p.s. toInt, toShort ... also returns tuple with converted value and boolean
(status of conversion)

Comment 1 Hugo Parente Lima 2011-02-09 15:30:26 EET

Prioritizing this as P4, it's a documentation issue.

Comment 2 Luciano Wolf 2011-03-02 20:10:28 EET

Fixed on commit: #17f2d83e0e33e146d8efd55a13328ace8afad152

I'm using the same return values as documented into PyQt documentation.

Comment 3 Hugo Parente Lima 2011-03-04 15:10:39 EET

Closing bug due to 1.0.0 release.