[Koha-bugs] [Bug 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 Katrin Fischer changed: What|Removed |Added Blocks||25790 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25790 [Bug 25790] [OMNIBUS] warnings removal -- 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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 Katrin Fischer changed: What|Removed |Added Blocks|25790 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25790 [Bug 25790] [OMNIBUS] warnings removal -- 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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 Aleisha Amohia changed: What|Removed |Added CC||alei...@catalyst.net.nz --- Comment #12 from Aleisha Amohia --- Does not apply cleanly to 19.11.x. Please rebase if needed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 Lucas Gass changed: What|Removed |Added CC||lu...@bywatersolutions.com Version(s)|20.11.00|20.11.00, 20.05.03 released in|| Status|Pushed to master|Pushed to stable --- Comment #11 from Lucas Gass --- backported to 20.05.x for 20.05.03 -- 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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 Jonathan Druart changed: What|Removed |Added Version(s)||20.11.00 released in|| Status|Passed QA |Pushed to 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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 --- Comment #10 from Jonathan Druart --- Pushed to master for 20.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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 Martin Renvoize 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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 Martin Renvoize changed: What|Removed |Added CC||martin.renvoize@ptfs-europe ||.com Status|Needs Signoff |Signed Off --- Comment #9 from Martin Renvoize --- A nice bit of tidying up.. works as expected, thanks! Going straight for a QA here.. PQA -- 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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 Martin Renvoize changed: What|Removed |Added Attachment #105999|0 |1 is obsolete|| --- Comment #8 from Martin Renvoize --- Created attachment 106387 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106387=edit Bug 25695: merge of similar code blocks after circulation_rules update In onboarding.pl we have second block with adding just one more rule parameter (maxissueqty) which can be merged to first one after all parameters now stored in circulation_rules table. Also this eliminates the need of adding forgotten "warn $@" after the second eval block. Mentored-by: Andrew Nugged Signed-off-by: Martin Renvoize -- 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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 Slava Shishkin changed: What|Removed |Added Status|In Discussion |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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 Slava Shishkin changed: What|Removed |Added Attachment #105650|0 |1 is obsolete|| Attachment #105998|0 |1 is obsolete|| --- Comment #7 from Slava Shishkin --- Created attachment 105999 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=105999=edit Bug 25695: merge of similar code blocks after circulation_rules update In onboarding.pl we have second block with adding just one more rule parameter (maxissueqty) which can be merged to first one after all parameters now stored in circulation_rules table. Also this eliminates the need of adding forgotten "warn $@" after the second eval block. Mentored-by: Andrew Nugged -- 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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695
--- Comment #6 from Andrew Nugged ---
have we saved a "success" message? Seems it's lost :).
This one:
push @messages, { code => 'success_on_insert_circ_rule' };
let's update :).
--
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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 Andrew Nugged changed: What|Removed |Added Status|Needs Signoff |In Discussion -- 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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 Slava Shishkin changed: What|Removed |Added Status|In Discussion |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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 --- Comment #5 from Slava Shishkin --- Created attachment 105998 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=105998=edit Bug 25695: merge of similar code blocks after circulation_rules update In onboarding.pl we have second block with adding just one more rule parameter (maxissueqty) which can be merged to first one after all parameters now stored in circulation_rules table. Also this eliminates the need of adding forgotten "warn $@" after the second eval block. Mentored-by: Andrew Nugged -- 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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 Andrew Nugged changed: What|Removed |Added Blocks||25790 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25790 [Bug 25790] [OMNIBUS] warnings removal -- 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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 Andrew Nugged changed: What|Removed |Added Status|Needs Signoff |In Discussion --- Comment #4 from Andrew Nugged --- Yes, that was the second option but more "distant from the original". I agree, we will. Slava: let's do it. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 --- Comment #3 from Jonathan Druart --- It's coming from commit 3c4232e746cef574de9d1a346be7704dc3cd2e33 Bug 22489: Update onboarding tool for bug 18925 But now we have all our rules moved to circulation_rules, I think maxissueqty can be moved into the same call as the other rules. -- 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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 Jonathan Druart changed: What|Removed |Added CC||jonathan.dru...@bugs.koha-c ||ommunity.org --- Comment #2 from Jonathan Druart --- Why are not we actually setting maxissueqty in the same set_rules call? -- 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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 Andrew Nugged changed: What|Removed |Added Status|NEW |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 25695] Missing logging of $@ in onboarding.pl after eval block
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25695 --- Comment #1 from Slava Shishkin --- Created attachment 105650 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=105650=edit Bug 25695: add a missing warning in onboarding.pl There is a forgotten warning in the second eval block just after the same eval block which has warning. This makes the error reason not logged. The solution is to add exactly the same 'warn $@' in consecutive eval block. Mentored-by: Andrew Nugged -- 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/
