Let's inch towards valid XHTML by making POST and GET lowercase in our forms
authorJames Fournie <jfournie@sitka.bclibraries.ca>
Mon, 18 Feb 2013 20:41:20 +0000 (12:41 -0800)
committerDan Scott <dan@coffeecode.net>
Mon, 18 Feb 2013 22:37:58 +0000 (17:37 -0500)
Signed-off-by: James Fournie <jfournie@sitka.bclibraries.ca>
Signed-off-by: Dan Scott <dan@coffeecode.net>

26 files changed:
Open-ILS/src/templates/opac/myopac/circ_history.tt2
Open-ILS/src/templates/opac/myopac/circs.tt2
Open-ILS/src/templates/opac/myopac/holds.tt2
Open-ILS/src/templates/opac/myopac/holds/edit.tt2
Open-ILS/src/templates/opac/myopac/lists.tt2
Open-ILS/src/templates/opac/myopac/main.tt2
Open-ILS/src/templates/opac/myopac/main_payment_form.tt2
Open-ILS/src/templates/opac/myopac/main_payments.tt2
Open-ILS/src/templates/opac/myopac/prefs.tt2
Open-ILS/src/templates/opac/myopac/prefs_notify.tt2
Open-ILS/src/templates/opac/myopac/prefs_settings.tt2
Open-ILS/src/templates/opac/myopac/update_email.tt2
Open-ILS/src/templates/opac/myopac/update_password.tt2
Open-ILS/src/templates/opac/myopac/update_username.tt2
Open-ILS/src/templates/opac/parts/advanced/expert.tt2
Open-ILS/src/templates/opac/parts/advanced/numeric.tt2
Open-ILS/src/templates/opac/parts/advanced/search.tt2
Open-ILS/src/templates/opac/parts/anon_list.tt2
Open-ILS/src/templates/opac/parts/login/form.tt2
Open-ILS/src/templates/opac/parts/myopac/main_base.tt2
Open-ILS/src/templates/opac/parts/place_hold.tt2
Open-ILS/src/templates/opac/parts/place_hold_result.tt2
Open-ILS/src/templates/opac/parts/searchbar.tt2
Open-ILS/src/templates/opac/results.tt2
Open-ILS/src/templates/opac/sms_cn.tt2
Open-ILS/src/templates/opac/temp_warn.tt2

index 9b6d78d..3bb04bb 100644 (file)
@@ -27,7 +27,7 @@
                [% IF ctx.circs.size < limit %] class='invisible' [% END %] >[% l('Next') %]<span class="nav_arrow_fix">&#9658;</span></a>
         </span>
         <span class="float-left">
-            <form action="[% mkurl(ctx.opac_root _ '/myopac/circ_history/export') %]" method="POST">
+            <form action="[% mkurl(ctx.opac_root _ '/myopac/circ_history/export') %]" method="post">
                 <div>
                     [%- INCLUDE "opac/parts/preserve_params.tt2" %]
                     <input type="hidden" name="filename" value="[% l('circ_history.csv') %]"/> 
index 8ff44ac..b4f0ea2 100644 (file)
@@ -31,7 +31,7 @@
     </div>
     [% END %]
     <div id='checked_main'>
-        <form method="POST" id="circ-form"
+        <form method="post" id="circ-form"
             onsubmit="return confirm('[% l("Are you sure you wish to renew the selected item(s)?") %]');">
         <table cellpadding='0' cellspacing='0' class="item_list_padding">
             <tr>
index 5b5a36a..2da8cef 100644 (file)
@@ -52,7 +52,7 @@
     </div>
     <div class="clear-both"></div>
     <div id='holds_main'>
-        <form method="POST">
+        <form method="post">
         <table cellpadding='0' cellspacing='0' class="opac-auto-097">
             <tr>
                 <td width="1">
index 175d319..3415546 100644 (file)
@@ -37,7 +37,7 @@
             <p>
                 <strong>[% l('Status') %]</strong>: [% hold.human_status %]
             </p>
-            <form method="POST">
+            <form method="post">
                 <table id="hold_editor_table">
                     <tr>
                         <th>
index 2103ae0..52e0689 100644 (file)
@@ -8,7 +8,7 @@
 <div id='myopac_bookbag_div' style="padding:5px;">
 
     <!-- new list creation -->
-    <form action="[% mkurl(ctx.opac_root _ '/myopac/list/update') %]" method="POST" id="create_form">
+    <form action="[% mkurl(ctx.opac_root _ '/myopac/list/update') %]" method="post" id="create_form">
         <h1>[% l('Create new list') %]</h1><a name="createnewlist"></a>
         <table cellpadding="0" border="0" id="list_create_table">
             <tr>
@@ -96,7 +96,7 @@
                 [% IF bbag.description %]<div class="bookbag-description">[% bbag.description | html %]</div>[% END %]
             </div>
             [% IF ctx.add_rec %]
-            <form action="[% mkurl(ctx.opac_root _ '/myopac/list/update', {}, 1) %]" method="POST">
+            <form action="[% mkurl(ctx.opac_root _ '/myopac/list/update', {}, 1) %]" method="post">
                 <div class="bookbag-controls">
                     <input type="hidden" name="action" value="add_rec" />
                     <input type="hidden" name="list" value="[% bbag.id %]" />
                 </div>
             </form>
             [% END %]
-            <form action="[% mkurl(ctx.opac_root _ '/myopac/list/update') %]" method="POST">
+            <form action="[% mkurl(ctx.opac_root _ '/myopac/list/update') %]" method="post">
                 <div class="bookbag-share">
                     <input type="hidden" name="list" value="[% bbag.id %]" />
                     [%- INCLUDE "opac/parts/preserve_params.tt2"; %]
                     [% END %]
                 </div>
             </form>
-            <form action="[% mkurl(ctx.opac_root _ '/myopac/list/update') %]" method="POST">
+            <form action="[% mkurl(ctx.opac_root _ '/myopac/list/update') %]" method="post">
                 <div class="bookbag-controls">
                     <input type="hidden" name="list" value="[% bbag.id %]" />
                     <input type="hidden" name="action" value="delete" />
                     <input type="submit" value="[% l('Delete List') %]" />
                 </div>
             </form>
-            <form action="[% mkurl(ctx.opac_root _ '/myopac/list/print') %]" method="POST">
+            <form action="[% mkurl(ctx.opac_root _ '/myopac/list/print') %]" method="post">
                 <div class="bookbag-controls">
                     <input type="hidden" name="list" value="[% bbag.id %]" />
                     <input type="hidden" name="sort" value="[% CGI.param('sort') | html %]" />
                 </div>
             </form>
             [% setting = 'opac.default_list'; %]
-            <form action="[% mkurl(ctx.opac_root _ '/myopac/list/update') %]" method="POST">
+            <form action="[% mkurl(ctx.opac_root _ '/myopac/list/update') %]" method="post">
                 <div class="bookbag-controls">
                     <input type="hidden" name="list" value="[% bbag.id %]" />
                     [%- INCLUDE "opac/parts/preserve_params.tt2"; %]
         [% IF CGI.param("bbid") == bbag.id %]
         <div class="bookbag-specific">
             <div class="sort">
-                <form method="GET">
+                <form method="get">
                     <label for="opac.result.sort">[% l("Sort list items by: ") %]</label>
                     [%- INCLUDE "opac/parts/preserve_params.tt2" params=['loc', 'query', 'qtype']; %]
                     [% INCLUDE "opac/parts/filtersort.tt2"
                 </form>
             </div>
             <div class="meta">
-                <form method="POST">
+                <form method="post">
                     <input type="hidden" name="bbid" value="[% bbag.id %]" />
                     <input type="hidden" name="action" value="editmeta" />
                     <input type="hidden" name="limit" value="[% limit %]" />
             </div>
         </div>
         <br class="clear-both" />
-        <form action="[% mkurl(ctx.opac_root _ '/myopac/list/update') %]" method="POST">
+        <form action="[% mkurl(ctx.opac_root _ '/myopac/list/update') %]" method="post">
         <input type="hidden" name="list" value="[% bbag.id %]" />
         <input type="hidden" name="sort" value="[% CGI.param('sort') | uri %]" />
         <table class="bookbag-specific" cellpadding='0' cellspacing='0' border='0'>
index 793653b..38d5b4f 100644 (file)
@@ -8,7 +8,7 @@
         myopac_cc_allowed = 1;
     END
 %]
-<form action="[% ctx.opac_root %]/myopac/main_payment_form" method="GET">
+<form action="[% ctx.opac_root %]/myopac/main_payment_form" method="get">
     [% IF ctx.fines.circulation.size > 0 %]
     <div id='myopac_circ_trans_div'>
         <table width='100%' class='data_grid'>
index 3bb08c2..8da0287 100644 (file)
@@ -16,7 +16,7 @@
 <div id="pay_fines_now">
     [% IF last_chance %]
     <p><big>[% l("Are you sure you are ready to charge [_1] to your credit card?", money(ctx.fines.balance_owed)) %]</big></p>
-    <form action="[% ctx.opac_root %]/myopac/main_pay_init" method="POST">
+    <form action="[% ctx.opac_root %]/myopac/main_pay_init" method="post">
         [% FOR k IN CGI.Vars;
             NEXT UNLESS k;
             FOR val IN CGI.param(k) %]
@@ -25,7 +25,7 @@
         <input type="submit" value="[% l('Submit Payment') %]" />
         <a href="[% mkurl(ctx.opac_root _ '/myopac/main', {}, 1) %]">[% l('Cancel') %]</a>
     [% ELSE %]
-    <form method="POST">
+    <form method="post">
         <input type="hidden" name="last_chance" value="1" />
         [% FOR xact IN CGI.param('xact') %]
         <input type="hidden" name="xact" value="[% xact | html %]" />
index a30edeb..c3caa58 100644 (file)
                 %]</td>
                 <td>[% money(payment.mp.amount) %]</td>
                 <td>
-                    <form action="[% ctx.opac_root %]/myopac/receipt_print" method="POST">
+                    <form action="[% ctx.opac_root %]/myopac/receipt_print" method="post">
                         <input type="hidden" name="payment" value="[% payment.mp.id %]" />
                         <input type="submit" value="[% l('Print') %]" />
                     </form>
-                    <form action="[% ctx.opac_root %]/myopac/receipt_email" method="POST">
+                    <form action="[% ctx.opac_root %]/myopac/receipt_email" method="post">
                         <input type="hidden" name="payment" value="[% payment.mp.id %]" />
                         <input type="submit" value="[% l('Email') %]" />
                     </form>
index d3178a7..38b78b4 100644 (file)
                 [% IF pending_addr OR edit_addr_id == cur_id;
                     update_addr = pending_addr || addr %]
                 <td>
-                    <form method='POST'>
+                    <form method='post'>
                         [% IF pending_addr %]
                             <input type='hidden' name='pending_addr' value='[% pending_addr.id %]'/>
                         [% ELSE %]
index fcd9f47..57eed22 100644 (file)
@@ -3,7 +3,7 @@
     myopac_page = "prefs";
     prefs_page = 'notify' %]
 
-<form method='POST'>
+<form method='post'>
     [% setting = 'opac.hold_notify' %]
     <input name='[% setting %]' type="hidden"
         [% IF ctx.user_setting_map.$setting; %] value='[% ctx.user_setting_map.$setting | html %]' [% END %]/>
index 84b2554..cd3ba17 100644 (file)
@@ -5,7 +5,7 @@
     prefs_page = 'settings' %]
 
 
-    <form method='POST'>
+    <form method='post'>
 
        <table class="full-width data_grid" id="acct_search_main">
             <tbody>
index ceb1a6b..4be2ffe 100644 (file)
@@ -17,7 +17,7 @@
 
 [% END %]
 
-<form method='POST' id='account-update-email' autocomplete='off'>
+<form method='post' id='account-update-email' autocomplete='off'>
     [% IF CGI.param("return_to_referer") %]
     <input type="hidden" name="redirect_to" value="[% ctx.referer | html %]" />
     [% END %]
index 7b77fe7..8655faa 100644 (file)
@@ -21,7 +21,7 @@
     </div>
 [% END %]
 
-<form method='POST' id='account-update-email' autocomplete='off'>
+<form method='post' id='account-update-email' autocomplete='off'>
     <table> 
         <tr><td>[% l('Current Password') %]</td><td><input type='password' name='current_pw'/></td></tr>
         <tr><td>[% l('New Password') %]</td><td><input type='password' name='new_pw'/></td></tr>
index 546c071..3dbb908 100644 (file)
@@ -25,7 +25,7 @@
 
 [% END %]
 
-<form method='POST' id='account-update-email' autocomplete='off'> 
+<form method='post' id='account-update-email' autocomplete='off'> 
     <table> 
         <tr><td>[% l('Current Username') %]</td><td>[% ctx.user.usrname | html %]</td></tr>
         <tr><td>[% l('Current Password') %]</td><td><input type='password' name='current_pw'/></td></tr>
index b5abd34..9d80fe4 100644 (file)
@@ -1,4 +1,4 @@
-<form action="[% ctx.opac_root %]/results" method="GET">
+<form action="[% ctx.opac_root %]/results" method="get">
     <div class="header_middle">[% l("Expert Search") %]</div>
     <input type="hidden" name="_special" value="1" />
     <table class="expert-search">
index 819fd79..8c1ad4a 100644 (file)
@@ -1,4 +1,4 @@
-<form action="[% ctx.opac_root %]/results" method="GET">
+<form action="[% ctx.opac_root %]/results" method="get">
     <div class="header_middle">[% l("Numeric Search") %]</div>
     <input type="hidden" name="contains" value="contains" />
     <input type="hidden" name="_special" value="1" />
index e90b6d3..ad8cc44 100644 (file)
@@ -2,7 +2,7 @@
     PROCESS "opac/parts/misc_util.tt2";
     PROCESS get_library;
 %]
-<form action="[% ctx.opac_root %]/results" method="GET">
+<form action="[% ctx.opac_root %]/results" method="get">
 <table id='adv_global_search' class='data_grid data_grid_center' width='100%'>
     <tr style='border-bottom: none;'>
         <!-- Contains the user-addable(?) rows to define search class, containment and text -->
index 00a898d..89e771c 100644 (file)
@@ -1,6 +1,6 @@
         [% IF ctx.mylist.size %]
         <div class="bookbag-specific">
-            <form method="GET">
+            <form method="get">
                 <label for="anonsort">[% l("Sort list items by: ") %]</label>
                 [% INCLUDE "opac/parts/filtersort.tt2" mode='bookbag'
                     id="anonsort" name="anonsort" value=CGI.param("anonsort") %]
@@ -11,7 +11,7 @@
             </form>
         </div>
 
-        <form action="[% mkurl(ctx.opac_root _ '/mylist/move') %]" method="GET">
+        <form action="[% mkurl(ctx.opac_root _ '/mylist/move') %]" method="get">
         <div>
             <p class="big-strong">[% l('Temporary List') %]</p>
             <table cellpadding='0' cellspacing='0' border='0'>
index e4c633c..0bdebc0 100644 (file)
@@ -16,7 +16,7 @@
 <div id='login-form-box' class='login_boxes left_brain' style='float:left'>
     <h1>[% l('Log in to Your Account') %]</h1>
     [% l('Please enter the following information:') %]
-    <form method='POST'>
+    <form method='post'>
         <table>
             <tr>
                 <td style="width: 20em; vertical-align: top; padding-top: 1.5em;" class='lbl1'><label for='username_field'>[% l('Library Card Number or Username') %]</label>
index 5cddf45..f0685e0 100644 (file)
@@ -41,7 +41,7 @@
             [% money(ctx.user_stats.fines.balance_owed) %]
         </span><br />
         [% IF (ctx.fines.grocery.size OR ctx.fines.circulation.size) AND ctx.get_org_setting(ctx.user.home_ou, 'credit.payments.allow') == 1 %]
-        <form action="[% ctx.opac_root %]/myopac/main_payment_form" method="GET"><input
+        <form action="[% ctx.opac_root %]/myopac/main_payment_form" method="get"><input
             type="submit" title="[% l('Pay Fines') %]" alt="[% l('Pay Fines') %]"
             value="[% l('Pay Fines') %]" class="pos-rel-top-5 opac-button" /></form>
         [% END %]
index 5b6171b..1162aed 100644 (file)
@@ -4,7 +4,7 @@
 
 <div id='holds_box' class='canvas' style='margin-top: 6px;'>
     <h1>[% l('Place Hold') %]</h1>
-    <form method="POST">
+    <form method="post">
         <input type="hidden" name="hold_type" value="[% CGI.param('hold_type') | html %]" />
         [%  
             redirect = CGI.param('hold_source_page') || CGI.param('redirect_to') || CGI.referer;
index bf2a654..017a163 100644 (file)
@@ -9,7 +9,7 @@
 <div id='holds_box' class='canvas' style='margin-top: 6px;'>
     <h1>[% l('Place Hold') %]</h1>
 
-    <form method="POST">
+    <form method="post">
         <input type="hidden" name="override" value="1" />
         [% FOR k IN ctx.orig_params.keys %]
         <input type="hidden" name="[% k %]" value="[% ctx.orig_params.$k | html %]" />
index b75752f..c3886b1 100644 (file)
@@ -1,7 +1,7 @@
 [% PROCESS "opac/parts/org_selector.tt2" %]
 <div id="search-box">    
     [% UNLESS took_care_of_form -%]
-    <form action="[% ctx.opac_root %]/results" method="GET">
+    <form action="[% ctx.opac_root %]/results" method="get">
     [%- END %]
     <div>
         <span class="search_catalog_lbl">[% l('Search the Catalog') %]</span>
index 7b2917e..cdbdc19 100644 (file)
@@ -17,7 +17,7 @@
     PROCESS "opac/parts/misc_util.tt2";
     PROCESS get_library;
 %]
-    <form action="[% ctx.opac_root %]/results" method="GET">
+    <form action="[% ctx.opac_root %]/results" method="get">
     <div id="search-wrapper">
         [% INCLUDE "opac/parts/searchbar.tt2" took_care_of_form=1 %]
     </div>
index c136dc9..4281d5d 100644 (file)
@@ -21,7 +21,7 @@
                     <a href="[% ctx.origin %]">[% l('Return to record') %]</a>
                     <pre>[% ctx.event.template_output.data %]</pre>
                     <blockquote>
-                        <form method="POST">
+                        <form method="post">
                             <blockquote>
                                 <input type="hidden" name="copy_id" value="[% ctx.copy_id %]"/>
                                 <input type="hidden" name="origin" value="[% ctx.origin %]"/>
index d6be0b3..4747f6c 100644 (file)
@@ -15,7 +15,7 @@
                       l('This information will disappear when you end your session, unless you login and save it to a permanent list.');
                    END
                 %]</p>
-             <form method="POST" action="[% mkurl(ctx.opac_root _ '/temp_warn/post', {}, 1) %]">
+             <form method="post" action="[% mkurl(ctx.opac_root _ '/temp_warn/post', {}, 1) %]">
              <input type="hidden" name="redirect_to" value="[% ctx.redirect_to %]" />
              <input type="checkbox" name="no_temp_list_warn" value="on" />
              <label for="no_temp_list_warn">[% l('Do not show this warning again.') %]</label>