Attachment 172222 Details for Bug 37891 – Bug 37891: (QA follow-up): Fix tt_valid QA test

[patch] Bug 37891: (QA follow-up): Fix tt_valid QA test

Bug-37891-QA-follow-up-Fix-ttvalid-QA-test.patch (text/plain), 7.58 KB, created by Pedro Amorim on 2024-09-30 13:51:00 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Pedro Amorim

Created: 2024-09-30 13:51:00 UTC

Size: 7.58 KB

patch

obsolete

>From 779e379a7ec61b3810143c9dd1fa9f1ce8ebe948 Mon Sep 17 00:00:00 2001
>From: Pedro Amorim <pedro.amorim@ptfs-europe.com>
>Date: Mon, 30 Sep 2024 13:47:06 +0000
>Subject: [PATCH] Bug 37891: (QA follow-up): Fix tt_valid QA test
>
>1) Apply both patches
>2) $ qa -c 2 -v 2
>---
> .../prog/en/modules/tools/letter.tt           | 36 +++++++++++++++----
> 1 file changed, 30 insertions(+), 6 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt
>index 00609180ab4..5763163e303 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt
>@@ -640,14 +640,26 @@
>                             <li>
>                                 <label for="is_html_[% letter.message_transport_type | html %]_[% lang | html %]">HTML message:</label>
>                                 [% IF letter.is_html %]
>-                                    <input [% IF sms_readonly %] disabled="disabled" [% END %] type="checkbox" name="is_html_[% letter.message_transport_type | html %]_[% lang | html %]" id="is_html_[% letter.message_transport_type | html %]_[% lang | html %]" value="1" checked="checked" />
>+                                    [% IF sms_readonly %]
>+                                        <input disabled="disabled" type="checkbox" name="is_html_[% letter.message_transport_type | html %]_[% lang | html %]" id="is_html_[% letter.message_transport_type | html %]_[% lang | html %]" value="1" checked="checked" />
>+                                    [% ELSE %]
>+                                        <input type="checkbox" name="is_html_[% letter.message_transport_type | html %]_[% lang | html %]" id="is_html_[% letter.message_transport_type | html %]_[% lang | html %]" value="1" checked="checked" />
>+                                    [% END %]
>                                 [% ELSE %]
>-                                    <input [% IF sms_readonly %] disabled="disabled" [% END %] type="checkbox" name="is_html_[% letter.message_transport_type | html %]_[% lang | html %]" id="is_html_[% letter.message_transport_type | html %]_[% lang | html %]" value="1" />
>+                                    [% IF sms_readonly %]
>+                                        <input disabled="disabled" type="checkbox" name="is_html_[% letter.message_transport_type | html %]_[% lang | html %]" id="is_html_[% letter.message_transport_type | html %]_[% lang | html %]" value="1" />
>+                                    [% ELSE %]
>+                                        <input type="checkbox" name="is_html_[% letter.message_transport_type | html %]_[% lang | html %]" id="is_html_[% letter.message_transport_type | html %]_[% lang | html %]" value="1" />
>+                                    [% END %]
>                                 [% END %]
>                             </li>
>                             <li>
>                                 <label for="title_[% letter.message_transport_type | html %]_[% lang | html %]">Message subject:</label>
>-                                <input [% IF sms_readonly %] readonly [% END %] type="text" id="title_[% letter.message_transport_type | html %]_[% lang | html %]" name="title" size="60" value="[% letter.title | html %]" />
>+                                [% IF sms_readonly %]
>+                                    <input readonly type="text" id="title_[% letter.message_transport_type | html %]_[% lang | html %]" name="title" size="60" value="[% letter.title | html %]" />
>+                                [% ELSE %]
>+                                    <input type="text" id="title_[% letter.message_transport_type | html %]_[% lang | html %]" name="title" size="60" value="[% letter.title | html %]" />
>+                                [% END %]
>                             </li>
>                             <li>
>                                 <label for="SQLfieldname_[% letter.message_transport_type | html %]_[% lang | html %]">Message body:</label>
>@@ -665,17 +677,29 @@
>                                 <table>
>                                     <tr>
>                                         <td>
>-                                            <select [% IF sms_readonly %] disabled="disabled" [% END %] name="SQLfieldname" id="SQLfieldname_[% letter.message_transport_type | html %]_[% lang | html %]" multiple="multiple" size="9">
>+                                            [% IF sms_readonly %]
>+                                                <select disabled="disabled" name="SQLfieldname" id="SQLfieldname_[% letter.message_transport_type | html %]_[% lang | html %]" multiple="multiple" size="9">
>+                                            [% ELSE%]
>+                                                <select name="SQLfieldname" id="SQLfieldname_[% letter.message_transport_type | html %]_[% lang | html %]" multiple="multiple" size="9">
>+                                            [% END %]
>                                                 [% FOREACH SQLfieldname IN SQLfieldnames %]
>                                                     <option value="[% SQLfieldname.value | html %]">[% SQLfieldname.text | html %]</option>
>                                                 [% END %]
>                                             </select>
>                                         </td>
>                                         <td class="actions">
>-                                            <button [% IF sms_readonly %] disabled="disabled" [% END %] type="button" data-containerid="[% letter.message_transport_type | html %]_[% lang | html %]" class="btn btn-default btn-sm insert">Insert <i class="fa-solid fa-right-long"></i></button>
>+                                            [% IF sms_readonly %]
>+                                                <button disabled="disabled" type="button" data-containerid="[% letter.message_transport_type | html %]_[% lang | html %]" class="btn btn-default btn-sm insert">Insert <i class="fa-solid fa-right-long"></i></button>
>+                                            [% ELSE %]
>+                                                <button type="button" data-containerid="[% letter.message_transport_type | html %]_[% lang | html %]" class="btn btn-default btn-sm insert">Insert <i class="fa-solid fa-right-long"></i></button>
>+                                            [% END %]
>                                         </td>
>                                         <td>
>-                                            <textarea [% IF sms_readonly %] readonly [% END %] name="content" data-lang="[% lang | html %]" class="content_[% letter.message_transport_type | html %]" id="content_[% letter.message_transport_type | html %]_[% lang | html %]" cols="80" rows="15">[% letter.content | html %]</textarea>
>+                                        [% IF sms_readonly %]
>+                                            <textarea readonly name="content" data-lang="[% lang | html %]" class="content_[% letter.message_transport_type | html %]" id="content_[% letter.message_transport_type | html %]_[% lang | html %]" cols="80" rows="15">[% letter.content | html %]</textarea>
>+                                        [% ELSE %]
>+                                            <textarea name="content" data-lang="[% lang | html %]" class="content_[% letter.message_transport_type | html %]" id="content_[% letter.message_transport_type | html %]_[% lang | html %]" cols="80" rows="15">[% letter.content | html %]</textarea>
>+                                        [% END %]
>                                         </td>
>                                     </tr>
>                                 </table>
>-- 
>2.39.5

Actions: View | Diff | Splinter Review

Attachments on bug 37891: 171300 | 172037 | 172222 | 172224 | 172225 | 172812 | 172813 | 172818