From a67e81b8d0d35a823f1d90e55caf9aa9d6360ac4 Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Jun 13 2018 12:29:27 +0000
Subject: Fix showing the project overview if head is undefined, assume `master` then


If `head` is undefined, the ajax request listing the branches available
for PRs will fail because the `branch_to` argument will be `None` and
thus ignored while it's actually required. So instead of bailing out
just assume the head is `master`, the rest of the code should know how
to deal with this.

Fixes https://pagure.io/pagure/issue/3305

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

---

diff --git a/pagure/templates/repo_master.html b/pagure/templates/repo_master.html
index 8afb061..c9bce25 100644
--- a/pagure/templates/repo_master.html
+++ b/pagure/templates/repo_master.html
@@ -407,7 +407,7 @@ window.addEventListener("load", function(event) {
 </script>
 <script>
 
-{% if g.authenticated and not g.repo_obj.is_empty%}
+{% if g.authenticated and not g.repo_obj.is_empty %}
 $(function() {
   $.ajax({
     url: '{{ url_for("internal_ns.get_pull_request_ready_branch") }}' ,
@@ -433,7 +433,7 @@ $(function() {
             repo=repo.name,
             username=g.fas_user.username,
             namespace=repo.namespace,
-            branch_to=head,
+            branch_to=head or 'master',
             branch_from='') }}" + branch;
           html = '<a class="dropdown-item" \
           href="' + url + '">From <span class="badge badge-secondary" style="font-size:0.9em;">'