Changeset 63

Timestamp:

2009年02月24日 11时06分34秒 (3 years ago)

Author:

jiangx

Message:

Repository admin can not add / remove repository in repos controller and authz controller.

See #15.

Location:

trunk/pysvnmanager

Files:

• controllers/authz.py (modified) (2 diffs)
• controllers/repos.py (modified) (3 diffs)
• templates/authz/index.mako (modified) (1 diff)
• templates/repos/hooks.mako (modified) (1 diff)

trunk/pysvnmanager/controllers/authz.py

@@ -55 +55 @@
     def index(self):
         c.revision = self.authz.version
+        c.is_super_user = self.is_super_user
         # used for functional test.
         c.reposlist = self.own_reposlist
@@ -181 +182 @@
         try:
             if isAddRepos:
+                if not self.is_super_user:
+                    raise Exception("Access denied.")
                 repos = self.authz.add_repos(reposname)
             else:
                 repos = self.authz.get_repos(reposname)
                 if not repos:
+                    if not self.is_super_user:
+                        raise Exception("Access denied.")
                     log.warning("Repos '%s' not exists. Create authz config automatically." % reposname)
                     repos = self.authz.add_repos(reposname)

trunk/pysvnmanager/controllers/repos.py

@@ -56 +56 @@
         
     def index(self):
+        c.is_super_user = self.is_super_user
         return render('/repos/hooks.mako')
 
@@ -206 +207 @@
         
     def create(self):
+        if not self.is_super_user:
+            return redirect_to(h.url_for(controller='security', action='failed'))
         return render('/repos/create.mako')
 
@@ -223 +226 @@
     
     def remove(self):
+        if not self.is_super_user:
+            return redirect_to(h.url_for(controller='security', action='failed'))
         return render('/repos/remove.mako')

trunk/pysvnmanager/templates/authz/index.mako

@@ -598 +598 @@
     <select name="repos_list" size="1" onChange='repos_changed()'>
     </select>
-<a href="#" onclick='new_repos()'>[+${_("New repository")}]</a>
+% if c.is_super_user:
+    <a href="#" onclick='new_repos()'>[+${_("New repository")}]</a>
+% endif
 </DIV>
 

trunk/pysvnmanager/templates/repos/hooks.mako

@@ -189 +189 @@
     <select name="repos_list" size="1" onChange='repos_changed()'>
     </select>
-${h.link_to(_("Add repository"), h.url_for(action="create"))}
-${h.link_to(_("Remove repository"), h.url_for(action="remove"))}
+% if c.is_super_user:
+    ${h.link_to(_("Add repository"), h.url_for(action="create"))}
+    ${h.link_to(_("Remove repository"), h.url_for(action="remove"))}
+% endif
 </DIV>