From 1e781fb133e95a8ac3523797081bcae6b21f10fe Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Feb 23 2016 09:35:35 +0000
Subject: Put the sorting as an extra argument rather than in the url


and remove debugging code

---

diff --git a/pagure/templates/_render_repo.html b/pagure/templates/_render_repo.html
index 7d73f22..b325d0d 100644
--- a/pagure/templates/_render_repo.html
+++ b/pagure/templates/_render_repo.html
@@ -81,13 +81,15 @@
                 Sort
               </button>
               <div class="dropdown-menu">
-                <a class="dropdown-item" href="{{ url_for('browse_projects') }}">
+                <a class="dropdown-item" href="{{
+                    url_for('browse_projects') }}">
                   {% if sorting == 'name'%}
                   <span class="oi" data-glyph="check"></span>
                   {%endif%}
                   Name
                 </a>
-                <a class="dropdown-item" href="{{ url_for('browse_projects') }}latest">
+                <a class="dropdown-item" href="{{
+                    url_for('browse_projects', sorting='latest') }}">
                   {% if sorting == 'latest'%}
                   <span class="oi" data-glyph="check"></span>
                   {%endif%}
diff --git a/pagure/ui/app.py b/pagure/ui/app.py
index d23ae7d..c8f8153 100644
--- a/pagure/ui/app.py
+++ b/pagure/ui/app.py
@@ -29,11 +29,11 @@ from pagure import (APP, SESSION, login_required,
 
 @APP.route('/browse/projects', endpoint='browse_projects')
 @APP.route('/browse/projects/', endpoint='browse_projects')
-@APP.route('/browse/projects/<sorting>')
 @APP.route('/')
-def index(sorting=None):
+def index():
     """ Front page of the application.
     """
+    sorting = flask.request.args.get('sorting') or None
     page = flask.request.args.get('page', 1)
     try:
         page = int(page)
@@ -64,8 +64,6 @@ def index(sorting=None):
         SESSION,
         fork=False,
         count=True)
-    print "app.py"
-    print sorting
     total_page = int(ceil(num_repos / float(limit)))
 
     if authenticated() and flask.request.path == '/':