From ec0ceec0c5ffb20e1be0c2ea8b0d8c954ea90ea1 Mon Sep 17 00:00:00 2001
From: Tim Krehan <mail@timkrehan.de>
Date: Sun, 26 May 2019 17:01:30 +0200
Subject: [PATCH] added selector variable

---
 js/list.js | 34 +++++++++++++++++-----------------
 1 file changed, 17 insertions(+), 17 deletions(-)

diff --git a/js/list.js b/js/list.js
index 4eda01f..1cb3895 100644
--- a/js/list.js
+++ b/js/list.js
@@ -47,29 +47,29 @@ function checkItem() {
       status: $(this).prop("checked")
     },
     success: function () {
-      var color = $("[data-id='" + dataId + "'").data("color");
-      console.log(color);
-      $("[data-id='" + dataId + "'").parent().parent().removeClass("bg-danger");
+      var dataIdSelector = ("[data-id='" + dataId + "'");
+      var color = $(dataIdSelector).data("color");
+      $(dataIdSelector).parent().parent().removeClass("bg-danger");
 
-      if($("[data-id='" + dataId + "'").prop("checked")){
-        $("[data-id='" + dataId + "'").parent().parent().removeClass(color);
-        $("[data-id='" + dataId + "'").parent().parent().addClass("bg-success");
+      if($(dataIdSelector).prop("checked")){
+        $(dataIdSelector).parent().parent().removeClass(color);
+        $(dataIdSelector).parent().parent().addClass("bg-success");
       }
       else{
-        $("[data-id='" + dataId + "'").parent().parent().removeClass("bg-success");
-        $("[data-id='" + dataId + "'").parent().parent().addClass(color);
+        $(dataIdSelector).parent().parent().removeClass("bg-success");
+        $(dataIdSelector).parent().parent().addClass(color);
       }
     },
     error: function () {
-      $("[data-id='" + dataId + "'").parent().parent().addClass("bg-danger");
-      $("[data-id='" + dataId + "'").parent().parent().data("toggle", "popover");
-      $("[data-id='" + dataId + "'").parent().parent().data("container", "body");
-      $("[data-id='" + dataId + "'").parent().parent().data("placement", "top");
-      $("[data-id='" + dataId + "'").parent().parent().data("html", true);
-      $("[data-id='" + dataId + "'").parent().parent().data("trigger", "focus");
-      $("[data-id='" + dataId + "'").parent().parent().data("content", "Dieses Element konnte nicht gespeichert werden.<br />Bitte die Seite Aktualisieren");
-      $("[data-id='" + dataId + "'").parent().parent().attr("title", "Fehler!");
-      $("[data-id='" + dataId + "'").parent().parent().popover('show');
+      $(dataIdSelector).parent().parent().addClass("bg-danger");
+      $(dataIdSelector).parent().parent().data("toggle", "popover");
+      $(dataIdSelector).parent().parent().data("container", "body");
+      $(dataIdSelector).parent().parent().data("placement", "top");
+      $(dataIdSelector).parent().parent().data("html", true);
+      $(dataIdSelector).parent().parent().data("trigger", "focus");
+      $(dataIdSelector).parent().parent().data("content", "Dieses Element konnte nicht gespeichert werden.<br />Bitte die Seite Aktualisieren");
+      $(dataIdSelector).parent().parent().attr("title", "Fehler!");
+      $(dataIdSelector).parent().parent().popover('show');
     }
   });
 }
\ No newline at end of file