Attachment 149787 Details for Bug 33478 – Bug 33478: [WIP] Allow formatting of notices

[patch] Bug 33478: [WIP] Allow formatting of notices

Bug-33478-WIP-Allow-formatting-of-notices.patch (text/plain), 10.67 KB, created by Aleisha Amohia on 2023-04-18 04:59:03 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Aleisha Amohia

Created: 2023-04-18 04:59:03 UTC

Size: 10.67 KB

patch

obsolete

>From d0a04e438bafd684c1ee40f183184917568d87bf Mon Sep 17 00:00:00 2001
>From: Aleisha Amohia <aleishaamohia@hotmail.com>
>Date: Tue, 18 Apr 2023 04:58:29 +0000
>Subject: [PATCH] Bug 33478: [WIP] Allow formatting of notices
>
>---
> .../prog/en/modules/tools/letter.tt           | 136 ++++++++++++++++++
> tools/letter.pl                               |   9 +-
> 2 files changed, 144 insertions(+), 1 deletion(-)
>
>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 910deebfef5..15987f31656 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt
>@@ -49,6 +49,14 @@
> [% END %]
> </head>
> 
>+[% BLOCK translate_justification_types %]
>+    [%  SWITCH type %]
>+        [%   CASE 'L' %]<span>Left</span>
>+        [%   CASE 'C' %]<span>Center</span>
>+        [%   CASE 'R' %]<span>Right</span>
>+    [%  END %]
>+[% END %]
>+
> <body id="tools_letter" class="tools">
>     [% WRAPPER 'header.inc' %]
>     [% INCLUDE 'letters-search.inc' %]
>@@ -648,6 +656,134 @@
>                 </div> 
>             </div> 
>         [% END # /FOR mtt %]
>+
>+        <div class="panel panel-default">
>+            <div class="panel-heading" role="tab" id="format_[% lang | html %]Heading">
>+                <h3 class="panel-title">
>+                    <a role="button" class="collapsed" data-toggle="collapse" data-parent="#group_[% lang | html %]" href="#format_[% lang | html %]" aria-expanded="false" aria-controls="format_[% lang | html %]">
>+                        <span>Print format</span>
>+                    </a>
>+                </h3>
>+            </div> 
>+            <div id="format_[% lang | html %]" class="panel-collapse collapse" role="tabpanel" aria-labelledby="format_[% lang | html %]Heading">
>+                <div class="panel-body">
>+                    <fieldset class="rows">
>+                        <input type="hidden" name="lang" value="[% lang | html %]" />
>+                        <div class="col-md-6">
>+                            <ol>
>+                                <li>
>+                                    <label for="text_justify">Text justification: </label>
>+                                    <select name="text_justify" id="text_justify">
>+                                        [% FOREACH text_justification_type IN text_justification_types %]
>+                                            [% IF ( text_justification_type.type == letter.text_justify ) %]
>+                                            <option value="[% text_justification_type.type | html %]" selected="selected">[% PROCESS translate_justification_types type=text_justification_type.type %]</option>
>+                                            [% ELSE %]
>+                                            <option value="[% text_justification_type.type | html %]">[% PROCESS translate_justification_types type=text_justification_type.type %]</option>
>+                                            [% END %]
>+                                        [% END %]
>+                                    </select>
>+                                </li>
>+                                <li>
>+                                    <label for="font">Font: </label>
>+                                    <select name="font" id="font">
>+                                        [% FOREACH font_type IN font_types %]
>+                                            [% IF ( font_type.type == letter.font ) %]
>+                                            <option value="[% font_type.type | html %]" selected="selected">[% font_type.name | html %]</option>
>+                                            [% ELSE %]
>+                                            <option value="[% font_type.type | html %]">[% font_type.name | html %]</option>
>+                                            [% END %]
>+                                        [% END %]
>+                                    </select>
>+                                </li>
>+                                <li>
>+                                    <label for="font_size">Font size: </label>
>+                                    <input type="text" name="font_size" id="font_size" size="2" value="[% letter.font_size | html %]" />
>+                                </li>
>+                                <li>
>+                                    <label for="units">Units:</label>
>+                                    <select id="units" name="units">
>+                                    [% FOREACH unit IN units %]
>+                                        [% IF ( unit.type == letter.units ) %]
>+                                        <option value="[% unit.type | html %]" selected="selected">
>+                                        [% ELSE %]
>+                                        <option value="[% unit.type | html %]">
>+                                        [% END %]
>+
>+                                            [% SWITCH unit.type %]
>+                                            [%   CASE 'POINT' %]
>+                                            <span>PostScript points</span>
>+                                            [%   CASE 'AGATE' %]
>+                                            <span>Agates</span>
>+                                            [%   CASE 'INCH' %]
>+                                            <span>US Inches</span>
>+                                            [%   CASE 'MM' %]
>+                                            <span>SI Millimeters</span>
>+                                            [%   CASE 'CM' %]
>+                                            <span>SI Centimeters</span>
>+                                            [% END %]
>+                                        </option>
>+                                    [% END %]
>+                                    </select>
>+                                </li>
>+                                <li>
>+                                    <label for="page_height">Page height:</label>
>+                                    <input type="text" size="4" name="page_height" id="page_height" value="[% letter.page_height | html %]" />
>+                                </li>
>+                                <li>
>+                                    <label for="page_width">Page width:</label>
>+                                    <input type="text" size="4" name="page_width" id="page_width" value="[% letter.page_width | html %]" />
>+                                </li>
>+                                <li>
>+                                    <label for="notice_width">Notice width:</label>
>+                                    <input type="text" size="4" name="notice_width" id="notice_width" value="[% letter.notice_width | html %]" />
>+                                </li>
>+                                <li>
>+                                    <label for="notice_height">Notice height:</label>
>+                                    <input type="text" size="4" name="notice_height" id="notice_height" value="[% letter.notice_height | html %]" />
>+                                </li>
>+                            </ol>
>+                        </div>
>+                        <div class="col-md-6">
>+                            <ol>
>+                                <li>
>+                                    <label for="top_margin">Top page margin:</label>
>+                                    <input type="text" size="4" name="top_margin" id="top_margin" value="[% letter.top_margin | html %]" />
>+                                </li>
>+                                <li>
>+                                    <label for="left_margin">Left page margin:</label>
>+                                    <input type="text" size="4" name="left_margin" id="left_margin" value="[% letter.left_margin | html %]" />
>+                                </li>
>+                                <li>
>+                                    <label for="top_text_margin">Top text margin:</label>
>+                                    <input type="text" size="4" name="top_text_margin" id="top_text_margin" value="[% letter.top_text_margin | html %]" />
>+                                </li>
>+                                <li>
>+                                    <label for="left_text_margin">Left text margin:</label>
>+                                    <input type="text" size="4" name="left_text_margin" id="left_text_margin" value="[% letter.left_text_margin | html %]" />
>+                                </li>
>+                                <li>
>+                                    <label for="cols">Number of columns:</label>
>+                                    <input type="text" size="4" name="cols" id="cols" value="[% letter.cols | html %]" />
>+                                </li>
>+                                <li>
>+                                    <label for="rows">Number of rows:</label>
>+                                    <input type="text" size="4" name="rows" id="rows" value="[% letter.rows | html %]" />
>+                                </li>
>+                                <li>
>+                                    <label for="col_gap">Gap between columns:</label>
>+                                    <input type="text" size="4" name="col_gap" id="col_gap" value="[% letter.col_gap | html %]" />
>+                                </li>
>+                                <li>
>+                                    <label for="row_gap">Gap between rows:</label>
>+                                    <input type="text" size="4" name="row_gap" id="row_gap" value="[% letter.row_gap | html %]" />
>+                                </li>
>+                            </ol>
>+                        </div>
>+                    </fieldset> 
>+                </div> 
>+            </div> 
>+        </div> 
>+
>     </div> 
> [% END %]
> 
>diff --git a/tools/letter.pl b/tools/letter.pl
>index c00cc6de3dc..d73ee03e138 100755
>--- a/tools/letter.pl
>+++ b/tools/letter.pl
>@@ -49,7 +49,11 @@ use C4::Context;
> use C4::Output qw( output_html_with_http_headers );
> use C4::Letters qw( GetMessageTransportTypes );
> use C4::Log qw( logaction );
>-
>+use C4::Creators qw(
>+    get_font_types
>+    get_text_justification_types
>+    get_unit_values
>+);
> use Koha::Notice::Templates;
> use Koha::Patron::Attribute::Types;
> 
>@@ -299,6 +303,9 @@ sub add_form {
>         SQLfieldnames => $field_selection,
>         branchcode => $branchcode,
>         preview_is_available => $preview_is_available,
>+        text_justification_types => get_text_justification_types(),
>+        font_types => get_font_types(),
>+        units => get_unit_values(),
>     );
>     return;
> }
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 33478: 149743 | 149785 | 149786 | 149787 | 149914 | 149915 | 149916 | 149917 | 149918 | 149999 | 150000 | 150001 | 150002 | 150003 | 150004 | 150005 | 150006 | 150007 | 150008 | 150009 | 150010 | 150011 | 150012 | 150265 | 150266 | 150267 | 150268 | 150269 | 150270 | 150271 | 150272 | 150273 | 150274 | 150275 | 150276 | 150277 | 150324 | 150325 | 150326 | 150327 | 150328 | 150329 | 150330 | 150331 | 150332 | 150333 | 150334 | 150335 | 150336 | 150337 | 150338 | 150339 | 150340 | 150341 | 150342 | 150343 | 150344 | 150345 | 151548 | 151580 | 151581 | 151582 | 151583 | 151584 | 151585 | 151586 | 151587 | 151588 | 151589 | 151590 | 151591 | 151592 | 151593 | 156469 | 157368 | 157369 | 157370 | 157371 | 157372 | 157373 | 157374 | 157375 | 157376 | 157377 | 157378 | 157379 | 157380 | 157381 | 157382 | 157387 | 158835 | 158836 | 158837 | 158838 | 158839 | 158840 | 158841 | 158842 | 158843 | 158844 | 158845 | 158846 | 158847 | 158848 | 158849 | 158856 | 162556 | 162557 | 162558 | 162559 | 162560 | 162561 | 162562 | 162563 | 162564 | 162565 | 162566 | 162567 | 162568 | 162569 | 163903 | 163904 | 163905 | 163906 | 163907 | 163908 | 163909 | 163910 | 163911 | 163912 | 163913 | 163914 | 163915 | 163916 | 163924 | 163925 | 163926 | 163927 | 163928 | 163929 | 163930 | 163931 | 163932 | 163933 | 163934 | 163935 | 163936 | 163937 | 163938 | 164719 | 164720 | 164721 | 164722 | 164723 | 164724 | 164725 | 164726 | 164727 | 164728 | 164729 | 164730 | 164731 | 164732 | 164733 | 164741 | 164742 | 164743 | 164744 | 164745 | 164746 | 164747 | 164748 | 164749 | 164750 | 164751 | 164752 | 164753 | 164754 | 164755