[Koha-bugs] [Bug 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Martin Renvoize changed: What|Removed |Added Blocks|14825 | CC||martin.renvoize@ptfs-europe ||.com Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14825 [Bug 14825] Accounts Rewrite Omnibus -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Barton Chittendenchanged: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=5216 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Kyle M Hallchanged: What|Removed |Added Blocks|17135 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17135 [Bug 17135] Fine for the previous overdue may get overwritten by the next one -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Kyle M Hallchanged: What|Removed |Added Blocks||17135 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17135 [Bug 17135] Fine for the previous overdue may get overwritten by the next one -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Barton Chittendenchanged: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=17135 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #39 from Jacek Ablewicz--- Had a look at 16.05.x branch on git to be sure, and yep, 16.05 is definitely affected by Bug 17135 too. Would be good to fix this issue ASAP, before angry mob of librarians shows up (with the pitchforks ;) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #38 from Katrin Fischer--- You are correct - I got my versions confused :( -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #37 from Jacek Ablewicz--- (In reply to Katrin Fischer from comment #36) > glad this didn't go into stable yet. I was under the impression that it did (comment #30), stable == 16.05 right now? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #36 from Katrin Fischer--- Please also take a look at bug 17135 - glad this didn't go into stable yet. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Jacek Ablewiczchanged: What|Removed |Added Blocks||17135 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17135 [Bug 17135] Fine for the previous overdue may get overwritten by the next one -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Julian Mauricechanged: What|Removed |Added Blocks||16496 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16496 [Bug 16496] accountlines.issue_id not set when new overdue is processed -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Katrin Fischerchanged: What|Removed |Added Blocks||16378 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16378 [Bug 16378] Overdues.pm: Can't call method "store" without a package or object reference -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Jacek Ablewiczchanged: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=16378 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #35 from Jacek Ablewicz--- One pretty nasty regression got introduced by this bug, see Bug 16378 for details. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Julian Mauricechanged: What|Removed |Added Resolution|--- |FIXED Status|Pushed to Master|RESOLVED Severity|minor |enhancement --- Comment #34 from Julian Maurice --- (In reply to Katrin Fischer from comment #33) > Hm, feels a bit big for a maintenance release, might be worth the wait. That was my thinking too, thanks for the confirmation -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #33 from Katrin Fischer--- Hm, feels a bit big for a maintenance release, might be worth the wait. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #32 from Kyle M Hall--- *** Bug 15310 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 http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Julian Mauricechanged: What|Removed |Added CC||julian.maur...@biblibre.com --- Comment #31 from Julian Maurice --- It looks like an enhancement rather than a bug. Should it be backported to 3.22 ? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Brendan Gallagherchanged: What|Removed |Added CC||bren...@bywatersolutions.co ||m Status|Passed QA |Pushed to Master --- Comment #30 from Brendan Gallagher --- Pushed to Master - Should be in the May 2016. Thanks -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Jonathan Druartchanged: What|Removed |Added Attachment #47378|0 |1 is obsolete|| Attachment #47379|0 |1 is obsolete|| Attachment #47380|0 |1 is obsolete|| Attachment #47601|0 |1 is obsolete|| Attachment #47810|0 |1 is obsolete|| --- Comment #24 from Jonathan Druart --- Created attachment 48468 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48468=edit Bug 15675 - Add issue_id column to accountlines and use it for updating fines Right now, fines are updated based on the fine description. There are a number of areas where this can go wrong ( date or time format changing, title being modified, etc ). Now that issues has a unique identifier, we should use that for selection and updating of fines. Test Plan: 1) Apply this patch 2) Test creating and updating fines via fines.pl and checking in overdue items. No changes should be noted. 3) prove t/db_dependent/Circulation.t Signed-off-by: Marc Véron Signed-off-by: Mirko Tietgen -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #29 from Jonathan Druart--- Created attachment 48473 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48473=edit Bug 15675: Fix conflict with bug 15446 (type vs _type) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #25 from Jonathan Druart--- Created attachment 48469 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48469=edit Bug 15675 - Update Schema file Signed-off-by: Mirko Tietgen -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #28 from Jonathan Druart--- Created attachment 48472 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48472=edit Bug 15675 [QA Followup] - Switch from NOT IN to LEFT JOIN -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #27 from Jonathan Druart--- Created attachment 48471 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48471=edit Bug 15675 [QA Followup] - Close out accruing fines that are not really accruing, add issue_id to accruing fines -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #26 from Jonathan Druart--- Created attachment 48470 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48470=edit Bug 15675 - Fix some missed tab characters, remove old FIXMEs Signed-off-by: Marc Véron Signed-off-by: Mirko Tietgen -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Jonathan Druartchanged: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Kyle M Hallchanged: What|Removed |Added Blocks||15895 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15895 [Bug 15895] Add Koha::Account module, replace recordpayment with Koha::Account::pay -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #23 from Kyle M Hall--- Created attachment 47810 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47810=edit Bug 15675 [QA Followup] - Switch from NOT IN to LEFT JOIN -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #22 from Jonathan Druart--- The "NOT IN" will be terribly slow on this table. Try using a left join instead. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Kyle M Hallchanged: What|Removed |Added Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #20 from Kyle M Hall--- Created attachment 47600 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47600=edit Bug 15675 [QA Followup] - Close out accruing fines that are not really accruing, add issue_id to accruing fines -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Kyle M Hallchanged: What|Removed |Added Attachment #47600|0 |1 is obsolete|| --- Comment #21 from Kyle M Hall --- Created attachment 47601 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47601=edit Bug 15675 [QA Followup] - Close out accruing fines that are not really accruing, add issue_id to accruing fines -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Jacek Ablewiczchanged: What|Removed |Added CC||a...@biblos.pk.edu.pl -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Kyle M Hallchanged: What|Removed |Added Blocks||15732 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15732 [Bug 15732] Remove accountno column from accountlines -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Jonathan Druartchanged: What|Removed |Added CC||jonathan.dru...@bugs.koha-c ||ommunity.org --- Comment #14 from Jonathan Druart --- Could you please detail what do we need this new column? 1/ There is no fk on it 2/ can be null but C4::Overdues::UpdateFine requires it 3/ The prototype of UpdateFine is weird, you have to pass issue_id, itemnumber and borrowernumber. issue_id should be enough. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #15 from Jonathan Druart--- (In reply to Katrin Fischer from comment #10) > Hi Kyle, > after thinking some about this, how are we going to deal with multiple fine > lines for the same issue? If the item is renewed the due date in issues will > change and a new fine line should be generated. The old due date information > will then be lost. I wonder if we should store the due date that the fine > was generated for additionally to the link in accountlines - in a separate > date column. Katrin, since there is no constraint on this new column, I don't understand your comment. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #16 from Katrin Fischer--- Hi Joubu, not really a database structure problem I was worried about, having the issue_id certainly makes sense and no FK seems fine as the id can be in issues or old_issues. I was wondering as this is no 1:1 but a 1:n if the calcuation will be correct. If it will still add another separate line in accountlines if an item goes overdue - renew - overdue - renew... etc. If the due date is removed from the description or no longer used, this information will get lost etc. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #17 from Kyle M Hall--- (In reply to Jonathan Druart from comment #14) > Could you please detail what do we need this new column? > 1/ There is no fk on it Right, as Katrin wrote, since the issue_id can be in issues or old_issues, we can't give it a key constraint. > 2/ can be null but C4::Overdues::UpdateFine requires it I suppose we *could* make it not null, but then how do we deal with adding an issue_id to all the already existing account lines? We can definitely update the ones that are currently accruing. Those are the only ones where it really matters. > 3/ The prototype of UpdateFine is weird, you have to pass issue_id, > itemnumber and borrowernumber. issue_id should be enough. Agreed, I should change that. I was just trying to make the smallest change possible. It would make sense for it to just take issue_id ( or an issue object if possible ). If you'd like to see that change just set the status to failed qa and let me know! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #18 from Kyle M Hall--- (In reply to Katrin Fischer from comment #16) > I was wondering as this is no 1:1 but a 1:n if the calcuation will be > correct. If it will still add another separate line in accountlines if an > item goes overdue - renew - overdue - renew... etc. If the due date is > removed from the description or no longer used, this information will get > lost etc. You are right, it is a one to many join from issues to accountlines. One item may go overdue multiple times. However, there should only ever be one *accruing* fine for a given issue at a time, so it shouldn't be a problem. The current query already deals with that in that the FU fine will be the first selected and matched by issue_id. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Kyle M Hallchanged: What|Removed |Added Status|Signed Off |Failed QA --- Comment #19 from Kyle M Hall --- Per conversation with Katrin, we should make sure to clean up any multiple FU fines that haven't been closed out in a db update for this. Setting to failed qa until I get that done. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Marc Véronchanged: What|Removed |Added Attachment #47313|0 |1 is obsolete|| --- Comment #6 from Marc Véron --- Created attachment 47373 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47373=edit Bug 15675 - Add issue_id column to accountlines and use it for updating fines Right now, fines are updated based on the fine description. There are a number of areas where this can go wrong ( date or time format changing, title being modified, etc ). Now that issues has a unique identifier, we should use that for selection and updating of fines. Test Plan: 1) Apply this patch 2) Test creating and updating fines via fines.pl and checking in overdue items. No changes should be noted. 3) prove t/db_dependent/Circulation.t Signed-off-by: Marc Véron -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Marc Véronchanged: What|Removed |Added Status|Needs Signoff |Signed Off CC||ve...@veron.ch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Marc Véronchanged: What|Removed |Added Attachment #47316|0 |1 is obsolete|| --- Comment #8 from Marc Véron --- Created attachment 47375 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47375=edit Bug 15675 - Fix some missed tab characters, remove old FIXMEs Signed-off-by: Marc Véron -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Mirko Tietgenchanged: What|Removed |Added Attachment #47375|1 |0 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Marc Véronchanged: What|Removed |Added Attachment #47314|0 |1 is obsolete|| --- Comment #7 from Marc Véron --- Created attachment 47374 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47374=edit Bug 15675 - Update Schema file -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Katrin Fischerchanged: What|Removed |Added CC||katrin.fisc...@bsz-bw.de --- Comment #10 from Katrin Fischer --- Hi Kyle, after thinking some about this, how are we going to deal with multiple fine lines for the same issue? If the item is renewed the due date in issues will change and a new fine line should be generated. The old due date information will then be lost. I wonder if we should store the due date that the fine was generated for additionally to the link in accountlines - in a separate date column. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Mirko Tietgenchanged: What|Removed |Added Attachment #47376|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 http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Mirko Tietgenchanged: What|Removed |Added Attachment #47375|0 |1 is obsolete|| --- Comment #13 from Mirko Tietgen --- Created attachment 47380 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47380=edit Bug 15675 - Fix some missed tab characters, remove old FIXMEs Signed-off-by: Marc Véron Signed-off-by: Mirko Tietgen -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Mirko Tietgenchanged: What|Removed |Added Attachment #47375|0 |1 is obsolete|| --- Comment #9 from Mirko Tietgen --- Created attachment 47376 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47376=edit Bug 15675 - Fix some missed tab characters, remove old FIXMEs Signed-off-by: Mirko Tietgen -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Mirko Tietgenchanged: What|Removed |Added Attachment #47374|0 |1 is obsolete|| --- Comment #12 from Mirko Tietgen --- Created attachment 47379 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47379=edit Bug 15675 - Update Schema file Signed-off-by: Mirko Tietgen -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Mirko Tietgenchanged: What|Removed |Added Attachment #47373|0 |1 is obsolete|| --- Comment #11 from Mirko Tietgen --- Created attachment 47378 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47378=edit Bug 15675 - Add issue_id column to accountlines and use it for updating fines Right now, fines are updated based on the fine description. There are a number of areas where this can go wrong ( date or time format changing, title being modified, etc ). Now that issues has a unique identifier, we should use that for selection and updating of fines. Test Plan: 1) Apply this patch 2) Test creating and updating fines via fines.pl and checking in overdue items. No changes should be noted. 3) prove t/db_dependent/Circulation.t Signed-off-by: Marc Véron Signed-off-by: Mirko Tietgen -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Kyle M Hallchanged: 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 http://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 15675] Add issue_id column to accountlines and use it for updating fines
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Kyle M Hallchanged: What|Removed |Added Severity|enhancement |minor -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Kyle M Hallchanged: What|Removed |Added Assignee|koha-b...@lists.koha-commun |k...@bywatersolutions.com |ity.org | -- 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 http://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 15675] Add issue_id column to accountlines and use it for updating fines
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Kyle M Hallchanged: What|Removed |Added CC||bar...@bywatersolutions.com --- Comment #1 from Kyle M Hall --- *** Bug 15334 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 http://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 15675] Add issue_id column to accountlines and use it for updating fines
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Kyle M Hallchanged: 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 http://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 15675] Add issue_id column to accountlines and use it for updating fines
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #3 from Kyle M Hall--- Created attachment 47314 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47314=edit Bug 15675 - Update Schema file -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #2 from Kyle M Hall--- Created attachment 47313 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47313=edit Bug 15675 - Add issue_id column to accountlines and use it for updating fines Right now, fines are updated based on the fine description. There are a number of areas where this can go wrong ( date or time format changing, title being modified, etc ). Now that issues has a unique identifier, we should use that for selection and updating of fines. Test Plan: 1) Apply this patch 2) Test creating and updating fines via fines.pl and checking in overdue items. No changes should be noted. 3) prove t/db_dependent/Circulation.t -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 --- Comment #4 from Kyle M Hall--- Created attachment 47315 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47315=edit Bug 15675 - Fix some missed tab characters -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Kyle M Hallchanged: What|Removed |Added Attachment #47315|0 |1 is obsolete|| --- Comment #5 from Kyle M Hall --- Created attachment 47316 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47316=edit Bug 15675 - Fix some missed tab characters, remove old FIXMEs -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Kyle M Hallchanged: What|Removed |Added Blocks||14825 Referenced Bugs: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14825 [Bug 14825] Accounts Rewrite Omnibus -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Nicole C. Engardchanged: What|Removed |Added CC||neng...@gmail.com, ||nic...@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Mirko Tietgenchanged: What|Removed |Added CC||mi...@abunchofthings.net -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Josef Moravecchanged: What|Removed |Added CC||josef.mora...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 15675] Add issue_id column to accountlines and use it for updating fines
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15675 Barton Chittendenchanged: What|Removed |Added Status|Needs Signoff |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 http://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/