From 1da958f59bdb6fdf6ec1a87969a468f7b6583416 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Fri, 3 Jun 2022 15:22:06 +0000 Subject: [PATCH] Bug 30936: Reindent authority detail template This patch updates the authority detail template so that indentation is consistent. To test, apply the patch and go to Authorities. - Locate an authority record and view the detail page. - Everything should look correct, with working numbered tabs. - If the AuthDisplayHierarchy preference is enabled, you should see a collapsible tree of elements in the authority hierarchy. Signed-off-by: David Nind Signed-off-by: Katrin Fischer Signed-off-by: Tomas Cohen Arazi --- .../prog/en/modules/authorities/detail.tt | 147 +++++++++--------- 1 file changed, 75 insertions(+), 72 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/detail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/detail.tt index 8b18746d6f..550046cf69 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/detail.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/detail.tt @@ -3,10 +3,11 @@ [% SET footerjs = 1 %] [% PROCESS 'authorities.inc' %] [% INCLUDE 'doc-head-open.inc' %] -[% IF ( unknownauthid ) %] - Unknown authority record +<title> + [% IF ( unknownauthid ) %] + Unknown authority record [% ELSE %] - Details for authority #[% authid | html %] ([% authtypetext | html %]) + Details for authority #[% authid | html %] ([% authtypetext | html %]) [% END %] › Authorities › Koha [% INCLUDE 'doc-head-close.inc' %] @@ -45,83 +46,85 @@
- [% INCLUDE 'authorities-toolbar.inc' %] - -[% IF ( displayhierarchy ) %] -
-[% PROCESS showhierarchy trees = loophierarchies %] -
-[% END %] + [% INCLUDE 'authorities-toolbar.inc' %] + + [% IF ( displayhierarchy ) %] +
+ [% PROCESS showhierarchy trees = loophierarchies %] +
+ [% END %] -[% IF ( unknownauthid ) %] -

Authority does not exist

-
The authority record you requested does not exist ([% errauthid | html %]).
-[% ELSE %] -

Authority #[% authid | html %] ([% authtypetext | html %])

+ [% IF ( unknownauthid ) %] +

Authority does not exist

+
The authority record you requested does not exist ([% errauthid | html %]).
+ [% ELSE %] -
- [% IF count %] - Used in [% count | html %] record(s) - [% ELSE %] - This authority is not used in any records. - [% END %] -
- -
- -
-[% FOREACH BIG_LOO IN BIG_LOOP %] -
- [% FOREACH innerloo IN BIG_LOO.innerloop %] - [% IF ( innerloo.tag_number ) %] -
-
- [% innerloo.tag_number | html %] - [% IF (innerloo.defined("ind1") ) %] - [% innerloo.ind1 | html %][% innerloo.ind2 | html %] - [% END %] - [% IF (innerloo.tag_desc) %] - - [% innerloo.tag_desc | html %] +

Authority #[% authid | html %] ([% authtypetext | html %])

+ +
+ [% IF count %] + Used in [% count | html %] record(s) + [% ELSE %] + This authority is not used in any records. [% END %]
- [% ELSE %] -
- [% END %] - [% FOREACH subfiel IN innerloo.subfield %] -
-
- [% UNLESS ( subfiel.hide_marc ) %] - [% subfiel.marc_subfield | html %] - [% END %] - [% subfiel.long_desc | html %] -
-
- - [% IF ( subfiel.is_url ) %] - [% subfiel.marc_value | html %] - [% ELSE %] - [% subfiel.marc_value | html %] + +
+
-
- [% END %] + +
+ [% FOREACH BIG_LOO IN BIG_LOOP %] +
+ [% FOREACH innerloo IN BIG_LOO.innerloop %] + [% IF ( innerloo.tag_number ) %] +
+
+ [% innerloo.tag_number | html %] + [% IF (innerloo.defined("ind1") ) %] + [% innerloo.ind1 | html %][% innerloo.ind2 | html %] + [% END %] + [% IF (innerloo.tag_desc) %] + - [% innerloo.tag_desc | html %] + [% END %] +
+ [% ELSE %] +
+ [% END %] + + [% FOREACH subfiel IN innerloo.subfield %] +
+
+ [% UNLESS ( subfiel.hide_marc ) %] + [% subfiel.marc_subfield | html %] + [% END %] + [% subfiel.long_desc | html %] +
+
+ + [% IF ( subfiel.is_url ) %] + [% subfiel.marc_value | html %] + [% ELSE %] + [% subfiel.marc_value | html %] + [% END %] + +
+
+ [% END %] +
+ [% END %] +
+ [% END %] +
+
+ [% END %]
- [% END %]
-[% END %] -
-
-
-
-[% END %] [% MACRO jsinclude BLOCK %] [% INCLUDE 'authorities_js.inc' %] -- 2.39.5