Attachment 95307 Details for Bug 23900 – Bug 23900: Teach label maker to concat database fields

[patch] Bug 23900: Teach label maker to concat database fields

Bug-23900-Teach-label-maker-to-concat-database-fie.patch (text/plain), 1.62 KB, created by ByWater Sandboxes on 2019-11-12 13:21:16 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: ByWater Sandboxes

Created: 2019-11-12 13:21:16 UTC

Size: 1.62 KB

patch

obsolete

>From 143405ba7278c9ef5ac9d9a32ec2b84f08626386 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Mon, 11 Nov 2019 17:54:56 +0100
>Subject: [PATCH] Bug 23900: Teach label maker to concat database fields
>
>Test plan:
>- define a label layout with this field list: "100a 245s","enumchron copynumber"
>- find or create a record with values for all of those fields
>- generate a label for that record
>- the 100a and 245s successfully display on one line
>- the enumchron and copynumber are displayed and separated by a space
>
>Try other combinations
>
>Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
>---
> C4/Labels/Label.pm | 13 +++++++++----
> 1 file changed, 9 insertions(+), 4 deletions(-)
>
>diff --git a/C4/Labels/Label.pm b/C4/Labels/Label.pm
>index 9ce6c0e580..ac5055fb85 100644
>--- a/C4/Labels/Label.pm
>+++ b/C4/Labels/Label.pm
>@@ -148,11 +148,16 @@ sub _get_barcode_data {
>             next FIELD_LIST;
>         }
>         elsif ( $f =~ /^($match_kohatable).*/ ) {
>-            if ($item->{$f}) {
>-                $datastring .= $item->{$f};
>-            } else {
>-                $debug and warn sprintf("The '%s' field contains no data.", $f);
>+            my @fields = split ' ', $f;
>+            my @data;
>+            for my $field ( @fields ) {
>+                if ($item->{$field}) {
>+                    push @data, $item->{$field};
>+                } else {
>+                    $debug and warn sprintf("The '%s' field contains no data.", $field);
>+                }
>             }
>+            $datastring .= join ' ', @data;
>             $f = $';
>             next FIELD_LIST;
>         }
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 23900: 95273 | 95307 | 99641 | 101072 | 101073