Description
George Veranis
2023-02-08 12:09:33 UTC
Created attachment 146387 [details]
Staff_Interface-Bibliographic_record_details-Item_type_icon_OK
Steps to reproduce: 1) (if needed) Configure the following System preferences: - item-level_itypes => 'specific item' - noItemTypeImages => 'Show' 2) (if needed) In Koha Administration > Item types, create an item type and associate it with an icon 3) (if needed) Create a test bibliographic record, and add an item to it that has an item type with an icon Then, visit the bibliographic record's details page in the Staff interface. In the 'Item type' column of the 'Holdings' tab, notice that instead of seeing the icon associated with the item type of each item you get the default 'broken image URL' image of your web browser. [If you try the above in a Dockerised Koha instance the icon will -most likely- be displayed just fine. However, you can still verify that it's being loaded from the wrong path by right-clicking on the icon and selecting 'Inspect'.] Created attachment 146388 [details] [review] Bug 32908: Load item type icons from the correct path In the Staff interface, the details page of a bibliographic record should display the icon associated with the item type of an item ('Item type' column in the 'Holdings' tab). This feature broke in v22.11.00 (for Debian package installs): instead of the item type icon you get the default 'broken image URL' image of your web browser. This patch fixes that. Test plan: 1) (if needed) Configure the following System preferences: - item-level_itypes => 'specific item' - noItemTypeImages => 'Show' 2) (if needed) In Koha Administration > Item types, create an item type and associate it with an icon 3) (if needed) Create a test bibliographic record, and add an item to it that has an item type with an icon 4) Visit the bibliographic record's details page in the Staff interface. Notice that in the 'Item type' column of the 'Holdings' tab instead of seeing the icon associated with the item type of each item you get the default 'broken image URL' image of your web browser [*]. [*] In non-Debian package installs, the 'href' property of the item type icon's <img> tag should start with '/opac-tmpl/bootstrap/itemtypeimg' 5) Apply this patch, and refresh the page with CTRL-F5. The item type icon should now be displayed correctly. [*] KTD and Koha Sandboxes users: To verify the patch has worked, you should right-click on the item type icon and click 'Inspect: the 'href' property of the <img> tag should now start with: '/intranet-tmpl/prog/img/itemtypeimg' Mentored by : Andreas Roussos Created attachment 146393 [details] [review] Bug 32908: Load item type icons from the correct path In the Staff interface, the details page of a bibliographic record should display the icon associated with the item type of an item ('Item type' column in the 'Holdings' tab). This feature broke in v22.11.00 (for Debian package installs): instead of the item type icon you get the default 'broken image URL' image of your web browser. This patch fixes that. Test plan: 1) (if needed) Configure the following System preferences: - item-level_itypes => 'specific item' - noItemTypeImages => 'Show' 2) (if needed) In Koha Administration > Item types, create an item type and associate it with an icon 3) (if needed) Create a test bibliographic record, and add an item to it that has an item type with an icon 4) Visit the bibliographic record's details page in the Staff interface. Notice that in the 'Item type' column of the 'Holdings' tab instead of seeing the icon associated with the item type of each item you get the default 'broken image URL' image of your web browser [*]. [*] In non-Debian package installs, the 'href' property of the item type icon's <img> tag should start with '/opac-tmpl/bootstrap/itemtypeimg' 5) Apply this patch, and refresh the page with CTRL-F5. The item type icon should now be displayed correctly. [*] KTD and Koha Sandboxes users: To verify the patch has worked, you should right-click on the item type icon and click 'Inspect: the 'href' property of the <img> tag should now start with: '/intranet-tmpl/prog/img/itemtypeimg' Signed-off-by: Frédéric Demians <f.demians@tamil.fr> Created attachment 146493 [details] [review] Bug 32908: Load item type icons from the correct path In the Staff interface, the details page of a bibliographic record should display the icon associated with the item type of an item ('Item type' column in the 'Holdings' tab). This feature broke in v22.11.00 (for Debian package installs): instead of the item type icon you get the default 'broken image URL' image of your web browser. This patch fixes that. Test plan: 1) (if needed) Configure the following System preferences: - item-level_itypes => 'specific item' - noItemTypeImages => 'Show' 2) (if needed) In Koha Administration > Item types, create an item type and associate it with an icon 3) (if needed) Create a test bibliographic record, and add an item to it that has an item type with an icon 4) Visit the bibliographic record's details page in the Staff interface. Notice that in the 'Item type' column of the 'Holdings' tab instead of seeing the icon associated with the item type of each item you get the default 'broken image URL' image of your web browser [*]. [*] In non-Debian package installs, the 'href' property of the item type icon's <img> tag should start with '/opac-tmpl/bootstrap/itemtypeimg' 5) Apply this patch, and refresh the page with CTRL-F5. The item type icon should now be displayed correctly. [*] KTD and Koha Sandboxes users: To verify the patch has worked, you should right-click on the item type icon and click 'Inspect: the 'href' property of the <img> tag should now start with: '/intranet-tmpl/prog/img/itemtypeimg' Signed-off-by: Frédéric Demians <f.demians@tamil.fr> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Thanks for the fix, Passing QA Did you check reserve/request? (In reply to Jonathan Druart from comment #8) > Did you check reserve/request? forget that, just seen bug 32909 now! Pushed to master for 23.05. Nice work everyone, thanks! Nice work everyone! Pushed to 22.11.x for next release Missing dependencies for 22.05.x, no backport. |