#2070 ForgeBlog ACL update

v1.0.0
closed
Wolf
sf-1 (616)
Admin
nobody
2015-08-20
2011-05-04
Dave Brondsema
No

Probably should grep for other cases of this too.

Module /var/local/allura/Allura/allura/ext/admin/templates/project_tools.html:52 in block "content"
<<          {% if 'ac' in mount %}
                   {% set app = mount['ac'].load()(c.project, mount['ac']) %}
                   {% set links = app.admin_menu() %}
                   {% set label = mount['ac'].options['mount_label'] if mount['ac'].options['mount_label'] != 'Tool Name' else mount['ac'].options['mount_point'] %}
                   <div class="fleft">
>>  {% set links = app.admin_menu() %}
Module forgeblog.main:89 in admin_menu
<<          admin_url = c.project.url()+'admin/'+self.config.options.mount_point+'/'
               links = []
               if self.permissions and has_access(self, 'configure', app=self)():
                   links.append(SitemapEntry('Permissions', admin_url + 'permissions', className='nav_child'))
               return links
>>  if self.permissions and has_access(self, 'configure', app=self)():
TypeError: has_access() got an unexpected keyword argument 'app'

Discussion

  • John Hoffman - 2011-05-04
    • status: open --> in-progress
    • assigned_to: John Hoffmann
    • component: General --> Admin
     

  • John Hoffman - 2011-05-04
    • status: in-progress --> code-review
    • assigned_to: John Hoffmann --> Wolf
     

  • John Hoffman - 2011-05-04

    allura: jwh/2070

    • Update the has_access call in the ForgeBlog admin_menu now that the acl system has changed.
    • Create a new project
    • allurapaste set-tool-access production.ini myproject alpha
    • Add the blog tool
    • Verify the tools page still loads
     

  • Wolf - 2011-05-04

    merged to forge:dev @ [408f81e]

     

  • Wolf - 2011-05-04
    • status: code-review --> closed
     

Log in to post a comment.