[Koha-bugs] [Bug 30933] Add a designated owner for shared and public lists at patron deletion
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Wed Jul 20 13:36:07 CEST 2022
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30933
Marcel de Rooy <m.de.rooy at rijksmuseum.nl> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #137880|0 |1
is obsolete| |
--- Comment #24 from Marcel de Rooy <m.de.rooy at rijksmuseum.nl> ---
Created attachment 137907
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=137907&action=edit
Bug 30933: Add lists methods for disowning
This patch aims to tidy things a bit by doing the following changes:
* Koha::Virtualshelf gets the 'transfer_ownership' method
* Koha::Virtualshelves gets the 'disown_or_delete' method
* Koha::Patron->delete gets rewritten/simplified by using
disown_or_delete
The idea is to capture the current behavior in more fine grained
methods. So current tests should be passing as they do now.
To test:
1. Run:
$ kshell
k$ prove t/db_dependent/Koha/Patrons.t \
t/db_dependent/Virtualshelves.t
=> SUCCESS: Tests pass
2. Apply this patches
3. Repeat 1
=> SUCCESS: Tests still pass!
4. Run:
k$ prove t/db_dependent/Koha/Virtualshelf.t \
t/db_dependent/Koha/Virtualshelves.t
=> SUCCESS: Tests pass!
5. Sign off :-D
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list