Loading...

XML

Word

Printable

Type: Task
Resolution: Fixed
Priority: P2: Important
Fix Version/s: 6.5.4, 6.6.1, 6.7.0 FF
Affects Version/s: 6.5, 6.6.0 FF
Component/s: Core: Other
Labels:
None

Epic Link:
Do not register property setters as dependencies of the properties they set
Story Points:
3
Commits:
5743837a2 (dev), e87daa3f3 (6.6), 3da854a9a (6.5), f5a5c5991 (dev), b799a2107 (6.6), 66987ebe5 (tqtc/lts-6.5)
Sprint:
Foundation Sprint 89, Foundation Sprint 90, Foundation Sprint 91, Foundation Sprint 92, Foundation Sprint 93, Foundations Sprint 94

As mentioned in the epic, the typical pattern of writing a setter for a bindable property causes a binding loop.

The scope of this task is to identify all such bindable properties.

One way to achieve it is to take a unit-test from https://codereview.qt-project.org/c/qt/qtbase/+/497620, re-write it in a more general way, and add this test to QTestPrivate::testReadWritePropertyBasics() test method.

Applying such patch locally would allow to identify all the problematic property. Once these are fixed (in all submodules), the updated test can be merged.

is required for

QTBUG-116346 [qtbase] Fix all binding loops caused by misuse of bindable properties

Closed

QTBUG-116540 [qtdeclarative] Fix all binding loops caused by misuse of bindable properties

Closed

QTBUG-116541 [QtPositioning] Fix all binding loops caused by misuse of bindable properties

Closed

QTBUG-116542 [QtScxml] Fix all binding loops caused by misuse of bindable properties

Closed

QTBUG-116543 [QtSensors] Fix all binding loops caused by misuse of bindable properties

Closed

QTBUG-116544 [Other] Fix all binding loops caused by misuse of bindable properties

Closed

(1 is required for)

- Issue Only
- Show All Reviews
- Show Open Reviews
- Show All Issues
- Show Open Issues

For Gerrit Dashboard: QTBUG-116345
#	Subject	Branch	Project	Status	CR	V
499136,15	Extend QTestPrivate property tests - actual implementation [2/2]	dev	qt/qtbase	Status: MERGED	+2	0
505101,2	Extend QTestPrivate property tests - update signatures [1/2]	dev	qt/qtbase	Status: MERGED	+2	0
507733,2	Extend QTestPrivate property tests - update signatures [1/2]	6.6	qt/qtbase	Status: MERGED	+2	0
507802,2	Extend QTestPrivate property tests - update signatures [1/2]	6.5	qt/qtbase	Status: MERGED	+2	0
515442,2	Extend QTestPrivate property tests - actual implementation [2/2]	6.6	qt/qtbase	Status: MERGED	+2	0
515471,2	Extend QTestPrivate property tests - actual implementation [2/2]	tqtc/lts-6.5	qt/tqtc-qtbase	Status: MERGED	+2	0