diff options
author | Juha Vuolle <juha.vuolle@insta.fi> | 2021-06-10 13:27:19 +0300 |
---|---|---|
committer | Juha Vuolle <juha.vuolle@insta.fi> | 2021-06-15 11:36:11 +0300 |
commit | c3cc1253ba159e6f4c9bd4c92da2b8c300ee94f2 (patch) | |
tree | 97085cbb23d2fc78bfb9ce4bff1c4148a0e71a32 /src | |
parent | f8445fdcbf75e455443304bc290c48c37961e9f1 (diff) |
Sensor identifier autotest and fix related findings
This commit adds a (QML) test for sensor identifier (and activation
whose logic is closely related). The related changes:
* Change sensor 'identifier' and 'type' properties to QByteArray. This
now matches with C++ side that uses QByteArray
* Allow changing of 'identifier' after componentComplete. This is now
aligned with C++ side. Changing identifier is fine as long as the
backend is not connected.
Pick-to: 6.2
Task-number: QTBUG-92513
Task-number: QTBUG-92505
Change-Id: I326d840d5a4efb13a3a6578711563e8054cc9961
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
Diffstat (limited to 'src')
-rw-r--r-- | src/sensors/doc/src/qt6-changes.qdoc | 5 | ||||
-rw-r--r-- | src/sensorsquick/qmlsensor.cpp | 31 | ||||
-rw-r--r-- | src/sensorsquick/qmlsensor_p.h | 12 |
3 files changed, 20 insertions, 28 deletions
diff --git a/src/sensors/doc/src/qt6-changes.qdoc b/src/sensors/doc/src/qt6-changes.qdoc index f71cde15..ffde4d19 100644 --- a/src/sensors/doc/src/qt6-changes.qdoc +++ b/src/sensors/doc/src/qt6-changes.qdoc @@ -79,4 +79,9 @@ that in most if not all cases the changes are not mandatory. Instead it is up to the individual sensor backend implementations to decide if the changes are useful. + \section2 Sensor::type and and QSensor::type properties marked as constant + + The \c type property cannot change after instantiation and is marked as constant + in both C++ and QML. + */ diff --git a/src/sensorsquick/qmlsensor.cpp b/src/sensorsquick/qmlsensor.cpp index 14dc21c7..2ff544a5 100644 --- a/src/sensorsquick/qmlsensor.cpp +++ b/src/sensorsquick/qmlsensor.cpp @@ -104,16 +104,14 @@ QmlSensor::~QmlSensor() Please see QSensor::identifier for information about this property. */ -QString QmlSensor::identifier() const +QByteArray QmlSensor::identifier() const { - return m_identifier; + return sensor()->identifier(); } -void QmlSensor::setIdentifier(const QString &identifier) +void QmlSensor::setIdentifier(const QByteArray &identifier) { - if (m_componentComplete) return; - m_identifier = identifier; - Q_EMIT identifierChanged(); + sensor()->setIdentifier(identifier); } /*! @@ -121,9 +119,9 @@ void QmlSensor::setIdentifier(const QString &identifier) This property holds the type of the sensor. */ -QString QmlSensor::type() const +QByteArray QmlSensor::type() const { - return QString::fromLatin1(sensor()->type()); + return sensor()->type(); } /*! @@ -448,26 +446,17 @@ void QmlSensor::componentComplete() connect(sensor(), SIGNAL(maxBufferSizeChanged(int)), this, SIGNAL(maxBufferSizeChanged(int))); connect(sensor(), SIGNAL(efficientBufferSizeChanged(int)), this, SIGNAL(efficientBufferSizeChanged(int))); connect(sensor(), &QSensor::busyChanged, this, &QmlSensor::busyChanged); - - // We need to set this on the sensor object now - sensor()->setIdentifier(m_identifier.toLocal8Bit()); + connect(sensor(), &QSensor::identifierChanged, this, &QmlSensor::identifierChanged); // These can change! - QByteArray oldIdentifier = sensor()->identifier(); int oldDataRate = dataRate(); int oldOutputRange = outputRange(); - bool ok = sensor()->connectToBackend(); - if (ok) { + if (sensor()->connectToBackend()) Q_EMIT connectedToBackendChanged(); - m_reading = createReading(); - m_reading->setParent(this); - } - if (oldIdentifier != sensor()->identifier()) { - m_identifier = QString::fromLatin1(sensor()->identifier()); - Q_EMIT identifierChanged(); - } + m_reading = createReading(); + m_reading->setParent(this); if (oldDataRate != dataRate()) Q_EMIT dataRateChanged(); if (oldOutputRange != outputRange()) diff --git a/src/sensorsquick/qmlsensor_p.h b/src/sensorsquick/qmlsensor_p.h index 2b893c99..276b6e76 100644 --- a/src/sensorsquick/qmlsensor_p.h +++ b/src/sensorsquick/qmlsensor_p.h @@ -72,8 +72,8 @@ class Q_SENSORSQUICK_PRIVATE_EXPORT QmlSensor : public QObject, public QQmlParse Q_OBJECT Q_DECLARE_PRIVATE(QmlSensor) Q_INTERFACES(QQmlParserStatus) - Q_PROPERTY(QString identifier READ identifier WRITE setIdentifier NOTIFY identifierChanged) - Q_PROPERTY(QString type READ type NOTIFY typeChanged) + Q_PROPERTY(QByteArray identifier READ identifier WRITE setIdentifier NOTIFY identifierChanged) + Q_PROPERTY(QByteArray type READ type CONSTANT) Q_PROPERTY(bool connectedToBackend READ isConnectedToBackend NOTIFY connectedToBackendChanged) Q_PROPERTY(QQmlListProperty<QmlSensorRange> availableDataRates READ availableDataRates NOTIFY availableDataRatesChanged) Q_PROPERTY(int dataRate READ dataRate WRITE setDataRate NOTIFY dataRateChanged) @@ -108,10 +108,10 @@ public: explicit QmlSensor(QObject *parent = 0); ~QmlSensor(); - QString identifier() const; - void setIdentifier(const QString &identifier); + QByteArray identifier() const; + void setIdentifier(const QByteArray &identifier); - QString type() const; + QByteArray type() const; bool isConnectedToBackend() const; @@ -162,7 +162,6 @@ public Q_SLOTS: Q_SIGNALS: void identifierChanged(); - void typeChanged(); void connectedToBackendChanged(); void availableDataRatesChanged(); void dataRateChanged(); @@ -196,7 +195,6 @@ private: virtual void _update(); bool m_componentComplete = false; bool m_activateOnComplete = false; - QString m_identifier; QmlSensorReading *m_reading = nullptr; }; |