Bug 8349

Summary: Attempted string substitution in C4::Breeding fails on undefined
Product: Koha Reporter: Mark Tompsett <mtompset>
Component: MARC Bibliographic record staging/importAssignee: Galen Charlton <gmcharlt>
Status: RESOLVED DUPLICATE QA Contact:
Severity: enhancement    
Priority: P5 - low    
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Moved the substitution inside the if statement

Description Mark Tompsett 2012-07-03 11:02:35 UTC
Encountered the bug in 3.6.3 and confirmed it in my development environment.
By moving the line, this can be compensated for. Patch to follow.
Comment 1 Mark Tompsett 2012-07-03 11:23:50 UTC
Created attachment 10623 [details] [review]
Moved the substitution inside the if statement

While generating this patch, I realized that the if statement needs fixing too. Bug report to follow for that problem. However, the if statement modification won't affect the NULL case, which is being handled by moving this. It does differ technically in the case of 0 or '0', however the result is the same.
Comment 2 Mark Tompsett 2012-07-03 12:22:16 UTC
Marking as duplicate, because my patch on 8350 will solve this problem and the poorly defined if issues.

*** This bug has been marked as a duplicate of bug 8350 ***