Bug 34425 - Fix breadcrumbs vue component
Summary: Fix breadcrumbs vue component
Status: RESOLVED DUPLICATE of bug 34418
Alias: None
Product: Koha
Classification: Unclassified
Component: Staff interface (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-07-26 19:57 UTC by Agustín Moyano
Modified: 2023-07-27 08:25 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Attachments
Bug 34425: fix Breadcrumbs vue component (3.62 KB, patch)
2023-07-26 20:28 UTC, Agustín Moyano 		Details | Diff | Splinter Review
[DO NOT PUSH] Bug 34425: Implementation example for Preservation module (12.62 KB, patch)
2023-07-26 20:41 UTC, Agustín Moyano 		Details | Diff | Splinter Review
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Agustín Moyano 2023-07-26 19:57:58 UTC 
When a node in the routes tree has no title, breadcrumbs do not display nicely, for example, "Preservation > Trains > Show train > > Add item to train" and when the links in each step of the breadcrumbs have parameters and the route node has no name, they are not replaced with the actual values
Comment 1 Agustín Moyano 2023-07-26 20:28:15 UTC 
Created attachment 153937 [details] [review]
Bug 34425: fix Breadcrumbs vue component

This patch fixes Breadcrumbs component when there is no title in a node in the routes tree,
and fixes the links when there are parameters and the node has no name defined
Comment 2 Agustín Moyano 2023-07-26 20:41:59 UTC 
Created attachment 153938 [details] [review]
[DO NOT PUSH] Bug 34425: Implementation example for Preservation module
Comment 3 Jonathan Druart 2023-07-27 08:25:58 UTC 
Thanks Agustin, did you see bug 34418?

We have decided to have a specific flag, see bug 34418 comment 10.

*** This bug has been marked as a duplicate of bug 34418 ***