From 74bb2a1c4a35ede73a8d41096305a3a0d830dd47 Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Dec 07 2015 16:39:15 +0000
Subject: Make the SHOW_PROJECTS_INDEX list what should be showing in the index page


Until now despite its name, we were using this configuration key as a way
to hide sections from the index page.
With this change, we actually use it to list what should show on the index
page. This makes it easier to write the proper if statements to show the
sections and the corresponding switch buttons.

---

diff --git a/pagure/templates/_render_repo.html b/pagure/templates/_render_repo.html
index eb74a3a..f45cb70 100644
--- a/pagure/templates/_render_repo.html
+++ b/pagure/templates/_render_repo.html
@@ -64,7 +64,7 @@
 {% macro render_repos(
         list, total, pagetitle, page, title, count, id, username=None, hide=True) %}
     <section class="project_list" id="{{ id }}"{%
-        if g.fas_user and id in config.get('SHOW_PROJECTS_INDEX', []) and hide
+        if g.fas_user and id not in config.get('SHOW_PROJECTS_INDEX', []) and hide
         %} style="display:none;" {% endif %}>
         <header>
             {% set title_lvl = 3 if username else 2 %}
@@ -89,34 +89,37 @@
         <strong>
             {% if all %}My repos:{% else %}Repos:{% endif %}
         </strong>
-
-            <label class="switch">
-                <input type="checkbox" class="switch-input"
-                    name="{% if all %}my{% endif %}repos" {%
-                    if 'myrepos' not in config.get('SHOW_PROJECTS_INDEX', []) and not hide
-                    %} checked {% endif %}/>
-                <span class="switch-label" data-on="On" data-off="Off"></span>
-                <span class="switch-handle"></span>
-            </label>
+        <label class="switch">
+            <input type="checkbox" class="switch-input"
+                name="{% if all %}my{% endif %}repos"
+                {%- if (
+                    (all and 'myrepos' in config.get('SHOW_PROJECTS_INDEX', []))
+                    or
+                    (not all and 'repos' not in config.get('SHOW_PROJECTS_INDEX', []))
+                  )
+                  or not hide %} checked {% endif %}/>
+            <span class="switch-label" data-on="On" data-off="Off"></span>
+            <span class="switch-handle"></span>
+        </label>
         <strong>Forks:</strong>
-            <label class="switch">
-                <input type="checkbox" class="switch-input"
-                    name="{% if all %}my{% endif %}forks" {%
-                    if 'myforks' not in config.get('SHOW_PROJECTS_INDEX', []) and not hide
-                    %} checked {% endif %}/>
-                <span class="switch-label" data-on="On" data-off="Off"></span>
-                <span class="switch-handle"></span>
-            </label>
+        <label class="switch">
+            <input type="checkbox" class="switch-input"
+                name="{% if all %}my{% endif %}forks" {%
+                if 'myforks' in config.get('SHOW_PROJECTS_INDEX', []) or not hide
+                %} checked {% endif %}/>
+            <span class="switch-label" data-on="On" data-off="Off"></span>
+            <span class="switch-handle"></span>
+        </label>
         {% if all %}
         <strong>All repos:</strong>
-            <label class="switch">
-                <input type="checkbox" class="switch-input"
-                    name="repos" id="allrepos" {%
-                    if 'repos' not in config.get('SHOW_PROJECTS_INDEX', []) and not hide
-                    %} checked {% endif %}/>
-                <span class="switch-label" data-on="On" data-off="Off"></span>
-                <span class="switch-handle"></span>
-            </label>
+        <label class="switch">
+            <input type="checkbox" class="switch-input"
+                name="repos" id="allrepos" {%
+                if 'repos' in config.get('SHOW_PROJECTS_INDEX', []) or not hide
+                %} checked {% endif %}/>
+            <span class="switch-label" data-on="On" data-off="Off"></span>
+            <span class="switch-handle"></span>
+        </label>
         {% endif %}
         </div>
     </aside>