[Koha-bugs] [Bug 31458] Remove Koha::Patron::Debarment::AddDebarment and use $patron->add_restriction in preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31458 Martin Renvoize changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=25996 -- 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/
[Koha-bugs] [Bug 31458] Remove Koha::Patron::Debarment::AddDebarment and use $patron->add_restriction in preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31458 Bug 31458 depends on bug 31095, which changed state. Bug 31095 Summary: Remove Koha::Patron::Debarment::GetDebarments and use $patron->restrictions in preference https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31095 What|Removed |Added Status|Pushed to stable|RESOLVED Resolution|--- |FIXED -- 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/
[Koha-bugs] [Bug 31458] Remove Koha::Patron::Debarment::AddDebarment and use $patron->add_restriction in preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31458 --- Comment #10 from Katrin Fischer --- We might need to put a warning again in any reports using these fields so people can see that they need to fix on update (I think we did this before with biblio_metadata)? Also: will the patron import change? People possibly rely on being able to set debarments on import. -- 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/
[Koha-bugs] [Bug 31458] Remove Koha::Patron::Debarment::AddDebarment and use $patron->add_restriction in preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31458 --- Comment #9 from Tomás Cohen Arazi --- (In reply to Martin Renvoize from comment #8) > OK, So Kyle suggests we can/should remove the debarred and debarredcomment > fields from the borrowers table. > > We do however return a restricted boolean field in the current API.. I think > it makes a fair bit of sense to continue to return this piece of primary > information about the borrower there.. but it opens up the question.. should > we actually remove the field from the table now and thus require a join for > API queries or should I go ahead of convert the current date field to a > proper boolean that's maintained by add_restriction and friends? I did it a boolean so we can remove those heh. The idea is that we would just embed the restriction objects, each of which should include its own information. -- 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/
[Koha-bugs] [Bug 31458] Remove Koha::Patron::Debarment::AddDebarment and use $patron->add_restriction in preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31458 Martin Renvoize changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |martin.renvoize@ptfs-europe |ity.org |.com -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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/
[Koha-bugs] [Bug 31458] Remove Koha::Patron::Debarment::AddDebarment and use $patron->add_restriction in preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31458 --- Comment #8 from Martin Renvoize --- OK, So Kyle suggests we can/should remove the debarred and debarredcomment fields from the borrowers table. We do however return a restricted boolean field in the current API.. I think it makes a fair bit of sense to continue to return this piece of primary information about the borrower there.. but it opens up the question.. should we actually remove the field from the table now and thus require a join for API queries or should I go ahead of convert the current date field to a proper boolean that's maintained by add_restriction and friends? -- You are receiving this mail because: You are the assignee for the bug. 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/
[Koha-bugs] [Bug 31458] Remove Koha::Patron::Debarment::AddDebarment and use $patron->add_restriction in preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31458 --- Comment #7 from Martin Renvoize --- Must admit.. I'd also be more motivated to continue if the dependant bug was moving ;P.. Bug 31095 seems to have stalled waiting for someone to test. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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/
[Koha-bugs] [Bug 31458] Remove Koha::Patron::Debarment::AddDebarment and use $patron->add_restriction in preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31458 --- Comment #6 from Martin Renvoize --- 'Still on my list' is the status ;P I need to swing back around to it soon, but have had other priorities the last couple of weeks :S -- You are receiving this mail because: You are the assignee for the bug. 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/
[Koha-bugs] [Bug 31458] Remove Koha::Patron::Debarment::AddDebarment and use $patron->add_restriction in preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31458 Tomás Cohen Arazi changed: What|Removed |Added CC||tomasco...@gmail.com --- Comment #5 from Tomás Cohen Arazi --- What is the status of this bug? -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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/
[Koha-bugs] [Bug 31458] Remove Koha::Patron::Debarment::AddDebarment and use $patron->add_restriction in preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31458 --- Comment #4 from Kyle M Hall --- I believe we've discussed this elsewhere, but just to make sure it's public; The field borrowers.debarred was retained so that existing code utilizing the debarred field would continue to work. In practice, sometimes that field gets out of sync with the actual debarments. At this point in time, it makes a lot more sense to remove borrowers.debarred from the database. I think the same can be said for debarredcomment as well. (In reply to Martin Renvoize from comment #3) > Created attachment 139800 [details] [review] > Add _update_patron_restriction_fields > > This still needs some work.. picking which of the two constructs for > ordering works and removing new lines from the debarredcomment field. > > Do we even need the local debarredcomment field any more... and should > the debarred field just become a boolean.. or should we just drop the > fields from the borrowers table entirely and rely on relations. -- You are receiving this mail because: You are the assignee for the bug. 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/
[Koha-bugs] [Bug 31458] Remove Koha::Patron::Debarment::AddDebarment and use $patron->add_restriction in preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31458 Martin Renvoize changed: What|Removed |Added CC||k...@bywatersolutions.com, ||n...@bywatersolutions.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/
[Koha-bugs] [Bug 31458] Remove Koha::Patron::Debarment::AddDebarment and use $patron->add_restriction in preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31458 --- Comment #3 from Martin Renvoize --- Created attachment 139800 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=139800=edit Add _update_patron_restriction_fields This still needs some work.. picking which of the two constructs for ordering works and removing new lines from the debarredcomment field. Do we even need the local debarredcomment field any more... and should the debarred field just become a boolean.. or should we just drop the fields from the borrowers table entirely and rely on relations. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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/
[Koha-bugs] [Bug 31458] Remove Koha::Patron::Debarment::AddDebarment and use $patron->add_restriction in preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31458 Martin Renvoize changed: What|Removed |Added Attachment #139764|0 |1 is obsolete|| --- Comment #2 from Martin Renvoize --- Created attachment 139799 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=139799=edit Bug 31458: Add add_restriction to Koha::Patron This patch adds the beggnings of an add_restriction method to Koha::Patron. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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/
[Koha-bugs] [Bug 31458] Remove Koha::Patron::Debarment::AddDebarment and use $patron->add_restriction in preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31458 --- Comment #1 from Martin Renvoize --- Created attachment 139764 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=139764=edit Bug 31458: Add add_restriction to Koha::Patron This patch adds the beggnings of an add_restriction method to Koha::Patron. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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/
[Koha-bugs] [Bug 31458] Remove Koha::Patron::Debarment::AddDebarment and use $patron->add_restriction in preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31458 Martin Renvoize changed: What|Removed |Added Depends on||31095 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31095 [Bug 31095] Remove Koha::Patron::Debarment::GetDebarments and use $patron->restrictions in preference -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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/
[Koha-bugs] [Bug 31458] Remove Koha::Patron::Debarment::AddDebarment and use $patron->add_restriction in preference
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31458 Martin Renvoize changed: What|Removed |Added Blocks||31457 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31457 [Bug 31457] Move Koha::Patron::Debarment into Koha::Object based classes -- You are receiving this mail because: You are the assignee for the bug. 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/