[Koha-bugs] [Bug 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Sara Brown changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=37700 -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Sara Brown changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=37699 -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Sara Brown changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=37698 -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Caroline Cyr La Rose changed: What|Removed |Added Resolution|--- |FIXED Documentation||https://gitlab.com/koha-com submission||munity/koha-manual/-/merge_ ||requests/766 Status|Needs documenting |RESOLVED CC||caroline.cyr-la-rose@inlibr ||o.com --- Comment #121 from Caroline Cyr La Rose --- This was already documented it seems. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Katrin Fischer changed: What|Removed |Added Status|Pushed to master|Needs documenting -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Fridolin Somers changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m --- Comment #120 from Fridolin Somers --- Enhancement not pushed to 23.05.x -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #119 from Tomás Cohen Arazi --- Pushed to master for 23.11. Nice work everyone, thanks! -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Tomás Cohen Arazi changed: What|Removed |Added Status|Passed QA |Pushed to master Version(s)||23.11.00 released in|| -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #118 from Tomás Cohen Arazi --- Created attachment 156850 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156850&action=edit Bug 16223: Tidy -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #117 from Tomás Cohen Arazi --- Created attachment 156849 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156849&action=edit Bug 16223: Mark lift_after_payment as boolean in schema -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #116 from Tomás Cohen Arazi --- Created attachment 156848 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156848&action=edit Bug 16223: DBIC schema Signed-off-by: Tomas Cohen Arazi -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #115 from Tomás Cohen Arazi --- Created attachment 156847 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156847&action=edit Bug 16223: (QA follow-up) Adjust DB changes Signed-off-by: Tomas Cohen Arazi -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #114 from Tomás Cohen Arazi --- Created attachment 156846 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156846&action=edit Bug 16223: (QA follow-up) Add filter, remove superfluous condition Signed-off-by: Marcel de Rooy Signed-off-by: Tomas Cohen Arazi -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #113 from Tomás Cohen Arazi --- Created attachment 156845 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156845&action=edit Bug 16223: Require fee limit only if "Lift after payment" is set as "Yes" Fee limit should only be required if restriction is meant to be lifted after payment. This patch makes "Fee limit" input field required based on value of "Lift after payment" select field. To test: 1. Create a new restriction. 2. "Lift after payment" should be set as "No". 3. Change value as "Yes". => "Fee limit" should now have "Required" text after input field. 4. Try to save restriction. => Text "This field is required" should be displayed after "Fee limit". 5. Change "Lift after payment" back to "No". => Both texts should now be lifted. Also test with existing restriction and make sure restrictions are still saved without problems. Sponsored-by: Koha-Suomi Oy Signed-off-by: Martin Renvoize Signed-off-by: Marcel de Rooy Signed-off-by: Tomas Cohen Arazi -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #112 from Tomás Cohen Arazi --- Created attachment 156844 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156844&action=edit Bug 16223: (QA follow-up) Move trigger to apply This patch moves the del_restrictions_after_payment from before the actual credit application call inside Koha::Account::pay to after the application of credits to debits in Koha::Account::Line::apply. Signed-off-by: Martin Renvoize Signed-off-by: Marcel de Rooy Signed-off-by: Tomas Cohen Arazi -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #111 from Tomás Cohen Arazi --- Created attachment 156843 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156843&action=edit Bug 16223: (QA follow-up) Remove GetDebarments We remove the GetDebarments routine in 31095. Signed-off-by: Martin Renvoize Signed-off-by: Marcel de Rooy Signed-off-by: Tomas Cohen Arazi -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #110 from Tomás Cohen Arazi --- Created attachment 156842 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156842&action=edit Bug 16223: Add ability to define liftable restrictions Some libraries debar Patrons at the end of the year for having unpaid fines, like in Bug 15157. Currently librarians have to manually remove this type of debarments after Patron has paid his/her fines. This patch adds ability to create restrictions which are lifted after patron pays ceratain amount of fines. To test: 1. Apply this patch. 2. Restart your services if needed. 3. Navigate to page restrictions.pl. => Note that table has two new colums in it, "Lift after payment?" and "Fee limit". 4. Add new restriction which has "Lift after payment?" set as Yes and fee limit as 5. 5. Create fees for a patron so they exceed fee limit e.g. 10 6. Add restriction made in step 2. for the patron 7. Pay patrons fees partially so that they go under fee limit => Note that patrons restriction should now be lifted. Also prove t/db_dependent/Patron/Borrower_Debarments.t. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman Signed-off-by: Martin Renvoize Signed-off-by: Marcel de Rooy Signed-off-by: Tomas Cohen Arazi -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Tomás Cohen Arazi changed: What|Removed |Added Attachment #155644|0 |1 is obsolete|| Attachment #155645|0 |1 is obsolete|| Attachment #155646|0 |1 is obsolete|| Attachment #155647|0 |1 is obsolete|| Attachment #155648|0 |1 is obsolete|| Attachment #155649|0 |1 is obsolete|| Attachment #155650|0 |1 is obsolete|| --- Comment #109 from Tomás Cohen Arazi --- Created attachment 156841 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156841&action=edit Bug 16223: Add new columns to table restriction_types This patch adds two new columns, lift_after_payment and fee_limit, to table restriction_types. These colums are used to control lifting patrons restrictions after paying their fees. To test: 1. Apply this patch. 2. Update your database via updatedatabase.pl. => Confirm your table restriction_types has two new columns. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman Signed-off-by: Martin Renvoize Signed-off-by: Marcel de Rooy Signed-off-by: Tomas Cohen Arazi -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #108 from Marcel de Rooy --- Note for RM: Please watch for conflicts: Bug 29145 - Allow patrons to have overdue items that would not result in debarment when removing overdue debarments -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #107 from Marcel de Rooy --- Looks good to me. FAIL koha-tmpl/intranet-tmpl/prog/en/modules/admin/restrictions.tt FAIL filters missing_filter at line 217 ([% IF restriction.fee_limit %][% restriction.fee_limit %][% END %]) Added follow-up. -if ($dupe->count) { +if ($dupe->count && $dupe->unblessed->{code} ne $code) { my $dupe = Koha::Patron::Restriction::Types->search( { code => { '!=' => $code }, display_text => $display_text, } ); if ($dupe->count && $dupe->unblessed->{code} ne $code) { Look above. This change is unneeded. Removed it. I was just wondering a bit about the column name fee_limit. The crux to understanding what it does, is this test: $total_due <= $restriction->type->fee_limit But I dont have a better name at hand. Could imagine that this field name is not immediately clear to people. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #106 from Marcel de Rooy --- Created attachment 155650 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155650&action=edit Bug 16223: (QA follow-up) Add filter, remove superfluous condition Signed-off-by: Marcel de Rooy -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Marcel de Rooy changed: What|Removed |Added Attachment #153029|0 |1 is obsolete|| --- Comment #105 from Marcel de Rooy --- Created attachment 155649 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155649&action=edit Bug 16223: Require fee limit only if "Lift after payment" is set as "Yes" Fee limit should only be required if restriction is meant to be lifted after payment. This patch makes "Fee limit" input field required based on value of "Lift after payment" select field. To test: 1. Create a new restriction. 2. "Lift after payment" should be set as "No". 3. Change value as "Yes". => "Fee limit" should now have "Required" text after input field. 4. Try to save restriction. => Text "This field is required" should be displayed after "Fee limit". 5. Change "Lift after payment" back to "No". => Both texts should now be lifted. Also test with existing restriction and make sure restrictions are still saved without problems. Sponsored-by: Koha-Suomi Oy Signed-off-by: Martin Renvoize Signed-off-by: Marcel de Rooy -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Marcel de Rooy changed: What|Removed |Added Attachment #153028|0 |1 is obsolete|| --- Comment #104 from Marcel de Rooy --- Created attachment 155648 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155648&action=edit Bug 16223: (QA follow-up) Move trigger to apply This patch moves the del_restrictions_after_payment from before the actual credit application call inside Koha::Account::pay to after the application of credits to debits in Koha::Account::Line::apply. Signed-off-by: Martin Renvoize Signed-off-by: Marcel de Rooy -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Marcel de Rooy changed: What|Removed |Added Attachment #153027|0 |1 is obsolete|| --- Comment #103 from Marcel de Rooy --- Created attachment 155647 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155647&action=edit Bug 16223: (QA follow-up) Remove GetDebarments We remove the GetDebarments routine in 31095. Signed-off-by: Martin Renvoize Signed-off-by: Marcel de Rooy -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Marcel de Rooy changed: What|Removed |Added Attachment #153026|0 |1 is obsolete|| --- Comment #102 from Marcel de Rooy --- Created attachment 155646 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155646&action=edit Bug 16223: Add ability to define liftable restrictions Some libraries debar Patrons at the end of the year for having unpaid fines, like in Bug 15157. Currently librarians have to manually remove this type of debarments after Patron has paid his/her fines. This patch adds ability to create restrictions which are lifted after patron pays ceratain amount of fines. To test: 1. Apply this patch. 2. Restart your services if needed. 3. Navigate to page restrictions.pl. => Note that table has two new colums in it, "Lift after payment?" and "Fee limit". 4. Add new restriction which has "Lift after payment?" set as Yes and fee limit as 5. 5. Create fees for a patron so they exceed fee limit e.g. 10 6. Add restriction made in step 2. for the patron 7. Pay patrons fees partially so that they go under fee limit => Note that patrons restriction should now be lifted. Also prove t/db_dependent/Patron/Borrower_Debarments.t. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman Signed-off-by: Martin Renvoize Signed-off-by: Marcel de Rooy -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Marcel de Rooy changed: What|Removed |Added Attachment #153025|0 |1 is obsolete|| --- Comment #101 from Marcel de Rooy --- Created attachment 155645 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155645&action=edit Bug 16223: DO NOT PUSH! Schema changes Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman Signed-off-by: Martin Renvoize Signed-off-by: Marcel de Rooy -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Marcel de Rooy changed: What|Removed |Added Attachment #153024|0 |1 is obsolete|| --- Comment #100 from Marcel de Rooy --- Created attachment 155644 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155644&action=edit Bug 16223: Add new columns to table restriction_types This patch adds two new columns, lift_after_payment and fee_limit, to table restriction_types. These colums are used to control lifting patrons restrictions after paying their fees. To test: 1. Apply this patch. 2. Update your database via updatedatabase.pl. => Confirm your table restriction_types has two new columns. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman Signed-off-by: Martin Renvoize Signed-off-by: Marcel de Rooy -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Marcel de Rooy changed: What|Removed |Added Patch complexity|--- |Small patch Status|BLOCKED |Passed QA -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Marcel de Rooy changed: What|Removed |Added Status|Signed Off |BLOCKED QA Contact|testo...@bugs.koha-communit |m.de.r...@rijksmuseum.nl |y.org | --- Comment #99 from Marcel de Rooy --- Looking here -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Status|Patch doesn't apply |Signed Off --- Comment #98 from Emmi Takkinen --- Rebased patches, setting status back to "Signed off". -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #150029|0 |1 is obsolete|| --- Comment #97 from Emmi Takkinen --- Created attachment 153029 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153029&action=edit Bug 16223: Require fee limit only if "Lift after payment" is set as "Yes" Fee limit should only be required if restriction is meant to be lifted after payment. This patch makes "Fee limit" input field required based on value of "Lift after payment" select field. To test: 1. Create a new restriction. 2. "Lift after payment" should be set as "No". 3. Change value as "Yes". => "Fee limit" should now have "Required" text after input field. 4. Try to save restriction. => Text "This field is required" should be displayed after "Fee limit". 5. Change "Lift after payment" back to "No". => Both texts should now be lifted. Also test with existing restriction and make sure restrictions are still saved without problems. Sponsored-by: Koha-Suomi Oy Signed-off-by: Martin Renvoize -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #153028|Bug 16223: Require fee |Bug 16223: (QA follow-up) description|limit only if "Lift after |Move trigger to apply |payment" is set as "Yes"| --- Comment #96 from Emmi Takkinen --- Comment on attachment 153028 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153028 Bug 16223: (QA follow-up) Move trigger to apply This patch moves the del_restrictions_after_payment from before the actual credit application call inside Koha::Account::pay to after the application of credits to debits in Koha::Account::Line::apply. Signed-off-by: Martin Renvoize -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #150028|0 |1 is obsolete|| --- Comment #95 from Emmi Takkinen --- Created attachment 153028 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153028&action=edit Bug 16223: Require fee limit only if "Lift after payment" is set as "Yes" Fee limit should only be required if restriction is meant to be lifted after payment. This patch makes "Fee limit" input field required based on value of "Lift after payment" select field. To test: 1. Create a new restriction. 2. "Lift after payment" should be set as "No". 3. Change value as "Yes". => "Fee limit" should now have "Required" text after input field. 4. Try to save restriction. => Text "This field is required" should be displayed after "Fee limit". 5. Change "Lift after payment" back to "No". => Both texts should now be lifted. Also test with existing restriction and make sure restrictions are still saved without problems. Sponsored-by: Koha-Suomi Oy Signed-off-by: Martin Renvoize -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #150027|0 |1 is obsolete|| --- Comment #94 from Emmi Takkinen --- Created attachment 153027 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153027&action=edit Bug 16223: (QA follow-up) Remove GetDebarments We remove the GetDebarments routine in 31095. Signed-off-by: Martin Renvoize -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #150026|0 |1 is obsolete|| --- Comment #93 from Emmi Takkinen --- Created attachment 153026 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153026&action=edit Bug 16223: Add ability to define liftable restrictions Some libraries debar Patrons at the end of the year for having unpaid fines, like in Bug 15157. Currently librarians have to manually remove this type of debarments after Patron has paid his/her fines. This patch adds ability to create restrictions which are lifted after patron pays ceratain amount of fines. To test: 1. Apply this patch. 2. Restart your services if needed. 3. Navigate to page restrictions.pl. => Note that table has two new colums in it, "Lift after payment?" and "Fee limit". 4. Add new restriction which has "Lift after payment?" set as Yes and fee limit as 5. 5. Create fees for a patron so they exceed fee limit e.g. 10 6. Add restriction made in step 2. for the patron 7. Pay patrons fees partially so that they go under fee limit => Note that patrons restriction should now be lifted. Also prove t/db_dependent/Patron/Borrower_Debarments.t. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman Signed-off-by: Martin Renvoize -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #150025|0 |1 is obsolete|| --- Comment #92 from Emmi Takkinen --- Created attachment 153025 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153025&action=edit Bug 16223: DO NOT PUSH! Schema changes Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman Signed-off-by: Martin Renvoize -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #148312|0 |1 is obsolete|| Attachment #150024|0 |1 is obsolete|| --- Comment #91 from Emmi Takkinen --- Created attachment 153024 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153024&action=edit Bug 16223: Add new columns to table restriction_types This patch adds two new columns, lift_after_payment and fee_limit, to table restriction_types. These colums are used to control lifting patrons restrictions after paying their fees. To test: 1. Apply this patch. 2. Update your database via updatedatabase.pl. => Confirm your table restriction_types has two new columns. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman Signed-off-by: Martin Renvoize -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #90 from Emmi Takkinen --- First patch has been resolved in bug 33574. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Marcel de Rooy changed: What|Removed |Added Status|Signed Off |Patch doesn't apply -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Bug 16223 depends on bug 33574, which changed state. Bug 33574 Summary: Restriction type is not stored, all restrictions fall back to MANUAL https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33574 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Martin Renvoize changed: What|Removed |Added QA Contact|martin.renvoize@ptfs-europe |testo...@bugs.koha-communit |.com|y.org Status|Needs Signoff |Signed Off -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Martin Renvoize changed: What|Removed |Added Attachment #148313|0 |1 is obsolete|| --- Comment #89 from Martin Renvoize --- Created attachment 150029 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150029&action=edit Bug 16223: Require fee limit only if "Lift after payment" is set as "Yes" Fee limit should only be required if restriction is meant to be lifted after payment. This patch makes "Fee limit" input field required based on value of "Lift after payment" select field. To test: 1. Create a new restriction. 2. "Lift after payment" should be set as "No". 3. Change value as "Yes". => "Fee limit" should now have "Required" text after input field. 4. Try to save restriction. => Text "This field is required" should be displayed after "Fee limit". 5. Change "Lift after payment" back to "No". => Both texts should now be lifted. Also test with existing restriction and make sure restrictions are still saved without problems. Sponsored-by: Koha-Suomi Oy Signed-off-by: Martin Renvoize -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Martin Renvoize changed: What|Removed |Added Attachment #148311|0 |1 is obsolete|| --- Comment #88 from Martin Renvoize --- Created attachment 150028 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150028&action=edit Bug 16223: (QA follow-up) Move trigger to apply This patch moves the del_restrictions_after_payment from before the actual credit application call inside Koha::Account::pay to after the application of credits to debits in Koha::Account::Line::apply. Signed-off-by: Martin Renvoize -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Martin Renvoize changed: What|Removed |Added Attachment #148310|0 |1 is obsolete|| --- Comment #87 from Martin Renvoize --- Created attachment 150027 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150027&action=edit Bug 16223: (QA follow-up) Remove GetDebarments We remove the GetDebarments routine in 31095. Signed-off-by: Martin Renvoize -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Martin Renvoize changed: What|Removed |Added Attachment #148309|0 |1 is obsolete|| --- Comment #86 from Martin Renvoize --- Created attachment 150026 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150026&action=edit Bug 16223: Add ability to define liftable restrictions Some libraries debar Patrons at the end of the year for having unpaid fines, like in Bug 15157. Currently librarians have to manually remove this type of debarments after Patron has paid his/her fines. This patch adds ability to create restrictions which are lifted after patron pays ceratain amount of fines. To test: 1. Apply this patch. 2. Restart your services if needed. 3. Navigate to page restrictions.pl. => Note that table has two new colums in it, "Lift after payment?" and "Fee limit". 4. Add new restriction which has "Lift after payment?" set as Yes and fee limit as 5. 5. Create fees for a patron so they exceed fee limit e.g. 10 6. Add restriction made in step 2. for the patron 7. Pay patrons fees partially so that they go under fee limit => Note that patrons restriction should now be lifted. Also prove t/db_dependent/Patron/Borrower_Debarments.t. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman Signed-off-by: Martin Renvoize -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Martin Renvoize changed: What|Removed |Added Attachment #148308|0 |1 is obsolete|| --- Comment #85 from Martin Renvoize --- Created attachment 150025 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150025&action=edit Bug 16223: DO NOT PUSH! Schema changes Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman Signed-off-by: Martin Renvoize -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Martin Renvoize changed: What|Removed |Added Attachment #148307|0 |1 is obsolete|| --- Comment #84 from Martin Renvoize --- Created attachment 150024 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150024&action=edit Bug 16223: Add new columns to table restriction_types This patch adds two new columns, lift_after_payment and fee_limit, to table restriction_types. These colums are used to control lifting patrons restrictions after paying their fees. To test: 1. Apply this patch. 2. Update your database via updatedatabase.pl. => Confirm your table restriction_types has two new columns. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman Signed-off-by: Martin Renvoize -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Martin Renvoize changed: What|Removed |Added Depends on|31095 |33574 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 https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33574 [Bug 33574] Restriction type is not stored, all restrictions fall back to MANUAL -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #83 from Emmi Takkinen --- Created attachment 148313 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148313&action=edit Bug 16223: Require fee limit only if "Lift after payment" is set as "Yes" Fee limit should only be required if restriction is meant to be lifted after payment. This patch makes "Fee limit" input field required based on value of "Lift after payment" select field. To test: 1. Create a new restriction. 2. "Lift after payment" should be set as "No". 3. Change value as "Yes". => "Fee limit" should now have "Required" text after input field. 4. Try to save restriction. => Text "This field is required" should be displayed after "Fee limit". 5. Change "Lift after payment" back to "No". => Both texts should now be lifted. Also test with existing restriction and make sure restrictions are still saved without problems. Sponsored-by: Koha-Suomi Oy -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #146057|0 |1 is obsolete|| --- Comment #82 from Emmi Takkinen --- Created attachment 148312 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148312&action=edit Bug 16223: Add correct value attributes to restriction options in select element Restrictions weren't lift after payment because value attributes for restrictions code were missing from restriction select element in patron-restrictions-tab.inc. Due this restrictions were always set as type "MANUAL". To test: 1. Add new restriction which has "Lift after payment?" set as Yes and fee limit as 5. 2. Create fees for a patron so they exceed fee limit e.g. 10 3. Add restriction made in step 1. for the patron 4. Pay patrons fees partially so that they go under fee limit => Note that patrons restriction should now be lifted. Also prove t/db_dependent/Patron/Borrower_Debarments.t. Sponsored-by: Koha-Suomi Oy -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #146055|0 |1 is obsolete|| Attachment #146056|0 |1 is obsolete|| --- Comment #81 from Emmi Takkinen --- Created attachment 148311 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148311&action=edit Bug 16223: (QA follow-up) Move trigger to apply This patch moves the del_restrictions_after_payment from before the actual credit application call inside Koha::Account::pay to after the application of credits to debits in Koha::Account::Line::apply. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #146054|0 |1 is obsolete|| --- Comment #80 from Emmi Takkinen --- Created attachment 148310 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148310&action=edit Bug 16223: (QA follow-up) Remove GetDebarments We remove the GetDebarments routine in 31095. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #146053|0 |1 is obsolete|| --- Comment #79 from Emmi Takkinen --- Created attachment 148309 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148309&action=edit Bug 16223: Add ability to define liftable restrictions Some libraries debar Patrons at the end of the year for having unpaid fines, like in Bug 15157. Currently librarians have to manually remove this type of debarments after Patron has paid his/her fines. This patch adds ability to create restrictions which are lifted after patron pays ceratain amount of fines. To test: 1. Apply this patch. 2. Restart your services if needed. 3. Navigate to page restrictions.pl. => Note that table has two new colums in it, "Lift after payment?" and "Fee limit". 4. Add new restriction which has "Lift after payment?" set as Yes and fee limit as 5. 5. Create fees for a patron so they exceed fee limit e.g. 10 6. Add restriction made in step 2. for the patron 7. Pay patrons fees partially so that they go under fee limit => Note that patrons restriction should now be lifted. Also prove t/db_dependent/Patron/Borrower_Debarments.t. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #146052|0 |1 is obsolete|| --- Comment #78 from Emmi Takkinen --- Created attachment 148308 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148308&action=edit Bug 16223: DO NOT PUSH! Schema changes Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #146051|0 |1 is obsolete|| --- Comment #77 from Emmi Takkinen --- Created attachment 148307 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148307&action=edit Bug 16223: Add new columns to table restriction_types This patch adds two new columns, lift_after_payment and fee_limit, to table restriction_types. These colums are used to control lifting patrons restrictions after paying their fees. To test: 1. Apply this patch. 2. Update your database via updatedatabase.pl. => Confirm your table restriction_types has two new columns. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #76 from Emmi Takkinen --- I got a comment that "Fee limit" should probably not be always required. Which got me thinking that maybe it should be required only if "Lift after payment" is set to "Yes". Or then it should always default to zero. Does anyone have any other suggestions? -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Bug 16223 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #143936|0 |1 is obsolete|| --- Comment #75 from Emmi Takkinen --- Created attachment 146057 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=146057&action=edit Bug 16223: Add correct value attributes to restriction options in select element Restrictions weren't lift after payment because value attributes for restrictions code were missing from restriction select element in patron-restrictions-tab.inc. Due this restrictions were always set as type "MANUAL". To test: 1. Add new restriction which has "Lift after payment?" set as Yes and fee limit as 5. 2. Create fees for a patron so they exceed fee limit e.g. 10 3. Add restriction made in step 1. for the patron 4. Pay patrons fees partially so that they go under fee limit => Note that patrons restriction should now be lifted. Also prove t/db_dependent/Patron/Borrower_Debarments.t. Sponsored-by: Koha-Suomi Oy -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #143770|0 |1 is obsolete|| --- Comment #74 from Emmi Takkinen --- Created attachment 146056 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=146056&action=edit Bug 16223: (QA follow-up) Move trigger to apply This patch moves the del_restrictions_after_payment from before the actual credit application call inside Koha::Account::pay to after the application of credits to debits in Koha::Account::Line::apply. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #143769|0 |1 is obsolete|| --- Comment #73 from Emmi Takkinen --- Created attachment 146055 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=146055&action=edit Bug 16223: (QA follow-up) Remove GetDebarments We remove the GetDebarments routine in 31095. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #143768|0 |1 is obsolete|| --- Comment #72 from Emmi Takkinen --- Created attachment 146054 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=146054&action=edit Bug 16223: (QA follow-up) Fix kohastructure bug -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #143767|0 |1 is obsolete|| --- Comment #71 from Emmi Takkinen --- Created attachment 146053 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=146053&action=edit Bug 16223: Add ability to define liftable restrictions Some libraries debar Patrons at the end of the year for having unpaid fines, like in Bug 15157. Currently librarians have to manually remove this type of debarments after Patron has paid his/her fines. This patch adds ability to create restrictions which are lifted after patron pays ceratain amount of fines. To test: 1. Apply this patch. 2. Restart your services if needed. 3. Navigate to page restrictions.pl. => Note that table has two new colums in it, "Lift after payment?" and "Fee limit". 4. Add new restriction which has "Lift after payment?" set as Yes and fee limit as 5. 5. Create fees for a patron so they exceed fee limit e.g. 10 6. Add restriction made in step 2. for the patron 7. Pay patrons fees partially so that they go under fee limit => Note that patrons restriction should now be lifted. Also prove t/db_dependent/Patron/Borrower_Debarments.t. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #143766|0 |1 is obsolete|| --- Comment #70 from Emmi Takkinen --- Created attachment 146052 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=146052&action=edit Bug 16223: DO NOT PUSH! Schema changes Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #143765|0 |1 is obsolete|| --- Comment #69 from Emmi Takkinen --- Created attachment 146051 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=146051&action=edit Bug 16223: Add new columns to table restriction_types This patch adds two new columns, lift_after_payment and fee_limit, to table restriction_types. These colums are used to control lifting patrons restrictions after paying their fees. To test: 1. Apply this patch. 2. Update your database via updatedatabase.pl. => Confirm your table restriction_types has two new columns. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Status|Failed QA |Needs Signoff -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #68 from Emmi Takkinen --- Created attachment 143936 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143936&action=edit Bug 16223: Add correct value attributes to restriction options in select element Restrictions weren't lift after payment because value attributes for restrictions code were missing from restriction select element in patron-restrictions-tab.inc. Due this restrictions were always set as type "MANUAL". To test: 1. Add new restriction which has "Lift after payment?" set as Yes and fee limit as 5. 2. Create fees for a patron so they exceed fee limit e.g. 10 3. Add restriction made in step 1. for the patron 4. Pay patrons fees partially so that they go under fee limit => Note that patrons restriction should now be lifted. Also prove t/db_dependent/Patron/Borrower_Debarments.t. Sponsored-by: Koha-Suomi Oy -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #67 from Emmi Takkinen --- Hmm, it seems code returns only "MANUAL" restriction. Tests however return correct type. So for some reason Koha saves restrictions incorrectly from UI. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #66 from Emmi Takkinen --- Thank you Martin and Andrew for looking into this. I'll try to work on this again tomorrow. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Andrew Fuerste-Henry changed: What|Removed |Added Status|Needs Signoff |Failed QA --- Comment #65 from Andrew Fuerste-Henry --- With these new patches the restriction is no longer being lifting on payment. I see that my restriction type exists, is marked to lift on payment, with a fee limit of 5.00. On two different patrons I added a $10 fee, created a restriction of my created type, and then paid the fine down to under $5. In both cases, the restriction was not removed. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #64 from Martin Renvoize --- I've rebased this and taken bug 31095 into account too.. I'd love to get an independent sign-off on this if your still interested in testing it Andrew Fuerste-Henry ? -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Martin Renvoize changed: What|Removed |Added Status|Signed Off |Needs Signoff -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 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. ___ 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #63 from Martin Renvoize --- Created attachment 143770 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143770&action=edit Bug 16223: (QA follow-up) Move trigger to apply This patch moves the del_restrictions_after_payment from before the actual credit application call inside Koha::Account::pay to after the application of credits to debits in Koha::Account::Line::apply. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #62 from Martin Renvoize --- Created attachment 143769 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143769&action=edit Bug 16223: (QA follow-up) Remove GetDebarments We remove the GetDebarments routine in 31095. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #61 from Martin Renvoize --- Created attachment 143768 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143768&action=edit Bug 16223: (QA follow-up) Fix kohastructure bug -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Martin Renvoize changed: What|Removed |Added Attachment #143690|0 |1 is obsolete|| --- Comment #60 from Martin Renvoize --- Created attachment 143767 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143767&action=edit Bug 16223: Add ability to define liftable restrictions Some libraries debar Patrons at the end of the year for having unpaid fines, like in Bug 15157. Currently librarians have to manually remove this type of debarments after Patron has paid his/her fines. This patch adds ability to create restrictions which are lifted after patron pays ceratain amount of fines. To test: 1. Apply this patch. 2. Restart your services if needed. 3. Navigate to page restrictions.pl. => Note that table has two new colums in it, "Lift after payment?" and "Fee limit". 4. Add new restriction which has "Lift after payment?" set as Yes and fee limit as 5. 5. Create fees for a patron so they exceed fee limit e.g. 10 6. Add restriction made in step 2. for the patron 7. Pay patrons fees partially so that they go under fee limit => Note that patrons restriction should now be lifted. Also prove t/db_dependent/Patron/Borrower_Debarments.t. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Martin Renvoize changed: What|Removed |Added Attachment #143689|0 |1 is obsolete|| --- Comment #59 from Martin Renvoize --- Created attachment 143766 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143766&action=edit Bug 16223: DO NOT PUSH! Schema changes Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Martin Renvoize changed: What|Removed |Added Attachment #143688|0 |1 is obsolete|| --- Comment #58 from Martin Renvoize --- Created attachment 143765 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143765&action=edit Bug 16223: Add new columns to table restriction_types This patch adds two new columns, lift_after_payment and fee_limit, to table restriction_types. These colums are used to control lifting patrons restrictions after paying their fees. To test: 1. Apply this patch. 2. Update your database via updatedatabase.pl. => Confirm your table restriction_types has two new columns. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Martin Renvoize changed: What|Removed |Added Attachment #143672|0 |1 is obsolete|| --- Comment #57 from Martin Renvoize --- Created attachment 143690 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143690&action=edit Bug 16223: Add ability to define liftable restrictions Some libraries debar Patrons at the end of the year for having unpaid fines, like in Bug 15157. Currently librarians have to manually remove this type of debarments after Patron has paid his/her fines. This patch adds ability to create restrictions which are lifted after patron pays ceratain amount of fines. To test: 1. Apply this patch. 2. Restart your services if needed. 3. Navigate to page restrictions.pl. => Note that table has two new colums in it, "Lift after payment?" and "Fee limit". 4. Add new restriction which has "Lift after payment?" set as Yes and fee limit as 5. 5. Create fees for a patron so they exceed fee limit e.g. 10 6. Add restriction made in step 2. for the patron 7. Pay patrons fees partially so that they go under fee limit => Note that patrons restriction should now be lifted. Also prove t/db_dependent/Patron/Borrower_Debarments.t. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Martin Renvoize changed: What|Removed |Added Attachment #143671|0 |1 is obsolete|| --- Comment #56 from Martin Renvoize --- Created attachment 143689 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143689&action=edit Bug 16223: DO NOT PUSH! Schema changes Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Martin Renvoize changed: What|Removed |Added Attachment #143670|0 |1 is obsolete|| --- Comment #55 from Martin Renvoize --- Created attachment 143688 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143688&action=edit Bug 16223: Add new columns to table restriction_types This patch adds two new columns, lift_after_payment and fee_limit, to table restriction_types. These colums are used to control lifting patrons restrictions after paying their fees. To test: 1. Apply this patch. 2. Update your database via updatedatabase.pl. => Confirm your table restriction_types has two new columns. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #54 from Martin Renvoize --- Hmm... I've not actually tested this yet.. just applied it and done a minor rebase.. reading the code it look like it won't work quite yet and it also looks like the follow-up to move from 'pay' to 'apply' is missing. I'll test and provide a follow-up myself now. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #53 from Martin Renvoize --- Oh wow, Emmi, you star! Thanks for the full rework here.. my apologies I missed the action had taken place.. the code it looking great.. I'm going to work through it now.. I'm afraid it's just missed 22.11 freezes but hopefully we can get it pushed as one of the first features for 23.05! -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Andrew Fuerste-Henry changed: What|Removed |Added Status|Needs Signoff |Signed Off -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 PTFS Europe Sandboxes changed: What|Removed |Added Attachment #143462|0 |1 is obsolete|| --- Comment #52 from PTFS Europe Sandboxes --- Created attachment 143672 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143672&action=edit Bug 16223: Add ability to define liftable restrictions Some libraries debar Patrons at the end of the year for having unpaid fines, like in Bug 15157. Currently librarians have to manually remove this type of debarments after Patron has paid his/her fines. This patch adds ability to create restrictions which are lifted after patron pays ceratain amount of fines. To test: 1. Apply this patch. 2. Restart your services if needed. 3. Navigate to page restrictions.pl. => Note that table has two new colums in it, "Lift after payment?" and "Fee limit". 4. Add new restriction which has "Lift after payment?" set as Yes and fee limit as 5. 5. Create fees for a patron so they exceed fee limit e.g. 10 6. Add restriction made in step 2. for the patron 7. Pay patrons fees partially so that they go under fee limit => Note that patrons restriction should now be lifted. Also prove t/db_dependent/Patron/Borrower_Debarments.t. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 PTFS Europe Sandboxes changed: What|Removed |Added Attachment #143389|0 |1 is obsolete|| --- Comment #51 from PTFS Europe Sandboxes --- Created attachment 143671 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143671&action=edit Bug 16223: DO NOT PUSH! Schema changes Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 PTFS Europe Sandboxes changed: What|Removed |Added Attachment #143388|0 |1 is obsolete|| --- Comment #50 from PTFS Europe Sandboxes --- Created attachment 143670 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143670&action=edit Bug 16223: Add new columns to table restriction_types This patch adds two new columns, lift_after_payment and fee_limit, to table restriction_types. These colums are used to control lifting patrons restrictions after paying their fees. To test: 1. Apply this patch. 2. Update your database via updatedatabase.pl. => Confirm your table restriction_types has two new columns. Sponsored-by: Koha-Suomi Oy Signed-off-by: Anneli Österman -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #49 from Andrew Fuerste-Henry --- I got the same error on the BibLibre sandboxes for some reason. Spinning one up on PTFS-Europe's site now. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Status|Patch doesn't apply |Needs Signoff --- Comment #48 from Emmi Takkinen --- Test plan updated. Setting this back to "Needs Signoff" since patches seem to apply nicely apart from ByWaters sandbox. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #143390|0 |1 is obsolete|| --- Comment #47 from Emmi Takkinen --- Created attachment 143462 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143462&action=edit Bug 16223: Add ability to define liftable restrictions Some libraries debar Patrons at the end of the year for having unpaid fines, like in Bug 15157. Currently librarians have to manually remove this type of debarments after Patron has paid his/her fines. This patch adds ability to create restrictions which are lifted after patron pays ceratain amount of fines. To test: 1. Apply this patch. 2. Restart your services if needed. 3. Navigate to page restrictions.pl, Administration->Patron restrictions. => Note that table has two new colums in it, "Lift after payment?" and "Fee limit". 4. Add new restriction which has "Lift after payment?" set as Yes and fee limit as 5. 5. Create fees for a patron so they exceed fee limit e.g. 10 6. Add restriction made in step 4. for the patron 7. Pay patrons fees partially so that they go under fee limit => Note that patrons restriction should now be lifted. Also prove t/db_dependent/Patron/Borrower_Debarments.t. Sponsored-by: Koha-Suomi Oy -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #46 from Emmi Takkinen --- (In reply to Andrew Fuerste-Henry from comment #45) > The third patch here just failed to apply for me on a sandbox. Not sure what > might have changed, given how recently you posted it. That's odd, rebasing or applying patches via bz-tool doesn't give merge conflict on my test environment. I also asked one our librarians to test this on sandbox (she used ptfs-europen sandbox) and she had no issues. But in Bywaters sandbox she got following error when she opened restrictions.pl: Template process failed: undef error - The method Koha::Patron::Restriction::Type->lift_after_payment is not covered by tests! Seems like schema update problem to me. Also there seems to be some typos and need for improvement on last patches test plan. Fixing that asap. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Andrew Fuerste-Henry changed: What|Removed |Added Status|Needs Signoff |Patch doesn't apply --- Comment #45 from Andrew Fuerste-Henry --- The third patch here just failed to apply for me on a sandbox. Not sure what might have changed, given how recently you posted it. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Status|ASSIGNED|Needs Signoff -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #44 from Emmi Takkinen --- Created attachment 143390 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143390&action=edit Bug 16223: Add ability to define liftable restrictions Some libraries debar Patrons at the end of the year for having unpaid fines, like in Bug 15157. Currently librarians have to manually remove this type of debarments after Patron has paid his/her fines. This patch adds ability to create restrictions which are lifted after patron pays ceratain amount of fines. To test: 1. Apply this patch. 2. Restart your services if needed. 3. Navigate to page restrictions.pl. => Note that table has two new colums in it, "Lift after payment?" and "Fee limit". 4. Add new restriction which has "Lift after payment?" set as Yes and fee limit as 5. 5. Create fees for a patron so they exceed fee limit e.g. 10 6. Add restriction made in step 2. for the patron 7. Pay patrons fees partially so that they go under fee limit => Note that patrons restriction should now be lifted. Also prove t/db_dependent/Patron/Borrower_Debarments.t. Sponsored-by: Koha-Suomi Oy -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 --- Comment #43 from Emmi Takkinen --- Created attachment 143389 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143389&action=edit Bug 16223: DO NOT PUSH! Schema changes Sponsored-by: Koha-Suomi Oy -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Attachment #132284|0 |1 is obsolete|| Attachment #132285|0 |1 is obsolete|| Attachment #132286|0 |1 is obsolete|| Attachment #132287|0 |1 is obsolete|| --- Comment #42 from Emmi Takkinen --- Created attachment 143388 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143388&action=edit Bug 16223: Add new columns to table restriction_types This patch adds two new columns, lift_after_payment and fee_limit, to table restriction_types. These colums are used to control lifting patrons restrictions after paying their fees. To test: 1. Apply this patch. 2. Update your database via updatedatabase.pl. => Confirm your table restriction_types has two new columns. Sponsored-by: Koha-Suomi Oy -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Status|Failed QA |ASSIGNED --- Comment #41 from Emmi Takkinen --- (In reply to Katrin Fischer from comment #40) > Hi Emmi, > > I recently reviewed bug 23681 and I think this goes in a more maintainable > direction. It will allow to define restriction types in a new table with > some being Koha-internal and others being added in addition. > > And what you do with the YAML pref could then be moved to be columns in the > new table, which would make it easy to query, easy to handle for staff, etc. > and remove any issues with variations of spelling/whitespace by having a > list to pick from. > Bug 23681 is in master now, so I'll take a look at this. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Failed QA --- Comment #40 from Katrin Fischer --- Hi Emmi, thanks for the rebase, starting to dig a little deeper into this now: 1) DelDebarmentsAfterPayment According to coding guidelines the methods in Koha namespace should always be snake case. 2) Terminology debarment --> restriction In general, and that possibly applies to above as well, we should use the term "restrictions". See: bug 5268 for an explanation. borrower --> patron I know you can't change all spots as the table is called borrowers_debarments, but I think it would be good to change whereever possible. Example: Lift these debarments after Borrower has paid their charges > Lift these restrictions after a patron has paid their charges See also: https://wiki.koha-community.org/wiki/Terminology 3) Number of tests The number of tests needs to be updated to 37 for tests to pass :) 4) Functionality A thing I stumbled on is that this works on the restriction comment and not the type of restriction. I believe this is done, because we can't have different types of restrictions currently and this makes it more flexible. But I am not sure if it will not also create issues. Currently the comment can only be set, if the type of restriction is MANUAL, for DISCHARGE it will be empty, possibly for OVERDUES and SUSPENSION as well. We cannot edit restrictions to add a certain comment or fix a typo. I recently reviewed bug 23681 and I think this goes in a more maintainable direction. It will allow to define restriction types in a new table with some being Koha-internal and others being added in addition. And what you do with the YAML pref could then be moved to be columns in the new table, which would make it easy to query, easy to handle for staff, etc. and remove any issues with variations of spelling/whitespace by having a list to pick from. I know this is the older dev, but maybe have a look at bug 23681? It's currently waiting for some more work, but I think it would be still achievable for the next version, especially with a little help. -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Katrin Fischer changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=23681 --- Comment #39 from Katrin Fischer --- Hi Emmi, thanks for the rebase, starting to dig a little deeper into this now: 1) DelDebarmentsAfterPayment According to coding guidelines the methods in Koha namespace should always be snake case. 2) Terminology debarment --> restriction In general, and that possibly applies to above as well, we should use the term "restrictions". See: bug 5268 for an explanation. borrower --> patron I know you can't change all spots as the table is called borrowers_debarments, but I think it would be good to change where possible. Example: Lift these debarments after Borrower has paid their charges > Lift these restrictions after a patron has paid their charges See also: https://wiki.koha-community.org/wiki/Terminology 3) Functionality I recently reviewed bug -- 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 16223] Automatically remove any borrower debarments after a payment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16223 Emmi Takkinen changed: What|Removed |Added Status|Patch doesn't apply |Signed Off --- Comment #38 from Emmi Takkinen --- Rebased patches. Couldn't find trailing whitespaces after applying patches, but found a extra =cut on last patch. -- 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/