Attachment 55672 Details for Bug 17292 – Bug 17292: Follow up - tweak to make it pass cleaner

[patch] Bug 17292: Follow up - tweak to make it pass cleaner

Bug-17292-Follow-up---tweak-to-make-it-pass-cleane.patch (text/plain), 1.26 KB, created by Mark Tompsett on 2016-09-19 00:59:02 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Mark Tompsett

Created: 2016-09-19 00:59:02 UTC

Size: 1.26 KB

patch

obsolete

>From 50d36faca01c504eb48175bb4272bde4f1f1eeb2 Mon Sep 17 00:00:00 2001
>From: Mark Tompsett <mtompset@hotmail.com>
>Date: Sun, 18 Sep 2016 20:47:33 -0400
>Subject: [PATCH] Bug 17292: Follow up - tweak to make it pass cleaner
>
>remove a 'my'
>next if not defined subscriptions.
>---
> installer/data/mysql/updatedatabase.pl | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
>diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl
>index 54fcc53..42380f3 100755
>--- a/installer/data/mysql/updatedatabase.pl
>+++ b/installer/data/mysql/updatedatabase.pl
>@@ -11477,9 +11477,10 @@ if ( C4::Context->preference("Version") < TransformToNum($DBversion) ) {
>     my $sth3 = $dbh->prepare("UPDATE serials SET serialseq_x = ?, serialseq_y = ?, serialseq_z = ? WHERE serialid = ?");
> 
>     foreach my $subscription ( $sth->fetchrow_hashref() ) {
>+        next if !defined($subscription);
>         my $number_pattern = $subscription->numberpattern();
>         $sth2->execute( $subscription->{numberpattern} );
>-        my $number_pattern = $sth2->fetchrow_hashref();
>+        $number_pattern = $sth2->fetchrow_hashref();
> 
>         my $numbering_method = $number_pattern->{numberingmethod};
>         # Get all the data between the enumeration values, we need
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 17292: 55502 | 55672 | 55673 | 55674 | 55861 | 55862 | 56734