[Koha-bugs] [Bug 9312] strict perl for picture-upload.pl

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9312

Bernardo Gonzalez Kriegel <bgkriegel at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #32781|0                           |1
        is obsolete|                            |

--- Comment #45 from Bernardo Gonzalez Kriegel <bgkriegel at gmail.com> ---
Created attachment 33355
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=33355&action=edit
[SIGNED-OFF] Bug 9312: Resolve error triggered by mutli-type return value.

$results is 1 upon success or a hash on failure.
Rather than check %$results || %error, I changed it to
$results!=1 || %error. Strangely, this works without
messy warnings. I believe this resolves the problem Katrin
mentions in comment #43.

TEST PLAN
---------
0) Make sure the system preference patronimages is set to Allow.
1) Apply all the patches
2) Login to staff client
3) Tools -> Upload patron images (it's in the bottom left)
4) Choose an image file, browse for a photo, enter a patron #.
5) Click Upload
6) Click the card number link
   -- the uploaded photo should be visible on the left side of
      the screen.
7) Run koha qa test tools.

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>

Image uploaded without problems
No koha-qa errors

-- 
You are receiving this mail because:
You are watching all bug changes.