From ef569a5160891b30cc2aa4d614b39f4a266d7ad7 Mon Sep 17 00:00:00 2001
From: Vivek Anand <vivekanand1101@gmail.com>
Date: Sep 21 2017 12:15:51 +0000
Subject: give project: generate gitolite acls after giving a project


Signed-off-by: Vivek Anand <vivekanand1101@gmail.com>

---

diff --git a/pagure/static/pagure.css b/pagure/static/pagure.css
index 3033695..6542064 100644
--- a/pagure/static/pagure.css
+++ b/pagure/static/pagure.css
@@ -23,12 +23,6 @@ html
     border-bottom:1px solid #DDD;
 }
 
-.repo-header-read-only
-{
-    background:#ffc0cb;
-    border-bottom:1px solid #DDD;
-}
-
 .issue_comment table,
 .readme table,
 .card .m-a-2 table
diff --git a/pagure/templates/repo_master.html b/pagure/templates/repo_master.html
index 37331bb..1f372e2 100644
--- a/pagure/templates/repo_master.html
+++ b/pagure/templates/repo_master.html
@@ -18,11 +18,7 @@
 {% endif %}
 
 {% block content %}
-{% if repo.read_only == true %}
-<div class="repo-header-read-only p-t-1">
-{% else %}
 <div class="repo-header p-t-1">
-{% endif %}
   <div class="container">
     <header>
       <h2 class="repo-name m-b-0">
@@ -30,10 +26,6 @@
           <span class="oi red-icon" data-glyph="lock-locked"
               title="Private project"></span>
         {% endif %}
-        {% if repo.read_only == true %}
-            <span class="oi red-icon" data-glyph="file"
-                  title="Read Only"></span>
-        {% endif %}
         {% if repo.is_fork -%}
           <a href="{{ url_for(
             'view_user', username=repo.user.user)
@@ -349,6 +341,21 @@
 </div>
 
 <div class="container p-t-3">
+    {% if repo.read_only %}
+    <div class="container p-t-2">
+        <div class="alert alert-danger alert-dismissible" role="alert">
+          <button type="button" class="close" data-dismiss="alert" aria-label="Close">
+            <span aria-hidden="true">&times;</span>
+            <span class="sr-only">Close</span>
+          </button>
+          <span class="oi red-icon" data-glyph="file"
+              title="Read Only"></span>
+          The access of this repository are being updated, this may take a
+          little while and during this time you or some of your contributors
+          may not be able to push to the repository.
+        </div>
+    </div>
+    {% endif %}
 {% block repo %}
 {% endblock %}
 </div>
diff --git a/pagure/ui/repo.py b/pagure/ui/repo.py
index 52b93c2..a217ce0 100644
--- a/pagure/ui/repo.py
+++ b/pagure/ui/repo.py
@@ -1607,7 +1607,7 @@ def remove_user(repo, userid, username=None, namespace=None):
                 break
         try:
             # Mark the project as read_only, celery will unmark it
-            pagure.lib.update_read_only_mode(session, repo, read_only=True)
+            pagure.lib.update_read_only_mode(SESSION, repo, read_only=True)
             SESSION.commit()
             pagure.lib.git.generate_gitolite_acls(project=repo)
             flask.flash('User removed')
@@ -1821,7 +1821,7 @@ def remove_group_project(repo, groupid, username=None, namespace=None):
                 break
         try:
             # Mark the project as read_only, celery will unmark it
-            pagure.lib.update_read_only_mode(session, repo, read_only=True)
+            pagure.lib.update_read_only_mode(SESSION, repo, read_only=True)
             SESSION.commit()
             pagure.lib.git.generate_gitolite_acls(project=repo)
             flask.flash('Group removed')
@@ -2689,6 +2689,7 @@ def give_project(repo, username=None, namespace=None):
                     SESSION, repo, new_user=flask.g.fas_user.username,
                     user=flask.g.fas_user.username)
             SESSION.commit()
+            pagure.lib.git.generate_gitolite_acls(project=repo)
             flask.flash(
                 'The project has been transferred to %s' % new_username)
         except SQLAlchemyError:  # pragma: no cover