https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28843 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 123791 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123791&action=edit Bug 28843: Add view and edit buttons to result of MARC record import This patch adds "View" and "Edit" buttons to the output of the result of a MARC record import. To test, apply the patch and go to Tools -> State MARC for import. - Import a file of MARC records. - Click the "Manage staged records" button. - In the table of staged records the last column labeled "Records" should be empty. - Click "Import this batch into the catalog." - The table of records will be shown again, and this time the last column should contain "View" and "Edit" buttons for each row. - Confirm that the buttons work correctly, opening the correct record for viewing and editing. Test with both bibliographic and authority records to confirm that the correct view and edit pages open for each. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28843 sandboxes@biblibre.com <sandboxes@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #123791|0 |1 is obsolete| | --- Comment #2 from sandboxes@biblibre.com <sandboxes@biblibre.com> --- Created attachment 123804 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123804&action=edit Bug 28843: Add view and edit buttons to result of MARC record import This patch adds "View" and "Edit" buttons to the output of the result of a MARC record import. To test, apply the patch and go to Tools -> State MARC for import. - Import a file of MARC records. - Click the "Manage staged records" button. - In the table of staged records the last column labeled "Records" should be empty. - Click "Import this batch into the catalog." - The table of records will be shown again, and this time the last column should contain "View" and "Edit" buttons for each row. - Confirm that the buttons work correctly, opening the correct record for viewing and editing. Test with both bibliographic and authority records to confirm that the correct view and edit pages open for each. Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28843 Barbara Johnson <barbara.johnson@bedfordtx.gov> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |In Discussion --- Comment #3 from Barbara Johnson <barbara.johnson@bedfordtx.gov> --- I originally signed off on this bug, but had some concerns so I revisited it and changed the status to In Discussion. 1. A staff member who does not have edit permissions still sees the edit button. Clicking on the edit button takes them to a Koha login screen. It would be better if the staff member did not see the edit button at all if they can't edit anything. 2. When a staff member that does not have bib editing permissions clicks on the View button from manage-marc-import.pl they are taken to a view only display on detail.pl which seems correct. However, clicking on the Edit dropdown on this screen gives them the ability to edit the record using a MARC mod template even though they do not have the permission for editing these templates. A staff member with view only permissions should not be able to edit the bib record at all. 3. In discussing this issue with an expert cataloger she had the following to say: "I can't think of a single reason why I'd allow someone to load files without any editing permissions, frankly--looking at a loaded file is a quality control step, and if someone doesn't have the expertise to edit MARC (bib or aut), how are they going to have the expertise to, e.g., troubleshoot something that should have overlaid but didn't?" Based on this comment it seems she would question whether a view option is even needed. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28843 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28843 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fridolin.somers@biblibre.co | |m --- Comment #6 from Fridolin Somers <fridolin.somers@biblibre.com> --- Super, I like that -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28843 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #123938|0 |1 is obsolete| | --- Comment #7 from Fridolin Somers <fridolin.somers@biblibre.com> --- Created attachment 123986 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123986&action=edit Bug 28843: Add view and edit buttons to result of MARC record import This patch adds "View" and "Edit" buttons to the output of the result of a MARC record import. To test, apply the patch and go to Tools -> State MARC for import. - Import a file of MARC records. - Click the "Manage staged records" button. - In the table of staged records the last column labeled "Records" should be empty. - Click "Import this batch into the catalog." - The table of records will be shown again, and this time the last column should contain "View" and "Edit" buttons for each row. - Confirm that the buttons work correctly, opening the correct record for viewing and editing. Test with both bibliographic and authority records to confirm that the correct view and edit pages open for each. EDIT: Add permission check for "Edit" link Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28843 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28843 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |21.11.00 released in| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28843 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com Status|Pushed to master |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.