feat: can delete your ideas on other's lists

context/context.go

@@ -78,6 +78,16 @@ func (ctx *Context) UpdateForeignWishlist(w http.ResponseWriter, r *http.Request
 			http.Error(w, "Failed to add gift idea to other user...", http.StatusInternalServerError)
 			return
 		}
+	case "delete":
+		claims := r.Form["unclaimed"]
+		unclaims := r.Form["claimed"]
+		gifts := append(claims, unclaims...)
+		err := user.RemoveGifts(gifts...)
+		if err != nil {
+			log.Printf("Failed to remove gift idea for other user: %s\n", err)
+			http.Error(w, "Failed to remove gift idea for other user...", http.StatusInternalServerError)
+			return
+		}
 	default:
 		http.Error(w, "Invalid intent", http.StatusBadRequest)
 	}

templates/base.gotmpl

@@ -28,6 +28,12 @@
       const accepted = submissionNames.length > 0 && submissionNames.every((name) => acceptedNames.includes(name));
       submitter.disabled = !accepted;
     }
+    function acceptAttribute(form, submitId, acceptedAttribute) {
+      const checkedInputs = Array.from(form.querySelectorAll("input")).filter((i) => i.checked);
+      const submitter = document.getElementById(submitId);
+      const accepted = checkedInputs.length > 0 && checkedInputs.every((i) => i.hasAttribute(acceptedAttribute));
+      submitter.disabled = !accepted;
+    }
   </script>
 </head>
 

templates/foreign_wishlist.gotmpl

@@ -33,26 +33,19 @@
         <h2>{{.Username}}'s list</h2>
         {{with .Gifts}}
         <form method="post" action="/{{$.UserReference}}/update" autocomplete="off"
-          onchange="acceptNames(this, 'claimSubmit', 'claimed', 'unclaimed'); acceptNames(this, 'completeSubmit', 'claimed');">
-          <button id="claimSubmit" class="btn btn-warning" type="submit" name="intent" value="claim"
-            disabled>Claim/Unclaim</button>
-          <button id="completeSubmit" class="btn btn-success" type="submit" name="intent" value="complete"
-            disabled>Complete</button>
-          <ul class="list-group mt-3">
+          onchange="acceptNames(this, 'claimSubmit', 'claimed', 'unclaimed'); acceptNames(this, 'completeSubmit', 'claimed'); acceptAttribute(this, 'deleteSubmit', 'data-deletable')">
+          <ul class="list-group mb-3">
             {{range .}}
+            {{$isMine := eq .ClaimantId $.CurrentUserId}}
+            {{$createdByMe := eq .CreatorId $.CurrentUserId}}
+            {{$outsideIdea := ne .RecipientId .CreatorId}}
             <li class="list-group-item{{if .Sent}} list-group-item-light{{end}}">
-              {{if .ClaimantId}}
-              {{if eq .ClaimantId $.CurrentUserId}}
+
               <input id="foreignlist_select_{{.Id}}" class="form-check-input" type="checkbox"
-                aria-describedby="wish_detail_{{.Id}}" name="claimed" value="{{.Id}}" {{if .Sent}}disabled{{end}}>
-              {{else}}
-              <input id="foreignlist_select_{{.Id}}" class="form-check-input" type="checkbox"
-                aria-describedby="wish_detail_{{.Id}}" disabled>
-              {{end}}
-              {{else}}
-              <input id="foreignlist_select_{{.Id}}" class="form-check-input" type="checkbox"
-                aria-describedby="wish_detail_{{.Id}}" name="unclaimed" value="{{.Id}}" {{if .Sent}}disabled{{end}}>
-              {{end}}
+                aria-describedby="wish_detail_{{.Id}}" {{if $isMine}} name="claimed" value="{{.Id}}" {{else if
+                .ClaimantId}} disabled {{else}} name="unclaimed" value="{{.Id}}" {{end}} {{if .Sent}} disabled {{end}}
+                {{if $createdByMe}}data-deletable{{end}}>
+
               <label class="form-check-label stretched-link" for="foreignlist_select_{{.Id}}">
                 {{if .Sent}}
                 <s>{{.Name}}</s>
@@ -60,29 +53,30 @@
                 {{.Name}}
                 {{end}}
               </label>
-              {{if or .ClaimantId (ne .RecipientId .CreatorId)}}
+
+              {{if or .ClaimantId $outsideIdea}}
               <div class="d-inline" id="wish_detail_{{.Id}}">
                 {{if .ClaimantId}}
-                {{if eq .ClaimantId $.CurrentUserId}}
-                <span style="color: blue;">{{if .Sent}}Completed{{else}}Claimed{{end}} by
-                  <em>you</em></span>
-                {{else}}
-                <span style="color: red;">{{if .Sent}}Completed{{else}}Claimed{{end}} by
-                  {{.ClaimantName}}</span>
-                {{end}}
-                {{end}}
-                {{if ne .RecipientId .CreatorId}}
-                {{if eq .CreatorId $.CurrentUserId}}
-                <span style="color: green;">Added by <em>you</em></span>
-                {{else}}
-                <span style="color: green;">Added by {{.CreatorName}}</span>
+                <span style="color: {{if $isMine}}blue{{else}}red{{end}};">{{if .Sent}}Completed{{else}}Claimed{{end}}
+                  by
+                  {{if $isMine}}<em>you</em>{{else}}{{.ClaimantName}}{{end}}</span>
                 {{end}}
+
+                {{if $outsideIdea}}
+                <span style="color: green;">Added by {{if
+                  $createdByMe}}<em>you</em>{{else}}{{.CreatorName}}{{end}}</span>
                 {{end}}
               </div>
               {{end}}
             </li>
             {{end}}
           </ul>
+          <button id="claimSubmit" class="btn btn-warning" type="submit" name="intent" value="claim"
+            disabled>Claim/Unclaim</button>
+          <button id="completeSubmit" class="btn btn-success" type="submit" name="intent" value="complete"
+            disabled>Complete</button>
+          <button id="deleteSubmit" class="btn btn-danger" type="submit" name="intent" value="delete"
+            disabled>Delete</button>
         </form>
         {{else}}
         <p>They don't have any gift ideas. Ask them to think of something, or add an idea yourself! 👇 (everyone

templates/home.gotmpl

@@ -30,8 +30,6 @@
         {{with .Gifts}}
         <form method="post" action="/wishlist/delete" onchange="acceptNames(this, 'deleteSubmit', 'gift')"
           autocomplete="off">
-          <button id="deleteSubmit" class="btn btn-danger mb-3" type="submit" name="mode" value="delete"
-            disabled>Delete</button>
           <ul class="list-group mb-3">
             {{range .}}
             <li class="list-group-item">
@@ -42,6 +40,8 @@
             </li>
             {{end}}
           </ul>
+          <button id="deleteSubmit" class="btn btn-danger mb-3" type="submit" name="mode" value="delete"
+            disabled>Delete</button>
         </form>
         {{else}}
         <p>Your list is empty. Think of some things to add!</p>
@@ -62,11 +62,7 @@
         <form method="post" action="/todo/update"
           onchange="acceptNames(this, 'unclaimSubmit', 'gift'); acceptNames(this, 'completeSubmit', 'gift')"
           autocomplete="off">
-          <button id="unclaimSubmit" class="btn btn-warning" type="submit" name="mode" value="unclaim"
-            disabled>Unclaim</button>
-          <button id="completeSubmit" class="btn btn-success" type="submit" name="mode" value="complete"
-            disabled>Complete</button>
-          <ul class="list-group mt-3">
+          <ul class="list-group mb-3">
             {{range .}}
             <li class="list-group-item{{if .Sent}} list-group-item-light{{end}}">
               <input id="todo_select_{{.Id}}" class="form-check-input" type="checkbox" {{if .Sent}}
@@ -86,10 +82,14 @@
             </li>
             {{end}}
           </ul>
+          <button id="unclaimSubmit" class="btn btn-warning" type="submit" name="mode" value="unclaim"
+            disabled>Unclaim</button>
+          <button id="completeSubmit" class="btn btn-success" type="submit" name="mode" value="complete"
+            disabled>Complete</button>
         </form>
       </div>
     </section>
     {{end}}
   </div>
 </div>
-{{end}}
+{{end}}