search

hassanakbar4 / test-usernames

0 stars 0 forks source link

Allow editing existing liaisons #362

Closed hassanakbar4 closed 14 years ago

hassanakbar4 commented 14 years ago

resolution_fixed type_defect | by esanchez@yaco.es

Users from groups 'Secretariat' and 'Liaison_Manager' are allowed to edit existing liaisons.

Issue migrated from trac:362 at 2021-12-13 14:49:41 +0500

hassanakbar4 commented 14 years ago

@esanchez@yaco.es commented

From a0b2ce8efef4a2f107917986fe18ba70722117c6:

Liaison edit view. See #362

hassanakbar4 commented 14 years ago

@esanchez@yaco.es changed status from new to closed

hassanakbar4 commented 14 years ago

@esanchez@yaco.es set resolution to fixed

hassanakbar4 commented 14 years ago

@esanchez@yaco.es commented

Fixed in d6ace441a77a291c2d5febaf4b26d4ee609b7efd:

Edition of existing liaisons. Fixes #362

hassanakbar4 commented 14 years ago

@esanchez@yaco.es changed resolution from fixed to ``

hassanakbar4 commented 14 years ago

@esanchez@yaco.es changed status from closed to reopened

hassanakbar4 commented 14 years ago

@esanchez@yaco.es commented

Changeset a0b2ce8efef4a2f107917986fe18ba70722117c6 breaks the liaison view for non authenticated users.

The problem is that the group manager of an anonymous user is not able to filter so request.user.groups.filter(namein=LIAISON_EDIT_GROUPS) is raising an exception.

hassanakbar4 commented 14 years ago

@esanchez@yaco.es changed status from reopened to closed

hassanakbar4 commented 14 years ago

@esanchez@yaco.es changed resolution from ` tofixed`

hassanakbar4 commented 14 years ago

@esanchez@yaco.es commented

Fixed in e7dcdd9727a4b17077719fc5097e9605e6babfd5:

Fixes anonymous liaison detail view. Fixes Secretariat group name. Fixes #362