From e4096460ebf90812007244a9011602070c88ce85 Mon Sep 17 00:00:00 2001
From: evazion <noizave@gmail.com>
Date: Thu, 12 Apr 2018 21:48:09 -0500
Subject: [PATCH] Fix #3622: Approval links on mod queue have same ID
 attribute.

---
 app/assets/javascripts/posts.js                     | 2 +-
 app/views/moderator/post/queues/_quick_mod.html.erb | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/app/assets/javascripts/posts.js b/app/assets/javascripts/posts.js
index 6aba8f69c..1485e1626 100644
--- a/app/assets/javascripts/posts.js
+++ b/app/assets/javascripts/posts.js
@@ -223,7 +223,7 @@
 
       if (Danbooru.meta("current-user-can-approve-posts") === "true") {
         Danbooru.keydown("shift+o", "approve", function(e) {
-          $("#quick-mod-approve").click();
+          $(".approve-link").click();
         });
       }
 
diff --git a/app/views/moderator/post/queues/_quick_mod.html.erb b/app/views/moderator/post/queues/_quick_mod.html.erb
index f0a043f13..246ba4e95 100644
--- a/app/views/moderator/post/queues/_quick_mod.html.erb
+++ b/app/views/moderator/post/queues/_quick_mod.html.erb
@@ -1,7 +1,7 @@
 <div class="quick-mod">
-  <%= link_to_if post.is_approvable?, "Approve", moderator_post_approval_path(post_id: post.id), id: "quick-mod-approve", method: :post, remote: true, class: "btn" %> |
-  <%= link_to "Breaks Rules", moderator_post_disapproval_path(post_disapproval: { post_id: post.id, reason: "breaks_rules" }), method: :post, remote: true, class: "btn" %> |
-  <%= link_to "Poor Quality", moderator_post_disapproval_path(post_disapproval: { post_id: post.id, reason: "poor_quality" }), method: :post, remote: true, class: "btn" %> |
-  <%= link_to "No Interest",  moderator_post_disapproval_path(post_disapproval: { post_id: post.id, reason: "disinterest" }), method: :post, remote: true, class: "btn" %> |
+  <%= link_to_if post.is_approvable?, "Approve", moderator_post_approval_path(post_id: post.id), method: :post, remote: true, class: "approve-link btn" %> |
+  <%= link_to "Breaks Rules", moderator_post_disapproval_path(post_disapproval: { post_id: post.id, reason: "breaks_rules" }), method: :post, remote: true, class: "disapprove-link btn" %> |
+  <%= link_to "Poor Quality", moderator_post_disapproval_path(post_disapproval: { post_id: post.id, reason: "poor_quality" }), method: :post, remote: true, class: "disapprove-link btn" %> |
+  <%= link_to "No Interest",  moderator_post_disapproval_path(post_disapproval: { post_id: post.id, reason: "disinterest" }), method: :post, remote: true, class: "disapprove-link btn" %> |
   <%= link_to "Detailed Rejection", "#", "data-post-id" => post.id, class: "detailed-rejection-link btn" %>
 </div>