[Bug 44068] New: Log Viewer gives 500 error, if output "to screen in the browser"
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=44068 Bug ID: 44068 Summary: Log Viewer gives 500 error, if output "to screen in the browser" Initiative type: --- Sponsorship --- status: Product: Koha Version: 26.05 Hardware: All OS: Linux Status: NEW Severity: normal Priority: P5 - low Component: Tools Assignee: koha-bugs@lists.koha-community.org Reporter: esa-pekka.keskitalo@helsinki.fi QA Contact: testopia@bugs.koha-community.org Target Milestone: --- Confirmed in our production servers 26.05 as well as a Openfifth sandbox Koha 26.06.00.050 1. Log in with permission to Log viewer 2. Select any combination of parameters. 3. Select Output: To screen in the browser 4. Click Submit Expected result: the log dispalyed on the screen What happens: error message "Something went wrong when loading the table. 500: Internal Server Error." Closing the error dialogue box, there is a throbber sayinig "processing". Where the log entries should show, it says "Loading..." N.B. Select Output: To a File works normally. -- 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=44068 E-P Keskitalo <esa-pekka.keskitalo@helsinki.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Hardware|All |Macintosh -- 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=44068 Andrii Nugged <nugged@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |nugged@gmail.com |ity.org | -- 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=44068 --- Comment #1 from E-P Keskitalo <esa-pekka.keskitalo@helsinki.fi> --- Further narrowed down. 500 only occurs if either Display from or Display to value is give, or both. If the dates are empty, output to screen works normally. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=44068 --- Comment #2 from Andrii Nugged <nugged@gmail.com> --- Created attachment 211473 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=211473&action=edit Bug 44068: Qualify log viewer date filters The log viewer's screen output joins item data, so an unqualified timestamp condition is ambiguous between action_logs and items. Setting either date limit therefore causes an API error. Qualify all three date-filter shapes with the main resultset alias. Keep the existing inclusive day boundaries and CSV behavior unchanged. Add an end-to-end Cypress test covering no dates, either date limit, and a date range against real action log entries. Test plan: Before applying the patch: 1. Go to Tools > Log viewer. 2. Click Submit with both date fields empty. Log entries should appear. 3. Click the Log viewer breadcrumb to return to the search form. 4. Set Display from to a date containing a known log entry. 5. Click Submit. The table fails to load with a 500 error. Apply the patch. After applying the patch: 1. Repeat steps 1-5 above. The dated entries should appear without an error, and the processing indicator should disappear. 2. Return to the search form using the Log viewer breadcrumb. 3. Clear Display from. 4. Set Display to to the same date. 5. Click Submit. Entries up to that date should appear. 6. Return to the search form using the Log viewer breadcrumb. 7. Set both date fields to the known entry's date. 8. Click Submit. Only entries from that day should appear. 9. Return to the search form using the Log viewer breadcrumb. 10. Select To a file with the same dates. 11. Click Submit. A CSV file should download. Automated tests: yarn cypress run --spec t/cypress/integration/Tools/LogViewer_spec.ts Co-Authored-By: Loontik <loontik@nugged.team> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=44068 Andrii Nugged <nugged@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=44068 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|26.05 |Main OS|Linux |All Depends on| |43868 Hardware|Macintosh |All Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43868 [Bug 43868] Log viewer search does not find circulation entries by item barcode -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org