[Bug 23971] New: Add logging for basket relation actions
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Bug ID: 23971 Summary: Add logging for basket relation actions Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Acquisitions Assignee: koha-bugs@lists.koha-community.org Reporter: andrew.isherwood@ptfs-europe.com QA Contact: testopia@bugs.koha-community.org Acquisitions currently creates no action_logs. Some of our customers are requesting the addition of logs. The first stage of this work is to add the following logging: 1. Who created a basket and when Basket number, Basket name, Creation date, Vendor name, Creator name 2. Who edited a basket and when Basket number, Basket name, Edited date, Vendor name, Editor name 3. Who approved a basket and when Basket number, Basket name, Approval date, Vendor name, Who approved 4. Who closed a basket and when Basket number, Basket name, Closed date, Vendor name, Who closed (all should include EDI order information) This bug is related to Bug 13325, which requests logging of fund related actions -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |andrew.isherwood@ptfs-europ |ity.org |e.com Status|NEW |ASSIGNED Change sponsored?|--- |Sponsored -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |22794 CC| |katrin.fischer@bsz-bw.de Summary|Add logging for basket |Add logging for basket |relation actions |related actions Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22794 [Bug 22794] Omnibus: Improve and extend logging in Koha -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #1 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 95285 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95285&action=edit Bug 23971: Add Acq action logging This patch adds logging for the following Acq actions: - Basket creation - Basket editing - Basket approval (via EDI) - Basket closure -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #95285|0 |1 is obsolete| | --- Comment #2 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 95286 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95286&action=edit Bug 23971: Add Acq action logging This patch adds logging for the following Acq actions: - Basket creation - Basket editing - Basket approval (via EDI) - Basket closure -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff --- Comment #3 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Test plan: - Apply the patch - Create a basket => TEST: Observe that an Acquisitions action log has been created containing: => Basket number => Basket name => Creation date => Vendor name => Creator name - Modify a basket => TEST: Observe that an Acquisitions action log has been created containing: => Basket number => Basket name => Edited date => Vendor name => Editor name - Approve a basket via EDI => TEST: Observe that an Acquisitions action log has been created containing: => Basket number => Basket name => Approval date => Vendor name => Approver name - Close a basket => TEST: Observe that an Acquisitions action log has been created containing: => Basket number => Basket name => Closed date => Vendor name => Closer name -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Séverine Queune <severine.queune@bulac.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |severine.queune@bulac.fr --- Comment #4 from Séverine Queune <severine.queune@bulac.fr> --- I did not manage to test the EDI part, it's a feature that I know who to use. Except that point, the 3 others work as described, even if I'm always surprised by the random order of the informations in actions_logs, but it's certainly out of scope of this patch. I think I could be good if libraries could disable this log, like it's possible for the other modules using the "Logging" sysprefs. I would probably add the reopening of a basket in the log, for, in my library, this is THE action out of workflow and I'm always quite annoy not knowing who did it. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #5 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I think Severine is right - logging the reopening of a basket would be really helpful. Andrew, could this be added to this patch set? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #6 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I think the missing pref could be considered a "Failed QA" as we should stick with existing patterns and it makes sense here. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |22794 Depends on|22794 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22794 [Bug 22794] Omnibus: Improve and extend logging in Koha -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #95286|0 |1 is obsolete| | --- Comment #7 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 95885 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95885&action=edit Bug 23971: Add Acq action logging This patch adds logging for the following Acq actions: - Basket creation - Basket editing - Basket approval (via EDI) - Basket closure -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff --- Comment #8 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- The nature of the logging has now changed according to customer requirements. I am logging less, and it is necessary now to use table joins in reports in order to gain access to the more detailed information, this will make the logging done here much more usable and flexible. Katrin and Séverine - I agree that basket reopening would be a useful thing to log, but it is outside of the scope of the requirements for this bug. There are many Acq actions that could be logged, I'm hoping these will be added in time. Katrin - I have put the logging behind an AcqLog syspref, thanks for highlighting this. Revised test plan: - Apply the patch => TEST: Observe there is a new "AcqLog" syspref - Enable the AcqLog syspref - Create a basket => TEST: Observe that an Acquisitions action log has been created containing: => Basket number - Modify a basket => TEST: Observe that an Acquisitions action log has been created containing: => Basket number => Borrowernumber of the user who modified the basket, this is stored in "info" as a left padded 10 digit number which can be joined on to the borrowers table - Approve a basket via EDI => TEST: Observe that an Acquisitions action log has been created containing: => Basket number => Borrowernumber of the user who approved the basket, this is stored in "info" as a left padded 10 digit number which can be joined on to the borrowers table - Close a basket => TEST: Observe that an Acquisitions action log has been created containing: => Basket number => Borrowernumber of the user who closed the basket, this is stored in "info" as a left padded 10 digit number which can be joined on to the borrowers table -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #9 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 95888 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=95888&action=edit Bug 23971 - Add logging for basket related actions This patch adds logging for the following Acq actions: - Basket creation - Basket editing - Basket approval (via EDI) - Basket closure -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #95885|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #10 from Séverine Queune <severine.queune@bulac.fr> --- (In reply to Andrew Isherwood from comment #8) > Katrin and Séverine - I agree that basket reopening would be a useful thing > to log, but it is outside of the scope of the requirements for this bug. > There are many Acq actions that could be logged, I'm hoping these will be > added in time. I will open a new bug to add this action. > Revised test plan: > > - Modify a basket > => TEST: Observe that an Acquisitions action log has been created containing: > => Basket number > => Borrowernumber of the user who modified the basket, this is stored in > "info" as a left padded 10 digit number which can be joined on to the > borrowers table 3 things about this MODIFY_BASKET log : - as the borrowernumber is saved in the 'user' field, why duplicating it in the 'info' field ? - in this 'info' field, we usually have the details of the action made by the user. To me, the information displays for now in not really usefull. - I only get log when I modify the "Managing library" but not when I update the managers of the basket, not when I edit the basket. Is that the expected behaviour ? > - Approve a basket via EDI > => TEST: Observe that an Acquisitions action log has been created containing: > => Basket number > => Borrowernumber of the user who approved the basket, this is stored in > "info" as a left padded 10 digit number which can be joined on to the > borrowers table I tried several EDI settings but it still doesn't work so one mode ime, I didn't test this part (would be happy if someone could help !). -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #11 from Katrin Fischer <katrin.fischer@bsz-bw.de> ---
Katrin and Séverine - I agree that basket reopening would be a useful thing to log, but it is outside of the scope of the requirements for this bug. There are many Acq actions that could be logged, I'm hoping these will be added in time.
I am sorry... but I'd like to at least try to explain my point of view a bit more. This bug states to add logging for basket related actions and of all the basket related actions one is missing. I don't feel this is out of scope at all. I've thought about why this is bothering me so much. I think it comes down to this: One of the biggest challenges we face is to develop Koha to be consistent and intuitive, to work as one piece of software, for everyone using it. When I have a feeling that something will cause question marks, and have me say: 'I know that doesn't make sense' to a library later, I feel I have to point it out.
Katrin - I have put the logging behind an AcqLog syspref, thanks for highlighting this.
Thank you. I am wondering if we should make the pref description a bit more specific to not lead to false expectations for now. Easy to change the text when coverage is extended beyond baskets. Switching to FQA because of Severine's comment#10 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #12 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 96063 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=96063&action=edit Bug 23961: (follow-up) Remove data duplication In response to Séverine observations in comment #10, this patch removes the duplicate logging of the borrowernumber -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #13 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- (In reply to Séverine Queune from comment #10)
3 things about this MODIFY_BASKET log : - as the borrowernumber is saved in the 'user' field, why duplicating it in the 'info' field ?
Thanks for point this out, I have attached a patch that removes this duplication.
- in this 'info' field, we usually have the details of the action made by the user. To me, the information displays for now in not really usefull.
The idea is that we're logging the minimal amount of information in the action logs but, through a report, it would be possible to obtain additional information about the related objects as required, such as basket, borrower etc. This logging is very much a starting point, I'll be elaborating on this point in response to Katrin's message.
- I only get log when I modify the "Managing library" but not when I update the managers of the basket, not when I edit the basket. Is that the expected behaviour ?
Again, this logging is a starting point
- Approve a basket via EDI => TEST: Observe that an Acquisitions action log has been created containing: => Basket number => Borrowernumber of the user who approved the basket, this is stored in "info" as a left padded 10 digit number which can be joined on to the borrowers table I tried several EDI settings but it still doesn't work so one mode ime, I didn't test this part (would be happy if someone could help !).
Thanks, I'll do some testing and fix any problems. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #14 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- (In reply to Katrin Fischer from comment #11)
I am sorry... but I'd like to at least try to explain my point of view a bit more. This bug states to add logging for basket related actions and of all the basket related actions one is missing. I don't feel this is out of scope at all. I've thought about why this is bothering me so much. I think it comes down to this: One of the biggest challenges we face is to develop Koha to be consistent and intuitive, to work as one piece of software, for everyone using it. When I have a feeling that something will cause question marks, and have me say: 'I know that doesn't make sense' to a library later, I feel I have to point it out.
Hi Katrin Thanks for your comments :) Firstly, I *completely* understand what you're saying and agree with everything. The challenge I face is that we have very specific requirements from the customer who is sponsoring this work and the patches that I have created address those requirements. So my comment about additional logging being out of scope meant that it was out of scope of those requirements. You're right, it is not out of scope of basket logging. There are many actions that can be performed on a basket, Séverine highlighted this by pointing out that logging did not occur when a basket manager is changed. Our requirement was to add logging of basket modification in the context of modifying the basket contents. Ideally, this bug would log every possible action on a basket, but that would significantly increase development time. I feel that this work is a foundation which will facilitate the addition of further logging. That said, I am about to create a patch to add basket reopening and basket header modification logging since those were specifically mentioned by yourself and Séverine. It's also worth me pointing out that I am about to embark on a further Acq logging development that is being sponsored by another customer, this will add substantial logging to other areas of Acq and will hopefully partly address some of your concerns. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #15 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to Andrew Isherwood from comment #14)
(In reply to Katrin Fischer from comment #11)
I am sorry... but I'd like to at least try to explain my point of view a bit more. This bug states to add logging for basket related actions and of all the basket related actions one is missing. I don't feel this is out of scope at all. I've thought about why this is bothering me so much. I think it comes down to this: One of the biggest challenges we face is to develop Koha to be consistent and intuitive, to work as one piece of software, for everyone using it. When I have a feeling that something will cause question marks, and have me say: 'I know that doesn't make sense' to a library later, I feel I have to point it out.
Hi Katrin
Thanks for your comments :) Firstly, I *completely* understand what you're saying and agree with everything. The challenge I face is that we have very specific requirements from the customer who is sponsoring this work and the patches that I have created address those requirements. So my comment about additional logging being out of scope meant that it was out of scope of those requirements. You're right, it is not out of scope of basket logging.
There are many actions that can be performed on a basket, Séverine highlighted this by pointing out that logging did not occur when a basket manager is changed. Our requirement was to add logging of basket modification in the context of modifying the basket contents. Ideally, this bug would log every possible action on a basket, but that would significantly increase development time. I feel that this work is a foundation which will facilitate the addition of further logging.
That said, I am about to create a patch to add basket reopening and basket header modification logging since those were specifically mentioned by yourself and Séverine.
It's also worth me pointing out that I am about to embark on a further Acq logging development that is being sponsored by another customer, this will add substantial logging to other areas of Acq and will hopefully partly address some of your concerns.
Thx Andrew for not getting this wrong most of all :) Also for considering an additon - you are right, Severine made a good point about the basket users. I am not going to block this and secretly hope the path of logging might lead you in direction of funds and budgetes - I know this was often wished for. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #16 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- (In reply to Katrin Fischer from comment #15)
[..] I am not going to block this and secretly hope the path of logging might lead you in direction of funds and budgetes - I know this was often wished for.
Well, you will hopefully be pleased. These are the requirements for the next work I am doing: 1. Order line cancellations This would show order line details including who made the cancellation and when the cancellation was made 2. Order line creations This would show order line details including who created the order line, when it was created, the fund, and amount 3. Invoice adjustment additions This would show the adjustment details, who added the adjustment, when it was added, the fund, and amount 4. Invoice adjustment amendments This would show the adjustment details, who amended the adjustment, when it was amended, the fund, and amount 5. Invoice adjustment deletions This would show the adjustment details, who deleted the adjustment, when it was deleted, the fund, and amount 6. Order line receipts against an invoice This would show the order line details, who received the order line, when it was received, the fund, and amount 7. Budget Adjustments This would show amounts added or removed from a budget (excluding spent), who, when, and amount 8. Fund adjustments This is essentially a bank statement. It would show additions and subtractions to a fund (excluding spent), who, when, amount, previous total, new total 9. Order release date (1) This would show individual order lines from an EDI message, when they were sent, who sent it, fund, and amount 10. Order release date (2) This would show individual order lines from a basket, when it was closed, who closed it, fund, and amount -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #17 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 96064 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=96064&action=edit Bug 23971: (follow-up) Add additional basket logging This patch adds additional basket logging as discussed in comment #14 - Basket reopening - Basket users (manager) modification - Basket header modification -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff --- Comment #18 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- I can't find any problems with the EDI approval basket closure. I'm going to set this back to "Needs Signoff" as I think I've addressed the additional requirements. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |24190 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190 [Bug 24190] Add additional Acquisition logging -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Marjorie Barry-Vila <marjorie.barry-vila@collecto.ca> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |marjorie.barry-vila@collect | |o.ca -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #95888|0 |1 is obsolete| | --- Comment #19 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 100239 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100239&action=edit Bug 23971 - Add logging for basket related actions This patch adds logging for the following Acq actions: - Basket creation - Basket editing - Basket approval (via EDI) - Basket closure -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #20 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 100240 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100240&action=edit Bug 23971: (follow-up) Remove data duplication In response to Séverine observations in comment #10, this patch removes the duplicate logging of the borrowernumber -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #96064|0 |1 is obsolete| | --- Comment #21 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 100241 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=100241&action=edit Bug 23971: (follow-up) Add additional basket logging This patch adds additional basket logging as discussed in comment #14 - Basket reopening - Basket users (manager) modification - Basket header modification -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #96063|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Michal Denar <black23@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |black23@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Holly <hc@interleaf.ie> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA CC| |hc@interleaf.ie --- Comment #22 from Holly <hc@interleaf.ie> --- I tested this patch but the following information is not appearing in the Log Viewer. - Basket reopening - Basket users (manager) modification - Basket header modification Other actions are appearing in the Acquisitions log. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #23 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 103911 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103911&action=edit Bug 23971: Add logging for basket related actions This patch adds logging for the following Acq actions: - Basket creation - Basket editing - Basket approval (via EDI) - Basket closure -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100240|0 |1 is obsolete| | --- Comment #24 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 103912 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103912&action=edit Bug 23971: (follow-up) Remove data duplication In response to Séverine observations in comment #10, this patch removes the duplicate logging of the borrowernumber -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100241|0 |1 is obsolete| | --- Comment #25 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 103913 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103913&action=edit Bug 23971: (follow-up) Add additional basket logging This patch adds additional basket logging as discussed in comment #14 - Basket reopening - Basket users (manager) modification - Basket header modification -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off --- Comment #26 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Seems there was a problem with applying this bug, which the latest patches attached by Martin should fix. Hopefully this will now pass QA. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@ptfs-europe | |.com Status|Signed Off |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #100239|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 ByWater Sandboxes <bws.sandboxes@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103913|0 |1 is obsolete| | --- Comment #27 from ByWater Sandboxes <bws.sandboxes@gmail.com> --- Created attachment 103945 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=103945&action=edit Bug 23971: (follow-up) Add additional basket logging This patch adds additional basket logging as discussed in comment #14 - Basket reopening - Basket users (manager) modification - Basket header modification Signed-off-by: Holly <hc@interleaf.ie> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Holly <hc@interleaf.ie> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 rkuiper@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.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org --- Comment #28 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Some comments from bug 24190 comment 14 (&15) apply here. Additionally I don't think it's a good idea to pass the flag to CloseBasket. 577 # Close the basket, passing a flag indicating that this action 578 # originated from an approval 579 CloseBasket($b, 1); Cannot we imagine that Koha::EDI deals with this specific situation? (UPDATE the action_log with the desire action) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #29 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- 2 things to note, but I think not blocker: - When a basket is created, a "MODIFY_BASKET_HEADER" is logged at the same time - Deleting a basket is not logged yet - The info column is not logged here, so the information known is the basket affected and the users involved, but no information about the change itself. I think this is worth knowing, but could be easily extended later. I see that the question from Jonathan in comment#28 remained unanswered - can you please check, so we can move this forward? Testing looks good so far. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #30 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 108962 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=108962&action=edit Bug 23971: (QA follow-up) Fix punctuation, doubled up return and syspref order -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #31 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- (In reply to Jonathan Druart from comment #28) Hi Jonathan
Some comments from bug 24190 comment 14 (&15) apply here.
I will review those comments when I get to that bug and respond there, any decisions made there I will also apply here where appropriate.
Cannot we imagine that Koha::EDI deals with this specific situation? (UPDATE the action_log with the desire action)
Sorry, I'm not 100% understanding. Are you suggesting that I drop the flag in the call to CloseBasket from Koha/EDI.pm (and consequently also remove it from the CloseBasket definition). And, instead, cause EDI::process_quote to update the action_log entry that was created in CloseBasket? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103911|0 |1 is obsolete| | --- Comment #32 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 111300 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111300&action=edit Bug 23971: Add logging for basket related actions This patch adds logging for the following Acq actions: - Basket creation - Basket editing - Basket approval (via EDI) - Basket closure -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103912|0 |1 is obsolete| | --- Comment #33 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 111301 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111301&action=edit Bug 23971: (follow-up) Remove data duplication In response to Séverine observations in comment #10, this patch removes the duplicate logging of the borrowernumber -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #103945|0 |1 is obsolete| | --- Comment #34 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 111302 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111302&action=edit Bug 23971: (follow-up) Add additional basket logging This patch adds additional basket logging as discussed in comment #14 - Basket reopening - Basket users (manager) modification - Basket header modification Signed-off-by: Holly <hc@interleaf.ie> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #108962|0 |1 is obsolete| | --- Comment #35 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 111303 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111303&action=edit Bug 23971: (QA follow-up) Fix punctuation, doubled up return and syspref order -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #36 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 111304 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111304&action=edit Bug 23971: (follow-up) Add unit tests This commit adds unit tests for the logging done in Acquisitions.pm -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #37 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- (In reply to Jonathan Druart from comment #28)
Hi Jonathan
Some comments from bug 24190 comment 14 (&15) apply here.
I will review those comments when I get to that bug and respond there, any decisions made there I will also apply here where appropriate.
Hi Jonathan - I've now added unit tests for the logging, as per your comments in bug 24190 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #38 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to Jonathan Druart from comment #28)
Some comments from bug 24190 comment 14 (&15) apply here.
Additionally I don't think it's a good idea to pass the flag to CloseBasket.
577 # Close the basket, passing a flag indicating that this action 578 # originated from an approval 579 CloseBasket($b, 1);
Cannot we imagine that Koha::EDI deals with this specific situation? (UPDATE the action_log with the desire action)
Hi Andrew, could you answer the question above about EDI please? Then should this be reset to Signed off or NSO? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off --- Comment #39 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- (In reply to Katrin Fischer from comment #38)
Hi Andrew, could you answer the question above about EDI please?
Hi Katrin - I can't answer it until Jonathan clarifies what he means, as I requested in comment #31
Then should this be reset to Signed off or NSO?
Well, it's signed off, so I guess "Signed off" -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #40 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Andrew Isherwood from comment #31)
Cannot we imagine that Koha::EDI deals with this specific situation? (UPDATE the action_log with the desire action)
Sorry, I'm not 100% understanding. Are you suggesting that I drop the flag in the call to CloseBasket from Koha/EDI.pm (and consequently also remove it from the CloseBasket definition). And, instead, cause EDI::process_quote to update the action_log entry that was created in CloseBasket?
Yes, it's what I meant! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #41 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Hi Jonathan OK, I've now made the change you suggested. We no longer pass anything to CloseBasket indicating whether the closure is tied to an EDI approval. I now log the closure and the approval separately, since they are actually distinct things. Revised test plan: - Apply the patch => TEST: Observe there is a new "AcqLog" syspref - Enable the AcqLog syspref - Create a basket => TEST: Observe that an Acquisitions "Create an acquisitions basket" action log has been created containing: => Basket number in "Object" column => Borrowernumber of staff member in "Librarian" column => TEST: Observe that an Acquisitions "Modify an acquisitions basket header" action log has been created containing: => Basket number in "Object" column => Borrowernumber of staff member in "Librarian" column - Modify a basket => TEST: Observe that an Acquisitions "Modify an acquisitions basket header" action log has been created containing: => Basket number in "Object" column => Borrowernumber of staff member in "Librarian" column - Approve a basket via EDI => TEST: Observe that an Acquisitions "Close an acquisitions basket" action log has been created containing: => Basket number in "Object" column => Borrowernumber of staff member in "Librarian" column => TEST: Observe that an Acquisitions "Approve an acquisitions basket" action log has been created containing: => Basket number in "Object" column => Borrowernumber of staff member in "Librarian" column - Close a basket => TEST: Observe that an Acquisitions "Close an acquisitions basket" action log has been created containing: => Basket number in "Object" column => Borrowernumber of staff member in "Librarian" column -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #111300|0 |1 is obsolete| | --- Comment #42 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 111955 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111955&action=edit Bug 23971: Add logging for basket related actions This patch adds logging for the following Acq actions: - Basket creation - Basket editing - Basket approval (via EDI) - Basket closure -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #111301|0 |1 is obsolete| | --- Comment #43 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 111956 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111956&action=edit Bug 23971: (follow-up) Remove data duplication In response to Séverine observations in comment #10, this patch removes the duplicate logging of the borrowernumber -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #111302|0 |1 is obsolete| | --- Comment #44 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 111957 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111957&action=edit Bug 23971: (follow-up) Add additional basket logging This patch adds additional basket logging as discussed in comment #14 - Basket reopening - Basket users (manager) modification - Basket header modification Signed-off-by: Holly <hc@interleaf.ie> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #111303|0 |1 is obsolete| | --- Comment #45 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 111958 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111958&action=edit Bug 23971: (QA follow-up) Fix punctuation, doubled up return and syspref order -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #111304|0 |1 is obsolete| | --- Comment #46 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 111959 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111959&action=edit Bug 23971: (follow-up) Add unit tests This commit adds unit tests for the logging done in Acquisitions.pm -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #47 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 111960 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111960&action=edit Bug 23971: (follow-up) Respond to feedback As requested by Jonathan in commit #28: - We now no longer pass a flag to CloseBasket indicating if the closure was due to an EDI basket approval, we log the closure and approval as separate events. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tomascohen@gmail.com QA Contact|testopia@bugs.koha-communit |tomascohen@gmail.com |y.org | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=26582 --- Comment #48 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Hi, it all looks ok. Things to look at: - Bug 26582 is PQA, should we rebase this one on top of it? I can help (it means moving the tests to the relevant file, etc). This is mainly a question for the RM. - I don't see tests for Koha::EDI related to the added APPROVE_BASKET action. They should be in t/db_dependent/Koha/EDI.t There's a minor rebase issue (tests count). -- You are receiving this mail because: You are watching all bug changes.
- Bug 26582 is PQA, should we rebase this one on top of it? I can help (it means moving the tests to the relevant file, etc). This is mainly a question for the RM. I think there's more to it than that. Looking at my code, currently the logging is embedded within C4::Acquisision::CloseBasket. So I'd need to move that to Koha::Acquisition::Basket::close. This isn't a problem, I just wanted to highlight it in case it would be problematic. Having logging as a side-effect in this way feels slightly weird, but it seems like it's less potentially error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #49 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- (In reply to Tomás Cohen Arazi from comment #48) prone than having to explicitly log at all the places where Koha::Acquisition::Basket::close is called. So, I think I'd have to move the logging as described above and modify the unit tests accordingly.
- I don't see tests for Koha::EDI related to the added APPROVE_BASKET action. They should be in t/db_dependent/Koha/EDI.t
As discussed on Slack, to be able to test this very, very small bit of functionality (is the approval of a basket logged) we'd have to implement a large amount of EDI testing infrastructure that doesn't currently exist, this seems out of scope to me. I'm not going to do anything more on this until we're all happy with how to move forward :-) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #111955|0 |1 is obsolete| | --- Comment #50 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 112277 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112277&action=edit Bug 23971: Add logging for basket related actions This patch adds logging for the following Acq actions: - Basket creation - Basket editing - Basket approval (via EDI) - Basket closure -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #111956|0 |1 is obsolete| | --- Comment #51 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 112278 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112278&action=edit Bug 23971: (follow-up) Remove data duplication In response to Séverine observations in comment #10, this patch removes the duplicate logging of the borrowernumber -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #111957|0 |1 is obsolete| | --- Comment #52 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 112279 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112279&action=edit Bug 23971: (follow-up) Add additional basket logging This patch adds additional basket logging as discussed in comment #14 - Basket reopening - Basket users (manager) modification - Basket header modification Signed-off-by: Holly <hc@interleaf.ie> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #111958|0 |1 is obsolete| | --- Comment #53 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 112280 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112280&action=edit Bug 23971: (QA follow-up) Fix punctuation, doubled up return and syspref order -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #111959|0 |1 is obsolete| | --- Comment #54 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 112281 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112281&action=edit Bug 23971: (follow-up) Add unit tests This commit adds unit tests for the logging done in Acquisitions.pm -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #111960|0 |1 is obsolete| | --- Comment #55 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 112282 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112282&action=edit Bug 23971: (follow-up) Respond to feedback As requested by Jonathan in commit #28: - We now no longer pass a flag to CloseBasket indicating if the closure was due to an EDI basket approval, we log the closure and approval as separate events. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #56 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 112283 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112283&action=edit Bug 23971: (follow-up) Fix tests Following a discussion with Tomás on Slack, this commit implements the following suggestions: - Switch from ->find to ->search in tests. ->find is only ever going to return 0 or 1 rows, which doesn't help us if there is breakage which results in more than 1 row being returned, the test would fail due to ->find not expecting more than 1 row, but we should be testing for it. So switching to ->search returns all matching rows. - Moved tests into their own subtest, they had the potential to pollute surrounding tests - Remove all action logs before each test, this ensures we're only testing the result of the current test -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112277|0 |1 is obsolete| | --- Comment #57 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 112284 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112284&action=edit Bug 23971: Add logging for basket related actions This patch adds logging for the following Acq actions: - Basket creation - Basket editing - Basket approval (via EDI) - Basket closure -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112278|0 |1 is obsolete| | --- Comment #58 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 112285 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112285&action=edit Bug 23971: (follow-up) Remove data duplication In response to Séverine observations in comment #10, this patch removes the duplicate logging of the borrowernumber -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112279|0 |1 is obsolete| | --- Comment #59 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 112286 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112286&action=edit Bug 23971: (follow-up) Add additional basket logging This patch adds additional basket logging as discussed in comment #14 - Basket reopening - Basket users (manager) modification - Basket header modification Signed-off-by: Holly <hc@interleaf.ie> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112280|0 |1 is obsolete| | --- Comment #60 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 112287 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112287&action=edit Bug 23971: (QA follow-up) Fix punctuation, doubled up return and syspref order -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112281|0 |1 is obsolete| | --- Comment #61 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 112288 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112288&action=edit Bug 23971: (follow-up) Add unit tests This commit adds unit tests for the logging done in Acquisitions.pm -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112282|0 |1 is obsolete| | --- Comment #62 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 112289 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112289&action=edit Bug 23971: (follow-up) Respond to feedback As requested by Jonathan in commit #28: - We now no longer pass a flag to CloseBasket indicating if the closure was due to an EDI basket approval, we log the closure and approval as separate events. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112283|0 |1 is obsolete| | --- Comment #63 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 112290 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112290&action=edit Bug 23971: (follow-up) Fix tests Following a discussion with Tomás on Slack, this commit implements the following suggestions: - Switch from ->find to ->search in tests. ->find is only ever going to return 0 or 1 rows, which doesn't help us if there is breakage which results in more than 1 row being returned, the test would fail due to ->find not expecting more than 1 row, but we should be testing for it. So switching to ->search returns all matching rows. - Moved tests into their own subtest, they had the potential to pollute surrounding tests - Remove all action logs before each test, this ensures we're only testing the result of the current test -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #64 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 112291 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112291&action=edit Bug 23971: (follow-up) Make changes for 26582 This commit makes changes necessary to allow this bug to be dependent on Bug 23562: - Move basket closure logging from C4::Acquisision::CloseBasket to Koha::Acquisition::Basket::close - Move basket closure unit test from t/db_dependent/Acquisition.t to t/db_dependent/Koha/Acquisition/Basket.t -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |26582 --- Comment #65 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Hi Tomás I've now made the changes we discussed on Slack. Should be ready for you again now. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26582 [Bug 26582] Add Koha::Acquisition::Basket->close -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112288|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112289|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112290|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112291|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112284|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112285|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112286|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112287|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #66 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Created attachment 112298 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112298&action=edit Bug 23971: Add logging for basket related actions This patch adds logging for the following Acq actions: - Basket creation - Basket editing - Basket approval (via EDI) - Basket closure Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #67 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Created attachment 112299 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112299&action=edit Bug 23971: (follow-up) Remove data duplication In response to Séverine observations in comment #10, this patch removes the duplicate logging of the borrowernumber https://bugs.koha-community.org/show_bug.cgi?id=23971 Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #68 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Created attachment 112300 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112300&action=edit Bug 23971: (follow-up) Add additional basket logging This patch adds additional basket logging as discussed in comment #14 - Basket reopening - Basket users (manager) modification - Basket header modification Signed-off-by: Holly <hc@interleaf.ie> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #69 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Created attachment 112301 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112301&action=edit Bug 23971: (QA follow-up) Fix punctuation, doubled up return and syspref order Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #70 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Created attachment 112302 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112302&action=edit Bug 23971: (follow-up) Add unit tests This commit adds unit tests for the logging done in Acquisitions.pm Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #71 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Created attachment 112303 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112303&action=edit Bug 23971: (follow-up) Respond to feedback As requested by Jonathan in commit #28: - We now no longer pass a flag to CloseBasket indicating if the closure was due to an EDI basket approval, we log the closure and approval as separate events. Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #72 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Created attachment 112304 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112304&action=edit Bug 23971: (follow-up) Fix tests Following a discussion with Tomás on Slack, this commit implements the following suggestions: - Switch from ->find to ->search in tests. ->find is only ever going to return 0 or 1 rows, which doesn't help us if there is breakage which results in more than 1 row being returned, the test would fail due to ->find not expecting more than 1 row, but we should be testing for it. So switching to ->search returns all matching rows. - Moved tests into their own subtest, they had the potential to pollute surrounding tests - Remove all action logs before each test, this ensures we're only testing the result of the current test Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #73 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Created attachment 112305 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112305&action=edit Bug 23971: (follow-up) Make changes for 26582 This commit makes changes necessary to allow this bug to be dependent on Bug 23562: - Move basket closure logging from C4::Acquisision::CloseBasket to Koha::Acquisition::Basket::close - Move basket closure unit test from t/db_dependent/Acquisition.t to t/db_dependent/Koha/Acquisition/Basket.t Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #74 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Created attachment 112306 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112306&action=edit Bug 23971: (QA follow-up) New DBrev syntax Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112306|0 |1 is obsolete| | --- Comment #75 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Created attachment 112307 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112307&action=edit Bug 23971: (QA follow-up) New DBrev syntax Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #76 from Tomás Cohen Arazi <tomascohen@gmail.com> --- I still feel the logs should include the objects, following what's been done so far with the rest of the logs. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Patch doesn't apply --- Comment #77 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- This is conflicting with bug 26584 CONFLICT (content): Merge conflict in acqui/basket.pl CONFLICT (content): Merge conflict in Koha/EDI.pm CONFLICT (content): Merge conflict in C4/Acquisition.pm Please rebase. (In reply to Tomás Cohen Arazi from comment #76)
I still feel the logs should include the objects, following what's been done so far with the rest of the logs.
I am with Tomas, we will know who modified the basket, but not the changes that have been made. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112298|0 |1 is obsolete| | --- Comment #78 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 113384 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=113384&action=edit Bug 23971: Add logging for basket related actions This patch adds logging for the following Acq actions: - Basket creation - Basket editing - Basket approval (via EDI) - Basket closure Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112299|0 |1 is obsolete| | --- Comment #79 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 113385 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=113385&action=edit Bug 23971: (follow-up) Remove data duplication In response to Séverine observations in comment #10, this patch removes the duplicate logging of the borrowernumber https://bugs.koha-community.org/show_bug.cgi?id=23971 Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112300|0 |1 is obsolete| | --- Comment #80 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 113386 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=113386&action=edit Bug 23971: (follow-up) Add additional basket logging This patch adds additional basket logging as discussed in comment #14 - Basket reopening - Basket users (manager) modification - Basket header modification Signed-off-by: Holly <hc@interleaf.ie> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112301|0 |1 is obsolete| | --- Comment #81 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 113387 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=113387&action=edit Bug 23971: (QA follow-up) Fix punctuation, doubled up return and syspref order Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112302|0 |1 is obsolete| | --- Comment #82 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 113388 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=113388&action=edit Bug 23971: (follow-up) Add unit tests This commit adds unit tests for the logging done in Acquisitions.pm Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112303|0 |1 is obsolete| | --- Comment #83 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 113389 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=113389&action=edit Bug 23971: (follow-up) Respond to feedback As requested by Jonathan in commit #28: - We now no longer pass a flag to CloseBasket indicating if the closure was due to an EDI basket approval, we log the closure and approval as separate events. Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112304|0 |1 is obsolete| | --- Comment #84 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 113390 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=113390&action=edit Bug 23971: (follow-up) Fix tests Following a discussion with Tomás on Slack, this commit implements the following suggestions: - Switch from ->find to ->search in tests. ->find is only ever going to return 0 or 1 rows, which doesn't help us if there is breakage which results in more than 1 row being returned, the test would fail due to ->find not expecting more than 1 row, but we should be testing for it. So switching to ->search returns all matching rows. - Moved tests into their own subtest, they had the potential to pollute surrounding tests - Remove all action logs before each test, this ensures we're only testing the result of the current test Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112305|0 |1 is obsolete| | --- Comment #85 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 113391 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=113391&action=edit Bug 23971: (follow-up) Make changes for 26582 This commit makes changes necessary to allow this bug to be dependent on Bug 23562: - Move basket closure logging from C4::Acquisision::CloseBasket to Koha::Acquisition::Basket::close - Move basket closure unit test from t/db_dependent/Acquisition.t to t/db_dependent/Koha/Acquisition/Basket.t Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112307|0 |1 is obsolete| | --- Comment #86 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 113392 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=113392&action=edit Bug 23971: (QA follow-up) New DBrev syntax Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #87 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 113393 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=113393&action=edit Bug 23971: (follow-up) Log entire objects As requested by Tomás and Jonathan, we now log the entire basket object when logging an action. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Signed Off --- Comment #88 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Hi Jonathan I've now rebased. I've also added logging of the entire basket object as requested. Revised test plan: - Apply the patch => TEST: Observe there is a new "AcqLog" syspref - Enable the AcqLog syspref - Create a basket => TEST: Observe that an Acquisitions "Create an acquisitions basket" action log has been created containing: => Basket number in "Object" column => Borrowernumber of staff member in "Librarian" column => Full basket object details in "Info" column => TEST: Observe that an Acquisitions "Modify an acquisitions basket header" action log has been created containing: => Basket number in "Object" column => Borrowernumber of staff member in "Librarian" column => Full basket object details in "Info" column - Modify a basket => TEST: Observe that an Acquisitions "Modify an acquisitions basket header" action log has been created containing: => Basket number in "Object" column => Borrowernumber of staff member in "Librarian" column => Full basket object details in "Info" column - Approve a basket via EDI => TEST: Observe that an Acquisitions "Close an acquisitions basket" action log has been created containing: => Basket number in "Object" column => Borrowernumber of staff member in "Librarian" column => Full basket object details in "Info" column => TEST: Observe that an Acquisitions "Approve an acquisitions basket" action log has been created containing: => Basket number in "Object" column => Borrowernumber of staff member in "Librarian" column => Full basket object details in "Info" column - Close a basket => TEST: Observe that an Acquisitions "Close an acquisitions basket" action log has been created containing: => Basket number in "Object" column => Borrowernumber of staff member in "Librarian" column -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Niamh Walker-Headon <Niamh.WalkerHeadon@hse.ie> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |Niamh.WalkerHeadon@hse.ie -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |release-notes-needed -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #89 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- acqui/addorder.pl is logging a MODIFY_BASKET, is that expected? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #90 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- I squashed the patches for review and found: +++ b/acqui/basketgroup.pl @@ -307,7 +307,8 @@ if ( $op eq "add" ) { my $basketno=$input->param('basketno'); my $basketgroupid=$input->param('basketgroupid'); ModBasket( { basketno => $basketno, - basketgroupid => $basketgroupid } ); + basketgroupid => $basketgroupid, + borrowernumber => $loggedinuser } ); Why are you adding borrowernumber? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #113384|0 |1 is obsolete| | --- Comment #91 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114857 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114857&action=edit Bug 23971: Add logging for basket related actions This patch adds logging for the following Acq actions: - Basket creation - Basket editing - Basket approval (via EDI) - Basket closure Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #113385|0 |1 is obsolete| | --- Comment #92 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114858 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114858&action=edit Bug 23971: (follow-up) Remove data duplication In response to Séverine observations in comment #10, this patch removes the duplicate logging of the borrowernumber https://bugs.koha-community.org/show_bug.cgi?id=23971 Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #113386|0 |1 is obsolete| | --- Comment #93 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114859 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114859&action=edit Bug 23971: (follow-up) Add additional basket logging This patch adds additional basket logging as discussed in comment #14 - Basket reopening - Basket users (manager) modification - Basket header modification Signed-off-by: Holly <hc@interleaf.ie> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #113387|0 |1 is obsolete| | --- Comment #94 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114860 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114860&action=edit Bug 23971: (QA follow-up) Fix punctuation, doubled up return and syspref order Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #113388|0 |1 is obsolete| | --- Comment #95 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114861 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114861&action=edit Bug 23971: (follow-up) Add unit tests This commit adds unit tests for the logging done in Acquisitions.pm Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #113389|0 |1 is obsolete| | --- Comment #96 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114862 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114862&action=edit Bug 23971: (follow-up) Respond to feedback As requested by Jonathan in commit #28: - We now no longer pass a flag to CloseBasket indicating if the closure was due to an EDI basket approval, we log the closure and approval as separate events. Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #113390|0 |1 is obsolete| | --- Comment #97 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114863 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114863&action=edit Bug 23971: (follow-up) Fix tests Following a discussion with Tomás on Slack, this commit implements the following suggestions: - Switch from ->find to ->search in tests. ->find is only ever going to return 0 or 1 rows, which doesn't help us if there is breakage which results in more than 1 row being returned, the test would fail due to ->find not expecting more than 1 row, but we should be testing for it. So switching to ->search returns all matching rows. - Moved tests into their own subtest, they had the potential to pollute surrounding tests - Remove all action logs before each test, this ensures we're only testing the result of the current test Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #113391|0 |1 is obsolete| | --- Comment #98 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114864 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114864&action=edit Bug 23971: (follow-up) Make changes for 26582 This commit makes changes necessary to allow this bug to be dependent on Bug 23562: - Move basket closure logging from C4::Acquisision::CloseBasket to Koha::Acquisition::Basket::close - Move basket closure unit test from t/db_dependent/Acquisition.t to t/db_dependent/Koha/Acquisition/Basket.t Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #113392|0 |1 is obsolete| | --- Comment #99 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114865 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114865&action=edit Bug 23971: (QA follow-up) New DBrev syntax Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #113393|0 |1 is obsolete| | --- Comment #100 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114866 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114866&action=edit Bug 23971: (follow-up) Log entire objects As requested by Tomás and Jonathan, we now log the entire basket object when logging an action. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #101 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114867 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114867&action=edit Bug 23971: (follow-up) Remove stale param As highlighted by Jonathan in comment #90, we were passing borrowernumber to ModBasket. This was a hangover from when we explicitly logged the borrower, which was later discovered to be unnecessary duplication, and was removed in commit "Remove data duplication". This commit removes this unnecessary parameter. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #102 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Hi Jonathan Thanks for that. I've rebased as I was getting conflicts on a couple of commits.
acqui/addorder.pl is logging a MODIFY_BASKET, is that expected?
I believe so. addorder processes the submission from the "New order" form when adding an order to a basket, so logging the basket modification makes sense here I think.
Why are you adding borrowernumber?
That was a hangover from when ModBasket was explicitly logging the borrower. This was highlighted as unnecessary duplication, so I removed it from the logging in the patch titled "Bug 23971: (follow-up) Remove data duplication", but missed removing it from the ModBasket call. I've now created a new commit that removes it, thanks for highlighting it. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Failed QA --- Comment #103 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Please double check the rebased patches, I get an error on tools/viewlog.pl I think we should squash all the patches. They are pretty simple to read once squashed but the history makes them very hard to follow. (In reply to Andrew Isherwood from comment #102)
acqui/addorder.pl is logging a MODIFY_BASKET, is that expected?
I believe so. addorder processes the submission from the "New order" form when adding an order to a basket, so logging the basket modification makes sense here I think.
It logs a "Modify an acquisitions basket" with the info of the basket when I create or update an order. It does not make sense to me. Can you confirm with the sponsor please? Should not we have ADD_ORDER/MODIFY_LOG instead? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114857|0 |1 is obsolete| | --- Comment #104 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114905 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114905&action=edit Bug 23971: Add logging for basket related actions This patch adds logging for the following Acq actions: - Basket creation - Basket editing - Basket approval (via EDI) - Basket closure Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114858|0 |1 is obsolete| | --- Comment #105 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114906 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114906&action=edit Bug 23971: (follow-up) Remove data duplication In response to Séverine observations in comment #10, this patch removes the duplicate logging of the borrowernumber https://bugs.koha-community.org/show_bug.cgi?id=23971 Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114859|0 |1 is obsolete| | --- Comment #106 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114907 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114907&action=edit Bug 23971: (follow-up) Add additional basket logging This patch adds additional basket logging as discussed in comment #14 - Basket reopening - Basket users (manager) modification - Basket header modification Signed-off-by: Holly <hc@interleaf.ie> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114860|0 |1 is obsolete| | --- Comment #107 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114908 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114908&action=edit Bug 23971: (QA follow-up) Fix punctuation, doubled up return and syspref order Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114861|0 |1 is obsolete| | --- Comment #108 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114909 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114909&action=edit Bug 23971: (follow-up) Add unit tests This commit adds unit tests for the logging done in Acquisitions.pm Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114862|0 |1 is obsolete| | --- Comment #109 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114910 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114910&action=edit Bug 23971: (follow-up) Respond to feedback As requested by Jonathan in commit #28: - We now no longer pass a flag to CloseBasket indicating if the closure was due to an EDI basket approval, we log the closure and approval as separate events. Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114863|0 |1 is obsolete| | --- Comment #110 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114911 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114911&action=edit Bug 23971: (follow-up) Fix tests Following a discussion with Tomás on Slack, this commit implements the following suggestions: - Switch from ->find to ->search in tests. ->find is only ever going to return 0 or 1 rows, which doesn't help us if there is breakage which results in more than 1 row being returned, the test would fail due to ->find not expecting more than 1 row, but we should be testing for it. So switching to ->search returns all matching rows. - Moved tests into their own subtest, they had the potential to pollute surrounding tests - Remove all action logs before each test, this ensures we're only testing the result of the current test Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114864|0 |1 is obsolete| | --- Comment #111 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114912 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114912&action=edit Bug 23971: (follow-up) Make changes for 26582 This commit makes changes necessary to allow this bug to be dependent on Bug 23562: - Move basket closure logging from C4::Acquisision::CloseBasket to Koha::Acquisition::Basket::close - Move basket closure unit test from t/db_dependent/Acquisition.t to t/db_dependent/Koha/Acquisition/Basket.t Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114865|0 |1 is obsolete| | --- Comment #112 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114913 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114913&action=edit Bug 23971: (QA follow-up) New DBrev syntax Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114866|0 |1 is obsolete| | --- Comment #113 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114914 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114914&action=edit Bug 23971: (follow-up) Log entire objects As requested by Tomás and Jonathan, we now log the entire basket object when logging an action. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114867|0 |1 is obsolete| | --- Comment #114 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114915 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114915&action=edit Bug 23971: (follow-up) Remove stale param As highlighted by Jonathan in comment #90, we were passing borrowernumber to ModBasket. This was a hangover from when we explicitly logged the borrower, which was later discovered to be unnecessary duplication, and was removed in commit "Remove data duplication". This commit removes this unnecessary parameter. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #115 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 114916 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114916&action=edit Bug 23971: (follow-up) Fix template typo As mentioned in comment #103, there was a typo in viewlog.tt -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #116 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- (In reply to Jonathan Druart from comment #103)
Please double check the rebased patches, I get an error on tools/viewlog.pl
Good spot, sorry about that. Now fixed.
I think we should squash all the patches. They are pretty simple to read once squashed but the history makes them very hard to follow.
Fair enough. Though I have found myself referring to individual patches during this process. Maybe squashing just prior to merging would be the best approach?
It logs a "Modify an acquisitions basket" with the info of the basket when I create or update an order. It does not make sense to me. Can you confirm with the sponsor please?
The original spec agreed with the customer requests "Who edited the basket and when", this was in the context of the basket contents. So I believe what we're doing is correct. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off --- Comment #117 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- I've asked Tomas another look on the last patch and the question about addorder.pl
Does it make sense to have a MODIFY_BASKET log when we add/update an order (without any info on the order)?
-- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114905|0 |1 is obsolete| | --- Comment #118 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 115976 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115976&action=edit Bug 23971: Add logging for basket related actions This patch adds logging for the following Acq actions: - Basket creation - Basket editing - Basket approval (via EDI) - Basket closure Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114906|0 |1 is obsolete| | --- Comment #119 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 115977 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115977&action=edit Bug 23971: (follow-up) Remove data duplication In response to Séverine observations in comment #10, this patch removes the duplicate logging of the borrowernumber https://bugs.koha-community.org/show_bug.cgi?id=23971 Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114907|0 |1 is obsolete| | --- Comment #120 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 115978 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115978&action=edit Bug 23971: (follow-up) Add additional basket logging This patch adds additional basket logging as discussed in comment #14 - Basket reopening - Basket users (manager) modification - Basket header modification Signed-off-by: Holly <hc@interleaf.ie> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114908|0 |1 is obsolete| | --- Comment #121 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 115979 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115979&action=edit Bug 23971: (QA follow-up) Fix punctuation, doubled up return and syspref order Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114909|0 |1 is obsolete| | --- Comment #122 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 115980 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115980&action=edit Bug 23971: (follow-up) Add unit tests This commit adds unit tests for the logging done in Acquisitions.pm Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114910|0 |1 is obsolete| | --- Comment #123 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 115981 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115981&action=edit Bug 23971: (follow-up) Respond to feedback As requested by Jonathan in commit #28: - We now no longer pass a flag to CloseBasket indicating if the closure was due to an EDI basket approval, we log the closure and approval as separate events. Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114911|0 |1 is obsolete| | --- Comment #124 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 115982 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115982&action=edit Bug 23971: (follow-up) Fix tests Following a discussion with Tomás on Slack, this commit implements the following suggestions: - Switch from ->find to ->search in tests. ->find is only ever going to return 0 or 1 rows, which doesn't help us if there is breakage which results in more than 1 row being returned, the test would fail due to ->find not expecting more than 1 row, but we should be testing for it. So switching to ->search returns all matching rows. - Moved tests into their own subtest, they had the potential to pollute surrounding tests - Remove all action logs before each test, this ensures we're only testing the result of the current test Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114912|0 |1 is obsolete| | --- Comment #125 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 115983 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115983&action=edit Bug 23971: (follow-up) Make changes for 26582 This commit makes changes necessary to allow this bug to be dependent on Bug 23562: - Move basket closure logging from C4::Acquisision::CloseBasket to Koha::Acquisition::Basket::close - Move basket closure unit test from t/db_dependent/Acquisition.t to t/db_dependent/Koha/Acquisition/Basket.t Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114913|0 |1 is obsolete| | --- Comment #126 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 115984 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115984&action=edit Bug 23971: (QA follow-up) New DBrev syntax Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114914|0 |1 is obsolete| | --- Comment #127 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 115985 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115985&action=edit Bug 23971: (follow-up) Log entire objects As requested by Tomás and Jonathan, we now log the entire basket object when logging an action. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114915|0 |1 is obsolete| | --- Comment #128 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Created attachment 115986 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115986&action=edit Bug 23971: (follow-up) Remove stale param As highlighted by Jonathan in comment #90, we were passing borrowernumber to ModBasket. This was a hangover from when we explicitly logged the borrower, which was later discovered to be unnecessary duplication, and was removed in commit "Remove data duplication". This commit removes this unnecessary parameter. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #129 from Andrew Isherwood <andrew.isherwood@ptfs-europe.com> --- Rebased so I can rebase the dependent bug -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <andrew.isherwood@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #114916|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #115976|0 |1 is obsolete| | Attachment #115977|0 |1 is obsolete| | Attachment #115978|0 |1 is obsolete| | Attachment #115979|0 |1 is obsolete| | Attachment #115980|0 |1 is obsolete| | Attachment #115981|0 |1 is obsolete| | Attachment #115982|0 |1 is obsolete| | Attachment #115983|0 |1 is obsolete| | Attachment #115984|0 |1 is obsolete| | Attachment #115985|0 |1 is obsolete| | Attachment #115986|0 |1 is obsolete| | --- Comment #130 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 119401 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119401&action=edit Bug 23971: Add logging for basket related actions This patch adds logging for the following Acq actions: - Basket creation - Basket editing - Basket approval (via EDI) - Basket closure Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #119401|0 |1 is obsolete| | --- Comment #131 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 119402 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119402&action=edit Bug 23971: Add logging for basket related actions This patch adds logging for the following Acq actions: - Basket creation - Basket editing - Basket approval (via EDI) - Basket closure Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #132 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 119403 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119403&action=edit Bug 23971: (follow-up) Remove data duplication In response to Séverine observations in comment #10, this patch removes the duplicate logging of the borrowernumber https://bugs.koha-community.org/show_bug.cgi?id=23971 Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #133 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 119404 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119404&action=edit Bug 23971: (follow-up) Add additional basket logging This patch adds additional basket logging as discussed in comment #14 - Basket reopening - Basket users (manager) modification - Basket header modification Signed-off-by: Holly <hc@interleaf.ie> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #134 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 119405 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119405&action=edit Bug 23971: (QA follow-up) Fix punctuation, doubled up return and syspref order Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #135 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 119406 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119406&action=edit Bug 23971: (follow-up) Add unit tests This commit adds unit tests for the logging done in Acquisitions.pm Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #136 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 119407 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119407&action=edit Bug 23971: (follow-up) Respond to feedback As requested by Jonathan in commit #28: - We now no longer pass a flag to CloseBasket indicating if the closure was due to an EDI basket approval, we log the closure and approval as separate events. Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #137 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 119408 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119408&action=edit Bug 23971: (follow-up) Fix tests Following a discussion with Tomás on Slack, this commit implements the following suggestions: - Switch from ->find to ->search in tests. ->find is only ever going to return 0 or 1 rows, which doesn't help us if there is breakage which results in more than 1 row being returned, the test would fail due to ->find not expecting more than 1 row, but we should be testing for it. So switching to ->search returns all matching rows. - Moved tests into their own subtest, they had the potential to pollute surrounding tests - Remove all action logs before each test, this ensures we're only testing the result of the current test Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #138 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 119409 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119409&action=edit Bug 23971: (follow-up) Make changes for 26582 This commit makes changes necessary to allow this bug to be dependent on Bug 23562: - Move basket closure logging from C4::Acquisision::CloseBasket to Koha::Acquisition::Basket::close - Move basket closure unit test from t/db_dependent/Acquisition.t to t/db_dependent/Koha/Acquisition/Basket.t Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #139 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 119410 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119410&action=edit Bug 23971: (QA follow-up) New DBrev syntax Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #140 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 119411 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119411&action=edit Bug 23971: (follow-up) Log entire objects As requested by Tomás and Jonathan, we now log the entire basket object when logging an action. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #141 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 119412 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119412&action=edit Bug 23971: (follow-up) Remove stale param As highlighted by Jonathan in comment #90, we were passing borrowernumber to ModBasket. This was a hangover from when we explicitly logged the borrower, which was later discovered to be unnecessary duplication, and was removed in commit "Remove data duplication". This commit removes this unnecessary parameter. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #142 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Release notes are missing. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #143 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 119944 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119944&action=edit Bug 23971: Rename pref with AcquisitionLog -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #144 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Please fill in the release notes fields with the list of what this patch set is going to log. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #145 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 119946 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119946&action=edit Bug 23971: Do not always flag AcquisitionLog as 'not enabled' -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This enhancement adds back release notes| |the `AcquisitionsLog` | |preference and when enabled | |it adds logging for the | |following actions: | | | |* | |Adding new baskets | |* | |Re-opening closed baskets | |* | |Modifying baskets | |* | |Modifying basket headers | |* | |Modifying basket users | |* | |Closing baskets | |* Approving | |baskets -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|release-notes-needed | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |21.05.00 released in| | Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #146 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Pushed to master for 21.05, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #147 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 120048 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120048&action=edit Bug 23971: Fix UsageStats.t Hum, what's the point of these tests? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 --- Comment #148 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Follow-up pushed to master. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fridolin.somers@biblibre.co | |m --- Comment #149 from Fridolin Somers <fridolin.somers@biblibre.com> --- Enhancement not pushed to 20.11.x -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |28872 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28872 [Bug 28872] AcquisitionLog, NewsLog, NoticesLog should use 1/0 for their values -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23971 Andrew Isherwood <bugzilla@warmlight.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|bugzilla@warmlight.co.uk |koha-bugs@lists.koha-commun | |ity.org -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org