If you are viewing a bibliographic record in the staff client (detail.pl, MARCdetail.pl, etc) and want to use the "Save" option to download the record in specific format (MODS, MARCXML, MARC etc.) the operation requires "export_catalog" permission. Since this operation is identical to one which OPAC users can perform without any privileges this must be incorrect. I believe the "export_catalog" permission only applies to tools/export.pl
Created attachment 15555 [details] [review] Bug 9663 - Individual bibliographic record "Save" option requires export permission This patch corrects permission on the export option available on catalogue pages in the staff client (detail.pl, MARCdetail.pl, etc) so that users no longer require "export_catalogue" permission. "export_catalogue" permission is required only for tools/export.pl. This patch also corrects some crazy nearby indentation. Sorry, couldn't resist. To test, visit a page like detail.pl with and without "export_catalogue" permission. It should be possible to save the record as MODS, MARCXML, MARC, etc. without error.
Comment on attachment 15555 [details] [review] Bug 9663 - Individual bibliographic record "Save" option requires export permission Review of attachment 15555 [details] [review]: ----------------------------------------------------------------- Correcting indentation is fine, the -w switch lets us ignore whitespace changes (for git diff and git blame etc)
Created attachment 15652 [details] [review] Bug 9663 - Individual bibliographic record "Save" option requires export permission This patch corrects permission on the export option available on catalogue pages in the staff client (detail.pl, MARCdetail.pl, etc) so that users no longer require "export_catalogue" permission. "export_catalogue" permission is required only for tools/export.pl. This patch also corrects some crazy nearby indentation. Sorry, couldn't resist. To test, visit a page like detail.pl with and without "export_catalogue" permission. It should be possible to save the record as MODS, MARCXML, MARC, etc. without error. Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
QA comment: Since this export script is only accessible from the catalogue/detail.pl page (which has the catalogue permission), I see no reason not to accept this patch. Marked as Passed QA.
Created attachment 15756 [details] [review] Bug 9663 - Individual bibliographic record "Save" option requires export permission This patch corrects permission on the export option available on catalogue pages in the staff client (detail.pl, MARCdetail.pl, etc) so that users no longer require "export_catalogue" permission. "export_catalogue" permission is required only for tools/export.pl. This patch also corrects some crazy nearby indentation. Sorry, couldn't resist. To test, visit a page like detail.pl with and without "export_catalogue" permission. It should be possible to save the record as MODS, MARCXML, MARC, etc. without error. Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
This patch has been pushed to master.
Pushed to 3.10.x and 3.8.x will be in 3.10.4 and 3.8.11