From e13b7a36504cacefe0cbb34a225708781cd5a640 Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Nov 02 2017 09:41:39 +0000
Subject: Move the forks into the stats tab


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

---

diff --git a/pagure/templates/repo_master.html b/pagure/templates/repo_master.html
index 27d3f27..13c21fb 100644
--- a/pagure/templates/repo_master.html
+++ b/pagure/templates/repo_master.html
@@ -301,25 +301,6 @@
           </a>
         </li>
 
-        {% if repo and repo.forks %}
-        <li class="nav-item">
-          <a {%
-            if select == 'forks' %}class="active nav-link" {%
-            else %}class="nav-link" {%
-            endif %}href="{{ url_for(
-                '.view_forks',
-                repo=repo.name,
-                username=username,
-                namespace=repo.namespace) }}">
-              <span class="oi hidden-md-up" data-glyph="fork"></span>
-              <span class="hidden-sm-down">Forks&nbsp;</span>
-              <span class="hidden-sm-down label label-default label-pill">
-                {{ repo.forks |length }}
-              </span>
-            </a>
-        </li>
-        {% endif %}
-
       {% if authenticated %}
         {% if g.repo_admin %}
           <li class="nav-item pull-xs-right">
diff --git a/pagure/templates/repo_stats.html b/pagure/templates/repo_stats.html
index a9c01c5..e18f9b2 100644
--- a/pagure/templates/repo_stats.html
+++ b/pagure/templates/repo_stats.html
@@ -22,6 +22,12 @@
         </a>
       </div>
       <div class="list-group">
+        <a class="list-group-item stats_btn" name="forks" href="#forks"
+          title="Forks of the project">
+          Forks
+          <span class="hidden-sm-down label label-default label-pill">
+            {{ repo.forks |length }}
+          </span>
         </a>
       </div>
       <div class="list-group">
@@ -36,6 +42,43 @@
         <span id="data_stats_spinner" style="display:none;"></span>
         <span id="data_stats" style="display:none;"></span>
       </section>
+      <section class="forks_list" id="forks_list">
+        <h2>Forks list</h2>
+        {% if repo.forks %}
+        <div class="list-group">
+          {% for fork in repo.forks %}
+            <div class="list-group-item">
+              <h5>
+                <strong><a href="{{ url_for('view_repo',
+                      repo=fork.name,
+                      username=fork.user.user,
+                      namespace=repo.namespace) }}">
+                    <span class="oi text-muted" data-glyph="fork"></span>
+                    {{ fork.user.user }}/{{ fork.name }}
+                </a></strong>
+                <div class="pull-xs-right">
+                  <span>
+                    <a href="{{ url_for('view_user', username=fork.user.user)}}">
+                      {{ fork.user.default_email | avatar(20) | safe }}
+                      {{ fork.user.fullname }}
+                    </a> forked this project
+                  </span>
+                  <span class="commitdate" data-toggle="tooltip" title="{{
+                      fork.date_created.strftime('%Y-%m-%d %H:%M:%S') }}">
+                      <strong>{{ fork.date_created|humanize }}</strong>
+                  </span>
+                </div>
+              </h5>
+            </div>
+
+          {% endfor %}
+        </div>
+        {% else %}
+        <p>
+          This project has not been forked.
+        </p>
+        {% endif %}
+      </section>
     </div>
   </div>
 {% endblock %}
@@ -112,23 +155,27 @@ commits_history_call = function() {
   commits_history(_stats_url, data);
 };
 
+toggle_forks = function() {
+  $("#forks_list").show();
+}
 
 
 $(document).ready(function() {
   $('.stats_btn').click(function(ev){
     var _b = $("#data_stats");
     _b.hide();
+    $("#forks_list").hide();
     if ($(this).attr('name') == 'issues') {
       issues_history_stats_plot_call();
     } else if ($(this).attr('name') == 'authors') {
       commits_authors_call();
     } else if ($(this).attr('name') == 'commits') {
       commits_history_call();
+    } else if ($(this).attr('name') == 'forks') {
+      toggle_forks();
     }
   });
 
-  commits_history_call();
-
 });
 </script>
 {% endblock %}
diff --git a/pagure/ui/repo.py b/pagure/ui/repo.py
index 9b4b514..6a0168f 100644
--- a/pagure/ui/repo.py
+++ b/pagure/ui/repo.py
@@ -898,27 +898,6 @@ def view_tree(repo, identifier=None, username=None, namespace=None):
     )
 
 
-@APP.route('/<repo>/forks/')
-@APP.route('/<repo>/forks')
-@APP.route('/<namespace>/<repo>/forks/')
-@APP.route('/<namespace>/<repo>/forks')
-@APP.route('/fork/<username>/<repo>/forks/')
-@APP.route('/fork/<username>/<repo>/forks')
-@APP.route('/fork/<username>/<namespace>/<repo>/forks/')
-@APP.route('/fork/<username>/<namespace>/<repo>/forks')
-def view_forks(repo, username=None, namespace=None):
-    """ Presents all the forks of the project.
-    """
-    repo = flask.g.repo
-
-    return flask.render_template(
-        'forks.html',
-        select='forks',
-        username=username,
-        repo=repo,
-    )
-
-
 @APP.route('/<repo>/releases/')
 @APP.route('/<repo>/releases')
 @APP.route('/<namespace>/<repo>/releases/')