[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Assignee|bugzi...@warmlight.co.uk|koha-b...@lists.koha-commun ||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 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Caroline Cyr La Rose changed: What|Removed |Added CC||caroline.cyr-la-rose@inlibr ||o.com Keywords|Manual | -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Katrin Fischer changed: What|Removed |Added Text to go in the|This enhancement adds |This enhancement adds release notes|additional logging of |additional logging of |acquisition-related changes |acquisition-related changes |including: |including: |- Order line|- Order line |creation and cancellation |creation and cancellation |- |- |Invoice adjustment |Invoice adjustment |additions, amendments and |additions, amendments and |deletions |deletions |- Order line|- Order line |receipts against an invoice |receipts against an invoice |- Budget adjustments |- Budget adjustments |- Fund |- Fund |adjustments |adjustments |- Order release |- Order release |date |date (EDIFACT) | | |The name of the |The name of |system preference that |the system preference that |enables logging of |enables logging of |acquisition-related changes |acquisition-related changes |was changed from AcqLog to |was changed from AcqLog to |AcquisitionLog. |AcquisitionLog. | | |Note: |Note: |Acquisition logging was |Acquisition logging was |added in Koha 21.05.|added in Koha 21.05. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 David Nind changed: What|Removed |Added Text to go in the||This enhancement adds release notes||additional logging of ||acquisition-related changes ||including: ||- Order line ||creation and cancellation ||- ||Invoice adjustment ||additions, amendments and ||deletions ||- Order line ||receipts against an invoice ||- Budget adjustments ||- Fund ||adjustments ||- Order release ||date || ||The name of the ||system preference that ||enables logging of ||acquisition-related changes ||was changed from AcqLog to ||AcquisitionLog. || ||Note: ||Acquisition logging was ||added in Koha 21.05. CC||da...@davidnind.com Keywords|release-notes-needed|Manual -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Jonathan Druart changed: What|Removed |Added Keywords||release-notes-needed -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Kyle M Hall changed: What|Removed |Added CC||k...@bywatersolutions.com Status|Pushed to master|RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #65 from Jonathan Druart --- Pushed to master for 21.11, thanks to everybody involved! -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Jonathan Druart changed: What|Removed |Added Status|Passed QA |Pushed to master Version(s)||21.11.00 released in|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Jonathan Druart changed: What|Removed |Added Attachment #123390|0 |1 is obsolete|| --- Comment #64 from Jonathan Druart --- Created attachment 125119 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=125119=edit Bug 24190: (follow-up) Rename AcqLog As requested in comment #49, renamed uses of AcqLog to AcquisitionLog Signed-off-by: Nick Clemens JD amended patch: replace one missing occurrence in Budgets.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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #63 from Nick Clemens --- Slight rebase and small follow-up to record when fund is not changed upon receipt. While I would like to see fuller logging of objects these patches add more information and will help track acquisitions so I think further enhancements can be on follow up bugs. Passing QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #62 from Nick Clemens --- Created attachment 123391 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123391=edit Bug 24190: (QA follow-up) record unchanged bookfund and fix typo Signed-off-by: Nick Clemens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #61 from Nick Clemens --- Created attachment 123390 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123390=edit Bug 24190: (follow-up) Rename AcqLog As requested in comment #49, renamed uses of AcqLog to AcquisitionLog Signed-off-by: Nick Clemens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #60 from Nick Clemens --- Created attachment 123389 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123389=edit Bug 24190: (follow-up) Modify order of logging Basket creation involves two steps in Koha, creating the basket then modifying its header. We were logging these two steps in the wrong order. This commit fixes that. Signed-off-by: Nick Clemens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #59 from Nick Clemens --- Created attachment 123388 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123388=edit Bug 24190: (follow-up) Add tests In response to Jonathan's request for tests, I have now added additional tests for the methods in C4/Budgets.pm that do logging Signed-off-by: Nick Clemens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #58 from Nick Clemens --- Created attachment 123387 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123387=edit Bug 24190: (follow-up) Respond to QA feedback This commit makes changes in response to Jonathan's feedback in comment - Moved from using zero padded strings to store log data to a JSON object - Stopped storing formatted dates in logged data Signed-off-by: Nick Clemens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #57 from Nick Clemens --- Created attachment 123386 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123386=edit Bug 24190: (follow-up) Cancel order del item bug If an order is cancelled but the associated bib / items are unable to be removed, we go into error handling. We still need to log the cancellation though. So this fix moves the logging to just after the cancellation, so it will be logged regardless of the outcome with associated things. Signed-off-by: Maura Stephens Signed-off-by: Nick Clemens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Nick Clemens changed: What|Removed |Added Attachment #122040|0 |1 is obsolete|| Attachment #122041|0 |1 is obsolete|| Attachment #122042|0 |1 is obsolete|| Attachment #122043|0 |1 is obsolete|| Attachment #122044|0 |1 is obsolete|| Attachment #122045|0 |1 is obsolete|| --- Comment #56 from Nick Clemens --- Created attachment 123385 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123385=edit Bug 24190: Add acquisition logging Signed-off-by: Maura Stephens Signed-off-by: Nick Clemens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Nick Clemens changed: 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 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Martin Renvoize changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |n...@bywatersolutions.com |y.org | -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #116152|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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: 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 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #55 from Andrew Isherwood --- Created attachment 122045 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122045=edit Bug 24190: (Follow-up) Rename AcqLog As requested in comment #49, renamed uses of AcqLog to AcquisitionLog -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #116153|0 |1 is obsolete|| --- Comment #54 from Andrew Isherwood --- Created attachment 122044 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122044=edit Bug 24190: (follow-up) Modify order of logging Basket creation involves two steps in Koha, creating the basket then modifying its header. We were logging these two steps in the wrong order. This commit fixes 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #116151|0 |1 is obsolete|| --- Comment #53 from Andrew Isherwood --- Created attachment 122043 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122043=edit Bug 24190: (follow-up) Add tests In response to Jonathan's request for tests, I have now added additional tests for the methods in C4/Budgets.pm that do logging -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #116150|0 |1 is obsolete|| --- Comment #52 from Andrew Isherwood --- Created attachment 122042 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122042=edit Bug 24190: (follow-up) Respond to QA feedback This commit makes changes in response to Jonathan's feedback in comment - Moved from using zero padded strings to store log data to a JSON object - Stopped storing formatted dates in logged data -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #116149|0 |1 is obsolete|| --- Comment #51 from Andrew Isherwood --- Created attachment 122041 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122041=edit Bug 24190: (follow-up) Cancel order del item bug If an order is cancelled but the associated bib / items are unable to be removed, we go into error handling. We still need to log the cancellation though. So this fix moves the logging to just after the cancellation, so it will be logged regardless of the outcome with associated things. Signed-off-by: Maura Stephens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #116148|0 |1 is obsolete|| --- Comment #50 from Andrew Isherwood --- Created attachment 122040 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122040=edit Bug 24190: Add acquisition logging Signed-off-by: Maura Stephens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #49 from Jonathan Druart --- Pref renamed AcquisitionLog on bug 23971, run % perl -p -i -e 's#AcqLog#AcquisitionLog#' **/*.t **/*.pl **/*.pm **/*.sql **/*.perl **/*.pref To apply it here. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Nick Clemens changed: What|Removed |Added Status|Signed Off |Failed QA --- Comment #48 from Nick Clemens --- I still don't like that there are columns not tracked. If the full object cannot be logged, then I would like to see modifications that create no tracked change are not logged, some of these are confusing: - I can change a budget name, or lock/unlock the budget and a modification with no changes is logged - Changing the fund for an order logs a modification but not the change When receiving an order the bookfund is coming through blank. I am not a huge fan of the name 'bookfund' - we could be ordering anything here, I would stick with fund. It looks like this comes from the fact that not changing the fund on receipt sends the value "" Several of the reports caused ISEs for me - I tested by simply checking the action logs full contents Adding an order to a basket logs a basket modification, but I cannot see what changes This is a super cool feature, I just think it needs a bit more to be production ready -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #47 from Andrew Isherwood --- (In reply to Nick Clemens from comment #39) > I will retest the rest once this applies Hi Nick - thanks so much for your efforts on this one! :-) I've now rebased it (having first rebased the bug it depends on). Hopefully this time it will work! -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Status|Patch doesn't apply |Signed Off --- Comment #46 from Andrew Isherwood --- Revised test plan: Setup: 1. Apply patch and run database update if necessary 2. Enable AcqLog syspref Order line cancellations: 1. Go to "Aquisitions", find an open basket with at least one order 2. Cancel the order and confirm the cancellation 3. From the staff main page, go to "Reports > Create from SQL" 4. Name the report "Order line cancellations" 5. In the SQL box, enter the report labelled "Order line cancellations" in comment #26 above 6. Save the report, the run it, choosing a start and end date 7. TEST => Observe that the order cancellation you just carried out is listed Order line creations: 1. Go to "Aquisitions", find or create an open basket 2. Create a new order in the basket 3. From the staff main page, go to "Reports > Create from SQL" 4. Name the report "Order line creations" 5. In the SQL box, enter the report labelled "Order line creations" in comment #26 above 6. Save the report, the run it, choosing a start and end date 7. TEST => Observe that the order creation you just carried out is listed Invoice adjustment additions: 1. Go to "Aquisitions", find or create an invoice 2. Click "Add an adjustment" 3. Fill in the details of the adjustment, then click "Update adjustments" 4. From the staff main page, go to "Reports > Create from SQL" 5. Name the report "Invoice adjustment additions" 6. In the SQL box, enter the report labelled "Invoice adjustment additions" in comment #26 above 7. Save the report, the run it, choosing a start and end date 8. TEST => Observe that the adjustment creation you just carried out is listed Invoice adjustment amendments: 1. Go to "Aquisitions", find the invoice from the previous steps 2. Find the adjustment you added and modify it 3. Click "Update adjustments" 4. From the staff main page, go to "Reports > Create from SQL" 5. Name the report "Invoice adjustment amendments" 6. In the SQL box, enter the report labelled "Invoice adjustment amendments" in comment #26 above 7. Save the report, the run it, choosing a start and end date 8. TEST => Observe that the adjustment amendment you just carried out is listed Invoice adjustment deletions: 1. Go to "Aquisitions", find the invoice from the previous steps 2. Find the adjustment you added and delete it 3. From the staff main page, go to "Reports > Create from SQL" 4. Name the report "Invoice adjustment deletions" 5. In the SQL box, enter the report labelled "Invoice adjustment deletions" in comment #26 above 6. Save the report, the run it, choosing a start and end date 7. TEST => Observe that the adjustment deletion you just carried out is listed Order line receipts against an invoice: 1. Go to "Aquisitions", find an open invoice with unreceived items 2. Go to the "Receipt summary" page 3. Click "Receive" for an item 4. Fill in the data as appropriate, then click "Save" 5. From the staff main page, go to "Reports > Create from SQL" 6. Name the report "Order line receipts against an invoice" 7. In the SQL box, enter the report labelled "Order line receipts against an invoice" in comment #26 above 8. Save the report, the run it, choosing a start and end date 9. TEST => Observe that the receipt you just carried out is listed Budget adjustments: 1. Go to "Aquisitions", find a budget 2. Click "Actions" > "Edit" 3. Modify some properties of the budget 4. Click "Save" 5. From the staff main page, go to "Reports > Create from SQL" 6. Name the report "Budget adjustments" 7. In the SQL box, enter the report labelled "Budget adjustments" in comment #26 above 8. Save the report, the run it, choosing a start and end date 9. TEST => Observe that the adjustment you just carried out is listed, including details of the "before" and "after" values Fund adjustments: 1. Go to "Aquisitions", find a fund 2. Click "Actions" > "Edit" 3. Modify some properties of the fund 4. Click "Submit" 5. From the staff main page, go to "Reports > Create from SQL" 6. Name the report "Fund adjustments" 7. In the SQL box, enter the report labelled "Fund adjustments" in comment #26 above 8. Save the report, the run it, choosing a start and end date 9. TEST => Observe that the adjustment you just carried out is listed, including details of the "before" and "after" values Order release date (1): 1. Go to "Aquisitions", find an open basket 2. Close the basket, noting it's ID 3. From the staff main page, go to "Reports > Create from SQL" 4. Name the report "Order release date (1)" 5. In the SQL box, enter the report labelled "Order release date (1)" in comment #26 above 6. Save the report, the run it, choosing a start and end date and basket ID 7. TEST => Observe that the details of the closed basket are listed Order release
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #112514|0 |1 is obsolete|| --- Comment #45 from Andrew Isherwood --- Created attachment 116153 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116153=edit Bug 24190: (follow-up) Modify order of logging Basket creation involves two steps in Koha, creating the basket then modifying its header. We were logging these two steps in the wrong order. This commit fixes 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #112513|0 |1 is obsolete|| --- Comment #44 from Andrew Isherwood --- Created attachment 116152 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116152=edit Bug 24190: (follow-up) Fix erroneous tabs The QA script was reporting forbidden tabs in some of the lines in this bug, these have now been removed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #112512|0 |1 is obsolete|| --- Comment #43 from Andrew Isherwood --- Created attachment 116151 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116151=edit Bug 24190: (follow-up) Add tests In response to Jonathan's request for tests, I have now added additional tests for the methods in C4/Budgets.pm that do logging -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #112511|0 |1 is obsolete|| --- Comment #42 from Andrew Isherwood --- Created attachment 116150 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116150=edit Bug 24190: (follow-up) Respond to QA feedback This commit makes changes in response to Jonathan's feedback in comment - Moved from using zero padded strings to store log data to a JSON object - Stopped storing formatted dates in logged data -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #112510|0 |1 is obsolete|| --- Comment #41 from Andrew Isherwood --- Created attachment 116149 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116149=edit Bug 24190: (follow-up) Cancel order del item bug If an order is cancelled but the associated bib / items are unable to be removed, we go into error handling. We still need to log the cancellation though. So this fix moves the logging to just after the cancellation, so it will be logged regardless of the outcome with associated things. Signed-off-by: Maura Stephens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #112509|0 |1 is obsolete|| --- Comment #40 from Andrew Isherwood --- Created attachment 116148 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=116148=edit Bug 24190: Add acquisition logging Signed-off-by: Maura Stephens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Nick Clemens changed: What|Removed |Added Status|Failed QA |Patch doesn't apply --- Comment #39 from Nick Clemens --- (In reply to Andrew Isherwood from comment #38) > (In reply to Nick Clemens from comment #31) > > Hi Nick > > Thanks for the feedback > > > [..] I wonder why the scope > > of information is limited for each object > required, but I felt it was more prudent to take a step by step approach > rather than just logging everything. On bug 23971 it looks like the decision was to log the full object, I feel like the same approach applies here > > Adding a new basket records two lines: > This is actually reflecting what Koha does, it creates the basket then > modifies the header (see: OK > > Cancelling receipt of an order is not logged > This was not on our customers list of requirements, hence has not been added. OK I will retest the rest once this applies -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #111084|0 |1 is obsolete|| --- Comment #36 from Andrew Isherwood --- Created attachment 112513 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112513=edit Bug 24190: (follow-up) Fix erroneous tabs The QA script was reporting forbidden tabs in some of the lines in this bug, these have now been removed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #111083|0 |1 is obsolete|| --- Comment #35 from Andrew Isherwood --- Created attachment 112512 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112512=edit Bug 24190: (follow-up) Add tests In response to Jonathan's request for tests, I have now added additional tests for the methods in C4/Budgets.pm that do logging -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #37 from Andrew Isherwood --- Created attachment 112514 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112514=edit Bug 24190: (follow-up) Modify order of logging Basket creation involves two steps in Koha, creating the basket then modifying its header. We were logging these two steps in the wrong order. This commit fixes 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #38 from Andrew Isherwood --- (In reply to Nick Clemens from comment #31) Hi Nick Thanks for the feedback > [..] I wonder why the scope > of information is limited for each object > if I modify a field that isn't logged a line is > added to the log, but no info on what was changed is recorded, e.g.: > - update order notes, change statistics > - add user to budget, lock budget, make budget active, change name > - change fund name, change statistics > Is the size concern reason enough to avoid logging the object unblessed? The development was funded by a few customers who had very specific requirements for what they wanted logging, the patches address these requirements. As you know, acquisitions is a vast area and even relatively simple objects (for example, a fund) have many properties. We **could** just do what has been done in other areas and just dump the entire object into the log, but another key requirement for our customers was that they actually had to be able to create meaningful reports on the logged data. I have therefore been discriminating in what data is logged and how it is logged to enable them to do this. I hope that the approach I've taken will make it relatively trivial for others to extend this logging as and what required, but I felt it was more prudent to take a step by step approach rather than just logging everything. > Adding a new basket records two lines: > Create an acquisitions basket > Modify an acquisitions basket header This is actually reflecting what Koha does, it creates the basket then modifies the header (see: https://github.com/PTFS-Europe/koha/blob/master/C4/Acquisition.pm#L190-L207) however, I did notice that the logging of the basket creation was happening after the header had been modified, so I've corrected that. > Cancelling receipt of an order is not logged This was not on our customers list of requirements, hence has not been added. > Creating a budget is logged as modification I can't replicate this, budget creation isn't logged at all, for the reason given above. > Budget/funds record a change of amount because of decimal difference: > "budget_amount_new":"6.00" > "budget_amount_old":"6.00" I'm just logging what Koha is using, it feels like it's asking for trouble to add or remove precision for the purpose of logging, particularly when those values will be used for reporting. -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #111080|0 |1 is obsolete|| --- Comment #32 from Andrew Isherwood --- Created attachment 112509 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112509=edit Bug 24190: Add acquisition logging Signed-off-by: Maura Stephens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #111082|0 |1 is obsolete|| --- Comment #34 from Andrew Isherwood --- Created attachment 112511 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112511=edit Bug 24190: (follow-up) Respond to QA feedback This commit makes changes in response to Jonathan's feedback in comment - Moved from using zero padded strings to store log data to a JSON object - Stopped storing formatted dates in logged data -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #111081|0 |1 is obsolete|| --- Comment #33 from Andrew Isherwood --- Created attachment 112510 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112510=edit Bug 24190: (follow-up) Cancel order del item bug If an order is cancelled but the associated bib / items are unable to be removed, we go into error handling. We still need to log the cancellation though. So this fix moves the logging to just after the cancellation, so it will be logged regardless of the outcome with associated things. Signed-off-by: Maura Stephens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Nick Clemens changed: What|Removed |Added CC||n...@bywatersolutions.com Status|Signed Off |Failed QA --- Comment #31 from Nick Clemens --- Hi Andrew, Overall I like this, it adds useful information, but I wonder why the scope of information is limited for each object, it seems to create some issues for Budgets/Funds/Orders - if I modify a field that isn't logged a line is added to the log, but no info on what was changed is recorded, e.g.: - update order notes, change statistics - add user to budget, lock budget, make budget active, change name - change fund name, change statistics Is the size concern reason enough to avoid logging the object unblessed? Other notes: Adding a new basket records two lines: Create an acquisitions basket Modify an acquisitions basket header Cancelling receipt of an order is not logged Creating a budget is logged as modification Budget/funds record a change of amount because of decimal difference: "budget_amount_new":"6.00" "budget_amount_old":"6.00" -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #30 from Katrin Fischer --- Waiting for the final QA on bug 23971. -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Status|BLOCKED |Signed Off --- Comment #29 from Andrew Isherwood --- Bug 23971 is no longer FQA, so setting this back to Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Katrin Fischer changed: What|Removed |Added Status|Signed Off |BLOCKED --- Comment #28 from Katrin Fischer --- Currently blocked by dependent bug 23971 being Failed QA. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Status|Patch doesn't apply |Signed Off --- Comment #27 from Andrew Isherwood --- Revised test plan: Setup: 1. Apply patch and run database update if necessary Order line cancellations: 1. Go to "Aquisitions", find an open basket with at least one order 2. Cancel the order and confirm the cancellation 3. From the staff main page, go to "Reports > Create from SQL" 4. Name the report "Order line cancellations" 5. In the SQL box, enter the report labelled "Order line cancellations" in comment #26 above 6. Save the report, the run it, choosing a start and end date 7. TEST => Observe that the order cancellation you just carried out is listed Order line creations: 1. Go to "Aquisitions", find or create an open basket 2. Create a new order in the basket 3. From the staff main page, go to "Reports > Create from SQL" 4. Name the report "Order line creations" 5. In the SQL box, enter the report labelled "Order line creations" in comment #26 above 6. Save the report, the run it, choosing a start and end date 7. TEST => Observe that the order creation you just carried out is listed Invoice adjustment additions: 1. Go to "Aquisitions", find or create an invoice 2. Click "Add an adjustment" 3. Fill in the details of the adjustment, then click "Update adjustments" 4. From the staff main page, go to "Reports > Create from SQL" 5. Name the report "Invoice adjustment additions" 6. In the SQL box, enter the report labelled "Invoice adjustment additions" in comment #26 above 7. Save the report, the run it, choosing a start and end date 8. TEST => Observe that the adjustment creation you just carried out is listed Invoice adjustment amendments: 1. Go to "Aquisitions", find the invoice from the previous steps 2. Find the adjustment you added and modify it 3. Click "Update adjustments" 4. From the staff main page, go to "Reports > Create from SQL" 5. Name the report "Invoice adjustment amendments" 6. In the SQL box, enter the report labelled "Invoice adjustment amendments" in comment #26 above 7. Save the report, the run it, choosing a start and end date 8. TEST => Observe that the adjustment amendment you just carried out is listed Invoice adjustment deletions: 1. Go to "Aquisitions", find the invoice from the previous steps 2. Find the adjustment you added and delete it 3. From the staff main page, go to "Reports > Create from SQL" 4. Name the report "Invoice adjustment deletions" 5. In the SQL box, enter the report labelled "Invoice adjustment deletions" in comment #26 above 6. Save the report, the run it, choosing a start and end date 7. TEST => Observe that the adjustment deletion you just carried out is listed Order line receipts against an invoice: 1. Go to "Aquisitions", find an open invoice with unreceived items 2. Go to the "Receipt summary" page 3. Click "Receive" for an item 4. Fill in the data as appropriate, then click "Save" 5. From the staff main page, go to "Reports > Create from SQL" 6. Name the report "Order line receipts against an invoice" 7. In the SQL box, enter the report labelled "Order line receipts against an invoice" in comment #26 above 8. Save the report, the run it, choosing a start and end date 9. TEST => Observe that the receipt you just carried out is listed Budget adjustments: 1. Go to "Aquisitions", find a budget 2. Click "Actions" > "Edit" 3. Modify some properties of the budget 4. Click "Save" 5. From the staff main page, go to "Reports > Create from SQL" 6. Name the report "Budget adjustments" 7. In the SQL box, enter the report labelled "Budget adjustments" in comment #26 above 8. Save the report, the run it, choosing a start and end date 9. TEST => Observe that the adjustment you just carried out is listed, including details of the "before" and "after" values Fund adjustments: 1. Go to "Aquisitions", find a fund 2. Click "Actions" > "Edit" 3. Modify some properties of the fund 4. Click "Submit" 5. From the staff main page, go to "Reports > Create from SQL" 6. Name the report "Fund adjustments" 7. In the SQL box, enter the report labelled "Fund adjustments" in comment #26 above 8. Save the report, the run it, choosing a start and end date 9. TEST => Observe that the adjustment you just carried out is listed, including details of the "before" and "after" values Order release date (1): 1. Go to "Aquisitions", find an open basket 2. Close the basket, noting it's ID 3. From the staff main page, go to "Reports > Create from SQL" 4. Name the report "Order release date (1)" 5. In the SQL box, enter the report labelled "Order release date (1)" in comment #26 above 6. Save the report, the run it, choosing a start and end date and basket ID 7. TEST => Observe that the details of the closed basket are listed Order release date (2): 1. Release a
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #26 from Andrew Isherwood --- Revised reports: 1. Order line cancellations: SELECT o.ordernumber AS 'Order number', bi.title AS 'Title', bi.author AS 'Author', ba.basketname AS 'Basket', f.budget_name AS 'Fund', CONCAT(b.firstname, ' ', b.surname) AS 'Cancelled by', a.timestamp AS 'Cancelled timestamp' FROM action_logs a INNER JOIN borrowers b ON a.user = b.borrowernumber INNER JOIN aqorders o ON a.object = o.ordernumber INNER JOIN aqbasket ba ON o.basketno = ba.basketno LEFT JOIN aqbudgets f ON o.budget_id = f.budget_id LEFT JOIN biblio bi ON o.biblionumber = bi.biblionumber WHERE a.module='ACQUISITIONS' AND a.action = 'CANCEL_ORDER' AND date(a.timestamp) BETWEEN <> AND <> 2. Order line creations: - SELECT o.ordernumber AS 'Order number', ba.basketname AS 'Basket', bi.title AS 'Title', bi.author AS 'Author', f.budget_name AS 'Fund', CONCAT(b.firstname, ' ', b.surname) AS 'Created by', a.timestamp AS 'Created timestamp' FROM action_logs a INNER JOIN borrowers b ON a.user = b.borrowernumber INNER JOIN aqorders o ON a.object = o.ordernumber INNER JOIN aqbasket ba ON o.basketno = ba.basketno LEFT JOIN aqbudgets f ON o.budget_id = f.budget_id LEFT JOIN biblio bi ON o.biblionumber = bi.biblionumber WHERE a.module='ACQUISITIONS' AND a.action = 'CREATE_ORDER' AND date(a.timestamp) BETWEEN <> AND <> 3. Invoice adjustment additions: SELECT f.budget_name AS 'Fund', i.invoicenumber, FORMAT(JSON_UNQUOTE(JSON_EXTRACT(a.info, '$.adjustment')), 2) AS 'Amount', ia.note AS 'Note', CASE WHEN JSON_UNQUOTE(JSON_EXTRACT(a.info, '$.adjustment')) = 0 THEN 'No' ELSE 'Yes' END AS 'Encumber open', av.lib AS 'Reason', CONCAT(b.firstname, ' ', b.surname) AS 'Created by', a.timestamp AS 'Created timestamp' FROM action_logs a INNER JOIN borrowers b ON a.user = b.borrowernumber INNER JOIN aqinvoice_adjustments ia ON a.object = ia.adjustment_id INNER JOIN aqinvoices i ON i.invoiceid = ia.invoiceid LEFT JOIN aqbudgets f ON JSON_UNQUOTE(JSON_EXTRACT(a.info, '$.budget_id')) = f.budget_id LEFT JOIN authorised_values av ON av.category = 'ADJ_REASON' AND av.authorised_value = JSON_UNQUOTE(JSON_EXTRACT(a.info, '$.reason')) WHERE a.module='ACQUISITIONS' AND a.action = 'CREATE_INVOICE_ADJUSTMENT' AND date(a.timestamp) BETWEEN <> AND <> 4. Invoice adjustment amendments: - SELECT i.invoicenumber AS 'Invoice number', f_before.budget_name AS 'Fund before', f_after.budget_name AS 'Fund after', FORMAT(JSON_UNQUOTE(JSON_EXTRACT(a.info, '$.adjustment_old')), 2) AS 'Amount before', FORMAT(JSON_UNQUOTE(JSON_EXTRACT(a.info, '$.adjustment')), 2) AS 'Amount after', ia.note AS 'Note', CASE WHEN JSON_UNQUOTE(JSON_EXTRACT(a.info, '$.encumber_open_old')) = 0 THEN 'No' ELSE 'Yes' END AS 'Encumber open before', CASE WHEN JSON_UNQUOTE(JSON_EXTRACT(a.info, '$.encumber_open')) = 0 THEN 'No' ELSE 'Yes' END AS 'Encumber open after', av_before.lib AS 'Reason before', av_after.lib AS 'Reason after', CONCAT(b.firstname, ' ', b.surname) AS 'Created by', a.timestamp AS 'Created timestamp' FROM action_logs a INNER JOIN borrowers b ON a.user = b.borrowernumber INNER JOIN aqinvoice_adjustments ia ON a.object = ia.adjustment_id INNER JOIN aqinvoices i ON i.invoiceid = ia.invoiceid LEFT JOIN aqbudgets f_before ON JSON_UNQUOTE(JSON_EXTRACT(a.info, '$.budget_id_old')) = f_before.budget_id LEFT JOIN aqbudgets f_after ON JSON_UNQUOTE(JSON_EXTRACT(a.info, '$.budget_id')) = f_after.budget_id LEFT JOIN authorised_values av_before ON av_before.category = 'ADJ_REASON' AND av_before.authorised_value = JSON_UNQUOTE(JSON_EXTRACT(a.info, '$.reason_old')) LEFT JOIN authorised_values av_after ON av_after.category = 'ADJ_REASON' AND av_after.authorised_value = JSON_UNQUOTE(JSON_EXTRACT(a.info, '$.reason')) WHERE a.module='ACQUISITIONS' AND a.action = 'UPDATE_INVOICE_ADJUSTMENT' AND date(a.timestamp) BETWEEN <> AND <> 5. Invoice adjustment deletions: SELECT i.invoicenumber AS 'Invoice number', f.budget_name AS 'Fund', FORMAT(JSON_UNQUOTE(JSON_EXTRACT(a.info, '$.adjustment')), 2) AS 'Amount', CASE WHEN JSON_UNQUOTE(JSON_EXTRACT(a.info, '$.encumber_open')) = 0 THEN 'No' ELSE 'Yes' END AS 'Encumber open', av.lib AS 'Reason', CONCAT(b.firstname, ' ', b.surname) AS 'Deleted by', a.timestamp AS 'Deleted timestamp' FROM action_logs a INNER JOIN borrowers b ON a.user = b.borrowernumber INNER JOIN aqinvoices i ON i.invoiceid = JSON_UNQUOTE(JSON_EXTRACT(a.info, '$.invoiceid')) LEFT JOIN aqbudgets f ON JSON_UNQUOTE(JSON_EXTRACT(a.info, '$.budget_id')) = f.budget_id LEFT JOIN authorised_values av ON av.category = 'ADJ_REASON' AND av.authorised_value =
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #25 from Andrew Isherwood --- Created attachment 111084 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111084=edit Bug 24190: (follow-up) Fix erroneous tabs The QA script was reporting forbidden tabs in some of the lines in this bug, these have now been removed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #106098|0 |1 is obsolete|| --- Comment #21 from Andrew Isherwood --- Created attachment 111080 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111080=edit Bug 24190: Add acquisition logging Signed-off-by: Maura Stephens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #23 from Andrew Isherwood --- Created attachment 111082 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111082=edit Bug 24190: (follow-up) Respond to QA feedback This commit makes changes in response to Jonathan's feedback in comment - Moved from using zero padded strings to store log data to a JSON object - Stopped storing formatted dates in logged data -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #24 from Andrew Isherwood --- Created attachment 111083 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111083=edit Bug 24190: (follow-up) Add tests In response to Jonathan's request for tests, I have now added additional tests for the methods in C4/Budgets.pm that do logging -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #106099|0 |1 is obsolete|| --- Comment #22 from Andrew Isherwood --- Created attachment 111081 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111081=edit Bug 24190: (follow-up) Cancel order del item bug If an order is cancelled but the associated bib / items are unable to be removed, we go into error handling. We still need to log the cancellation though. So this fix moves the logging to just after the cancellation, so it will be logged regardless of the outcome with associated things. Signed-off-by: Maura Stephens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #20 from Andrew Isherwood --- I gather you've spoken with Martin, I'm going to proceed with storing in JSON -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #19 from Andrew Isherwood --- Hi Jonathan - After creating the patches to use JSON rather than zero padded strings, I remembered the reason I went for zero padded strings rather than another format. Since the content of the logged data needs to be used in SQL reports, I need it to be in a format that can be parsed using SQL. JSON is the obvious choice, however JSON support doesn't exist on the older versions of MySQL that Koha supports (e.g. 5.5), so that is not a viable option. The only method I could come up with for storing arbitrary data was in padded strings, then using the SUBSTRING operator to extract the data in a report. > And why 10? rrp_tax_excluded is decimal(28,6) for instance OK, I'd not appreciated that. I can increase the padding to a much larger value if we're allowing values with precision of 28 to be stored... I wanted the padding to be sufficient, 10 seemed to be enough, but apparently not. -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #18 from Andrew Isherwood --- (In reply to Jonathan Druart from comment #14) Hi Jonathan > 1. You really should avoid indentation spaces outside of the block you are > editing It was a while ago, but this was almost certainly my editor. I'm very conscious of irrelevant changes, so I'm sure I wouldn't have done this deliberately. I'll keep a closer eye on it in future. > 2. Why are you using the "%010d" format? Since the data I was storing was pretty small, storing an entire Dumper output felt over the top. However, I have since discovered that a precedent has been set for storing action_log data in JSON, so I will move to using this as it is infinitely more readable than zero padded strings. > 3. I don't think we should store formatted dates. OK, fair enough, I'll change that > 4. You should also consider adding tests OK, I'll look at doing that. I don't think they'll be anything beyond "does this function call logaction with the correct payload", but it's better than nothing -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #17 from Katrin Fischer --- Please also check the comments! -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Patch doesn't apply --- Comment #16 from Katrin Fischer --- I just noticed that there is a dependent bug, that needs to be tested first, but worth noting, that this does no longer apply on top: Apply? [(y)es, (n)o, (i)nteractive] y Applying: Bug 24190: Add acquisition logging error: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/en/modules/tools/viewlog.tt). error: could not build fake ancestor Patch failed at 0001 Bug 24190: Add acquisition logging The copy of the patch that failed is found in: .git/rebase-apply/patch When you have resolved this problem run "git bz apply --continue". If you would prefer to skip this patch, instead run "git bz apply --skip". To restore the original branch and stop patching run "git bz apply --abort". Patch left in /tmp/Bug-24190-Add-acquisition-logging-mCG2UX.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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Jonathan Druart changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=13325 -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #15 from Jonathan Druart --- 4. You should also consider adding 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190
Jonathan Druart changed:
What|Removed |Added
CC||jonathan.dru...@bugs.koha-c
||ommunity.org
--- Comment #14 from Jonathan Druart
---
Some comments:
1. You really should avoid indentation spaces outside of the block you are
editing. It makes rebase a nightmare for you and for those who will have to
rebase on top of it.
2. Why are you using the "%010d" format?
Why are not you dumping (using Data::Dumper) the whole $budget?
I have not tested but
+sprintf("%010d", $del_adj->invoiceid) .
+sprintf("%010d", $del_adj->budget_id) .
That will produce a concat of 0101 (for invoiceid=1 and
budget_id=1
Is it really what we expect?
And why 10? rrp_tax_excluded is decimal(28,6) for instance
3. +eval { output_pref({ dt => dt_from_string(
$input->param('budget_period_startdate') ), dateformat => 'iso', dateonly => 1
} ); } .
I don't think we should store formatted dates.
--
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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 rkui...@roundrocktexas.gov changed: What|Removed |Added Blocks||25916 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25916 [Bug 25916] Create function and UI to modify budgets and funds. -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Martin Renvoize changed: What|Removed |Added Status|Needs Signoff |Signed Off --- Comment #13 from Martin Renvoize --- Tested and signed off on PTFS-E sandboxes by Maura Stephens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Martin Renvoize changed: What|Removed |Added Attachment #106097|0 |1 is obsolete|| --- Comment #12 from Martin Renvoize --- Created attachment 106099 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106099=edit Bug 24190: (follow-up) Cancel order del item bug If an order is cancelled but the associated bib / items are unable to be removed, we go into error handling. We still need to log the cancellation though. So this fix moves the logging to just after the cancellation, so it will be logged regardless of the outcome with associated things. Signed-off-by: Maura Stephens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Martin Renvoize changed: What|Removed |Added Attachment #106096|0 |1 is obsolete|| --- Comment #11 from Martin Renvoize --- Created attachment 106098 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106098=edit Bug 24190: Add acquisition logging Signed-off-by: Maura Stephens -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Martin Renvoize changed: What|Removed |Added Attachment #105082|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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Martin Renvoize changed: What|Removed |Added Attachment #105083|0 |1 is obsolete|| --- Comment #10 from Martin Renvoize --- Created attachment 106097 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106097=edit Bug 24190: (follow-up) Cancel order del item bug If an order is cancelled but the associated bib / items are unable to be removed, we go into error handling. We still need to log the cancellation though. So this fix moves the logging to just after the cancellation, so it will be logged regardless of the outcome with associated things. -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #9 from Martin Renvoize --- Created attachment 106096 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106096=edit Bug 24190: Add acquisition logging -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #100243|0 |1 is obsolete|| --- Comment #7 from Andrew Isherwood --- Created attachment 105082 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=105082=edit Bug 24190 - Add acquisition logging -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #8 from Andrew Isherwood --- Created attachment 105083 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=105083=edit Bug 24190: (follow-up) Cancel order del item bug If an order is cancelled but the associated bib / items are unable to be removed, we go into error handling. We still need to log the cancellation though. So this fix moves the logging to just after the cancellation, so it will be logged regardless of the outcome with associated things. -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Martin Renvoize changed: What|Removed |Added Version|18.11 |master -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Martin Renvoize changed: What|Removed |Added Version|18.11 |master 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Martin Renvoize changed: What|Removed |Added Version|master |18.11 -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #96369|0 |1 is obsolete|| --- Comment #6 from Andrew Isherwood --- Created attachment 100243 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100243=edit Bug 24190 - Add acquisition logging -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Status|ASSIGNED|Needs Signoff --- Comment #5 from Andrew Isherwood --- Test plan: Setup: 1. Apply patch and run database update if necessary Order line cancellations: 1. Go to "Aquisitions", find an open basket with at least one order 2. Cancel the order and confirm the cancellation 3. From the staff main page, go to "Reports > Create from SQL" 4. Name the report "Order line cancellations" 5. In the SQL box, enter the report labelled "Order line cancellations" in comment #4 above 6. Save the report, the run it, choosing a start and end date 7. TEST => Observe that the order cancellation you just carried out is listed Order line creations: 1. Go to "Aquisitions", find or create an open basket 2. Create a new order in the basket 3. From the staff main page, go to "Reports > Create from SQL" 4. Name the report "Order line creations" 5. In the SQL box, enter the report labelled "Order line creations" in comment #4 above 6. Save the report, the run it, choosing a start and end date 7. TEST => Observe that the order creation you just carried out is listed Invoice adjustment additions: 1. Go to "Aquisitions", find or create an invoice 2. Click "Add an adjustment" 3. Fill in the details of the adjustment, then click "Update adjustments" 4. From the staff main page, go to "Reports > Create from SQL" 5. Name the report "Invoice adjustment additions" 6. In the SQL box, enter the report labelled "Invoice adjustment additions" in comment #4 above 7. Save the report, the run it, choosing a start and end date 8. TEST => Observe that the adjustment creation you just carried out is listed Invoice adjustment amendments: 1. Go to "Aquisitions", find the invoice from the previous steps 2. Find the adjustment you added and modify it 3. Click "Update adjustments" 4. From the staff main page, go to "Reports > Create from SQL" 5. Name the report "Invoice adjustment amendments" 6. In the SQL box, enter the report labelled "Invoice adjustment amendments" in comment #4 above 7. Save the report, the run it, choosing a start and end date 8. TEST => Observe that the adjustment amendment you just carried out is listed Invoice adjustment deletions: 1. Go to "Aquisitions", find the invoice from the previous steps 2. Find the adjustment you added and delete it 3. From the staff main page, go to "Reports > Create from SQL" 4. Name the report "Invoice adjustment deletions" 5. In the SQL box, enter the report labelled "Invoice adjustment deletions" in comment #4 above 6. Save the report, the run it, choosing a start and end date 7. TEST => Observe that the adjustment deletion you just carried out is listed Order line receipts against an invoice: 1. Go to "Aquisitions", find an open invoice with unreceived items 2. Go to the "Receipt summary" page 3. Click "Receive" for an item 4. Fill in the data as appropriate, then click "Save" 5. From the staff main page, go to "Reports > Create from SQL" 6. Name the report "Order line receipts against an invoice" 7. In the SQL box, enter the report labelled "Order line receipts against an invoice" in comment #4 above 8. Save the report, the run it, choosing a start and end date 9. TEST => Observe that the receipt you just carried out is listed Budget adjustments: 1. Go to "Aquisitions", find a budget 2. Click "Actions" > "Edit" 3. Modify some properties of the budget 4. Click "Save" 5. From the staff main page, go to "Reports > Create from SQL" 6. Name the report "Budget adjustments" 7. In the SQL box, enter the report labelled "Budget adjustments" in comment #4 above 8. Save the report, the run it, choosing a start and end date 9. TEST => Observe that the adjustment you just carried out is listed, including details of the "before" and "after" values Fund adjustments: 1. Go to "Aquisitions", find a fund 2. Click "Actions" > "Edit" 3. Modify some properties of the fund 4. Click "Submit" 5. From the staff main page, go to "Reports > Create from SQL" 6. Name the report "Fund adjustments" 7. In the SQL box, enter the report labelled "Fund adjustments" in comment #4 above 8. Save the report, the run it, choosing a start and end date 9. TEST => Observe that the adjustment you just carried out is listed, including details of the "before" and "after" values Order release date (1): 1. Go to "Aquisitions", find an open basket 2. Close the basket, noting it's ID 3. From the staff main page, go to "Reports > Create from SQL" 4. Name the report "Order release date (1)" 5. In the SQL box, enter the report labelled "Order release date (1)" in comment #4 above 6. Save the report, the run it, choosing a start and end date and basket ID 7. TEST => Observe that the details of the closed basket are listed Order release date (2): 1. Release a basket via an
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #4 from Andrew Isherwood --- 1. Order line cancellations: SELECT o.ordernumber AS 'Order number', bi.title AS 'Title', bi.author AS 'Author', ba.basketname AS 'Basket', f.budget_name AS 'Fund', CONCAT(b.firstname, ' ', b.surname) AS 'Cancelled by', a.timestamp AS 'Cancelled timestamp' FROM action_logs a INNER JOIN borrowers b ON a.user = b.borrowernumber INNER JOIN aqorders o ON a.object = o.ordernumber INNER JOIN aqbasket ba ON o.basketno = ba.basketno LEFT JOIN aqbudgets f ON o.budget_id = f.budget_id LEFT JOIN biblio bi ON o.biblionumber = bi.biblionumber WHERE a.module='ACQUISITIONS' AND a.action = 'CANCEL_ORDER' AND date(a.timestamp) BETWEEN <> AND <> Order line creations: - SELECT o.ordernumber AS 'Order number', ba.basketname AS 'Basket', bi.title AS 'Title', bi.author AS 'Author', f.budget_name AS 'Fund', CONCAT(b.firstname, ' ', b.surname) AS 'Created by', a.timestamp AS 'Created timestamp' FROM action_logs a INNER JOIN borrowers b ON a.user = b.borrowernumber INNER JOIN aqorders o ON a.object = o.ordernumber INNER JOIN aqbasket ba ON o.basketno = ba.basketno LEFT JOIN aqbudgets f ON o.budget_id = f.budget_id LEFT JOIN biblio bi ON o.biblionumber = bi.biblionumber WHERE a.module='ACQUISITIONS' AND a.action = 'CREATE_ORDER' AND date(a.timestamp) BETWEEN <> AND <> 3. Invoice adjustment additions: SELECT f.budget_name AS 'Fund', i.invoicenumber, FORMAT(SUBSTRING(a.info, 1, 10), 2) AS 'Amount', ia.note AS 'Note', CASE WHEN SUBSTRING(a.info, 101, 10) = 0 THEN 'No' ELSE 'Yes' END AS 'Encumber open', av.lib AS 'Reason', CONCAT(b.firstname, ' ', b.surname) AS 'Created by', a.timestamp AS 'Created timestamp' FROM action_logs a INNER JOIN borrowers b ON a.user = b.borrowernumber INNER JOIN aqinvoice_adjustments ia ON a.object = ia.adjustment_id INNER JOIN aqinvoices i ON i.invoiceid = ia.invoiceid LEFT JOIN aqbudgets f ON SUBSTRING(a.info, 91, 10) = f.budget_id LEFT JOIN authorised_values av ON av.category = 'ADJ_REASON' AND av.authorised_value = TRIM(LEADING ' ' FROM SUBSTRING(a.info, 11, 80)) WHERE a.module='ACQUISITIONS' AND a.action = 'CREATE_INVOICE_ADJUSTMENT' AND date(a.timestamp) BETWEEN <> AND <> 4. Invoice adjustment amendments: - SELECT i.invoicenumber AS 'Invoice number', f_before.budget_name AS 'Fund before', f_after.budget_name AS 'Fund after', FORMAT(SUBSTRING(a.info, 131, 10), 2) AS 'Amount before', FORMAT(SUBSTRING(a.info, 1, 10), 2) AS 'Amount after', ia.note AS 'Note', CASE WHEN SUBSTRING(a.info, 121, 10) = 0 THEN 'No' ELSE 'Yes' END AS 'Encumber open before', CASE WHEN SUBSTRING(a.info, 101, 10) = 0 THEN 'No' ELSE 'Yes' END AS 'Encumber open after', av_before.lib AS 'Reason before', av_after.lib AS 'Reason after', CONCAT(b.firstname, ' ', b.surname) AS 'Created by', a.timestamp AS 'Created timestamp' FROM action_logs a INNER JOIN borrowers b ON a.user = b.borrowernumber INNER JOIN aqinvoice_adjustments ia ON a.object = ia.adjustment_id INNER JOIN aqinvoices i ON i.invoiceid = ia.invoiceid LEFT JOIN aqbudgets f_before ON SUBSTRING(a.info, 111, 10) = f_before.budget_id LEFT JOIN aqbudgets f_after ON SUBSTRING(a.info, 91, 10) = f_after.budget_id LEFT JOIN authorised_values av_before ON av_before.category = 'ADJ_REASON' AND av_before.authorised_value = TRIM(LEADING ' ' FROM SUBSTRING(a.info, 141, 80)) LEFT JOIN authorised_values av_after ON av_after.category = 'ADJ_REASON' AND av_after.authorised_value = TRIM(LEADING ' ' FROM SUBSTRING(a.info, 11, 80)) WHERE a.module='ACQUISITIONS' AND a.action = 'UPDATE_INVOICE_ADJUSTMENT' AND date(a.timestamp) BETWEEN <> AND <> 5. Invoice adjustment deletions: SELECT i.invoicenumber AS 'Invoice number', f.budget_name AS 'Fund', FORMAT(SUBSTRING(a.info, 31, 10), 2) AS 'Amount', CASE WHEN SUBSTRING(a.info, 21, 10) = 0 THEN 'No' ELSE 'Yes' END AS 'Encumber open', av.lib AS 'Reason', CONCAT(b.firstname, ' ', b.surname) AS 'Deleted by', a.timestamp AS 'Deleted timestamp', SUBSTRING(a.info, 31, 10) FROM action_logs a INNER JOIN borrowers b ON a.user = b.borrowernumber INNER JOIN aqinvoices i ON i.invoiceid = SUBSTRING(a.info, 1, 10) LEFT JOIN aqbudgets f ON SUBSTRING(a.info, 11, 10) = f.budget_id LEFT JOIN authorised_values av ON av.category = 'ADJ_REASON' AND av.authorised_value = TRIM(LEADING ' ' FROM SUBSTRING(a.info, 41, 80)) WHERE a.module='ACQUISITIONS' AND a.action = 'DELETE_INVOICE_ADJUSTMENT' AND date(a.timestamp) BETWEEN <> AND <> 6. Order line receipts against an invoice: - SELECT o.ordernumber AS 'Order number',
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Attachment #96364|0 |1 is obsolete|| --- Comment #2 from Andrew Isherwood --- Created attachment 96369 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=96369=edit Bug 24190 - Add acquisition logging -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #3 from Andrew Isherwood --- Associated reports: 1. Order line cancellations SELECT o.ordernumber AS 'Order number', ba.basketname AS 'Basket', f.budget_name AS 'Fund', CONCAT(b.firstname, ' ', b.surname) AS 'Cancelled by', a.timestamp AS 'Cancelled timestamp' FROM action_logs a INNER JOIN borrowers b ON a.user = b.borrowernumber INNER JOIN aqorders o ON a.object = o.ordernumber INNER JOIN aqbasket ba ON o.basketno = ba.basketno LEFT JOIN aqbudgets f ON o.budget_id = f.budget_id WHERE a.module='ACQUISITIONS' AND a.action = 'CANCEL_ORDER' AND date(a.timestamp) BETWEEN <> AND <> 2. Order line creations SELECT o.ordernumber AS 'Order number', ba.basketname AS 'Basket', f.budget_name AS 'Fund', CONCAT(b.firstname, ' ', b.surname) AS 'Created by', a.timestamp AS 'Created timestamp' FROM action_logs a INNER JOIN borrowers b ON a.user = b.borrowernumber INNER JOIN aqorders o ON a.object = o.ordernumber INNER JOIN aqbasket ba ON o.basketno = ba.basketno LEFT JOIN aqbudgets f ON o.budget_id = f.budget_id WHERE a.module='ACQUISITIONS' AND a.action = 'CREATE_ORDER' AND date(a.timestamp) BETWEEN <> AND <> 3. Invoice adjustment additions SELECT f.budget_name AS 'Fund', i.invoicenumber, FORMAT(SUBSTRING(a.info, 1, 10), 2) AS 'Amount', ia.note AS 'Note', CASE WHEN SUBSTRING(a.info, 101, 10) = 0 THEN 'No' ELSE 'Yes' END AS 'Encumber open', av.lib AS 'Reason', CONCAT(b.firstname, ' ', b.surname) AS 'Created by', a.timestamp AS 'Created timestamp' FROM action_logs a INNER JOIN borrowers b ON a.user = b.borrowernumber INNER JOIN aqinvoice_adjustments ia ON a.object = ia.adjustment_id INNER JOIN aqinvoices i ON i.invoiceid = ia.invoiceid LEFT JOIN aqbudgets f ON SUBSTRING(a.info, 91, 10) = f.budget_id LEFT JOIN authorised_values av ON av.category = 'ADJ_REASON' AND av.authorised_value = TRIM(LEADING ' ' FROM SUBSTRING(a.info, 11, 80)) WHERE a.module='ACQUISITIONS' AND a.action = 'CREATE_INVOICE_ADJUSTMENT' AND date(a.timestamp) BETWEEN <> AND <> 4. Invoice adjustment amendments SELECT i.invoicenumber AS 'Invoice number', f_before.budget_name AS 'Fund before', f_after.budget_name AS 'Fund after', FORMAT(SUBSTRING(a.info, 131, 10), 2) AS 'Amount before', FORMAT(SUBSTRING(a.info, 1, 10), 2) AS 'Amount after', ia.note AS 'Note', CASE WHEN SUBSTRING(a.info, 121, 10) = 0 THEN 'No' ELSE 'Yes' END AS 'Encumber open before', CASE WHEN SUBSTRING(a.info, 101, 10) = 0 THEN 'No' ELSE 'Yes' END AS 'Encumber open after', av_before.lib AS 'Reason before', av_after.lib AS 'Reason after', CONCAT(b.firstname, ' ', b.surname) AS 'Created by', a.timestamp AS 'Created timestamp' FROM action_logs a INNER JOIN borrowers b ON a.user = b.borrowernumber INNER JOIN aqinvoice_adjustments ia ON a.object = ia.adjustment_id INNER JOIN aqinvoices i ON i.invoiceid = ia.invoiceid LEFT JOIN aqbudgets f_before ON SUBSTRING(a.info, 111, 10) = f_before.budget_id LEFT JOIN aqbudgets f_after ON SUBSTRING(a.info, 91, 10) = f_after.budget_id LEFT JOIN authorised_values av_before ON av_before.category = 'ADJ_REASON' AND av_before.authorised_value = TRIM(LEADING ' ' FROM SUBSTRING(a.info, 141, 80)) LEFT JOIN authorised_values av_after ON av_after.category = 'ADJ_REASON' AND av_after.authorised_value = TRIM(LEADING ' ' FROM SUBSTRING(a.info, 11, 80)) WHERE a.module='ACQUISITIONS' AND a.action = 'UPDATE_INVOICE_ADJUSTMENT' AND date(a.timestamp) BETWEEN <> AND <> 5. Invoice adjustment deletions SELECT i.invoicenumber AS 'Invoice number', f.budget_name AS 'Fund', FORMAT(SUBSTRING(a.info, 31, 10), 2) AS 'Amount', CASE WHEN SUBSTRING(a.info, 21, 10) = 0 THEN 'No' ELSE 'Yes' END AS 'Encumber open', av.lib AS 'Reason', CONCAT(b.firstname, ' ', b.surname) AS 'Deleted by', a.timestamp AS 'Deleted timestamp', SUBSTRING(a.info, 31, 10) FROM action_logs a INNER JOIN borrowers b ON a.user = b.borrowernumber INNER JOIN aqinvoices i ON i.invoiceid = SUBSTRING(a.info, 1, 10) LEFT JOIN aqbudgets f ON SUBSTRING(a.info, 11, 10) = f.budget_id LEFT JOIN authorised_values av ON av.category = 'ADJ_REASON' AND av.authorised_value = TRIM(LEADING ' ' FROM SUBSTRING(a.info, 41, 80)) WHERE a.module='ACQUISITIONS' AND a.action = 'DELETE_INVOICE_ADJUSTMENT' AND date(a.timestamp) BETWEEN <> AND <> 6. Order line receipts against an invoice SELECT o.ordernumber AS 'Order number', FORMAT(SUBSTRING(a.info, 1, 10), 0) AS 'Quantity received', f.budget_name AS 'Fund', FORMAT(SUBSTRING(a.info, 21, 10), 2) AS 'Tax rate', FORMAT(SUBSTRING(a.info, 31, 10), 2) AS 'Replacement price', FORMAT(SUBSTRING(a.info, 41, 10), 2) AS 'Actual cost', CONCAT(b.firstname, ' ', b.surname) AS 'Received by', a.timestamp AS 'Received timestamp' FROM action_logs a INNER JOIN
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 --- Comment #1 from Andrew Isherwood --- Created attachment 96364 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=96364=edit Bug 24190 - Add acquisition logging -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Marjorie Barry-Vila changed: What|Removed |Added CC||marjorie.barry-vila@collect ||o.ca -- 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 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |andrew.isherwood@ptfs-europ |ity.org |e.com Change sponsored?|--- |Sponsored Status|NEW |ASSIGNED -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 24190] Add additional Acquisition logging
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 Andrew Isherwood changed: What|Removed |Added Depends on||23971 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 [Bug 23971] Add logging for basket related actions -- 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/
