From 05fa8bb45c966bd7649e08c8d3cf98fb21a06e15 Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Dec 24 2015 09:22:03 +0000
Subject: Rework the index endpoint to use two different endpoints if you're authenticated or not


---

diff --git a/pagure/templates/index_auth.html b/pagure/templates/index_auth.html
new file mode 100644
index 0000000..b376dd3
--- /dev/null
+++ b/pagure/templates/index_auth.html
@@ -0,0 +1,82 @@
+{% extends "master.html" %}
+
+{% block title %}Home{% endblock %}
+{% set tag = "projects" %}
+{% from "_browseheader.html" import browse_header %}
+
+
+{% block header %}
+  <link rel="stylesheet" type="text/css" media="screen"
+      href="{{ url_for('static', filename='toggle.css') }}"/>
+{% endblock %}
+
+
+{% from "_render_repo.html" import render_repos_as_card %}
+
+{% block content %}
+<div class="repo-header p-t-1">
+  <div class="container">
+    {{browse_header(select=tag)}}
+  </div>
+</div>
+<div class="p-t-2">
+  <div class="container">
+    <h2>
+      {{ username | avatar(36) | safe }} {{ username }}
+      {% if g.fas_user and g.fas_user.username == username %}
+        <a class="user_settings" href="{{ url_for('user_settings') }}"
+           title="Update your settings">
+          Settings
+        </a>
+      {% endif %}
+    </h2>
+
+    <div class="row">
+      <div class="col-md-8">
+        {{ render_repos_as_card(repos, repos_length, 'Projects') }}
+        {{ render_repos_as_card(forks, forks_length,'Forks',) }}
+      </div>
+      <div class="col-md-4">
+        <div class="card" id="group_list">
+            <div class="card-header">
+                Groups <span class="label label-default">{{ user.groups | length }}</span>
+            </div>
+            {% for group in user.groups %}
+              <div class="list-group">
+                <a class="list-group-item" href="{{ url_for(
+                    'view_group', group=group)}}">
+                  <div class=""><strong><span class="oi" data-glyph="people">
+                    </span> &nbsp;{{ group }}</strong>
+                  </div>
+                </a>
+              </div>
+            {% else %}
+            <p>No group found</p>
+            {% endfor %}
+        </div>
+      </div>
+    </div>
+    {% if config.get('DATAGREPPER_URL') %}
+    <section class="project_list" id="user_graph">
+        <header>
+            <h2>Contributions</h2>
+        </header>
+        <a href="{{config.get('DATAGREPPER_URL')}}/raw?category={{config.get('DATAGREPPER_CATEGORY', 'pagure')}}&user={{username}}" target="_blank">
+          <img src="{{config.get('DATAGREPPER_URL')}}/charts/line?category={{config.get('DATAGREPPER_CATEGORY', 'pagure')}}&user={{username}}&split_on=users&delta=31536000&N=12&title=&width=600&height=200&style=light&fill=True" alt="User activity graph"/>
+        </a>
+    </section>
+    {% endif %}
+  </div>
+</div>
+{% endblock %}
+
+{% block jscripts %}
+    {{ super() }}
+    <script type="text/javascript">
+      $(function(){
+        $('.show_parts input[type="checkbox"]').change(function(){
+          $('#' + $(this).attr('name')).toggle();
+        });
+      });
+    </script>
+{% endblock %}
diff --git a/pagure/ui/app.py b/pagure/ui/app.py
index 1420b18..15c1e1f 100644
--- a/pagure/ui/app.py
+++ b/pagure/ui/app.py
@@ -54,77 +54,79 @@ def index():
 
     total_page = int(ceil(num_repos / float(limit)))
 
-    repopage = None
-    forkpage = None
-    user_repos = None
-    user_forks = None
-    total_page_repos = None
-    total_page_forks = None
-    username = None
-    user_repos_length = None
-    user_forks_length = None
-
     if authenticated():
-        username = flask.g.fas_user.username
+        return index_auth()
 
-        repopage = flask.request.args.get('repopage', 1)
-        try:
-            repopage = int(repopage)
-        except ValueError:
-            repopage = 1
+    return flask.render_template(
+        'index.html',
+        select="projects",
+        repos=repos,
+        repos_length=num_repos,
+    )
 
-        forkpage = flask.request.args.get('forkpage', 1)
-        try:
-            forkpage = int(forkpage)
-        except ValueError:
-            forkpage = 1
-
-        repo_start = limit * (repopage - 1)
-        fork_start = limit * (forkpage - 1)
-
-        user_repos = pagure.lib.search_projects(
-            SESSION,
-            username=username,
-            fork=False,
-            start=repo_start,
-            limit=limit)
-        user_repos_length = pagure.lib.search_projects(
-            SESSION,
-            username=username,
-            fork=False,
-            count=True)
-
-        user_forks = pagure.lib.search_projects(
-            SESSION,
-            username=username,
-            fork=True,
-            start=fork_start,
-            limit=limit)
-        user_forks_length = pagure.lib.search_projects(
-            SESSION,
-            username=username,
-            fork=True,
-            count=True)
-
-        total_page_repos = int(ceil(user_repos_length / float(limit)))
-        total_page_forks = int(ceil(user_forks_length / float(limit)))
+
+def index_auth():
+    """ Front page for authenticated user.
+    """
+    user = pagure.lib.search_user(SESSION, username=flask.g.fas_user.username)
+    if not user:
+        flask.abort(404, 'No user `%s` found, re-login maybe?' % username)
+
+    repopage = flask.request.args.get('repopage', 1)
+    try:
+        repopage = int(repopage)
+    except ValueError:
+        repopage = 1
+
+    forkpage = flask.request.args.get('forkpage', 1)
+    try:
+        forkpage = int(forkpage)
+    except ValueError:
+        forkpage = 1
+
+    limit = APP.config['ITEM_PER_PAGE']
+    repo_start = limit * (repopage - 1)
+    fork_start = limit * (forkpage - 1)
+
+    repos = pagure.lib.search_projects(
+        SESSION,
+        username=flask.g.fas_user.username,
+        fork=False,
+        start=repo_start,
+        limit=limit)
+    repos_length = pagure.lib.search_projects(
+        SESSION,
+        username=flask.g.fas_user.username,
+        fork=False,
+        count=True)
+
+    forks = pagure.lib.search_projects(
+        SESSION,
+        username=flask.g.fas_user.username,
+        fork=True,
+        start=fork_start,
+        limit=limit)
+    forks_length = pagure.lib.search_projects(
+        SESSION,
+        username=flask.g.fas_user.username,
+        fork=True,
+        count=True)
+
+    total_page_repos = int(ceil(repos_length / float(limit)))
+    total_page_forks = int(ceil(forks_length / float(limit)))
 
     return flask.render_template(
-        'index.html',
+        'index_auth.html',
+        username=flask.g.fas_user.username,
+        user=user,
+        forks=forks,
         repos=repos,
-        total_page=total_page,
-        page=page,
-        username=username,
-        select="projects",
         repopage=repopage,
         forkpage=forkpage,
-        user_repos=user_repos,
-        user_forks=user_forks,
         total_page_repos=total_page_repos,
         total_page_forks=total_page_forks,
-        user_repos_length=user_repos_length,
-        user_forks_length=user_forks_length,
-        repos_length=num_repos,
+        repos_length=repos_length,
+        forks_length=forks_length,
     )