From cf46bf86022ff5bfcb9aba281d64a21e9244cf5a Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Jan 09 2018 16:58:50 +0000
Subject: [PATCH 1/2] Add namespace to the url_for() calls


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

---

diff --git a/template/commit.html b/template/commit.html
index 8762a4f..42d2a56 100644
--- a/template/commit.html
+++ b/template/commit.html
@@ -13,23 +13,23 @@
     <span>{{ split_message[0] }}</span>
     <div class="pull-xs-right btn-group">
       <a class="btn btn-secondary btn-sm" href="{{ url_for(
-        'view_raw_file',
+        'ui_ns.view_raw_file',
         repo=repo.name,
         username=username,
         namespace=repo.namespace,
         identifier=commitid) }}" title="View as raw">raw</a>
       <a class="btn btn-secondary btn-sm" href="{{ url_for(
-        'view_commit_patch',
+        'ui_ns.view_commit_patch',
         repo=repo.name,
         username=username,
         namespace=repo.namespace,
         commitid=commitid) }}">patch</a>
       <a class="btn btn-secondary btn-sm" href="{{ url_for(
-        'view_tree', username=username, namespace=repo.namespace,
+        'ui_ns.view_tree', username=username, namespace=repo.namespace,
         repo=repo.name, identifier=commitid) }}">tree</a>
       {% if (commit.parents|length) == 1 %}
       <a class="btn btn-secondary btn-sm" title={{commit.parents[0].oid.hex}} href="{{ url_for(
-        'view_commit',
+        'ui_ns.view_commit',
         repo=repo.name,
         username=username,
         namespace=repo.namespace,
@@ -43,7 +43,7 @@
         <div class="dropdown-menu dropdown-menu-right" aria-labelledby="parentsDrop">
           {% for parent in commit.parents %}
             <a class="dropdown-item" href="{{ url_for(
-                'view_commit',
+                'ui_ns.view_commit',
                 repo=repo.name,
                 username=username,
                 namespace=repo.namespace,
@@ -165,14 +165,14 @@
   <div class="card-header">
     {% if patch|hasattr('new_file_path') %}
       <a href="{{ url_for(
-          'view_file', username=username, namespace=repo.namespace,
+          'ui_ns.view_file', username=username, namespace=repo.namespace,
           repo=repo.name, identifier=commitid,
           filename=patch.new_file_path) }}" title="View file as of {{ commitid|short }}">
         {{  patch.new_file_path|unicode }}
       </a>
     {% elif patch|hasattr('delta') %}
       <a href="{{ url_for(
-          'view_file', username=username, namespace=repo.namespace,
+          'ui_ns.view_file', username=username, namespace=repo.namespace,
           repo=repo.name, identifier=commitid,
           filename=patch.delta.new_file.path) }}" title="View file as of {{ commitid|short }}">
         {{  patch.delta.new_file.path|unicode }}
diff --git a/template/commits.html b/template/commits.html
index 77c0dfe..079dfcc 100644
--- a/template/commits.html
+++ b/template/commits.html
@@ -52,7 +52,7 @@
           <div class="dropdown-menu dropdown-menu-right">
             {% for branch in g.branches %}
                 <a class="dropdown-item" href="{{ url_for(
-                    'view_commits',
+                    'ui_ns.view_commits',
                     repo=repo.name,
                     username=username,
                     namespace=repo.namespace,
@@ -89,7 +89,7 @@
         (repo.is_fork or repo.settings.get('pull_requests', True)) %}
           <div id="request_pull" class="pull-xs-right">
             <a class="btn btn-primary btn-sm"
-              href="{{ url_for('new_request_pull',
+              href="{{ url_for('ui_ns.new_request_pull',
                     repo=repo.name,
                     username=username,
                     namespace=repo.namespace,
@@ -101,7 +101,7 @@
 
       <div id="diff_commits" class="list-group m-t-1" style="display:none;">
         {% for diff_commit_full in diff_commits_full %}
-           <a href="{{ url_for('view_commit',
+           <a href="{{ url_for('ui_ns.view_commit',
                     repo=repo.name,
                     username=username,
                     namespace=repo.namespace,
@@ -143,7 +143,7 @@
               </div>
             {% endif %}
             <h5>
-              <a href="{{ url_for('view_commit',
+              <a href="{{ url_for('ui_ns.view_commit',
                     repo=repo.name,
                     username=username,
                     namespace=repo.namespace,
@@ -159,7 +159,7 @@
               </a>
             </h5>
               {{ commit.author|author2user_commits(
-                link=url_for('view_commits',
+                link=url_for('ui_ns.view_commits',
                     repo=repo.name,
                     branchname=branchname,
                     username=username,
diff --git a/template/index_auth.html b/template/index_auth.html
index ce670ef..ade3fb8 100644
--- a/template/index_auth.html
+++ b/template/index_auth.html
@@ -27,7 +27,8 @@
             {% if config.get('ENABLE_NEW_PROJECTS', True) and
                   config.get('ENABLE_UI_NEW_PROJECTS', True) and repos %}
             <span class="pull-xs-right">
-              <a class="btn btn-secondary btn-sm" href="{{ url_for('new_project') }}">
+              <a class="btn btn-secondary btn-sm" href="{{
+                    url_for('ui_ns.new_project') }}">
                 <span class="oi" data-glyph="plus" data-toggle="tooltip"
                     title="Create New Project" aria-hidden="true">
                 </span>
@@ -40,7 +41,7 @@
             {% for repo in repos %}
               <div class="list-group-item">
                 {% set url = url_for(
-                    'view_repo',
+                    'ui_ns.view_repo',
                     repo=repo.name,
                     username=repo.user.username if repo.is_fork else None,
                     namespace=repo.namespace) %}
@@ -61,7 +62,7 @@
             <div class="text-xs-center m-t-1">
               {% if config.get('ENABLE_NEW_PROJECTS', True) and
                   config.get('ENABLE_UI_NEW_PROJECTS', True) %}
-              <a class="btn btn-success" href="{{ url_for('new_project') }}">
+              <a class="btn btn-success" href="{{ url_for('ui_ns.new_project') }}">
                 <span class="oi" data-glyph="plus" data-toggle="tooltip"
                     title="Create New Project" aria-hidden="true">
                 </span> Create a Project
@@ -81,7 +82,7 @@
             {% for fork in forks %}
               <div class="list-group-item">
                 {% set url = url_for(
-                    'view_repo',
+                    'ui_ns.view_repo',
                     repo=fork.name,
                     username=fork.user.username if fork.is_fork else None,
                     namespace=fork.namespace) %}
@@ -93,7 +94,7 @@
                   <div class="pull-xs-right">
                   <span class="text-muted">forked from</span>
                   <a href="{{ url_for(
-                    'view_repo', repo=fork.parent.name,
+                    'ui_ns.view_repo', repo=fork.parent.name,
                     namespace=fork.parent.namespace) }}">
                     {{ fork.parent.namespace + '/' if fork.parent.namespace
                     }}{{fork.parent.name}}
@@ -117,7 +118,7 @@
                 My Groups <span class="label label-default">{{ user.groups | length }}</span>
                 <span class="pull-xs-right">
                   {% if config.get('ENABLE_GROUP_MNGT') %}
-                    <a class="btn btn-secondary btn-sm" href="{{ url_for('add_group') }}">
+                    <a class="btn btn-secondary btn-sm" href="{{ url_for('ui_ns.add_group') }}">
                       <span class="oi" data-glyph="plus" data-toggle="tooltip"
                         title="Create New Group" aria-hidden="true">
                       </span>
@@ -128,7 +129,7 @@
             {% for group in user.groups %}
               <div class="list-group">
                 <a class="list-group-item" href="{{ url_for(
-                    'view_group', group=group)}}">
+                    'ui_ns.view_group', group=group)}}">
                   <div class=""><strong><span class="oi" data-glyph="people">
                     </span> &nbsp;{{ group }}</strong>
                   </div>
@@ -148,7 +149,7 @@
               <div class="list-group">
                 {% if repo.is_fork %}
                   <a class="list-group-item" href="{{ url_for(
-                    'view_repo',
+                    'ui_ns.view_repo',
                     repo=repo.name,
                     username=repo.user.username,
                     namespace=repo.namespace) }}">
@@ -163,7 +164,7 @@
                   </a>
                 {% else %}
                   <a class="list-group-item" href="{{ url_for(
-                    'view_repo', repo=repo.name, namespace=repo.namespace) }}">
+                    'ui_ns.view_repo', repo=repo.name, namespace=repo.namespace) }}">
                     <div class="">
                       <strong>
                         <span class="oi" data-glyph="document"></span>
diff --git a/template/repo_info.html b/template/repo_info.html
index 8e09284..fbf17c0 100644
--- a/template/repo_info.html
+++ b/template/repo_info.html
@@ -144,7 +144,7 @@ git push -u origin master</pre>
             <h5><strong>Contributors</strong></h5>
             <div class="m-b-2">
               <div>
-                <a href="{{ url_for('view_user', username=repo.user.user)}}">
+                <a href="{{ url_for('ui_ns.view_user', username=repo.user.user)}}">
                   {{ repo.user.default_email | avatar(20) | safe }}
                   {{ repo.user.fullname }} ({{ repo.user.user }})
                 </a>
@@ -153,7 +153,7 @@ git push -u origin master</pre>
               {% for access in repo.contributors %}
                 {% for user in repo.contributors[access] %}
                   <div>
-                    <a href="{{ url_for('view_user', username=user.user)}}">
+                    <a href="{{ url_for('ui_ns.view_user', username=user.user)}}">
                       {{ user.default_email | avatar(20) | safe }}
                       {{ user.fullname }} ({{ user.user }})
                     </a>
@@ -164,7 +164,7 @@ git push -u origin master</pre>
               {% for access in repo.contributor_groups %}
                 {% for group in repo.contributor_groups[access] %}
                   <div>
-                    <a href="{{ url_for('view_group', group=group.group_name)}}">
+                    <a href="{{ url_for('ui_ns.view_group', group=group.group_name)}}">
                       @{{ group.group_name }}
                     </a>
                     - {{ access }}
@@ -184,7 +184,7 @@ git push -u origin master</pre>
                   {% else  %}
                   <span class="oi text-muted" data-glyph="random"
                     title="Default branch"></span>
-                  <a href="{{ url_for('view_repo_branch',
+                  <a href="{{ url_for('ui_ns.view_repo_branch',
                     repo=repo.name,
                     username=username,
                     namespace=repo.namespace,
@@ -206,7 +206,7 @@ git push -u origin master</pre>
                 <div class="branch_name col-md-8">
                   <span class="oi text-muted" data-glyph="random"></span>
                   {% if branchname != branch %}
-                  <a class="" href="{{ url_for('view_repo_branch',
+                  <a class="" href="{{ url_for('ui_ns.view_repo_branch',
                     repo=repo.name,
                     username=username,
                     namespace=repo.namespace,
@@ -221,7 +221,7 @@ git push -u origin master</pre>
                 <div class="branch_del col-md-4">
                   {% if g.repo_committer and branch != head and config.get('ALLOW_DELETE_BRANCH', True) %}
                   <form id="delete_branch_form-{{ branch | replace('/', '__') }}" action="{{
-                    url_for('.delete_branch',
+                    url_for('ui_ns.delete_branch',
                             repo=repo.name,
                             username=username,
                             namespace=repo.namespace,
@@ -329,7 +329,7 @@ git push -u origin master</pre>
               <div class="list-group list-group-flush">
                 {% for commit in last_commits %}
                   <a href="{{
-                      url_for('view_commit',
+                      url_for('ui_ns.view_commit',
                       repo=repo.name,
                       username=username,
                       namespace=repo.namespace,
@@ -390,7 +390,7 @@ $(function() {
       }
   });
 
-  {% if authenticated and g.repo_committer %}
+  {% if g.authenticated and g.repo_committer %}
   $.ajax({
     url: '{{ url_for("internal_ns.get_pull_request_ready_branch") }}' ,
     type: 'POST',
@@ -405,7 +405,7 @@ $(function() {
       if (res.code == 'OK'){
         for (branch in res.message.new_branch){
           var url = "{{ url_for(
-            'new_request_pull',
+            'ui_ns.new_request_pull',
             repo=repo.name,
             username=repo.user.user if repo.is_fork else None,
             namespace=repo.namespace,
@@ -445,7 +445,7 @@ $(function() {
         }
         for (branch in res.message.branch_w_pr){
           var url = "{{ url_for(
-              'request_pull',
+              'ui_ns.request_pull',
               repo=repo.name,
               username=None,
               namespace=repo.namespace,
diff --git a/template/user_info.html b/template/user_info.html
index ea32c77..87fc1a3 100644
--- a/template/user_info.html
+++ b/template/user_info.html
@@ -45,7 +45,7 @@
             {% for group in user.groups %}
               <div class="list-group">
                 <a class="list-group-item" href="{{ url_for(
-                    'view_group', group=group)}}">
+                    'ui_ns.view_group', group=group)}}">
                   <div class=""><strong><span class="oi" data-glyph="people">
                     </span> &nbsp;{{ group }}</strong>
                   </div>

From a61dda0464436e4a051b41e9fe2fa10ef3122bc5 Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Jan 10 2018 18:05:46 +0000
Subject: [PATCH 2/2] Make the branch name grey and remove the new PR button for disabled branches


Relates to https://pagure.io/pagure-dist-git/issue/28

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

---

diff --git a/template/repo_info.html b/template/repo_info.html
index fbf17c0..aedeb4c 100644
--- a/template/repo_info.html
+++ b/template/repo_info.html
@@ -35,6 +35,13 @@
     white-space: -o-pre-wrap;    /* Opera 7 */
     word-wrap: break-word;       /* Internet Explorer 5.5+ */
   }
+
+  .disabled_branch a {
+    color: #818a91;
+  }
+  .disabled_branch .new_pr_btn {
+    display: none;
+  }
   </style>
 
 {% endblock %}
@@ -381,6 +388,35 @@ $(document).ready(function() {
   });
 });
 
+function disable_branches(){
+  function get_branches(_url){
+    $.ajax({
+      url: _url,
+      type: 'GET',
+      dataType: 'json',
+      success: function(res) {
+        for (branch in res.results){
+          branch = res.results[branch];
+          var _it = $('#branch-' + branch.name);
+          _it.addClass('disabled_branch');
+        }
+        if (res.next){
+          get_branches(res.next);
+        }
+      }
+    })
+  }
+  var _ns = '{{ repo.namespace }}';
+  if (_ns == 'rpms') {
+    _ns = 'rpm';
+  } else if (_ns == 'modules') {
+    _ns = 'modules';
+  }
+  var _url = 'https://pdc.fedoraproject.org/rest_api/v1/component-branches/'
+        + '?active=false&type=' + _ns + '&global_component={{ repo.name }}';
+  get_branches(_url);
+}
+
 $(function() {
   $( "#more_gits" ).click(function() {
       if ($( "#more_gits" ).html() == 'more') {
@@ -424,16 +460,16 @@ $(function() {
           /*$($('.bodycontent').find('.row').children()[0]).before(html);*/
           {% if repo.is_fork %}
           html2 = ' \
-                <a class="small label label-sm label-primary" data-toggle="tooltip" \
-                href="' + url + branch + '"\
+                <a class="small label label-sm label-primary new_pr_btn" data-toggle="tooltip" \
+                 href="' + url + branch + '"\
                  title="' + branch +' contains '
                  + res.message.new_branch[branch].length+' \
                  commit not in the upstream project. Click to create new PR now.'+'"> New PR</a> \
                 </div>';
           {% else %}
           html2 = ' \
-                <a class="small label label-sm label-primary" data-toggle="tooltip" \
-                href="' + url + branch + '"\
+                <a class="small label label-sm label-primary new_pr_btn" data-toggle="tooltip" \
+                 href="' + url + branch + '"\
                  title="' + branch +' contains '
                  + res.message.new_branch[branch].length+' \
                  commit not in the main branch. Click to create new PR now.'+'"> New PR</a> \
@@ -454,7 +490,7 @@ $(function() {
           var html = ' \
               <a class="small label label-sm label-success" data-toggle="tooltip" \
               href="' + url + '"\
-               title="' + branch +' is already in an opened pull-request">'
+              title="' + branch +' is already in an opened pull-request">'
                + 'PR#' + res.message.branch_w_pr[branch] + ' </a> \
               </div>';
           $('#branch-' + branch.replace('.', '\\.').replace('/', '__')
@@ -465,6 +501,9 @@ $(function() {
     }
   });
   {% endif %}
+
+  disable_branches();
+
 });
 </script>
 {% endblock %}