From 629a13493e023e89ae6d22aacae6d50e319c887c Mon Sep 17 00:00:00 2001
From: omarcodex <omaralimalik@gmail.com>
Date: Wed, 6 Sep 2017 23:55:47 -0700
Subject: [PATCH 1/7] Add front-end check box for DoneWhop on admin-edit form

---
 admin.html | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/admin.html b/admin.html
index ab06d49..5585384 100644
--- a/admin.html
+++ b/admin.html
@@ -107,7 +107,7 @@ <h2 class="mdl-card__title-text">Create a new DoWhop</h2>
           </div>
         </div>
       </div>
-      
+
       <!-- Start Mini-Form for Admin Email Data Entry-->
       <div class="mdl-cell mdl-cell--6-col">
         <div class="mdl-card">
@@ -126,6 +126,12 @@ <h2 id="selected-for-edit" class="mdl-card__title-text">Edit your DoWhop</h2>
                 <label class="mdl-textfield__label" for="doer-description-email-update">Who is attending this DoWhop (email, please)?</label>
                 <span class="mdl-tooltip" for="doer-description-email-update">Enter multiple doers, seperated, by, Commas</span>
               </div>
+              <div class="mdl-card__supporting-text">
+               <label id ="checkall" class="mdl-checkbox mdl-js-checkbox mdl-js-ripple-effect" for="checkbox-0">
+                 <input type="checkbox" id="donewhop-status" class="mdl-checkbox__input">
+                 <span class="mdl-checkbox__label">This is a DoneWhop</span>
+               </label>
+             </div
               <!-- END NEW -->
               <div class="mdl-card__actions">
                 <button class="btn btn-default" id="emailSubmit">
@@ -143,9 +149,9 @@ <h2 id="selected-for-edit" class="mdl-card__title-text">Edit your DoWhop</h2>
 
     <script src="scripts/dowhop-description-form-admin.js"></script>
     <script>
-      auth.onAuthStateChanged(function(user) {
-        if (user) registerDoWhopDescriptionCallback();
-      });
-    </script>
+    auth.onAuthStateChanged(function(user) {
+  if (user) registerDoWhopDescriptionCallback();
+});
+</script>
   </body>
 </html>

From c2c1608bdeda6747f1aa02d832373edd90e5b12f Mon Sep 17 00:00:00 2001
From: omarcodex <omaralimalik@gmail.com>
Date: Thu, 7 Sep 2017 17:05:15 -0700
Subject: [PATCH 2/7] Fix checkbox MDL id, class naming

---
 admin.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/admin.html b/admin.html
index 5585384..3f1138d 100644
--- a/admin.html
+++ b/admin.html
@@ -127,7 +127,7 @@ <h2 id="selected-for-edit" class="mdl-card__title-text">Edit your DoWhop</h2>
                 <span class="mdl-tooltip" for="doer-description-email-update">Enter multiple doers, seperated, by, Commas</span>
               </div>
               <div class="mdl-card__supporting-text">
-               <label id ="checkall" class="mdl-checkbox mdl-js-checkbox mdl-js-ripple-effect" for="checkbox-0">
+               <label id ="donewhop-checkbox" class="mdl-checkbox mdl-js-checkbox mdl-js-ripple-effect" for="donewhop-checkbox">
                  <input type="checkbox" id="donewhop-status" class="mdl-checkbox__input">
                  <span class="mdl-checkbox__label">This is a DoneWhop</span>
                </label>

From 9d07f3060bc0830d622edd38da0ed17ff8bcc92f Mon Sep 17 00:00:00 2001
From: omarcodex <omaralimalik@gmail.com>
Date: Thu, 7 Sep 2017 17:05:43 -0700
Subject: [PATCH 3/7] Add code to toggle, check DoneWhop checkbox status

---
 scripts/dowhop-description-form-admin.js | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/scripts/dowhop-description-form-admin.js b/scripts/dowhop-description-form-admin.js
index 82881ab..a3a6940 100644
--- a/scripts/dowhop-description-form-admin.js
+++ b/scripts/dowhop-description-form-admin.js
@@ -250,6 +250,7 @@ function revealEditEmailForm(node) {
 
 function updateCreatorDoerEmails(e) {
   e.preventDefault();
+  console.log(document.getElementById('donewhop-checkbox').classList.contains('is-checked') === true);
   adminEditDoWhopForm.removeAttribute('hidden');
   doWhopDescriptionsRef.child(doWhopDescriptionKeyForUpdate).update({
     creatorDescription: creatorDescriptionUpdate.value.toLowerCase(),
@@ -275,3 +276,13 @@ function addToMyDoWhops(node) {
     .child(node.parentElement.id)
     .update({ doer: true });
 }
+
+// Adding funcationality to manage DoneWhop checkbox and status.
+
+document.getElementById('donewhop-checkbox').addEventListener('click', checkDoneWhop);
+
+function checkDoneWhop() {
+  document.getElementById('donewhop-checkbox').classList.contains('is-checked') === true
+    ? document.getElementById('donewhop-checkbox').MaterialCheckbox.uncheck()
+    : document.getElementById('donewhop-checkbox').MaterialCheckbox.check();
+}

From 22af5d043d71f634e818ce729d1d54db60c8ed4c Mon Sep 17 00:00:00 2001
From: omarcodex <omaralimalik@gmail.com>
Date: Thu, 7 Sep 2017 17:10:03 -0700
Subject: [PATCH 4/7] Split toggle and return DoneWhopStatus upon form submit

---
 scripts/dowhop-description-form-admin.js | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/scripts/dowhop-description-form-admin.js b/scripts/dowhop-description-form-admin.js
index a3a6940..824d9b9 100644
--- a/scripts/dowhop-description-form-admin.js
+++ b/scripts/dowhop-description-form-admin.js
@@ -250,7 +250,7 @@ function revealEditEmailForm(node) {
 
 function updateCreatorDoerEmails(e) {
   e.preventDefault();
-  console.log(document.getElementById('donewhop-checkbox').classList.contains('is-checked') === true);
+  console.log(returnDoneWhopStatus());
   adminEditDoWhopForm.removeAttribute('hidden');
   doWhopDescriptionsRef.child(doWhopDescriptionKeyForUpdate).update({
     creatorDescription: creatorDescriptionUpdate.value.toLowerCase(),
@@ -279,10 +279,14 @@ function addToMyDoWhops(node) {
 
 // Adding funcationality to manage DoneWhop checkbox and status.
 
-document.getElementById('donewhop-checkbox').addEventListener('click', checkDoneWhop);
+document.getElementById('donewhop-checkbox').addEventListener('click', toggleDoneWhopStatus);
 
-function checkDoneWhop() {
+function toggleDoneWhopStatus() {
   document.getElementById('donewhop-checkbox').classList.contains('is-checked') === true
     ? document.getElementById('donewhop-checkbox').MaterialCheckbox.uncheck()
     : document.getElementById('donewhop-checkbox').MaterialCheckbox.check();
 }
+
+function returnDoneWhopStatus() {
+  return document.getElementById('donewhop-checkbox').classList.contains('is-checked') === true;
+}

From e47635cad57d45f6b752f44dc12fdac6fd5fc06e Mon Sep 17 00:00:00 2001
From: omarcodex <omaralimalik@gmail.com>
Date: Thu, 7 Sep 2017 17:25:21 -0700
Subject: [PATCH 5/7] Add saving of DoneWhop status as flag in
 DoWhopDescription in database

---
 scripts/dowhop-description-form-admin.js | 27 ++++++++++++++++++------
 1 file changed, 21 insertions(+), 6 deletions(-)

diff --git a/scripts/dowhop-description-form-admin.js b/scripts/dowhop-description-form-admin.js
index 824d9b9..7e21746 100644
--- a/scripts/dowhop-description-form-admin.js
+++ b/scripts/dowhop-description-form-admin.js
@@ -30,7 +30,7 @@ var submitNewDoWhopBtn = document.getElementById('create-new-dowhop');
 var emailSubmitBtn = document.getElementById('emailSubmit');
 
 submitNewDoWhopBtn.addEventListener('click', submitNewDoWhopEntry);
-emailSubmitBtn.addEventListener('click', updateCreatorDoerEmails);
+emailSubmitBtn.addEventListener('click', updateCreatorDoerEmailsDoneWhop);
 
 function submitNewDoWhopEntry(e) {
   e.preventDefault();
@@ -248,16 +248,31 @@ function revealEditEmailForm(node) {
   });
 }
 
-function updateCreatorDoerEmails(e) {
+function updateCreatorDoerEmailsDoneWhop(e) {
   e.preventDefault();
   console.log(returnDoneWhopStatus());
+  var doneWhopStatus = returnDoneWhopStatus(); // Checking if DoWhop has been marked as 'done' via true, false.
   adminEditDoWhopForm.removeAttribute('hidden');
-  doWhopDescriptionsRef.child(doWhopDescriptionKeyForUpdate).update({
-    creatorDescription: creatorDescriptionUpdate.value.toLowerCase(),
-    doerDescription: doerDescriptionUpdate.value.toLowerCase()
+  var doWhopRef = doWhopDescriptionsRef.child(doWhopDescriptionKeyForUpdate);
+
+  if (creatorDescriptionUpdate.value) {
+    doWhopRef.update({
+      creatorDescription: creatorDescriptionUpdate.value.toLowerCase()
+    });
+  }
+
+  if (doerDescriptionUpdate.value) {
+    doWhopRef.update({
+      doerDescription: doerDescriptionUpdate.value.toLowerCase()
+    });
+  }
+
+  doWhopRef.update({
+    isDoneWhop: doneWhopStatus
   });
+
   selectedForEdit.innerHTML = 'Edit your DoWhop';
-  error.innerHTML = 'Emails have been updated!';
+  error.innerHTML = 'Details have been updated!';
   adminEditDoWhopForm.reset();
 }
 

From da70246a79035b18bfa32734a57d15914e3bf7ac Mon Sep 17 00:00:00 2001
From: omarcodex <omaralimalik@gmail.com>
Date: Thu, 7 Sep 2017 17:26:53 -0700
Subject: [PATCH 6/7] Refactor to common name for DOM element

---
 scripts/dowhop-description-form-admin.js | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/scripts/dowhop-description-form-admin.js b/scripts/dowhop-description-form-admin.js
index 7e21746..4285516 100644
--- a/scripts/dowhop-description-form-admin.js
+++ b/scripts/dowhop-description-form-admin.js
@@ -294,14 +294,16 @@ function addToMyDoWhops(node) {
 
 // Adding funcationality to manage DoneWhop checkbox and status.
 
-document.getElementById('donewhop-checkbox').addEventListener('click', toggleDoneWhopStatus);
+var DoneWhopCheckbox = document.getElementById('donewhop-checkbox');
+
+DoneWhopCheckBox.addEventListener('click', toggleDoneWhopStatus);
 
 function toggleDoneWhopStatus() {
-  document.getElementById('donewhop-checkbox').classList.contains('is-checked') === true
-    ? document.getElementById('donewhop-checkbox').MaterialCheckbox.uncheck()
-    : document.getElementById('donewhop-checkbox').MaterialCheckbox.check();
+  DoneWhopCheckBox.classList.contains('is-checked') === true
+    ? DoneWhopCheckBox.MaterialCheckbox.uncheck()
+    : DoneWhopCheckBox.MaterialCheckbox.check();
 }
 
 function returnDoneWhopStatus() {
-  return document.getElementById('donewhop-checkbox').classList.contains('is-checked') === true;
+  return DoneWhopCheckBox.classList.contains('is-checked') === true;
 }

From fa055be46d27e0a808f02c694d5e86707f653a12 Mon Sep 17 00:00:00 2001
From: omarcodex <omaralimalik@gmail.com>
Date: Thu, 7 Sep 2017 17:31:30 -0700
Subject: [PATCH 7/7] Add check to show if DoWhop is already DoneWhop in
 checkbox upon form load

---
 scripts/dowhop-description-form-admin.js | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

diff --git a/scripts/dowhop-description-form-admin.js b/scripts/dowhop-description-form-admin.js
index 4285516..c0ee56f 100644
--- a/scripts/dowhop-description-form-admin.js
+++ b/scripts/dowhop-description-form-admin.js
@@ -239,12 +239,19 @@ function revealEditEmailForm(node) {
   var currentNodeTitle = '';
   doWhopDescriptionKeyForUpdate = node.id;
   var doWhopDescriptionRef = doWhopDescriptionsRef.child(doWhopDescriptionKeyForUpdate);
-  doWhopDescriptionRef.once('value', function(data) {
+  doWhopDescriptionRef.once('value').then(function(data) {
     var doWhopDescription = data.val();
     var currentNodeTitle = doWhopDescription.titleDescription;
+    var DoneWhopStatus = doWhopDescription.isDoneWhop === true;
+    console.log(DoneWhopStatus);
     selectedForEdit.innerHTML = 'Edit: ' + currentNodeTitle;
     creatorDescriptionUpdate.value = doWhopDescription.creatorDescription;
     doerDescriptionUpdate.value = doWhopDescription.doerDescription;
+
+    // Pre-marking the checkbox in form:
+    DoneWhopStatus === true
+      ? document.getElementById('donewhop-checkbox').MaterialCheckbox.check()
+      : document.getElementById('donewhop-checkbox').MaterialCheckbox.uncheck();
   });
 }
 
@@ -296,14 +303,14 @@ function addToMyDoWhops(node) {
 
 var DoneWhopCheckbox = document.getElementById('donewhop-checkbox');
 
-DoneWhopCheckBox.addEventListener('click', toggleDoneWhopStatus);
+document.getElementById('donewhop-checkbox').addEventListener('click', toggleDoneWhopStatus);
 
 function toggleDoneWhopStatus() {
-  DoneWhopCheckBox.classList.contains('is-checked') === true
-    ? DoneWhopCheckBox.MaterialCheckbox.uncheck()
-    : DoneWhopCheckBox.MaterialCheckbox.check();
+  document.getElementById('donewhop-checkbox').classList.contains('is-checked') === true
+    ? document.getElementById('donewhop-checkbox').MaterialCheckbox.uncheck()
+    : document.getElementById('donewhop-checkbox').MaterialCheckbox.check();
 }
 
 function returnDoneWhopStatus() {
-  return DoneWhopCheckBox.classList.contains('is-checked') === true;
+  return document.getElementById('donewhop-checkbox').classList.contains('is-checked') === true;
 }