[Bug 18642] New: Internal Server Error in Guided reports wizard
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18642 Bug ID: 18642 Summary: Internal Server Error in Guided reports wizard Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: major Priority: P5 - low Component: Installation and upgrade (web-based installer) Assignee: koha-bugs@lists.koha-community.org Reporter: veron@veron.ch QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com Calling Guided reports wizard from Home > Reports results in an Internal Server Error. Plack error log says: Could not compile /home/marc/koha/reports/guided_reports.pl: Can't locate Data/Printer.pm in @INC It seems that the module Data::Printer is missing, but About page and installation tool do not complain about. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18642 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |17898 --- Comment #1 from Marc Véron <veron@veron.ch> --- Introduced by first patch in Bug 17898 - Add a way to automatically convert SQL reports Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17898 [Bug 17898] Add a way to automatically convert SQL reports -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18642 Josef Moravec <josef.moravec@gmail.com> 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.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18642 --- Comment #2 from Josef Moravec <josef.moravec@gmail.com> --- Created attachment 63607 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63607&action=edit Bug 18642: Add missing dependency for Data::Printer module Test plan: 1) Go to About page -- without patch you can't see Data::Printer listed -- with patch you'll see it 2) uninstall Data::Printer - apt-get remove libdata-printer-perl 3) go to Reports -> Use saved - you'll see ISE 4) install the module again - apt-get install libdata-printer-perl 5) go to Reports -> Use saved - everything should work fine -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18642 Josef Moravec <josef.moravec@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |josef.moravec@gmail.com |ity.org | Version|unspecified |master CC| |josef.moravec@gmail.com -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18642 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tomascohen@gmail.com --- Comment #3 from Tomás Cohen Arazi <tomascohen@gmail.com> --- When rebasing the original patches, I introduced the dep for debugging purposes only. We should just remove the code that uses Data::Printer instead! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18642 Josef Moravec <josef.moravec@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63607|0 |1 is obsolete| | --- Comment #4 from Josef Moravec <josef.moravec@gmail.com> --- Created attachment 63609 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63609&action=edit Bug 18642: Remove debug messages Test plan: 0) Apply the patch 1) git grep Data::Printer -> should return no results 2) Go to Reports -> Use saved - should work fine -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18642 --- Comment #5 from Josef Moravec <josef.moravec@gmail.com> --- (In reply to Tomás Cohen Arazi from comment #3)
When rebasing the original patches, I introduced the dep for debugging purposes only. We should just remove the code that uses Data::Printer instead!
Ok, I've attached new patch. Thanks! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18642 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63609|0 |1 is obsolete| | --- Comment #6 from Marc Véron <veron@veron.ch> --- Created attachment 63612 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63612&action=edit Bug 18642: Remove debug messages Test plan: 0) Apply the patch 1) git grep Data::Printer -> should return no results 2) Go to Reports -> Use saved - should work fine Signed-off-by: Marc Véron <veron@veron.ch> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18642 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18642 Jonathan Druart <jonathan.druart@bugs.koha-community.org> 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=18642 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63612|0 |1 is obsolete| | --- Comment #7 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 63615 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63615&action=edit Bug 18642: Remove debug messages Test plan: 0) Apply the patch 1) git grep Data::Printer -> should return no results 2) Go to Reports -> Use saved - should work fine Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18642 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org Summary|Internal Server Error in |Internal Server Error in |Guided reports wizard |Guided reports wizard | |caused by debug messages -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18642 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com Status|Passed QA |Pushed to Master --- Comment #8 from Kyle M Hall <kyle@bywatersolutions.com> --- Pushed to master for 17.05, thanks Josef! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18642 Bug 18642 depends on bug 17898, which changed state. Bug 17898 Summary: Add a way to automatically convert SQL reports https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17898 What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18642 Bug 18642 depends on bug 17898, which changed state. Bug 17898 Summary: Add a way to automatically convert SQL reports https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17898 What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|FIXED |--- -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18642 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|Pushed to Master |RESOLVED CC| |katrin.fischer@bsz-bw.de --- Comment #9 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Depends on bug 17898 which is not part of 16.11.x. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org