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

Héctor Eduardo Castro Avalos <hector.hecax...@gmail.com> changed:

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

--- Comment #5 from Héctor Eduardo Castro Avalos <hector.hecax...@gmail.com> ---
Created attachment 44844
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=44844&action=edit
[SIGNED-OFF]Bug 15182: Continue to consider Crypt::GCrypt optional

Caused by commit f25fe6ddb4a340f12613784dc841ab5bfd672d6b
Date:   Fri Nov 6 11:21:56 2015 -0300
    Bug 15151: Avoid DB access to load C4::Members

Make Koha::NorwegianPatronDB requires on loading C4::Members.
But Koha::NorwegianPatronDB uses Crypt::GCrypt which is marked as
optional in the deps list.
This patch suggest to load the module only if it can be loaded and does
not explode otherwise.

Another solution would be to surround the require with an eval, to avoid
to load Module::Load when C4::Members is used.

Test plan:
uninstall Crypt::GCrypt and try to access the staff interface.
Without this patch, you will get a software error.

Signed-off-by: Hector Castro <hector.hecax...@gmail.com>
Works as advertised. No software error without optional module.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to