From c86fa268c29a83fcd9a47eaf66f73d603b636458 Mon Sep 17 00:00:00 2001
From: Jonathan Rehm <jonathan@rehm.me>
Date: Fri, 28 Jul 2017 20:02:53 -0700
Subject: [PATCH] Fix/remove duplicate id's

---
 cps/templates/detail.html      | 10 +++++-----
 cps/templates/index.html       |  4 ++--
 cps/templates/search_form.html |  6 +++---
 3 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/cps/templates/detail.html b/cps/templates/detail.html
index 2859d64c..500d47f1 100644
--- a/cps/templates/detail.html
+++ b/cps/templates/detail.html
@@ -17,7 +17,7 @@
           {% if g.user.role_download() %}
             <div class="btn-group" role="group">
             {% if entry.data|length < 3 %}
-              <button id="btnGroupDrop1" type="button" class="btn btn-primary">
+              <button type="button" class="btn btn-primary">
                 {{_('Download')}} :
               </button>
                 {% for format in entry.data %}
@@ -44,11 +44,11 @@
             {% endif %}
           {% if (g.user.role_download() and g.user.is_anonymous()) or g.user.is_authenticated %}
               <div class="btn-group" role="group">
-                <button id="btnGroupDrop2" type="button" class="btn btn-primary dropdown-toggle" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
+                <button id="read-in-browser" type="button" class="btn btn-primary dropdown-toggle" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
                   <span class="glyphicon glyphicon-eye-open"></span> {{_('Read in browser')}}
                   <span class="caret"></span>
                 </button>
-                <ul class="dropdown-menu" aria-labelledby="btnGroupDrop2">
+                <ul class="dropdown-menu" aria-labelledby="read-in-browser">
                   {% for format in entry.data %}
                   {%if format.format|lower == 'epub' or format.format|lower == 'txt' or format.format|lower == 'pdf' or format.format|lower == 'cbr' or format.format|lower == 'cbt' or format.format|lower == 'cbz' %}
                   <li><a target="_blank" href="{{ url_for('read_book', book_id=entry.id, book_format=format.format|lower) }}">{{format.format}}</a></li>
@@ -184,11 +184,11 @@
       {% if g.user.shelf.all() or g.public_shelfes %}
       <div id="shelf-actions" class="btn-toolbar" role="toolbar">
         <div class="btn-group" role="group" aria-label="Add to shelves">
-          <button id="btnGroupDrop2" type="button" class="btn btn-primary btn-sm dropdown-toggle" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
+          <button id="add-to-shelf" type="button" class="btn btn-primary btn-sm dropdown-toggle" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
             <span class="glyphicon glyphicon-list"></span> {{_('Add to shelf')}}
             <span class="caret"></span>
           </button>
-          <ul id="add-to-shelves" class="dropdown-menu" aria-labelledby="btnGroupDrop2">
+          <ul id="add-to-shelves" class="dropdown-menu" aria-labelledby="add-to-shelf">
             {% for shelf in g.user.shelf %}
               {% if not shelf.id in books_shelfs and shelf.is_public != 1 %}
                 <li>
diff --git a/cps/templates/index.html b/cps/templates/index.html
index c5d156ce..9fb9291f 100755
--- a/cps/templates/index.html
+++ b/cps/templates/index.html
@@ -6,7 +6,7 @@
   <div class="row">
 
     {% for entry in random %}
-    <div id="books_rand" class="col-sm-3 col-lg-2 col-xs-6 book">
+    <div class="col-sm-3 col-lg-2 col-xs-6 book">
       <div class="cover">
           <a href="{{ url_for('show_book', book_id=entry.id) }}" data-toggle="modal" data-target="#bookDetailsModal" data-remote="false">
             {% if entry.has_cover %}
@@ -42,7 +42,7 @@
   <div class="row">
     {% if entries[0] %}
     {% for entry in entries %}
-    <div id="books" class="col-sm-3 col-lg-2 col-xs-6 book">
+    <div class="col-sm-3 col-lg-2 col-xs-6 book">
       <div class="cover">
           <a href="{{ url_for('show_book', book_id=entry.id) }}" data-toggle="modal" data-target="#bookDetailsModal" data-remote="false">
             {% if entry.has_cover %}
diff --git a/cps/templates/search_form.html b/cps/templates/search_form.html
index 6dbd17a8..d8c3a07d 100644
--- a/cps/templates/search_form.html
+++ b/cps/templates/search_form.html
@@ -28,7 +28,7 @@
     <div class="form-group">
       <div class="btn-toolbar btn-toolbar-lg" data-toggle="buttons">
         {% for tag in tags %}
-          <label id="tag_{{tag.id}}" class="btn btn-danger tags_click">
+          <label id="exclude_tag_{{tag.id}}" class="btn btn-danger tags_click">
             <input type="checkbox" autocomplete="off" name="exclude_tag" id="exclude_tag" value="{{tag.id}}">{{tag.name}}</input>
           </label>
         {% endfor %}
@@ -48,7 +48,7 @@
     <div class="form-group">
       <div class="btn-toolbar btn-toolbar-lg" data-toggle="buttons">
         {% for serie in series %}
-          <label id="serie_{{serie.id}}" class="btn btn-danger serie_click">
+          <label id="exclude_serie_{{serie.id}}" class="btn btn-danger serie_click">
             <input type="checkbox" autocomplete="off" name="exclude_serie" id="exclude_serie" value="{{serie.id}}">{{serie.name}}</input>
           </label>
         {% endfor %}
@@ -69,7 +69,7 @@
       <div class="form-group">
         <div class="btn-toolbar btn-toolbar-lg" data-toggle="buttons">
           {% for language in languages %}
-            <label id="language_{{language.id}}" class="btn btn-danger language_click">
+            <label id="exclude_language_{{language.id}}" class="btn btn-danger language_click">
               <input type="checkbox" autocomplete="off" name="exclude_language" id="exclude_language" value="{{language.id}}">{{language.name}}</input>
             </label>
           {% endfor %}