[Bug 26766] New: Don't show star rating in dialog when saving a checkout note
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26766 Bug ID: 26766 Summary: Don't show star rating in dialog when saving a checkout note Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: OPAC Assignee: oleonard@myacpl.org Reporter: katrin.fischer@bsz-bw.de QA Contact: testopia@bugs.koha-community.org When CheckoutNotes is enabled and you enter a note for a checkout in the OPAC, it displays the title and "has been saved and sent to the library." Between the title and the message the ratings appear now, which looks not quite right there. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26766 --- Comment #1 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- You'll also have to activate star ratings! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26766 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26766 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Trivial patch Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26766 --- Comment #2 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 112114 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112114&action=edit Bug 26766: Correct selection of title during OPAC note confirmation This patch modifies the JavaScript in opac-user.tt which executes when the user submits a checkout note. The script tries to grab the checkout's title from an earlier cell in the table row so that it can display it in the confirmation message. Before start ratings were enabled on this page, the title of each checked-out item was the only HTML inside this table cell, so it was necessary only to grab the HTML contents of the <td>. Now that star ratings might be displayed as well we must make the selection more specific. To test, apply the patch and enable both the OpacStarRatings and AllowCheckoutNotes preferences. - Log in to the OPAC as a user with checkouts. - On the "Your summary" page, fill out and submit the "Note" field for one of your checkouts. - In the confirmation message, confirm that the correct title information is displayed as a link to the bibliographic detail page. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26766 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26766 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112114|0 |1 is obsolete| | --- Comment #3 from David Nind <david@davidnind.com> --- Created attachment 112445 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112445&action=edit Bug 26766: Correct selection of title during OPAC note confirmation This patch modifies the JavaScript in opac-user.tt which executes when the user submits a checkout note. The script tries to grab the checkout's title from an earlier cell in the table row so that it can display it in the confirmation message. Before start ratings were enabled on this page, the title of each checked-out item was the only HTML inside this table cell, so it was necessary only to grab the HTML contents of the <td>. Now that star ratings might be displayed as well we must make the selection more specific. To test, apply the patch and enable both the OpacStarRatings and AllowCheckoutNotes preferences. - Log in to the OPAC as a user with checkouts. - On the "Your summary" page, fill out and submit the "Note" field for one of your checkouts. - In the confirmation message, confirm that the correct title information is displayed as a link to the bibliographic detail page. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26766 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |nick@bywatersolutions.com |y.org | CC| |nick@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26766 Katrin Fischer <katrin.fischer@bsz-bw.de> 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=26766 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #112445|0 |1 is obsolete| | --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 112447 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112447&action=edit Bug 26766: Correct selection of title during OPAC note confirmation This patch modifies the JavaScript in opac-user.tt which executes when the user submits a checkout note. The script tries to grab the checkout's title from an earlier cell in the table row so that it can display it in the confirmation message. Before start ratings were enabled on this page, the title of each checked-out item was the only HTML inside this table cell, so it was necessary only to grab the HTML contents of the <td>. Now that star ratings might be displayed as well we must make the selection more specific. To test, apply the patch and enable both the OpacStarRatings and AllowCheckoutNotes preferences. - Log in to the OPAC as a user with checkouts. - On the "Your summary" page, fill out and submit the "Note" field for one of your checkouts. - In the confirmation message, confirm that the correct title information is displayed as a link to the bibliographic detail page. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26766 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |20.11.00 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26766 --- Comment #5 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Pushed to master for 20.11, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26766 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)|20.11.00 |20.11.00, 20.05.06 released in| | CC| |lucas@bywatersolutions.com Status|Pushed to master |Pushed to stable --- Comment #6 from Lucas Gass <lucas@bywatersolutions.com> --- backported to 20.05.x for 20.05.06 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26766 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |aleisha@catalyst.net.nz --- Comment #7 from Aleisha Amohia <aleisha@catalyst.net.nz> --- enhancement, not backported to 19.11.x -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26766 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to stable |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org