On Thu, Aug 28, 2008 at 03:08:40PM +0200, Stein Magnus Jodal wrote:

On Thu, 2008-08-28 at 15:01 +0200, Morten Brekkevold wrote:

...

The simplest way (and least intrusive) way of adding the permission to a running NAV installation is to add a new web_access privilege to the following regex pattern: ^/ipdevinfo/?

That is also the only useful way to automatically update the permissions while upgrading to the next release, Stein Magnus. Upgrades should not touch existing privileges, only add new ones.

On second thought, this is rather obvious and clearly the best way. I've just changed the trunk.sql upgrade script in the ipdevinfo branch to insert a new privilege instead of updating the old one.

I edited the permissions as suggested, and now have a python error when clicking on the (details) link, so I guess some template file is missing somewhere or can't be read because wider permissions need to be added too?

Here's the error:

--- start ---

Mod_python error: "PythonHandler django.core.handlers.modpython"

Traceback (most recent call last):

File "/usr/lib/python2.4/site-packages/mod_python/apache.py", line 299, in HandlerDispatch result = object(req)

File "/var/lib/python-support/python2.4/django/core/handlers/modpython.py", line 189, in handler return ModPythonHandler()(req)

File "/var/lib/python-support/python2.4/django/core/handlers/modpython.py", line 162, in __call__ response = self.get_response(request)

File "/var/lib/python-support/python2.4/django/core/handlers/base.py", line 108, in get_response return callback(request, **param_dict)

File "/var/lib/python-support/python2.4/django/views/defaults.py", line 79, in page_not_found t = loader.get_template(template_name) # You need to create a 404.html template.

File "/var/lib/python-support/python2.4/django/template/loader.py", line 79, in get_template source, origin = find_template_source(template_name)

File "/var/lib/python-support/python2.4/django/template/loader.py", line 72, in find_template_source raise TemplateDoesNotExist, name

TemplateDoesNotExist: 404.html

--- end ---

I guess I should file a bug, but since we have a discussion running... :)