Skip to content

Commit d8c4b8a

Browse files
committed
Minor updates
1 parent 503636d commit d8c4b8a

File tree

4 files changed

+5
-13
lines changed

4 files changed

+5
-13
lines changed

CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
2222

2323
### 🔧 Changed
2424

25-
- Hue Bridge - Wizard updates to support bridge-ids
25+
- Hue Bridge - Wizard updates to support bridge-ids, overall code refactoring
2626

2727
- **Fixes:**
2828
- UI - Language is not selectable (#1877)

libsrc/leddevice/dev_net/LedDevicePhilipsHue.cpp

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -945,10 +945,7 @@ void LedDevicePhilipsHueBridge::setBridgeDetails(const QJsonDocument &doc, bool
945945

946946
// Check if bridge-id MAC prefix is known from Philips Hue devices
947947
// If not, we assume it is a DiyHue or other 3rd party bridge
948-
if (!DEV_DATA_MAC_PREFIXES.contains(getBridgeId().left(6)))
949-
{
950-
_isPhilipsHueBridge = false;
951-
}
948+
_isPhilipsHueBridge = DEV_DATA_MAC_PREFIXES.contains(getBridgeId().left(6));
952949

953950
// Check if bridge is DIYHue to apply workarounds
954951
if (_deviceName.startsWith("DiyHue", Qt::CaseInsensitive))

libsrc/leddevice/dev_net/ProviderRestApi.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
#ifndef PROVIDERRESTKAPI_H
2-
#define PROVIDERRESTKAPI_H
1+
#ifndef PROVIDERRESTAPI_H
2+
#define PROVIDERRESTAPI_H
33

44
// Local-Hyperion includes
55
#include <utils/Logger.h>
@@ -482,4 +482,4 @@ protected slots:
482482
bool _isSelfSignedCertificateAccpeted;
483483
};
484484

485-
#endif // PROVIDERRESTKAPI_H
485+
#endif // PROVIDERRESTAPI_H

libsrc/leddevice/schemas/schema-philipshue.json

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -169,11 +169,6 @@
169169
"useEntertainmentAPI": true
170170
}
171171
},
172-
"options": {
173-
"dependencies": {
174-
"useAPIv2": false
175-
}
176-
},
177172
"propertyOrder": 15
178173
},
179174
"brightnessFactor": {

0 commit comments

Comments
 (0)