[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Bug 28717 depends on bug 26205, which changed state. Bug 26205 Summary: News changes aren't logged https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26205 What|Removed |Added Status|Pushed to stable|RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Keywords|rel_21_11_candidate | -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Kyle M Hall changed: What|Removed |Added Resolution|--- |FIXED Status|Pushed to master|RESOLVED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 --- Comment #27 from Jonathan Druart --- Pushed to master for 21.11, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Status|Passed QA |Pushed to master Version(s)||21.11.00 released in|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Katrin Fischer changed: What|Removed |Added Attachment #125637|0 |1 is obsolete|| --- Comment #26 from Katrin Fischer --- Created attachment 125659 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=125659=edit Bug 28717: Fix additional content logs The logging for additional contents added by bug 26205 has been broken by but 22544. This patch is a revisited version as bug 24387 has been pushed. It does not log MODIFY if no modification has been made on a template (useful when only 1 version/lang of a content has been modified) Test plan: Turn on NewsLog Add/modify and delete additional contents/News and confirm that modification are logged. Signed-off-by: Lucas Gass Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Katrin Fischer changed: What|Removed |Added Severity|normal |major -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Katrin Fischer changed: What|Removed |Added Attachment #125543|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Attachment #122944|0 |1 is obsolete|| Attachment #122955|0 |1 is obsolete|| --- Comment #25 from Lucas Gass --- Created attachment 125637 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=125637=edit Bug 28717: Fix additional content logs The logging for additional contents added by bug 26205 has been broken by but 22544. This patch is a revisited version as bug 24387 has been pushed. It does not log MODIFY if no modification has been made on a template (useful when only 1 version/lang of a content has been modified) Test plan: Turn on NewsLog Add/modify and delete additional contents/News and confirm that modification are logged. Signed-off-by: Lucas Gass -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 --- Comment #24 from Jonathan Druart --- Sorry, Lucas! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Attachment #125471|0 |1 is obsolete|| --- Comment #23 from Jonathan Druart --- Created attachment 125543 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=125543=edit Bug 28717: Fix additional content logs The logging for additional contents added by bug 26205 has been broken by but 22544. This patch is a revisited version as bug 24387 has been pushed. It does not log MODIFY if no modification has been made on a template (useful when only 1 version/lang of a content has been modified) Test plan: Turn on NewsLog Add/modify and delete additional contents/News and confirm that modification are logged. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Status|Needs Signoff |Failed QA --- Comment #22 from Lucas Gass --- When I apply this patch and go to create a new HTML customization on additional-contents.pl I now get this error: Can't call method "code" on an undefined value at /kohadevbox/koha/tools/additional-contents.pl line 123 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Status|Patch doesn't apply |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Attachment #123964|0 |1 is obsolete|| --- Comment #21 from Jonathan Druart --- Created attachment 125471 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=125471=edit Bug 28717: Fix additional content logs The logging for additional contents added by bug 26205 has been broken by but 22544. This patch is a revisited version as bug 24387 has been pushed. It does not log MODIFY if no modification has been made on a template (useful when only 1 version/lang of a content has been modified) Test plan: Turn on NewsLog Add/modify and delete additional contents/News and confirm that modification are logged. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Status|Needs Signoff |Patch doesn't apply --- Comment #20 from Lucas Gass --- this patch, the one for master, no longer cleanly applies. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Attachment #122955|[20.05] Bug 28717: |[21.05] Bug 28717: description|(follow-up) add title to|(follow-up) add title to |logged information |logged information -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Attachment #122944|[20.05] Bug 28717: Add |[21.05] Bug 28717: Add description|logging logic to|logging logic to |koha-news.pl|koha-news.pl -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 --- Comment #19 from Jonathan Druart --- To clarify, only the last patch is for master. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 --- Comment #18 from Jonathan Druart --- (In reply to Jonathan Druart from comment #17) > We should actually log Dumper of the additional content object, but we need > bug 28692 to prevent logging DateTime object. I meant bug 28700. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 --- Comment #17 from Jonathan Druart --- We should actually log Dumper of the additional content object, but we need bug 28692 to prevent logging DateTime object. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Attachment #122955|Bug 28717: (follow-up) add |[20.05] Bug 28717: description|title to logged information |(follow-up) add title to ||logged information -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Attachment #122944|Bug 28717: Add logging |[20.05] Bug 28717: Add description|logic to koha-news.pl |logging logic to ||koha-news.pl -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Status|ASSIGNED|Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 --- Comment #16 from Jonathan Druart --- Created attachment 123964 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123964=edit Bug 28717: Fix additional content logs The logging for additional contents added by bug 26205 has been broken by but 22544. This patch is a revisited version as bug 24387 has been pushed. It does not log MODIFY if no modification has been made on a template (useful when only 1 version/lang of a content has been modified) Test plan: Turn on NewsLog Add/modify and delete additional contents/News and confirm that modification are logged. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Status|Patch doesn't apply |ASSIGNED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Assignee|lu...@bywatersolutions.com |jonathan.druart+koha@gmail. ||com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Owen Leonard changed: What|Removed |Added Status|Needs Signoff |Patch doesn't apply -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Kyle M Hall changed: What|Removed |Added CC||k...@bywatersolutions.com Depends on||26205 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26205 [Bug 26205] News changes aren't logged -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Bug 28717 depends on bug 28718, which changed state. Bug 28718 Summary: Can't delete multiple news items at once https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28718 What|Removed |Added Status|Pushed to master|RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Keywords||rel_21_11_candidate -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Patch complexity|--- |Small patch Text to go in the||This patch fixes a release notes||regression in the NewsLog ||caused by Bug 28718. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 --- Comment #15 from Lucas Gass --- Created attachment 122955 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122955=edit Bug 28717: (follow-up) add title to logged information -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Status|Signed Off |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 --- Comment #14 from Owen Leonard --- This works well, so I've signed off. I'd love to see a follow-up which adds title to the logged information. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Owen Leonard changed: What|Removed |Added Attachment #122943|0 |1 is obsolete|| --- Comment #13 from Owen Leonard --- Created attachment 122944 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122944=edit Bug 28717: Add logging logic to koha-news.pl To test: 1. have some news items 2. make sure NewsLog is turned on 3. Add, modify, and delete some news 4. Check the action logs, nothing is logged 5. Apply patch 6. do step 3 again 7. now changes should be logged 8. make sure you try deleting multiple news items at once and confirm that all deletions are logged Signed-off-by: Owen Leonard -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Owen Leonard changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Status|Failed QA |Needs Signoff --- Comment #12 from Jonathan Druart --- Sorry, it was caused by my rebase. I didn't get the conflict when I applied it this morning, it auto-solved using a previous resolution (that was wrong apparently!) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Attachment #122941|0 |1 is obsolete|| --- Comment #11 from Jonathan Druart --- Created attachment 122943 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122943=edit Bug 28717: Add logging logic to koha-news.pl To test: 1. have some news items 2. make sure NewsLog is turned on 3. Add, modify, and delete some news 4. Check the action logs, nothing is logged 5. Apply patch 6. do step 3 again 7. now changes should be logged 8. make sure you try deleting multiple news items at once and confirm that all deletions are logged -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Owen Leonard changed: What|Removed |Added Status|Needs Signoff |Failed QA --- Comment #10 from Owen Leonard --- When I try to delete multiple news entries at one time I get an error: DBIx::Class::Storage::DBI::_dbh_execute(): DBI Exception: DBD::mysql::st execute failed: Unknown column '29' in 'where clause' [for Statement "DELETE FROM `opac_news` WHERE ( ( `29` = ? AND `idnew` = ? ) )" with ParamValues: 0="27", 1="44"] at /kohadevbox/koha/Koha/Objects.pm line 208 at /usr/share/perl5/DBIx/Class/Exception.pm line 77 The deletions fail but are logged as if they were successful. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Attachment #122927|0 |1 is obsolete|| --- Comment #9 from Jonathan Druart --- Created attachment 122941 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122941=edit Bug 28717: Add logging logic to koha-news.pl To test: 1. have some news items 2. make sure NewsLog is turned on 3. Add, modify, and delete some news 4. Check the action logs, nothing is logged 5. Apply patch 6. do step 3 again 7. now changes should be logged 8. make sure you try deleting multiple news items at once and confirm that all deletions are logged -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Status|Patch doesn't apply |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Owen Leonard changed: What|Removed |Added Status|Needs Signoff |Patch doesn't apply --- Comment #8 from Owen Leonard --- Does not apply cleanly on top of Bug 28718 or master -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Attachment #122921|0 |1 is obsolete|| --- Comment #7 from Lucas Gass --- Created attachment 122927 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122927=edit Bug 28717: Add logging logic to koha-news.pl To test: 1. have some news items 2. make sure NewsLog is turned on 3. Add, modify, and delete some news 4. Check the action logs, nothing is logged 5. Apply patch 6. do step 3 again 7. now changes should be logged 8. make sure you try deleting multiple news items at once and confirm that all deletions are logged -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Status|Needs Signoff |Failed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 --- Comment #6 from Lucas Gass --- > I think one of the major reasons to log news changes, especially deletions, > is to offer a means of recovering a deleted entry. Because of that I think > the full text of the news item should be logged. Yes, very true. Marking this as FQA and coming back with another patch ASAP :) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 --- Comment #5 from Owen Leonard --- (In reply to Lucas Gass from comment #4) > Upon deletion I am just sending > the ID to the logs. Do we need to also log the content of the deleted news > item? I think one of the major reasons to log news changes, especially deletions, is to offer a means of recovering a deleted entry. Because of that I think the full text of the news item should be logged. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |lu...@bywatersolutions.com |ity.org | -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 --- Comment #4 from Lucas Gass --- (In reply to Jonathan Druart from comment #2) > Sorry about that, Lucas! > > I think there is something missing in your patch, the $lang won't be set for > DELETE. Can you have another look please? Thanks Jonathan! I submitted a new patch. Upon deletion I am just sending the ID to the logs. Do we need to also log the content of the deleted news item? -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Depends on||28718 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28718 [Bug 28718] Can't delete multiple news items at once -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Depends on|28718 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28718 [Bug 28718] Can't delete multiple news items at once -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Depends on||28718 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28718 [Bug 28718] Can't delete multiple news items at once -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Attachment #122864|0 |1 is obsolete|| --- Comment #3 from Lucas Gass --- Created attachment 122921 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122921=edit Bug 28717: Add logging logic to koha-news.pl To test: 1. have some news items 2. make sure NewsLog is turned on 3. Add, modify, and delete some news 4. Check the action logs, nothing is logged 5. Apply patch 6. do step 3 again 7. now changes should be logged 8. make sure you try deleting multiple news items at once and confirm that all deletions are logged -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Status|Needs Signoff |Failed QA CC||jonathan.druart+koha@gmail. ||com --- Comment #2 from Jonathan Druart --- Sorry about that, Lucas! I think there is something missing in your patch, the $lang won't be set for DELETE. Can you have another look please? -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Jonathan Druart changed: What|Removed |Added Depends on||22544 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22544 [Bug 22544] Move C4:NewsChannels to Koha namespace -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 --- Comment #1 from Lucas Gass --- Created attachment 122864 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=122864=edit Bug 28717: Add logging logic to koha-news.pl To test: 1. have some news items 2. make sure NewsLog is turned on 3. Add, modify, and delete some news 4. Check the action logs, nothing is logged 5. Apply patch 6. do step 3 again 7. now changes should be logged -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Depends on|22544 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22544 [Bug 22544] Move C4:NewsChannels to Koha namespace -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28717] NewsLog doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28717 Lucas Gass changed: What|Removed |Added Depends on||22544 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22544 [Bug 22544] Move C4:NewsChannels to Koha namespace -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
