[Bug 7894] New: Broken detail page for first link on page 2 from search result page in OPAC
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 Priority: P5 - low Change sponsored?: --- Bug ID: 7894 Assignee: oleonard@myacpl.org Summary: Broken detail page for first link on page 2 from search result page in OPAC QA Contact: koha.sekjal@gmail.com Severity: blocker Classification: Unclassified OS: All Reporter: arslanone@gmail.com Hardware: All Status: NEW Version: master Component: OPAC Product: Koha Hi, When you search the OPAC, clicking on the 1st result of page 2 always gives this error: ------------------------------------------- "Software error: Can't use string ("00800nam a22002537a 450000100050") as an ARRAY ref while "strict refs" in use at /usr/share/koha/lib/C4/Search.pm line 1493." ------------------------------------------- Screenshot: http://postimage.org/image/ouqivltox/ The error only happens when we search in the OPAC, go to page 2, and click on 1st record. Search a different term and, that will result in some different record on page 2 top, and clicking that record will give same error. Right click and copy the link of that record, and open it by directly pasting in another browser and the record will show properly. Meaning there is nothing wrong with the record itself. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 --- Comment #1 from MJ Ray (software.coop) <mjr@software.coop> --- Created attachment 8859 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8859&action=edit bug 7894: crude fix: convert results array into an arrayref -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 MJ Ray (software.coop) <mjr@software.coop> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff CC| |mjr@software.coop --- Comment #2 from MJ Ray (software.coop) <mjr@software.coop> --- Crude fix attached, but this functionality really should be optional and enabled by a system preference. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 --- Comment #3 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- Agreed about the syspref. This patch seems to break the results pagination on other records, though. With the patch applied, "Previous" and "Next" links are grayed out on other records. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 Magnus Enger <magnus@enger.priv.no> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |magnus@enger.priv.no --- Comment #4 from Magnus Enger <magnus@enger.priv.no> --- I tested the patch and while it does make the first record on the second page of results appear OK, the links in the "pager" thing on the right (Next/Previous) are somewhat broken. On record 20 the link to record 21 is inactive, and on record 21 the link to record 20 is inactive. So the next/previous links are noe working from one page of results to the next. This seems to be working on a 3.6.3 install though. Not sure if I should sign off so we can get the main fix in, or if it's worth waiting for a solution to the next/previous problem? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 MJ Ray (software.coop) <mjr@software.coop> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 MJ Ray (software.coop) <mjr@software.coop> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff --- Comment #5 from MJ Ray (software.coop) <mjr@software.coop> --- I've moved this back to Needs Signoff because I've confirmed that Next/Previous don't work across the boundary in 3.6.4 as running on http://catalog-test.nexpresslibrary.org/ - so that is a different bug and I think it's less severe than crashing out on the first result of later pages. I've also confirmed that Next/Previous works for me within the first and second results pages. I've discovered an entertaining bug which I will report now. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 Magnus Enger <magnus@enger.priv.no> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #8859|0 |1 is obsolete| | --- Comment #6 from Magnus Enger <magnus@enger.priv.no> --- Created attachment 8889 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8889&action=edit bug 7894: crude fix: convert results array into an arrayref Signed-off-by: Magnus Enger <magnus@enger.priv.no> This patch fixes the problem with the first record on the second page, but there is still a problem with the next/previous links, as described in comments 3 and 4 in the bug report. I'm signing off since it's unclear wether that problem is directly linked to the "first record on the second page" problem, and a partial fix seems better than no fix at all. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 Magnus Enger <magnus@enger.priv.no> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #8889|0 |1 is obsolete| | --- Comment #7 from Magnus Enger <magnus@enger.priv.no> --- Created attachment 8890 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8890&action=edit bug 7894 [SIGNED-OFF]: crude fix: convert results array into an arrayref Signed-off-by: Magnus Enger <magnus@enger.priv.no> This patch fixes the problem with the first record on the second page, but there is still a problem with the next/previous links, as described in comments 3 and 4 in the bug report. I'm signing off since it's unclear wether that problem is directly linked to the "first record on the second page" problem, and a partial fix seems better than no fix at all. Signed-off-by: Magnus Enger <magnus@enger.priv.no> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 Magnus Enger <magnus@enger.priv.no> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |paul.poulain@biblibre.com --- Comment #8 from Paul Poulain <paul.poulain@biblibre.com> --- QA comment: tiny patch, passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master Version|master |rel_3_8 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 Magnus Enger <magnus@enger.priv.no> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jcamins@cpbibliography.com --- Comment #9 from Magnus Enger <magnus@enger.priv.no> --- I'm seeing this in 3.6.4 too. Looks like the patch should be in 3.6.5, Jared? I'm also getting the same error from the RSS feeds in 3.6.4: Do a simple search from the front page, klick on the RSS icon and click on any title in the feed. I get this error: Can't use string ("02361 a2200565 450000100150") as an ARRAY ref while "strict refs" in use at /usr/share/koha/lib/C4/Search.pm line 1502. Which is the same one i get for the first record on the second page. (I'm guessing some other change to Search.pm made the line with error move after Arslan's first report of this problem.) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 Magnus Enger <magnus@enger.priv.no> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |vinod_librarian@rediffmail. | |com --- Comment #10 from Magnus Enger <magnus@enger.priv.no> --- *** Bug 8225 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=8293 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@biblibre.co | |m --- Comment #11 from Jonathan Druart <jonathan.druart@biblibre.com> --- See Bug 8293 for the next -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|oleonard@myacpl.org |mjr@software.coop -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7894 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable CC| |chris@bigballofwax.co.nz -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org