Bug 6088 - remove a perl warning
Summary: remove a perl warning
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: 3.4
Hardware: All All
: PATCH-Sent (DO NOT USE) minor
Assignee: Paul Poulain
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-06 12:58 UTC by Paul Poulain
Modified: 2013-12-05 20:05 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Attachments
proposed patch for neworderempty.pl (940 bytes, patch)
2011-04-06 12:59 UTC, Paul Poulain 		Details | Diff | Splinter Review
another patch for another warning (1.02 KB, patch)
2011-04-07 09:56 UTC, Paul Poulain 		Details | Diff | Splinter Review
Signed patch for moremember.pl (1.04 KB, patch)
2011-04-07 11:38 UTC, Marcel de Rooy 		Details | Diff | Splinter Review
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Poulain 2011-04-06 12:58:26 UTC 
remove a perl warning due to use scrict & use warnings, comparison on an unset variable
Comment 1 Paul Poulain 2011-04-06 12:59:42 UTC 
Created attachment 3743 [details] [review]
proposed patch for neworderempty.pl
Comment 2 Paul Poulain 2011-04-07 09:56:08 UTC Comment hidden (obsolete)
Comment 3 Marcel de Rooy 2011-04-07 11:15:58 UTC 
Signed off the first patch. Second patch does not look completely ok. Will check it further now.
Comment 4 Marcel de Rooy 2011-04-07 11:38:01 UTC 
Created attachment 3782 [details] [review]
Signed patch for moremember.pl

Small fix on proposed patch. Signed off.
Comment 5 Marcel de Rooy 2011-04-07 11:39:24 UTC 
Both patches tested and signed.
Comment 6 Chris Cormack 2011-04-08 01:18:13 UTC 
Pushed, please test
Comment 7 Jared Camins-Esakov 2012-12-31 00:30:29 UTC 
There have been no further reports of problems so I am marking this bug resolved.