From 539e2a3c7f824dc4b2167f2dcc25d0b1a35b70be Mon Sep 17 00:00:00 2001
From: Kyle M Hall <kyle@bywatersolutions.com>
Date: Tue, 17 Sep 2019 15:41:12 -0400
Subject: [PATCH] Bug 23625: Make new 'required' attributes match the currently
 used syntax in other templates

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit 0aeb5998481a28fb72c8f184bd3f85f7491dc858)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
(cherry picked from commit 267d0a90333006004f527fbc535556e04f6786de)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
---
 .../en/modules/opac-request-article.tt         | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-request-article.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-request-article.tt
index f5e6b78c96..9f62d2fc9b 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-request-article.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-request-article.tt
@@ -35,7 +35,7 @@
                         <li>
                             [% IF mandatory_fields.search('title') %]
                                 <label for="title" class="required">Title:</label>
-                                <input type="text" required name="title" id="title" size="50"/>
+                                <input type="text" required="required" name="title" id="title" size="50"/>
                             [% ELSE %]
                                 <label for="title">Title:</label>
                                 <input type="text" name="title" id="title" size="50"/>
@@ -45,7 +45,7 @@
                         <li>
                             [% IF mandatory_fields.search('author') %]
                                 <label for="author" class="required">Author:</label>
-                                <input type="text" required name="author" id="author" size="50"/>
+                                <input type="text" required="required" name="author" id="author" size="50"/>
                             [% ELSE %]
                                 <label for="author">Author:</label>
                                 <input type="text" name="author" id="author" size="50"/>
@@ -55,7 +55,7 @@
                         <li>
                             [% IF mandatory_fields.search('volume') %]
                                 <label for="volume" class="required">Volume:</label>
-                                <input type="text" required name="volume" id="volume" size="50"/>
+                                <input type="text" required="required" name="volume" id="volume" size="50"/>
                             [% ELSE %]
                                 <label for="volume">Volume:</label>
                                 <input type="text" name="volume" id="volume" size="50"/>
@@ -65,7 +65,7 @@
                         <li>
                             [% IF mandatory_fields.search('issue') %]
                                 <label for="issue" class="required">Issue:</label>
-                                <input type="text" required name="issue" id="issue" size="50"/>
+                                <input type="text" required="required" name="issue" id="issue" size="50"/>
                             [% ELSE %]
                                 <label for="issue">Issue:</label>
                                 <input type="text" name="issue" id="issue" size="50"/>
@@ -75,7 +75,7 @@
                         <li>
                             [% IF mandatory_fields.search('date') %]
                                 <label for="date" class="required">Date:</label>
-                                <input type="text" required name="date" id="date" size="50"/>
+                                <input type="text" required="required" name="date" id="date" size="50"/>
                             [% ELSE %]
                                 <label for="date">Date:</label>
                                 <input type="text" name="date" id="date" size="50"/>
@@ -85,7 +85,7 @@
                         <li>
                             [% IF mandatory_fields.search('pages') %]
                                 <label for="pages" class="required">Pages:</label>
-                                <input type="text" required name="pages" id="pages" size="50"/>
+                                <input type="text" required="required" name="pages" id="pages" size="50"/>
                             [% ELSE %]
                                 <label for="pages">Pages:</label>
                                 <input type="text" name="pages" id="pages" size="50"/>
@@ -95,7 +95,7 @@
                         <li>
                             [% IF mandatory_fields.search('chapters') %]
                                 <label for="chapters" class="required">Chapters:</label>
-                                <input type="text" required name="chapters" id="chapters" size="50"/>
+                                <input type="text" required="required" name="chapters" id="chapters" size="50"/>
                             [% ELSE %]
                                 <label for="chapters">Chapters:</label>
                                 <input type="text" name="chapters" id="chapters" size="50"/>
@@ -109,8 +109,8 @@
 
                         <li>
                             <label for="branchcode">Pickup library:</label>
-                            <select name="branchcode" id="branchcode">
-                                [% FOREACH b IN Branches.all %]required
+                            <select name="branchcode" id="branchcode" required="required">
+                                [% FOREACH b IN Branches.all %]
                                     [% IF b.branchcode == Branches.GetLoggedInBranchcode %]
                                         <option value="[% b.branchcode | html %]" selected="selected">[% b.branchname | html %]</option>
                                     [% ELSE %]
-- 
2.39.5