Summary: | Inconsistencies in MARC Bibliographic framework test page title and breadcrumbs | ||
---|---|---|---|
Product: | Koha | Reporter: | Caroline Cyr La Rose <caroline.cyr-la-rose> |
Component: | Templates | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | trivial | ||
Priority: | P5 - low | CC: | fridolin.somers, m.de.rooy |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
23.11.00
|
|
Circulation function: | |||
Bug Depends on: | |||
Bug Blocks: | 34396, 34635 | ||
Attachments: |
Bug 34394: Fix inconsistencies in MARC Bibliographic framework test page title and breadcrumbs
Bug 34394: Fix inconsistencies in MARC Bibliographic framework test page title and breadcrumbs Bug 34394: Fix inconsistencies in MARC Bibliographic framework test page title and breadcrumbs |
Description
Caroline Cyr La Rose
2023-07-25 18:54:10 UTC
Created attachment 154815 [details] [review] Bug 34394: Fix inconsistencies in MARC Bibliographic framework test page title and breadcrumbs This patch makes some changes to the MARC bibliographic framework test page so that breadcrumbs, page title, and page heading are consistent with each other. The patch also creates standard message and alert dialogs to show the status of the check. This is more consistent than having the message only at the bottom of the table. The patch also changes the term "itemnum" to "itemnumber." To test, apply the patch and go to Admnistration -> MARC bibliographic framework test. - The page title, breadcrumb navigation, and page heading should match: "MARC bibliographic framework test." - If your system has no errors, there should be a message-style dialog at the top of the page saying "Configuration okay." - Create an error in your MARC configuration: - Go to Administration -> MARC frameworks -> MARC structure - Locate the 952 tag and click Actions -> View subfields - Edit subfield 9 and set the "Managed in tab" dropdown to something other than "ignore" - Return to the framework test page. There should be an alert-style dialog at the top of the page, "Configuration errors found" Thanks Owen! It's much better now. I wonder if we should remove the "TOTAL" line of the table when there is an error? https://snipboard.io/VXsJ6I.jpg And also use the singular/plural thing... Not sure what it is called, but the thing that allows to have one string for 1 and one string for more than 1, like "you have 1 error" (singular error) vs "you have 2 errors" (plural errors)? I'm still singing off because I think it's OK as it is. But I can sign off any follow ups. :) Created attachment 154887 [details] [review] Bug 34394: Fix inconsistencies in MARC Bibliographic framework test page title and breadcrumbs This patch makes some changes to the MARC bibliographic framework test page so that breadcrumbs, page title, and page heading are consistent with each other. The patch also creates standard message and alert dialogs to show the status of the check. This is more consistent than having the message only at the bottom of the table. The patch also changes the term "itemnum" to "itemnumber." To test, apply the patch and go to Admnistration -> MARC bibliographic framework test. - The page title, breadcrumb navigation, and page heading should match: "MARC bibliographic framework test." - If your system has no errors, there should be a message-style dialog at the top of the page saying "Configuration okay." - Create an error in your MARC configuration: - Go to Administration -> MARC frameworks -> MARC structure - Locate the 952 tag and click Actions -> View subfields - Edit subfield 9 and set the "Managed in tab" dropdown to something other than "ignore" - Return to the framework test page. There should be an alert-style dialog at the top of the page, "Configuration errors found" Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Created attachment 155059 [details] [review] Bug 34394: Fix inconsistencies in MARC Bibliographic framework test page title and breadcrumbs This patch makes some changes to the MARC bibliographic framework test page so that breadcrumbs, page title, and page heading are consistent with each other. The patch also creates standard message and alert dialogs to show the status of the check. This is more consistent than having the message only at the bottom of the table. The patch also changes the term "itemnum" to "itemnumber." To test, apply the patch and go to Admnistration -> MARC bibliographic framework test. - The page title, breadcrumb navigation, and page heading should match: "MARC bibliographic framework test." - If your system has no errors, there should be a message-style dialog at the top of the page saying "Configuration okay." - Create an error in your MARC configuration: - Go to Administration -> MARC frameworks -> MARC structure - Locate the 952 tag and click Actions -> View subfields - Edit subfield 9 and set the "Managed in tab" dropdown to something other than "ignore" - Return to the framework test page. There should be an alert-style dialog at the top of the page, "Configuration errors found" Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Pushed to master for 23.11. Nice work everyone, thanks! Like others, not backported to 23.05.x |