Added error handling in frontend debug forms

homeassistant/components/http/www_static/polymer/dialogs/event-fire-dialog.html

@@ -10,7 +10,11 @@
 <polymer-element name="event-fire-dialog" attributes="api">
 <template>
 
-  <ha-action-dialog id="dialog" heading="Fire Event" class='two-column'>
+  <ha-action-dialog
+    id="dialog"
+    heading="Fire Event"
+    class='two-column'
+    closeSelector='[dismissive]'>
 
     <div layout horizontal>
       <div class='ha-form'>
@@ -68,15 +72,15 @@ Polymer({
   },
 
   clickFireEvent: function() {
-    var data;
+    try {
-
+      this.api.fire_event(
-    if(this.$.inputData.value !== "") {
+        this.$.inputType.value,
-      data = JSON.parse(this.$.inputData.value);
+        this.$.inputData.value ? JSON.parse(this.$.inputData.value) : {});
-    } else {
+      this.$.dialog.close();
-      data = {};
+    
+    } catch (err) {
+      alert("Error parsing JSON: " + err);
     }
-
-    this.api.fire_event(this.$.inputType.value, data);
   }
 });
 </script>

homeassistant/components/http/www_static/polymer/dialogs/service-call-dialog.html

@@ -10,7 +10,10 @@
 <polymer-element name="service-call-dialog" attributes="api">
 <template>
 
-  <ha-action-dialog id="dialog" heading="Call Service">
+  <ha-action-dialog
+    id="dialog"
+    heading="Call Service"
+    closeSelector='[dismissive]'>
 
     <core-style ref='ha-dialog'></core-style>
 
@@ -45,7 +48,7 @@
 Polymer({
   ready: function() {
     // to ensure callback methods work..
-    this.serviceSelected = this.serviceSelected.bind(this)
+    this.serviceSelected = this.serviceSelected.bind(this);
   },
 
   show: function(domain, service, serviceData) {
@@ -65,16 +68,17 @@ Polymer({
   },
 
   clickCallService: function() {
-    var data;
+    try {
-
-    if(this.$.inputData.value != "") {
-      data = JSON.parse(this.$.inputData.value);
-    }
-
     this.api.call_service(
       this.$.inputDomain.value,
       this.$.inputService.value,
-      data);
+      this.$.inputData.value ? JSON.parse(this.$.inputData.value) : {});
+
+      this.$.dialog.close();
+    
+    } catch (err) {
+      alert("Error parsing JSON: " + err);
+    }
   }
 });
 </script>

homeassistant/components/http/www_static/polymer/dialogs/state-set-dialog.html

@@ -9,12 +9,16 @@
 
 <polymer-element name="state-set-dialog" attributes="api">
 <template>
-  <ha-action-dialog id="dialog" heading="Set State">
+  <ha-action-dialog
+    id="dialog"
+    heading="Set State"
+    closeSelector='[dismissive]'>
 
     <core-style ref='ha-dialog'></core-style>
 
     <p>
-      This dialog will update the representation of the device within Home Assistant. This will not communicate with the actual device.
+      This dialog will update the representation of the device within Home Assistant.<br />
+      This will not communicate with the actual device.
     </p>
 
     <div layout horizontal>
@@ -81,12 +85,18 @@ Polymer({
     this.setStateData(state.attributes);
   },
 
-  clickSetState: function() {
+  clickSetState: function(ev) {
-    this.api.set_state(
+    try {
-      this.$.inputEntityID.value,
+      this.api.set_state(
-      this.$.inputState.value,
+        this.$.inputEntityID.value,
-      JSON.parse(this.$.inputData.value)
+        this.$.inputState.value,
+        this.$.inputData.value ? JSON.parse(this.$.inputData.value) : {}
       );
+
+      this.$.dialog.close();
+    } catch (err) {
+      alert("Error parsing JSON: " + err);
+    }
   }
 });
 </script>