[Koha-bugs] [Bug 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #50 from Tomás Cohen Arazi --- Created attachment 128665 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=128665=edit Bug 27946: Add UI handling of AR fees This patch: - Adds a notice about the fee to be charged to the patron (staff and OPAC) - Adds the right UI description for ARTICLE_REQUEST_FEE debit types in the UI. To test: 1. Have some article request fee rules 2. Play with placing an article request in OPAC and staff => SUCCESS: You see a message about the fee => SUCCESS: The fees are correctly applied to the patron 3. Sign off :-D -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #49 from Tomás Cohen Arazi --- Created attachment 128664 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=128664=edit Bug 27946: Make Koha::ArticleRequest->request add a fee if required This patch makes the ->request method add a fee for the patron if required. It relies on methods defined in Koha::Patron for the task. The debit line is linked to the AR if applies. To test: 1. Apply this patch 2. Run: $ kshell k$ prove t/db_dependent/Koha/ArticleRequest.t => SUCCESS: Tests pass! 3. Sign off :-D -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #48 from Tomás Cohen Arazi --- Created attachment 128663 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=128663=edit Bug 27946: Article request fee methods in Koha::Patron This patch adds article_request_fee() and add_article_request_fee_if_needed() methods to Koha::Patron. To test: 1. Apply this patch 2. Run: $ kshell k$ prove t/db_dependent/Koha/Patron.t => SUCCESS: Tests pass! 3. Sign off :-D -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #47 from Tomás Cohen Arazi --- Created attachment 128662 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=128662=edit Bug 27946: Add article request fee to circulation rules This patch adds a new circulation rule: article requests fee. It can be set per library and category. To test: 1. Try setting sdifferent numeric values, change, and delete them => SUCCESS: Things work as expected 2. Make sure rules are loaded correctly when re-entering the circ rules page => SUCCESS: All good 3. Verify the data on the DB is stored as it should on each of your tests: $ koha-mysql kohadev > SELECT * FROM circulation_rules \ WHERE rule_name='article_request_fee'; => SUCCESS: All good (remember NULL => 'All' in the UI) 4. Sign off :-D 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #46 from Tomás Cohen Arazi --- Created attachment 128661 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=128661=edit Bug 27946: DBIC schema changes [DO NOT PUSH] 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #45 from Tomás Cohen Arazi --- Created attachment 128660 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=128660=edit Bug 27946: Add article_requests.debit_id and ARTICLE_REQUEST account debit type 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Tomás Cohen Arazi changed: What|Removed |Added Depends on|28656 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28656 [Bug 28656] Cancellations should be allowed on paid/partially paid debts -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #44 from Tomás Cohen Arazi --- (In reply to Marcel de Rooy from comment #43) > Tomas: Add some patches ;) Good idea :-P -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #43 from Marcel de Rooy --- Tomas: Add some patches ;) -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Tomás Cohen Arazi 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #42 from Tomás Cohen Arazi --- (In reply to Marcel de Rooy from comment #41) > And what about the dependency ? I'm moving the cancellation handling to a separate bug. Martin and I have talked about working together on the dependency bug just after the release. -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #41 from Marcel de Rooy --- And what about the dependency ? -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Marcel de Rooy changed: What|Removed |Added Status|Needs Signoff |ASSIGNED --- Comment #40 from Marcel de Rooy --- Tomas, I am keen to signoff this one, but wouldnt it be helpful if you would add patches ;) -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Tomás Cohen Arazi changed: What|Removed |Added Attachment #123503|0 |1 is obsolete|| -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Tomás Cohen Arazi changed: What|Removed |Added Attachment #123507|0 |1 is obsolete|| -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Tomás Cohen Arazi changed: What|Removed |Added Attachment #123506|0 |1 is obsolete|| -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Tomás Cohen Arazi changed: What|Removed |Added Attachment #123505|0 |1 is obsolete|| -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Tomás Cohen Arazi changed: What|Removed |Added Attachment #123504|0 |1 is obsolete|| -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Tomás Cohen Arazi 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Tomás Cohen Arazi changed: What|Removed |Added Assignee|agustinmoy...@theke.io |tomasco...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Tomás Cohen Arazi changed: What|Removed |Added Status|BLOCKED |ASSIGNED -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #39 from Marcel de Rooy --- *** Bug 26088 has been marked as a duplicate of this 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Marcel de Rooy changed: What|Removed |Added Status|Signed Off |BLOCKED --- Comment #38 from Marcel de Rooy --- Changing status to BLOCKED. This bug depends on bug 28656 which is in Failed QA state. Could we do something to get this/that one further? Or remove the dependency on this report? @Martin: ping. -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Agustín Moyano changed: What|Removed |Added Blocks|27945 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27945 [Bug 27945] Limit the number of article requests per day per patron category -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Agustín Moyano 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #37 from Agustín Moyano --- Created attachment 123507 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123507=edit Bug 27946: (follow-up) Use bug 28656 -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #36 from Agustín Moyano --- Created attachment 123506 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123506=edit Bug 27946: [DO NOT PUSH] Update DBIx 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #35 from Agustín Moyano --- Created attachment 123505 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123505=edit Bug 27946: Add article request fee feature This patch adds the ability to charge a fee for article scan requests. To test: 1. apply patches 2. updatedatabase 3. restart_all 4. enable ArticleRequest in preferences 5. grab a patron category and set a fee for article requests 6. grab a patron of that category (patron a) and another fron different category (patron b) 7. place an article request for both patrons SUCCESS => only for patron a, a warning is displayed saying a fee will be charged 8. check the account for both patrons SUCCESS => only for patron a there is a debit of type ARTICLE_REQUEST 9. cancel the article request for patron a SUCCESS => the debit for patron a gets canceled 10. repeat step 7 only for patron a 11. change article request fee for the patron a's category 12. repeat step 10 13. check account for patron a SUCCESS => there are 2 debits with different amount of typ ARTICLE_REQUEST 14. cancel only one article request SUCCESS => only the corresponding debit gets canceled 15. prove t/db_dependent/ArticleRequests.t 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #34 from Agustín Moyano --- Created attachment 123504 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123504=edit Bug 27946: Add article_request_fee to categories table and ARTICLE_REQUEST account debit type 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Agustín Moyano changed: What|Removed |Added Attachment #122529|0 |1 is obsolete|| Attachment #122530|0 |1 is obsolete|| Attachment #122531|0 |1 is obsolete|| Attachment #122532|0 |1 is obsolete|| Attachment #122552|0 |1 is obsolete|| --- Comment #33 from Agustín Moyano --- Created attachment 123503 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123503=edit Bug 27946: Add tests 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Agustín Moyano changed: What|Removed |Added Status|Patch doesn't apply |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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Agustín Moyano changed: What|Removed |Added Blocks||27945 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27945 [Bug 27945] Limit the number of article requests per day per patron category -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #32 from Agustín Moyano --- (In reply to Marcel de Rooy from comment #31) > (In reply to Martin Renvoize from comment #29) > > Well.. I deliberately didn't set it as a hard dependency yet.. the follow-up > > was meant as a hint to where we might want to go with this having discussed > > it with Tomas :). > > Still seems to me that the idea of Agustin on its own is simpler. Perhaps it > needs a bit of attention (checks). But adding a credit and let the cronjob > reconcile the balance somehow, whats actually wrong with that? Seems that what I implemented did not took into account some cases, for example, what happens when a patron pays a part of what was owed. Besides, it seems reasonable for the logic of adding a credit to be inside of cancel -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Marcel de Rooy changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=26088 -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #31 from Marcel de Rooy --- (In reply to Martin Renvoize from comment #29) > Well.. I deliberately didn't set it as a hard dependency yet.. the follow-up > was meant as a hint to where we might want to go with this having discussed > it with Tomas :). Still seems to me that the idea of Agustin on its own is simpler. Perhaps it needs a bit of attention (checks). But adding a credit and let the cronjob reconcile the balance somehow, whats actually wrong with that? -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #30 from Martin Renvoize --- (In reply to Marcel de Rooy from comment #28) > Interesting! How important is branchcode for the accounts subsytem? > kohastructure refers to the place where a payment was made or an invoice > created, etc. > The article request contains a branchcode for the pickup library btw. > It might not be relevant indeed, but I feel that the charge is created at > the branch where the article is actually copied or scanned. Very good point.. I'm not sure the branchcode is at all relevant here.. but if it is, I think you're probably right that it should go to the library doing the actual copying. -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #29 from Martin Renvoize --- (In reply to Marcel de Rooy from comment #27) > Hmm a dependency on a NEW bug report? Well.. I deliberately didn't set it as a hard dependency yet.. the follow-up was meant as a hint to where we might want to go with this having discussed it with Tomas :). There's a little debate going on as to how we should record the cancellation over-credit if you cancel a debt in full when some of that debt has already been paid.. should we just create the single 'cancel' credit, offset it up to the amount outstanding on the original debt and then allow the rest of the outstanding to be 'paid out' or 'applied' to other debts.. or do we split the cancellation into 'cancellation' and 'overpayment credit'... -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #28 from Marcel de Rooy --- (In reply to Martin Renvoize from comment #23) > I think the `C4::Context->preference('ArticleRequests')` check here is > superfluous to requirements.. I would hope we are checking that the feature > is enabled prior to this point and also don't think it makes sense to check > them together. Yes, that seems fair. > > + library_id => C4::Context->userenv ? > > C4::Context->userenv->{'branch'} : undef, > > I wonder if this should default to the items home library? I presume > there's no requirement, for now at least, for these charges to be library > specific? Interesting! How important is branchcode for the accounts subsytem? kohastructure refers to the place where a payment was made or an invoice created, etc. The article request contains a branchcode for the pickup library btw. It might not be relevant indeed, but I feel that the charge is created at the branch where the article is actually copied or scanned. -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #27 from Marcel de Rooy --- Hmm a dependency on a NEW bug report? -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Agustín Moyano changed: What|Removed |Added Depends on||28656 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28656 [Bug 28656] Cancellations should be allowed on paid/partially paid debts -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #26 from Martin Renvoize --- As discussed with Tomas, follow-up attached that uses some updates to the 'cancel' method as detailed in bug 22435 that makes this process of refunding simpler. -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #25 from Martin Renvoize --- Created attachment 122552 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122552=edit Bug 27946: (follow-up) Use bug 28656 -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Martin Renvoize changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=28656 -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #24 from Martin Renvoize --- Comment on attachment 122531 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122531 Bug 27946: Add article request fee feature Review of attachment 122531: --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html=27946=122531) - ::: Koha/ArticleRequest.pm @@ +103,5 @@ > > $self->status(Koha::ArticleRequest::Status::Canceled); > $self->notes($notes) if $notes; > +if ( $self->debit_line_id ) { > +my $line = Koha::Account::Lines->find( $self->debit_line_id ); Hmm, having looked at this a little more closely now I've read the concerns/discussion above.. I think we may need to add some additional considerations here. a) What should happen if a fee was 'Written off' or 'Reduced' (You've taken care of 'Cancelled' by the looks). b) We could use the 'reduce' method to cover both cases and allow the passing of 'CANCELLED' credit type perhaps? I'm thinking something along the lines of ``` my $refundable = 0; if ( $line->amount != $line->amountoutstanding ) { my $refundable_offsets = Koha::Account::Offsets->search( { debit_id => $line->id, credit_id => { '!=' => undef }, # Not the debt itself type => { '!=' => [ 'Writeoff', 'Refund', 'Cancellation', 'Discount' ] }, amount=> { '<' => 0 }# credits are negative on the DB ); $refundable = ( $refundable_offsets->count > 0 ) ? $credits_offsets->total * -1# credits are negative on the DB : 0; } $refundable += $line->amountoutstanding; $line->reduce( reduction_type => REQUEST_CANCELLATION, amount => $refundable, staff_id => C4::Context->userenv ? C4::Context->userenv->{'number'} : undef, interface => C4::Context->interface, branchcode => C4::Context->userenv ? C4::Context->userenv->{'branch'} : undef ); } ``` -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #23 from Martin Renvoize --- Comment on attachment 122531 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122531 Bug 27946: Add article request fee feature Review of attachment 122531: --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html=27946=122531) - Generally this is looking good :) ::: Koha/ArticleRequest.pm @@ +50,4 @@ > my ($self) = @_; > > $self->status(Koha::ArticleRequest::Status::Pending); > +if(C4::Context->preference('ArticleRequests') && > $self->borrower->category->article_request_fee) { I think the `C4::Context->preference('ArticleRequests')` check here is superfluous to requirements.. I would hope we are checking that the feature is enabled prior to this point and also don't think it makes sense to check them together. @@ +55,5 @@ > +{ > + amount => $self->borrower->category->article_request_fee, > + user_id=> C4::Context->userenv ? > C4::Context->userenv->{'number'} : undef, > + interface => C4::Context->interface, > + library_id => C4::Context->userenv ? > C4::Context->userenv->{'branch'} : undef, I wonder if this should default to the items home library? I presume there's no requirement, for now at least, for these charges to be library specific? -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #22 from Marcel de Rooy --- (In reply to Tomás Cohen Arazi from comment #21) > (In reply to Marcel de Rooy from comment #20) > > Martin: Are you happy with the way this patch deals with the cancellation as > > to the accounts handling? Testing amountoutstanding and choosing > > line->cancel or account->add_credit? Balance wise, it works fine. > > I think ->cancel is correct but cancelling has some caveats: some exceptions > will be thrown if (say) the line is already offsetted (partially/fully > payed, writen off, etc). > > Those cases lead me to think ->void would suit better, but we need to > (maybe) think about possible refunds. Agustin compares amount with outstanding before calling cancel. So that should be fine, I guess. It fixes the earlier problem reported. -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #21 from Tomás Cohen Arazi --- (In reply to Marcel de Rooy from comment #20) > Martin: Are you happy with the way this patch deals with the cancellation as > to the accounts handling? Testing amountoutstanding and choosing > line->cancel or account->add_credit? Balance wise, it works fine. I think ->cancel is correct but cancelling has some caveats: some exceptions will be thrown if (say) the line is already offsetted (partially/fully payed, writen off, etc). Those cases lead me to think ->void would suit better, but we need to (maybe) think about possible refunds. -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #20 from Marcel de Rooy --- Martin: Are you happy with the way this patch deals with the cancellation as to the accounts handling? Testing amountoutstanding and choosing line->cancel or account->add_credit? Balance wise, it works fine. -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Marcel de Rooy changed: What|Removed |Added CC||martin.renvoize@ptfs-europe ||.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Marcel de Rooy 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #19 from Marcel de Rooy --- Created attachment 122532 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122532=edit Bug 27946: [DO NOT PUSH] Update DBIx 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Marcel de Rooy changed: What|Removed |Added Attachment #122522|0 |1 is obsolete|| --- Comment #18 from Marcel de Rooy --- Created attachment 122531 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122531=edit Bug 27946: Add article request fee feature This patch adds the ability to charge a fee for article scan requests. To test: 1. apply patches 2. updatedatabase 3. restart_all 4. enable ArticleRequest in preferences 5. grab a patron category and set a fee for article requests 6. grab a patron of that category (patron a) and another fron different category (patron b) 7. place an article request for both patrons SUCCESS => only for patron a, a warning is displayed saying a fee will be charged 8. check the account for both patrons SUCCESS => only for patron a there is a debit of type ARTICLE_REQUEST 9. cancel the article request for patron a SUCCESS => the debit for patron a gets canceled 10. repeat step 7 only for patron a 11. change article request fee for the patron a's category 12. repeat step 10 13. check account for patron a SUCCESS => there are 2 debits with different amount of typ ARTICLE_REQUEST 14. cancel only one article request SUCCESS => only the corresponding debit gets canceled 15. prove t/db_dependent/ArticleRequests.t 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Marcel de Rooy changed: What|Removed |Added Attachment #120145|0 |1 is obsolete|| --- Comment #17 from Marcel de Rooy --- Created attachment 122530 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122530=edit Bug 27946: Add article_request_fee to categories table and ARTICLE_REQUEST account debit type 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Marcel de Rooy changed: What|Removed |Added Attachment #120144|0 |1 is obsolete|| Attachment #120146|0 |1 is obsolete|| --- Comment #16 from Marcel de Rooy --- Created attachment 122529 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122529=edit Bug 27946: Add tests 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #15 from Marcel de Rooy --- (In reply to Agustín Moyano from comment #14) > Hi Marcel, I addressed points 1 to 4. Great! Thx > The 5th, even though I agree there > should be a limit, I believe is out of scope, and should be done in a > different bug. I could at least open a new report for that one. Bug 28652. -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Marcel de Rooy changed: What|Removed |Added Blocks||28652 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28652 [Bug 28652] Enforce maximum on article request charges -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #14 from Agustín Moyano --- (In reply to Marcel de Rooy from comment #12) > Some of these 5 points justify a status change here now. Hi Marcel, I addressed points 1 to 4. The 5th, even though I agree there should be a limit, I believe is out of scope, and should be done in a different bug. Cheers. -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Agustín Moyano changed: What|Removed |Added Attachment #120147|0 |1 is obsolete|| --- Comment #13 from Agustín Moyano --- Created attachment 122522 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122522=edit Bug 27946: Add article request fee feature This patch adds the ability to charge a fee for article scan requests. To test: 1. apply patches 2. updatedatabase 3. restart_all 4. enable ArticleRequest in preferences 5. grab a patron category and set a fee for article requests 6. grab a patron of that category (patron a) and another fron different category (patron b) 7. place an article request for both patrons SUCCESS => only for patron a, a warning is displayed saying a fee will be charged 8. check the account for both patrons SUCCESS => only for patron a there is a debit of type ARTICLE_REQUEST 9. cancel the article request for patron a SUCCESS => the debit for patron a gets canceled 10. repeat step 7 only for patron a 11. change article request fee for the patron a's category 12. repeat step 10 13. check account for patron a SUCCESS => there are 2 debits with different amount of typ ARTICLE_REQUEST 14. cancel only one article request SUCCESS => only the corresponding debit gets canceled 15. prove t/db_dependent/ArticleRequests.t -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Agustín Moyano 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Marcel de Rooy changed: What|Removed |Added Status|Needs Signoff |Failed QA --- Comment #12 from Marcel de Rooy --- Some of these 5 points justify a status change here 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #11 from Marcel de Rooy --- [5] There should be a limit where you cant add requests anymore. Similar to maxoutstanding OR OPACFineNoRenewals OR MaxFine OR noissuescharge. Even RentalsInNoissuesCharge. NOTE: This leaves the most room for discussion about scope borders.. $self->borrower->account->add_debit DOES NOT check a limit. Please note: Your account has outstanding fees & charges of 35.35. Holds are blocked because your fine balance is over the limit. Please note: You cannot renew your books online. Reason: Your fines exceed 100.00. Reducing fine for item $itemnum borrower $borrowernumber from $amount to $maxIncrease - MaxFine reached But I can just keep adding article requests without any limit applied. -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #10 from Marcel de Rooy --- [4] Test by putting negative amount in fee on patron category => add_debit should throw an exception.. Seems that I cannot save that value with category on the list view. value="[% category.article_request_fee | $Price on_editing => 1 %]" Koha::Number::Price->new( $value )->format_for_editing $VAR1 = '-10.00'; But -10 is saved in the DATABASE ! Not shown in STAFF (list view shows only a minus; detail form does show -10) So, yes adding a request now crashes: Debit amount passed is not positive at /usr/share/perl5/Exception/Class/Base.pm line 88 Conclusion: The categories form should enforce the amount to be zero or POSITIVE. -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #9 from Marcel de Rooy --- Read: cancel an article request after paying fee .. -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #8 from Marcel de Rooy --- [3] Cancel a fee after paying it Started with fee 0.35. Added one request. Changed fee to 35. Added four requests. Balance is now 140.35. I pay 105. Results in 35.35. OK I cancel in OPAC one of the requests (which already had amountoutstanding == 0 but you cant see that). What happened ? The balance stays 35.35 ! Wrong ! It should have gone down. NEEDS FIXING I change the fee again to 10. Add a request. Balance 45.35. OK Change fee back to 35. Now cancel the last request. Balance goes back to 35.35 Great. -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #7 from Marcel de Rooy --- [2] Warning Use of uninitialized value in string eq at /usr/share/koha/Koha/ArticleRequest.pm line 110. my $line = Koha::Account::Lines->find($self->debit_line_id); $line->cancel( { branch => C4::Context->userenv ? C4::Context->userenv->{'branch'} : undef, staff_id => C4::Context->userenv ? C4::Context->userenv->{'number'} : undef, } ) unless $line->status eq 'CANCELLED' || $line->amount != $line->amountoutstanding; } => status == NULL -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #6 from Marcel de Rooy --- [1] No item link. Shouldnt we save a link to item in the accounts when the article request contains one ? I added two article requests, one with an item and one without. Both were in the database without an item reference. Accountlines table has an itemnumber column. Showing up sometimes as a barcode on a form. -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #5 from Marcel de Rooy --- Tested. Looks like it would work when configured correctly. But found some cases that need to be addressed in some form. Adding a few comments for 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Marcel de Rooy changed: What|Removed |Added CC||n...@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #4 from Agustín Moyano --- Created attachment 120147 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120147=edit Bug 27946: Add article request fee feature This patch adds the ability to charge a fee for article scan requests. To test: 1. apply patches 2. updatedatabase 3. restart_all 4. enable ArticleRequest in preferences 5. grab a patron category and set a fee for article requests 6. grab a patron of that category (patron a) and another fron different category (patron b) 7. place an article request for both patrons SUCCESS => only for patron a, a warning is displayed saying a fee will be charged 8. check the account for both patrons SUCCESS => only for patron a there is a debit of type ARTICLE_REQUEST 9. cancel the article request for patron a SUCCESS => the debit for patron a gets canceled 10. repeat step 7 only for patron a 11. change article request fee for the patron a's category 12. repeat step 10 13. check account for patron a SUCCESS => there are 2 debits with different amount of typ ARTICLE_REQUEST 14. cancel only one article request SUCCESS => only the corresponding debit gets canceled 15. prove t/db_dependent/ArticleRequests.t -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #3 from Agustín Moyano --- Created attachment 120146 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120146=edit [DO NOT PUSH] Koha 27946: Update categories 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #2 from Agustín Moyano --- Created attachment 120145 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120145=edit Bug 27946: Add article_request_fee to categories table and ARTICLE_REQUEST account debit type -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 --- Comment #1 from Agustín Moyano --- Created attachment 120144 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120144=edit Bug 27946: Add tests -- 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 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Agustín Moyano changed: What|Removed |Added Status|NEW |Needs Signoff Patch complexity|--- |Small 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/
[Koha-bugs] [Bug 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Tomás Cohen Arazi changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |agustinmoy...@theke.io |ity.org | CC||tomasco...@gmail.com -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27946] Add a charge per article request to patron categories configuration
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27946 Nick Clemens changed: What|Removed |Added Change sponsored?|--- |Sponsored CC||m.de.r...@rijksmuseum.nl -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/