[Koha-bugs] [Bug 17767] Let Koha::Patron::Modification handle extended attributes

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Wed Dec 21 20:59:26 CET 2016


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

Owen Leonard <oleonard at myacpl.org> changed:

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

--- Comment #13 from Owen Leonard <oleonard at myacpl.org> ---
Created attachment 58367
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=58367&action=edit
[SIGNED-OFF] Bug 17767: Make Koha::Patron::Modification handle extended
attributes

This patch makes Koha::Patron::Modification aware of the new
extended_attributes
column, which is expected to contain valid JSON data.

The ->store method is modified so it validates the field value (i.e. the
content
is decoded using the JSON library) and raises a convenient exception in case of
failure.

This behaviour change is covered by the provided unit tests.

To test:
- Apply the patchset
- Run:
  $ prove t/db_dependent/Koha/Patron/Modifications.t
=> SUCCESS: Tests make sense, and they pass
- Sign off :-D

Signed-off-by: Owen Leonard <oleonard at myacpl.org>

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


More information about the Koha-bugs mailing list