From 7ce286124a0d76d53f17a1b8a4a35c06d3f946a9 Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Aug 29 2018 13:09:39 +0000
Subject: Add a button to show the inactive milestones


It can be useful to turn them back on or just to check if they exists
or not.

Fixes https://pagure.io/pagure/issue/3587

Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>

---

diff --git a/pagure/templates/settings.html b/pagure/templates/settings.html
index 77c3e69..359dd7a 100644
--- a/pagure/templates/settings.html
+++ b/pagure/templates/settings.html
@@ -1389,6 +1389,13 @@ $('.extend-form').click(function(e) {
       var _a = form.find('input[name=active_milestone_' + idx + ']');
       $(_a).attr('name', 'active_milestone_' + (idx + 1 ));
       $(_a).prop('checked', true);
+  } else if (tgt == '#milestones_show'){
+    var _el = $('.milestone_inactive')
+    if (_el.css('display') == 'none'){
+      _el.css('display', 'flex');
+    } else {
+     _el.hide();
+    }
   }
   $(tgt).append(form);
 });
diff --git a/pagure/templates/settings_milestones.html b/pagure/templates/settings_milestones.html
index af225dc..3c995bb 100644
--- a/pagure/templates/settings_milestones.html
+++ b/pagure/templates/settings_milestones.html
@@ -51,19 +51,25 @@
                 </div>
                 <div class="col-sm-1 p-r-0" >
                     <input type="checkbox" name="active_milestone_{{ loop.index
-                    }}"{% if not milestone or repo.milestones[milestone]['active']
+                    }}"{% if milestone and repo.milestones[milestone]['active']
                     %} checked{% endif %} />
                 </div>
               </div>
           {% endfor %}
           </div>
           <div class="row p-t-1">
-            <div class="col-sm-12">
+            <div class="col-sm-6">
               <a href="javascript:void(0)" class="btn btn-outline-primary btn-sm btn-block extend-form mt-1"
                 data-target="#milestones">
                   <i class="fa fa-plus"></i> Add new milestone
               </a>
             </div>
+            <div class="col-sm-6">
+              <a href="javascript:void(0)" class="btn btn-outline-primary btn-sm btn-block extend-form mt-1"
+                data-target="#milestones_show">
+                  <i class="fa fa-repeat"></i> Show all milestones
+              </a>
+            </div>
           </div>
           <div class="row p-t-1">
             <div class="col-sm-12">
diff --git a/tests/test_pagure_flask_ui_repo.py b/tests/test_pagure_flask_ui_repo.py
index 4c16481..e241b63 100644
--- a/tests/test_pagure_flask_ui_repo.py
+++ b/tests/test_pagure_flask_ui_repo.py
@@ -1449,7 +1449,7 @@ class PagureFlaskRepotests(tests.Modeltests):
                         data-stone="1"></span>
                 </div>
                 <div class="col-sm-1 p-r-0" >
-                    <input type="checkbox" name="active_milestone_1" checked />
+                    <input type="checkbox" name="active_milestone_1" />
                 </div>
               </div>''', output_text)
 
diff --git a/tests/test_pagure_flask_ui_repo_milestones.py b/tests/test_pagure_flask_ui_repo_milestones.py
index 5de5684..d587d42 100644
--- a/tests/test_pagure_flask_ui_repo_milestones.py
+++ b/tests/test_pagure_flask_ui_repo_milestones.py
@@ -84,7 +84,7 @@ class PagureFlaskRepoMilestonestests(tests.Modeltests):
                         data-stone="1"></span>
                 </div>
                 <div class="col-sm-1 p-r-0" >
-                    <input type="checkbox" name="active_milestone_1" checked />
+                    <input type="checkbox" name="active_milestone_1" />
                 </div>
               </div>''', output.get_data(as_text=True))