Attachment 37568 Details for Bug 6007 – [SIGNED-OFF] Bug 6007: Fix MARC framework display when only used tags/subfields

[patch] [SIGNED-OFF] Bug 6007: Fix MARC framework display when only used tags/subfields

SIGNED-OFF-Bug-6007-Fix-MARC-framework-display-whe.patch (text/plain), 3.73 KB, created by Josef Moravec on 2015-04-08 10:19:21 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Josef Moravec

Created: 2015-04-08 10:19:21 UTC

Size: 3.73 KB

patch

obsolete

>From 7f5b9de1d55ef253b57ca3c5a6bb5e1edc6435ee Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@biblibre.com>
>Date: Wed, 8 Apr 2015 10:48:26 +0200
>Subject: [PATCH] [SIGNED-OFF] Bug 6007: Fix MARC framework display when only
> used tags/subfields
>
>The "Display only used tags/subfields" output breaks DataTables and the
>output is unusable.
>
>Test plan:
>1/ Go on admin/marctagstructure.pl after you have selected a framework
>2/ Check the checkbox
>3/ The table should be correctly displayed.
>Before this patch, the table have 2 lines per tag, now all data are
>contained into the same line.
>
>Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
>---
> .../prog/en/modules/admin/marctagstructure.tt      |   26 +++++++++-----------
> 1 file changed, 12 insertions(+), 14 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt
>index 049f419..17ee420 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt
>@@ -188,13 +188,21 @@ $(document).ready(function() {
>         <th>Edit</th>
>         <th>Delete</th>
>         </tr>
>-	</thead>
>+    </thead>
>     <tbody>
>     [% IF ( select_display ) %]
>         [% FOREACH loo IN loop %]
>-            <tr>
>+        <tr>
>             <td>[% loo.tagfield %]</td>
>-            <td>[% loo.liblibrarian %]</td>
>+            <td>
>+                [% loo.liblibrarian %]
>+                [% FOREACH subfield IN loo.subfields %]
>+                    <p>
>+                        Tab:[% subfield.tab %] | $[% subfield.tagsubfield %]
>+                        [% subfield.liblibrarian %] [% IF ( subfield.kohafield ) %][% subfield.kohafield %][% END %][% IF ( subfield.repeatable ) %], repeatable[% END %][% IF ( subfield.mandatory ) %], Mandatory[% END %][% IF ( subfield.seealso ) %], See [% subfield.seealso %][% END %][% IF ( subfield.authorised_value ) %], [% subfield.authorised_value %][% END %][% IF ( subfield.authtypecode ) %], [% subfield.authtypecode %][% END %][% IF ( subfield.value_builder ) %], [% subfield.value_builder %][% END %]
>+                    </p>
>+                [% END %]
>+            </td>
>             <td>[% IF ( loo.repeatable ) %]Yes[% ELSE %]No[% END %]</td>
>             <td>[% IF ( loo.mandatory ) %]Yes[% ELSE %]No[% END %]</td>
>             <td>[% loo.authorised_value %]</td>
>@@ -202,20 +210,10 @@ $(document).ready(function() {
>             <td><a href="[% loo.edit %]">Edit</a></td>
>             <td><a href="[% loo.delete %]">Delete</a></td>
>         </tr>
>-      [% IF ( loop.odd ) %]<tr>[% ELSE %]<tr class="highlight">[% END %]
>-            <td>&nbsp;</td>
>-            <td colspan="7">
>-                [% FOREACH subfield IN loo.subfields %]
>-                    <p>	Tab:[% subfield.tab %] | $[% subfield.tagsubfield %]
>-                            [% subfield.liblibrarian %] [% IF ( subfield.kohafield ) %][% subfield.kohafield %][% END %][% IF ( subfield.repeatable ) %], repeatable[% END %][% IF ( subfield.mandatory ) %], Mandatory[% END %][% IF ( subfield.seealso ) %], See [% subfield.seealso %][% END %][% IF ( subfield.authorised_value ) %], [% subfield.authorised_value %][% END %][% IF ( subfield.authtypecode ) %], [% subfield.authtypecode %][% END %][% IF ( subfield.value_builder ) %], [% subfield.value_builder %][% END %]
>-                    </p>
>-                [% END %]
>-            </td>
>-        </tr>
>         [% END %]
>     [% ELSE %]
>     [% FOREACH loo IN loop %]
>-    [% IF ( loop.odd ) %]<tr>[% ELSE %]<tr class="highlight">[% END %]
>+    <tr>
>         <td>[% loo.tagfield %]</td>
>         <td>[% loo.liblibrarian %]</td>
>         <td>[% IF ( loo.repeatable ) %]Yes[% ELSE %]No[% END %]</td>
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 6007: 3531 | 29315 | 37563 | 37568 | 37659