Bug 7264 made it possible to enter a piece of "OPAC info" for each library, and have it displayed as a tooltip on the library name in the items table on the detail view in the OPAC. This functionality now seems to be broken, I can't get anything to display when I hover over the library name. The Firebug JS console does not show any errors. Tested in Firefox 30.0 on Ubuntu 14.04.
Git bisect blames this commit (Bug 7720): http://git.koha-community.org/gitweb/?p=koha.git;a=commit;h=09ce37d349340f9af4b7bb565381718f5c182e18
Created attachment 28877 [details] [review] Bug 12428 - "OPAC info" is not displayed in the OPAC The changes made by Bug 7720 ("Ambiguity in OPAC Details location") broke the display of OPAC branch information tooltips by making changes to the markup which the tooltip plugin needs to function. This patch corrects the markup to enable tooltips to work again. This patch also makes changes to Items.pm so that branch information tooltips can be shown for both home and holding branches (which are optionally displayed now via the changes by Bug 7720). Before this patch the tooltip would always display the information for the holding branch regardless of the OpacLocationBranchToDisplay setting. This patch also changes the footer include, adding an alias for the jQueryUI tooltip function to prevent conflict with Bootstrap's function of the same name. To test, you must have at least two libraries configured with "OPAC info" for display in the OPAC. Modify the holdings of a title so that there is at least one item which has different holding and home branches matching your library configured above. View the detail page for that record under various values of the OpacLocationBranchToDisplay system preference: - "holding library" : The table of holdings should show a column for only the holding library. Hovering your cursor over the library name should display the branch information you configured for that library in a tooltip . - "home and holding library" : The table of holdings should show columns for both home and holding library. Hovering your cursor over each should show the corresponding library information tooltips. - "home library" : The table of holdings should show a column for only the home library. Hovering your cursor over the library name should display the branch information you configured for that library in a tooltip.
Patch tested with a sandbox, by Christopher Brannon <cbrannon@cdalibrary.org>
Created attachment 29060 [details] [review] Bug 12428 - "OPAC info" is not displayed in the OPAC The changes made by Bug 7720 ("Ambiguity in OPAC Details location") broke the display of OPAC branch information tooltips by making changes to the markup which the tooltip plugin needs to function. This patch corrects the markup to enable tooltips to work again. This patch also makes changes to Items.pm so that branch information tooltips can be shown for both home and holding branches (which are optionally displayed now via the changes by Bug 7720). Before this patch the tooltip would always display the information for the holding branch regardless of the OpacLocationBranchToDisplay setting. This patch also changes the footer include, adding an alias for the jQueryUI tooltip function to prevent conflict with Bootstrap's function of the same name. To test, you must have at least two libraries configured with "OPAC info" for display in the OPAC. Modify the holdings of a title so that there is at least one item which has different holding and home branches matching your library configured above. View the detail page for that record under various values of the OpacLocationBranchToDisplay system preference: - "holding library" : The table of holdings should show a column for only the holding library. Hovering your cursor over the library name should display the branch information you configured for that library in a tooltip . - "home and holding library" : The table of holdings should show columns for both home and holding library. Hovering your cursor over each should show the corresponding library information tooltips. - "home library" : The table of holdings should show a column for only the home library. Hovering your cursor over the library name should display the branch information you configured for that library in a tooltip. Signed-off-by: Christopher Brannon <cbrannon@cdalibrary.org>
Everything works as outlined in the test plan. Thank you, Owen, for a great patch. Christopher
Created attachment 29062 [details] [review] [PASSED QA] Bug 12428 - "OPAC info" is not displayed in the OPAC The changes made by Bug 7720 ("Ambiguity in OPAC Details location") broke the display of OPAC branch information tooltips by making changes to the markup which the tooltip plugin needs to function. This patch corrects the markup to enable tooltips to work again. This patch also makes changes to Items.pm so that branch information tooltips can be shown for both home and holding branches (which are optionally displayed now via the changes by Bug 7720). Before this patch the tooltip would always display the information for the holding branch regardless of the OpacLocationBranchToDisplay setting. This patch also changes the footer include, adding an alias for the jQueryUI tooltip function to prevent conflict with Bootstrap's function of the same name. To test, you must have at least two libraries configured with "OPAC info" for display in the OPAC. Modify the holdings of a title so that there is at least one item which has different holding and home branches matching your library configured above. View the detail page for that record under various values of the OpacLocationBranchToDisplay system preference: - "holding library" : The table of holdings should show a column for only the holding library. Hovering your cursor over the library name should display the branch information you configured for that library in a tooltip . - "home and holding library" : The table of holdings should show columns for both home and holding library. Hovering your cursor over each should show the corresponding library information tooltips. - "home library" : The table of holdings should show a column for only the home library. Hovering your cursor over the library name should display the branch information you configured for that library in a tooltip. Signed-off-by: Christopher Brannon <cbrannon@cdalibrary.org> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Passes QA script and tests. Works with all settings of OpacLocationBranchToDisplay.
Created attachment 29179 [details] [review] Bug 12428: (QA followup) regression tests. Just that. Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Patch pushed to master. Thanks Owen!
Will this patch get back ported?
(In reply to Ed Veal from comment #9) > Will this patch get back ported? Please ping Galen on this matter.
Created attachment 34094 [details] [review] Bug 12428 [3.16.x] "OPAC info" is not displayed in the OPAC This patch changes the footer include, adding an alias for the jQueryUI tooltip function to prevent conflict with Bootstrap's function of the same name. To test, you must have at least two libraries configured with "OPAC info" for display in the OPAC. Modify the holdings of a title so that there is at least one item which has different holding and home branches matching your library configured above. View the detail page for that record. Hovering your cursor over the library name in the "Location" column should display the branch information you configured for that library in a tooltip.
I applied and tested this patch on a sandbox. It works like expected. But I was not able to change status in "signed off". When I tried, I had the following message "It seems you don't have applied a patch, so you cannot sign it off. If you applied patches from the right report, check the commit message of the last patch. It should start with "Bug XXXXX", if not, please inform the author of the patch."
This patch is targetted at 3.16 specifically, so I think it might not be testable on a sandbox successfully.
Created attachment 34692 [details] [review] [SIGNED OFF]Bug 12428 [3.16.x] "OPAC info" is not displayed in the OPAC This patch changes the footer include, adding an alias for the jQueryUI tooltip function to prevent conflict with Bootstrap's function of the same name. To test, you must have at least two libraries configured with "OPAC info" for display in the OPAC. Modify the holdings of a title so that there is at least one item which has different holding and home branches matching your library configured above. View the detail page for that record. Hovering your cursor over the library name in the "Location" column should display the branch information you configured for that library in a tooltip. Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Created attachment 34722 [details] [review] [PASSED QA] Bug 12428 [3.16.x] "OPAC info" is not displayed in the OPAC This patch changes the footer include, adding an alias for the jQueryUI tooltip function to prevent conflict with Bootstrap's function of the same name. To test, you must have at least two libraries configured with "OPAC info" for display in the OPAC. Modify the holdings of a title so that there is at least one item which has different holding and home branches matching your library configured above. View the detail page for that record. Hovering your cursor over the library name in the "Location" column should display the branch information you configured for that library in a tooltip. Signed-off-by: Nick Clemens <nick@quecheelibrary.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to 3.16.x, patch will be in 3.16.7
Patch is in 3.16.x, I set to Pushed to stable
Pushed to 3.14.x will be in 3.14.13