[Koha-bugs] [Bug 16685] Use eval instead of do for .perl atomicupdates

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Tue Aug 23 02:33:29 CEST 2016


https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16685

Aleisha Amohia <aleishaamohia at hotmail.com> changed:

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

--- Comment #9 from Aleisha Amohia <aleishaamohia at hotmail.com> ---
Created attachment 54729
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=54729&action=edit
[SIGNED-OFF] Bug 16685 - Use eval instead of do for .perl atomicupdates

If we use eval instead of do for our .perl atomic update files, it will
allow developers to put in the exact code that should go into
updatedatabase.pl. The problem with do is that none of the variables
defined in updatadatabase.pl are available, whereas with eval they are.

Test Plan:
1) Apply this patch
2) Create a .perl file in atomicupdates with the following in it:
say "DBversion: $DBversion";
3) Run updatadatabase.pl
4) Note the output
5) Add a syntax error to your atomic update
6) Run updatedatabase.pl
7) Note the error is displayed

Signed-off-by: Aleisha Amohia <aleishaamohia at hotmail.com>

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


More information about the Koha-bugs mailing list