Details
-
Task
-
Resolution: Fixed
-
P2: Important
-
6.5, 6.6.0 FF
-
None
-
3
-
e0bb5f497 (dev), d529400fe (dev), 53a8aebc7 (6.6), 817388e7c (6.6), dbd5162e0 (6.5), bbea23164 (6.5), c953767e2 (tqtc/lts-6.5)
-
Foundation Sprint 92
Description
Fix all properties that will fail the test implemented in QTBUG-116345.
In general case, the fix would be as simple as calling valueBypassingBindings() instead of simply accessing the property in the setter (see the epic for a code example).
The scope of this task is to implement fixes in all submodules that are not covered by the other tasks in the epic.
Using git-grep, I managed to get the following list of files in three submodules.
Qt Remote Objects
~/qt5/qtremoteobjects$ git grep -rI --name-only --recurse-submodules "Q_OBJECT_COMPAT_PROPERTY\|Q_OBJECT_COMPUTED_PROPERTY\|Q_OBJECT_BINDABLE_PROPERTY"
src/remoteobjects/qremoteobjectregistry.cpp
Qt Serial Port
~/qt5/qtserialport$ git grep -rI --name-only --recurse-submodules "Q_OBJECT_COMPAT_PROPERTY\|Q_OBJECT_COMPUTED_PROPERTY\|Q_OBJECT_BINDABLE_PROPERTY"
src/serialport/qserialport_p.h
Qt WebChannel
~/qt5/qtwebchannel$ git grep -rI --name-only --recurse-submodules "Q_OBJECT_COMPAT_PROPERTY\|Q_OBJECT_COMPUTED_PROPERTY\|Q_OBJECT_BINDABLE_PROPERTY"
src/webchannel/qmetaobjectpublisher_p.h
tests/auto/qml/testobject.h
tests/auto/webchannel/tst_webchannel.h
If it turns out that some of the classes are missing the tests for bindable properties, or if the tests do not use the QTestPrivate helpers, then the missing tests should be added.
Attachments
Issue Links
- clones
-
QTBUG-116543 [QtSensors] Fix all binding loops caused by misuse of bindable properties
- Closed
- depends on
-
QTBUG-116345 Identify all bindable properties that cause binding loops
- Closed
- resulted in
-
QTBUG-117887 QSerialPort::bindableStopBits has a wrong return value
- Closed
For Gerrit Dashboard: QTBUG-116544 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
510045,2 | SerialPort: fix bindable properties | dev | qt/qtserialport | Status: MERGED | +2 | 0 |
510070,2 | Extend tests for bindable properties | dev | qt/qtwebchannel | Status: MERGED | +2 | 0 |
510097,2 | SerialPort: fix bindable properties | 6.6 | qt/qtserialport | Status: MERGED | +2 | 0 |
510113,2 | Extend tests for bindable properties | 6.6 | qt/qtwebchannel | Status: MERGED | +2 | 0 |
510172,2 | SerialPort: fix bindable properties | 6.5 | qt/qtserialport | Status: MERGED | +2 | 0 |
510177,2 | Extend tests for bindable properties | 6.5 | qt/qtwebchannel | Status: MERGED | +2 | 0 |
512728,2 | SerialPort: fix bindable properties | tqtc/lts-6.5 | qt/tqtc-qtserialport | Status: MERGED | +2 | 0 |