allow hue edit

RicoSuter · Nov 22, 2023 · 878904c · 878904c
1 parent fc55fc1
commit 878904c
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 3 deletions.
diff --git a/src/HomeBlaze.Philips.Hue/HueBridge.cs b/src/HomeBlaze.Philips.Hue/HueBridge.cs
@@ -20,7 +20,7 @@ namespace HomeBlaze.Philips.Hue
 {
     [DisplayName("Philips Hue Bridge")]
     [Description("The brains of the Philips Hue smart lighting system, the Hue Bridge allows you to connect and control up to 50 lights and accessories.")]
-    [ThingSetup(typeof(HueBridgeSetup))]
+    [ThingSetup(typeof(HueBridgeSetup), CanEdit = true)]
     public class HueBridge : PollingThing, IIconProvider, ILastUpdatedProvider,
         IConnectedThing, IHubDevice, IPowerConsumptionSensor
     {

diff --git a/src/HomeBlaze.Philips.Hue/HueBridgeSetup.razor b/src/HomeBlaze.Philips.Hue/HueBridgeSetup.razor
@@ -88,4 +88,28 @@ else
 
         return null;
     }
+
+    public override async Task<bool> UpdateEditedThingAsync(CancellationToken cancellationToken)
+    {
+        try
+        {
+            if (EditedThing != null &&
+                _selectedBridge is LocatedBridge selectedBridge)
+            {
+                var client = new LocalHueClient(selectedBridge.IpAddress);
+                var appKey = await client.RegisterAsync("HomeBlaze", "Local");
+
+                EditedThing.AppKey = appKey;
+                EditedThing.BridgeId = selectedBridge.BridgeId;
+
+                return true;
+            }
+        }
+        catch (Exception e)
+        {
+            await DialogService.ShowMessageBox("Failed to add Philips Hue Bridge", e.Message);
+        }
+
+        return false;
+    }
 }
diff --git a/src/HomeBlaze.Services/ReflectionUtilities.cs b/src/HomeBlaze.Services/ReflectionUtilities.cs
@@ -35,7 +35,7 @@ public class StatePropertyInfo
                 }
                 catch (Exception e)
                 {
-                    logger.LogWarning($"Getter of property threw an exception.", e);
+                    logger.LogWarning(e, $"Getter of property threw an exception.");
                     return null;
                 }
             }

diff --git a/src/HomeBlaze.Xbox.SmartGlass/SmartGlass/Messaging/Session/SessionMessageTransport.cs b/src/HomeBlaze.Xbox.SmartGlass/SmartGlass/Messaging/Session/SessionMessageTransport.cs
@@ -89,7 +89,7 @@ public void StartHeartbeat()
                     }
                     catch (Exception ex)
                     {
-                        logger.LogError("Error sending heartbeat", ex);
+                        logger.LogError(ex, "Error sending heartbeat");
                     }
                     await Task.Delay(_heartbeatInterval);
                     if (DateTime.Now - _lastReceived > _heartbeatTimeout)