Attachment 115932 Details for Bug 27493 – Bug 27493: (follow-up) Remove redundant class

[patch] Bug 27493: (follow-up) Remove redundant class

Bug-27493-follow-up-Remove-redundant-class.patch (text/plain), 14.61 KB, created by Owen Leonard on 2021-01-28 12:59:56 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2021-01-28 12:59:56 UTC

Size: 14.61 KB

patch

obsolete

>From 764d616ec2c8abd1411d8e00611707993f47807e Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Thu, 28 Jan 2021 12:45:09 +0000
>Subject: [PATCH] Bug 27493: (follow-up) Remove redundant class
>
>This patch modifies the templates so that instances of "class='select
>selectcol' are replaced with "class='selectcol'". I could find no
>instances of the "select" class being used in CSS or JS.
>---
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-basket.tt        |  2 +-
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt     | 10 +++++-----
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt       |  2 +-
> .../opac-tmpl/bootstrap/en/modules/opac-search-history.tt      |  8 ++++----
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-shelves.tt       |  2 +-
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt   |  2 +-
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-tags.tt          |  2 +-
> 7 files changed, 14 insertions(+), 14 deletions(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-basket.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-basket.tt
>index 89932cb36b..72969e98d5 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-basket.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-basket.tt
>@@ -268,7 +268,7 @@
>                                     <tbody>
>                                     [% FOREACH BIBLIO_RESULT IN BIBLIO_RESULTS %]
>                                         <tr>
>-                                                <td class="noprint select selectcol">
>+                                                <td class="noprint selectcol">
>                                                 <input type="checkbox" class="cb" value="[% BIBLIO_RESULT.biblionumber | html %]" name="bib[% BIBLIO_RESULT.biblionumber | html %]" id="bib[% BIBLIO_RESULT.biblionumber | html %]">
>                                                 </td>
>                                             <td>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt
>index f6766b2d8f..ef0b7af5d5 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt
>@@ -85,7 +85,7 @@
>                                             [% END %]
>                                             [% IF ( SMSSendDriver ) %]
>                                                 [% IF ( messaging_preference.transport_sms ) %]
>-                                                    <td class="select selectcol">
>+                                                    <td class="selectcol">
>                                                         [% IF ( messaging_preference.transports_sms ) %]
>                                                         <input type="checkbox" id="sms[% messaging_preference.message_attribute_id | html %]" name="[% messaging_preference.message_attribute_id | html %]" value="sms" checked="checked" />
>                                                         [% ELSE %]
>@@ -98,7 +98,7 @@
>                                             [% END %]
>                                             [% IF Koha.Preference('PhoneNotification') %]
>                                                 [% IF ( messaging_preference.transport_phone ) %]
>-                                                    <td class="select selectcol">
>+                                                    <td class="selectcol">
>                                                         [% IF ( messaging_preference.transports_phone ) %]
>                                                             <input type="checkbox"
>                                                                    id="phone[% messaging_preference.message_attribute_id | html %]"
>@@ -117,7 +117,7 @@
>                                             [% END %]
>                                             [% IF ( TalkingTechItivaPhone ) %]
>                                                 [% IF ( messaging_preference.transport_itiva ) %]
>-                                                <td class="select selectcol">
>+                                                <td class="selectcol">
>                                                 [% IF ( messaging_preference.transports_itiva ) %]
>                                                 <input type="checkbox"
>                                                             id="itiva[% messaging_preference.message_attribute_id | html %]"
>@@ -136,7 +136,7 @@
>                                             [% END %]
> 
>                                             [% IF ( messaging_preference.transport_email ) %]
>-                                                <td class="select selectcol">
>+                                                <td class="selectcol">
>                                                     [% IF ( messaging_preference.transports_email ) %]
>                                                         <input type="checkbox" id="email[% messaging_preference.message_attribute_id | html %]" name="[% messaging_preference.message_attribute_id | html %]" value="email" checked="checked" onclick = "document.opacmessaging.none[% messaging_preference.message_attribute_id | html %].checked=false;" />
>                                                     [% ELSE %]
>@@ -148,7 +148,7 @@
>                                             [% END %]
> 
>                                             [% IF ( messaging_preference.has_digest ) %]
>-                                                <td class="select selectcol">
>+                                                <td class="selectcol">
>                                                     [% IF ( messaging_preference.digest ) %]
>                                                         <input type="checkbox" id="digest[% messaging_preference.message_attribute_id | html %]" value="[% messaging_preference.message_attribute_id | html %]" name="digest" checked="checked" onclick = "document.opacmessaging.none[% messaging_preference.message_attribute_id | html %].checked=false;" />
>                                                     [% ELSE %]
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>index 188af083bd..b5f84eb01d 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>@@ -253,7 +253,7 @@
>                                             [% ELSE %]
>                                                 [% check_title = SEARCH_RESULT.biblionumber %]
>                                             [% END %]
>-                                            <td class="select selectcol">
>+                                            <td class="selectcol">
>                                                 [% IF Koha.Preference( 'opacbookbag' ) == 1 %]
>                                                     <input type="checkbox" class="cb" id="bib[% SEARCH_RESULT.biblionumber | html %]" name="biblionumber" value="[% SEARCH_RESULT.biblionumber | html %]" aria-label="Select search result: [% check_title | html %]" /> <label for="bib[% SEARCH_RESULT.biblionumber | html %]"></label>
>                                                 [% ELSE %]
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-search-history.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-search-history.tt
>index 5c4d6fe8f5..6d8e9e3f71 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-search-history.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-search-history.tt
>@@ -80,7 +80,7 @@
>                                             <tbody>
>                                                 [% FOREACH s IN current_biblio_searches %]
>                                                     <tr>
>-                                                        <td class="select selectcol"><input type="checkbox" name="id" value="[% s.id | html %]" /></td>
>+                                                        <td class="selectcol"><input type="checkbox" name="id" value="[% s.id | html %]" /></td>
>                                                         <td><span title="[% s.time | html %]">[% s.time |$KohaDates  with_hours => 1 %]</span></td>
>                                                         <td>
>                                                             <a href="[% OPACBaseURL | url %]/cgi-bin/koha/opac-search.pl?[% query_cgi | $raw %][% limit_cgi | $raw %]&amp;[% s.query_cgi | $raw %]&amp;count=[% countrss | uri %]&amp;sort_by=acqdate_dsc&amp;format=rss" aria-label="Subscribe to this search" class="rsssearchlink">
>@@ -125,7 +125,7 @@
>                                             <tbody>
>                                             [% FOREACH s IN previous_biblio_searches %]
>                                                 <tr>
>-                                                    <td class="select selectcol"><input type="checkbox" name="id" value="[% s.id | html %]" /></td>
>+                                                    <td class="selectcol"><input type="checkbox" name="id" value="[% s.id | html %]" /></td>
>                                                     <td><span title="[% s.time | html %]">[% s.time |$KohaDates  with_hours => 1 %]</span></td>
>                                                     <td>
>                                                         <a href="[% OPACBaseURL | url %]/cgi-bin/koha/opac-search.pl?[% query_cgi | $raw %][% limit_cgi | $raw %]&amp;[% s.query_cgi | $raw %]&amp;count=[% countrss | uri %]&amp;sort_by=acqdate_dsc&amp;format=rss" aria-label="Subscribe to this search" class="rsssearchlink">
>@@ -177,7 +177,7 @@
>                                             <tbody>
>                                                 [% FOREACH s IN current_authority_searches %]
>                                                     <tr>
>-                                                        <td class="select selectcol"><input type="checkbox" name="id" value="[% s.id | html %]" /></td>
>+                                                        <td class="selectcol"><input type="checkbox" name="id" value="[% s.id | html %]" /></td>
>                                                         <td><span title="[% s.time | html %]">[% s.time |$KohaDates  with_hours => 1 %]</span></td>
>                                                         <td><a href="/cgi-bin/koha/opac-authorities-home.pl?[% s.query_cgi | $raw %]">[% s.query_desc | html %]</a></td>
>                                                         <td>[% s.total | html %]</td>
>@@ -214,7 +214,7 @@
>                                             <tbody>
>                                                 [% FOREACH s IN previous_authority_searches %]
>                                                     <tr>
>-                                                        <td class="select selectcol"><input type="checkbox" name="id" value="[% s.id | html %]" /></td>
>+                                                        <td class="selectcol"><input type="checkbox" name="id" value="[% s.id | html %]" /></td>
>                                                         <td><span title="[% s.time | html %]">[% s.time |$KohaDates  with_hours => 1 %]</span></td>
>                                                         <td><a href="/cgi-bin/koha/opac-authorities-home.pl?[% s.query_cgi | $raw %]">[% s.query_desc | html %]</a></td>
>                                                         <td>[% s.total | html %]</td>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-shelves.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-shelves.tt
>index 4ee6813fc7..85d27922e2 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-shelves.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-shelves.tt
>@@ -369,7 +369,7 @@
>                                                     [% ELSE %]
>                                                         [% check_title = itemsloo.biblionumber %]
>                                                     [% END %]
>-                                                    <td class="select selectcol"><input type="checkbox" class="cb" name="biblionumber" value="[% itemsloo.biblionumber | html %]" aria-label="[% check_title | html %]"/></td>
>+                                                    <td class="selectcol"><input type="checkbox" class="cb" name="biblionumber" value="[% itemsloo.biblionumber | html %]" aria-label="[% check_title | html %]"/></td>
>                                                     [% UNLESS ( item_level_itypes ) %]
>                                                         <td>
>                                                             [% UNLESS ( Koha.Preference('OpacNoItemTypeImages') ) %]
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt
>index 8249a03ef0..92b1a4fd47 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt
>@@ -354,7 +354,7 @@
>                                             [% FOREACH suggestions_loo IN suggestions_loop %]
>                                                 <tr>
>                                                     [% IF ( loggedinusername ) %]
>-                                                        <td class="select selectcol">
>+                                                        <td class="selectcol">
>                                                             [% IF ( suggestions_loo.showcheckbox ) %]
>                                                                 [% SET can_delete_suggestion = 1 %]
>                                                                 <input type="checkbox" class="cb" id="id[% suggestions_loo.suggestionid | html %]" name="delete_field" data-title="[% suggestions_loo.title | html %]" value="[% suggestions_loo.suggestionid | html %]" />
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-tags.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-tags.tt
>index 41f8962a66..5e926ce77c 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-tags.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-tags.tt
>@@ -138,7 +138,7 @@
>                                     [% FOREACH MY_TAG IN MY_TAGS %]
>                                         [% IF MY_TAG.visible %]
>                                         <tr>
>-                                            <td class="select selectcol">
>+                                            <td class="selectcol">
>                                                 <input type="checkbox" name="del[% MY_TAG.tag_id | html %]" value="del[% MY_TAG.tag_id | html %]">
>                                             </td>
>                                             <td class="tagterm">
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 27493: 115416 | 115428 | 115753 | 115754 | 115847 | 115922 | 115923 | 115932