To continue the process started by Bug 32507, this patch will update cataloging templates so that they use the new template wrapper for breadcrumbs. - cataloguing/addbiblio.tt - cataloguing/addbooks.tt - cataloguing/additem.tt - cataloguing/cataloging-home.tt - cataloguing/editor.tt - cataloguing/linkitem.tt - cataloguing/merge.tt - cataloguing/moveitem.tt
Created attachment 147791 [details] [review] Bug 33148: Use template wrapper for breadcrumbs: Cataloging This patch updates several cataloging templates so that they use the new WRAPPER for displaying breadcrumbs. To test, apply the patch and test each page and its variations. Breadcrumbs should look correct, and each link should be correct. - Cataloging -> Home page - Cataloging search results - Select two results and 'Merge selected' - Basic MARC editor - Advanced MARC editor - Item edit page - Catalog -> Search -> Bibliographic record details - Edit -> Attach item - Edit -> Link to host record (must have EasyAnalyticalRecords preference enabled)
I have one comment regarding the advanced editor vs basic edtior: New record in the basic editor: -Cataloging > Add MARC record New record in the advanced editor: -Cataloging > Editor Existing record in the basic editor: -Cataloging > CSS : the definitive guide / Eric A. Meyer. (Record number 12) Existing record in the advanced editor: -Cataloging > Editor Should these made to match? And if so should we do it here or in a different bug?
Created attachment 148883 [details] [review] Bug 33148: Use template wrapper for breadcrumbs: Cataloging This patch updates several cataloging templates so that they use the new WRAPPER for displaying breadcrumbs. To test, apply the patch and test each page and its variations. Breadcrumbs should look correct, and each link should be correct. - Cataloging -> Home page - Cataloging search results - Select two results and 'Merge selected' - Basic MARC editor - Advanced MARC editor - Item edit page - Catalog -> Search -> Bibliographic record details - Edit -> Attach item - Edit -> Link to host record (must have EasyAnalyticalRecords preference enabled) Signed-off-by: Clemens Tubach <clemens.tubach@kit.edu>
Created attachment 149070 [details] [review] Bug 33148: Use template wrapper for breadcrumbs: Cataloging This patch updates several cataloging templates so that they use the new WRAPPER for displaying breadcrumbs. To test, apply the patch and test each page and its variations. Breadcrumbs should look correct, and each link should be correct. - Cataloging -> Home page - Cataloging search results - Select two results and 'Merge selected' - Basic MARC editor - Advanced MARC editor - Item edit page - Catalog -> Search -> Bibliographic record details - Edit -> Attach item - Edit -> Link to host record (must have EasyAnalyticalRecords preference enabled) Signed-off-by: Clemens Tubach <clemens.tubach@kit.edu> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(In reply to Lucas Gass from comment #2) > I have one comment regarding the advanced editor vs basic edtior: > > New record in the basic editor: > -Cataloging > Add MARC record > > New record in the advanced editor: > -Cataloging > Editor > > Existing record in the basic editor: > -Cataloging > CSS : the definitive guide / Eric A. Meyer. (Record number > 12) > > Existing record in the advanced editor: > -Cataloging > Editor > Should these made to match? And if so should we do it here or in a different > bug? Yes, please file a new bug, these seem to exist before these patches
Pushed to master for 23.05. Nice work everyone, thanks!
Nice work, thanks everyone! Pushed to 22.11.x for the next release.