From e58095c0357e18fb79d2cfd6a10ba3eaf1d5c7df Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Fri, 25 Aug 2023 14:15:10 +0000 Subject: [PATCH] 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 --- .../prog/en/modules/admin/checkmarc.tt | 30 ++++++++++--------- 1 file changed, 16 insertions(+), 14 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/checkmarc.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/checkmarc.tt index e4040beb314..0dd34d64467 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/checkmarc.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/checkmarc.tt @@ -4,11 +4,7 @@ [% SET footerjs = 1 %] [% INCLUDE 'doc-head-open.inc' %] [% FILTER collapse %] - [% IF ( total ) %] - [% tx("MARC check: {total} errors found", { total = total }) | html %] - [% ELSE %] - [% t("MARC check: Configuration OK!") | html %] - [% END %] › + [% t("MARC bibliographic framework test") | html %] › [% t("Administration") | html %] › [% t("Koha") | html %] [% END %] @@ -25,11 +21,7 @@ Administration [% END %] [% WRAPPER breadcrumb_item bc_active= 1 %] - [% IF ( total ) %] - MARC check: [% total | html %] errors found - [% ELSE %] - MARC check: Configuration OK! - [% END %] + MARC bibliographic framework test [% END %] [% END #/ WRAPPER breadcrumbs %] [% END #/ WRAPPER sub-header.inc %] @@ -41,6 +33,17 @@

MARC bibliographic framework test

+[% IF ( total ) %] +
+

Configuration errors found

+ You have [% total | html %] error(s) in your MARC configuration. Please fix them before using Koha +
+[% ELSE %] +
+

Configuration okay

You don't have errors in your MARC configuration +
+[% END %] +
@@ -50,14 +53,14 @@ [% IF ( itemnum ) %] - - + [% ELSE %] - + [% END %] @@ -290,7 +293,6 @@
itemnum
  • The field itemnum MUST be mapped
  • +
itemnumber
  • The field itemnumber MUST be mapped
  • The corresponding subfield MUST be in with -1 (ignore) tab
OKitemnum : the field itemnumber is mapped to a field in tab -1itemnumber : the field itemnumber is mapped to a field in tab -1
[% ELSE %] -

Configuration OK, you don't have errors in your MARC parameters table

[% END %] -- 2.34.1