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

--- Comment #214 from Liz Rea <wizzy...@gmail.com> ---
Created attachment 92308
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=92308&action=edit
Bug 14570: Add error handling to Koha::Patron::Relationship->store

This patch adds checks on the values for the 'relationship'. This is
done to avoid future problems when migrating relationships from the
plain text syspref into (why not) a proper table. And to preserve
consistency.

There's also catching on possible broken constraints and throwing a new
exception

Tests are added for both the new exceptions and the changes to
Koha::Patron::Relationship.

To test:
- Apply this patches
- Run:
  $ kshell
 k$ prove t/Koha/Exceptions.t \
          t/db_dependent/Koha/Patron.t \
          t/db_dependent/Koha/Patron/Relationship.t
=> SUCCESS: Tests pass!

Signed-off-by: Tomas Cohen Arazi <tomasco...@theke.io>

Signed-off-by: Kyle M Hall <k...@bywatersolutions.com>
Signed-off-by: Agustin Moyano <agustinmoy...@theke.io>

Signed-off-by: Liz Rea <wizzy...@gmail.com>

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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