Bug 33148 - Use template wrapper for breadcrumbs: Cataloging
Summary: Use template wrapper for breadcrumbs: Cataloging
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: master
Hardware: All All
: P5 - low minor (vote)
Assignee: Owen Leonard
QA Contact: Nick Clemens
URL:
Keywords:
Depends on: 32507
Blocks:
  Show dependency treegraph
 
Reported: 2023-03-06 14:49 UTC by Owen Leonard
Modified: 2023-12-28 20:42 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Small patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.05.00,22.11.05


Attachments
Bug 33148: Use template wrapper for breadcrumbs: Cataloging (15.19 KB, patch)
2023-03-06 14:59 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 33148: Use template wrapper for breadcrumbs: Cataloging (15.25 KB, patch)
2023-03-29 08:42 UTC, Clemens Tubach
Details | Diff | Splinter Review
Bug 33148: Use template wrapper for breadcrumbs: Cataloging (15.30 KB, patch)
2023-04-01 22:59 UTC, Nick Clemens
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2023-03-06 14:49:10 UTC
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
Comment 1 Owen Leonard 2023-03-06 14:59:22 UTC
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)
Comment 2 Lucas Gass 2023-03-16 21:04:22 UTC
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?
Comment 3 Clemens Tubach 2023-03-29 08:42:42 UTC
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>
Comment 4 Nick Clemens 2023-04-01 22:59:16 UTC
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>
Comment 5 Nick Clemens 2023-04-01 23:00:10 UTC
(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
Comment 6 Tomás Cohen Arazi 2023-04-06 12:30:29 UTC
Pushed to master for 23.05.

Nice work everyone, thanks!
Comment 7 Jacob O'Mara 2023-04-16 17:24:33 UTC
Nice work, thanks everyone!

Pushed to 22.11.x for the next release.