On the "Manage staged MARC records" page the list of staged records currently displays the title (or authority heading), author, and ISBN/ISSN together in one link. I think this makes the information hard to parse. I propose to send author, ISBN, and ISSN as separate variables, and move some of the display logic into the DataTables configuration.
Created attachment 94479 [details] [review] Bug 23856: Split author and ISBN/ISSN out of citation in staged MARC record management This patch modifies the way data is passed to the staged MARC record management template, splitting author, ISBN, and ISSN out of the "citation" variable and passing them separately. The DataTables configuration for the staged MARC record management page is modified so that those now-separate variables are conditionally displayed. Unrelated minor change: Added a class to the MARC preview modal so that it displays wider. To test, apply the patch and view the "Staged MARC management" page for both batches of bibliographic records and batches of authority records. In each case the title or authority heading should be displayed as a link while author, ISBN, and ISSN (if any) should not.
Created attachment 97953 [details] [review] Bug 23856: Split author and ISBN/ISSN out of citation in staged MARC record management This patch modifies the way data is passed to the staged MARC record management template, splitting author, ISBN, and ISSN out of the "citation" variable and passing them separately. The DataTables configuration for the staged MARC record management page is modified so that those now-separate variables are conditionally displayed. Unrelated minor change: Added a class to the MARC preview modal so that it displays wider. To test, apply the patch and view the "Staged MARC management" page for both batches of bibliographic records and batches of authority records. In each case the title or authority heading should be displayed as a link while author, ISBN, and ISSN (if any) should not. Signed-off-by: David Nind <david@davidnind.com>
Created attachment 98005 [details] [review] Bug 23856: Split author and ISBN/ISSN out of citation in staged MARC record management This patch modifies the way data is passed to the staged MARC record management template, splitting author, ISBN, and ISSN out of the "citation" variable and passing them separately. The DataTables configuration for the staged MARC record management page is modified so that those now-separate variables are conditionally displayed. Unrelated minor change: Added a class to the MARC preview modal so that it displays wider. To test, apply the patch and view the "Staged MARC management" page for both batches of bibliographic records and batches of authority records. In each case the title or authority heading should be displayed as a link while author, ISBN, and ISSN (if any) should not. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Nice work everyone! Pushed to master for 20.05
Enhancement not backported to 19.11.x