---- Reported by mason@kohaaloha.com 2008-05-27 08:41:01 ---- This mod basically splits the existing 'edit-catalogue' user-permission into 2 permissions ('edit-catalogue' and 'edititems') ---- Additional Comments From mason@kohaaloha.com 2008-05-27 09:13:22 ---- Created an attachment functional example of both 'editcatalogue' and 'edititems' user-perms ---- Additional Comments From mason@kohaaloha.com 2008-05-27 09:14:58 ---- Created an attachment functional example of 'editcatalogue' user-perm only ---- Additional Comments From mason@kohaaloha.com 2008-05-27 09:17:28 ---- Created an attachment functional example of 'edititems' user-perm only ---- Additional Comments From mason@kohaaloha.com 2008-05-27 09:27:33 ---- FYI: this user-perm works with the independent-branches syspref staff can only edit items with the same branchcode as them, with this syspref on ---- Additional Comments From mason@kohaaloha.com 2008-05-27 09:30:29 ---- Created an attachment 1) example of 'edititems' user-perm with independent-branches ---- Additional Comments From mason@kohaaloha.com 2008-05-27 09:31:04 ---- Created an attachment 2) example of 'edititems' user-perm with independent-branches ---- Additional Comments From mason@kohaaloha.com 2008-05-27 11:36:33 ---- patches sent ---- Additional Comments From nengard@gmail.com 2009-12-17 00:48:46 ---- edititems does not appear as a permission in the most recent version of HEAD ---- Additional Comments From nengard@gmail.com 2010-02-07 15:04:50 ---- I don't see this patch in the patches email - was it ever submitted? --- Bug imported by chris@bigballofwax.co.nz 2010-05-21 00:47 UTC --- This bug was previously known as _bug_ 2170 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=2170 Imported an attachment (id=524) Imported an attachment (id=525) Imported an attachment (id=526) Imported an attachment (id=527) Imported an attachment (id=528) Unknown Component Authentication Using default product and component set in Parameters Actual time not defined. Setting to 0.0 The original reporter of this bug does not have an account here. Reassigning to the person who moved it here: chris@bigballofwax.co.nz. Previous reporter was mason@kohaaloha.com. The original assignee of this bug does not have an account here. Reassigning to the default assignee for the component, chris@bigballofwax.co.nz. Previous assignee was mason@kohaaloha.com. Bug reassigned, setting status to "NEW". Previous status was "ASSIGNED". The original submitter of attachment 524 [details] is unknown. Reassigning to the person who moved it here: chris@bigballofwax.co.nz. The original submitter of attachment 525 [details] is unknown. Reassigning to the person who moved it here: chris@bigballofwax.co.nz. The original submitter of attachment 526 [details] is unknown. Reassigning to the person who moved it here: chris@bigballofwax.co.nz. The original submitter of attachment 527 [details] is unknown. Reassigning to the person who moved it here: chris@bigballofwax.co.nz. The original submitter of attachment 528 [details] is unknown. Reassigning to the person who moved it here: chris@bigballofwax.co.nz.
This feature is now working in HEAD, although I see that there are still places in the template which do not respect the setting: Links to "edit items" which should be missing or disabled when the user lacks that permission.
Created attachment 2413 [details] [review] Follow-up patch with template revisions This patch follows up on the addition of the 'edititems' permission by adding a check of the new permission to places in the interface which link to the edit item screen. The catalogue toolbar had to be extensively re-worked in order to ensure that the "New" and "Edit" menus would appear (or not) whether or not edit_catalogue, edit_items, or create_subscription permissions are on.
Original patch will probably not apply cleanly anymore. A revised version of this fix can be found here: http://gitorious.org/koha-dev/koha-dev/commits/ps-Bug-2170-eddititems-permission-markup-2010-07-23
Created attachment 2837 [details] [review] Revised fix Revised for current HEAD.
Signed off Owen's revised patch. Also sent a patch adding the permission check to another location where edit items can be called
Patches pushed, please test
This bug is mentioned in: Updated fix for Bug 2170, Adding 'edititems' user-permission http://lists.koha-community.org/pipermail/koha-patches/2010-December/013247.html Updated fix for Bug 2170, Adding 'edititems' user-permission http://lists.koha-community.org/pipermail/koha-patches/2010-December/013249.html Bug 2170 Supplementary Fix Wrap link in permissions check http://lists.koha-community.org/pipermail/koha-patches/2010-December/013250.html
I see something that needs a correction: If a user has 'editcatalogue" permission but not 'edititems' permission they should not be offered the option on addbiblio.pl to 'Save and edit items.' The default should be 'Save and view record.'
Created attachment 3126 [details] [review] Follow-up fix Save button and duplicate confirmation redirects must respect the edititems permission: Users without permission to edit items should not be redirected to the edit items screen.
On testing this patch I found the following: I logged in as a staff member with editcatalog permissions but not edititems and when i clicked save when editing a bib record it said I didn't have permission and logged me out. I think it should have brought me to the edited bib record. Also - less major - a whitespace error: .dotest/patch:31: space before tab in indent. background : #FFF none; warning: 1 line adds whitespace errors.
Nicole, you signed off on the patch, but you have some comments, is it worth pushing it, or should I wait til the rest is addressed?
Owen and I talked on IRC and found that I didn't really have a problem - so that's why I signed off :) No need to wait.
Pushed, please test and mark resolved