From 5879fe94cc047fa820aa57b1c9856dd9cc05ba9f Mon Sep 17 00:00:00 2001
From: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Date: Fri, 11 Oct 2013 00:53:52 -0400
Subject: [PATCH] If payment processor is Stripe, only show payment form if
 Javascript enabled.

Stripe doesn't work without it.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
---
 Open-ILS/src/templates/opac/myopac/main_payment_form.tt2 | 13 ++++++++++---
 Open-ILS/src/templates/opac/parts/js.tt2                 |  4 ++++
 2 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/Open-ILS/src/templates/opac/myopac/main_payment_form.tt2 b/Open-ILS/src/templates/opac/myopac/main_payment_form.tt2
index cecb607205..b1dd27c4e7 100644
--- a/Open-ILS/src/templates/opac/myopac/main_payment_form.tt2
+++ b/Open-ILS/src/templates/opac/myopac/main_payment_form.tt2
@@ -16,7 +16,14 @@ END %]
         "total is non-positive.  We cannot process non-positive amounts.") %]
 </div>
 [% ELSE %]
-<div id="pay_fines_now">
+[% IF ctx.use_stripe %]
+<noscript>
+    [% l("Your browser does not have Javascript enabled, and we cannot " _
+        "process credit card payments without it.  Please change your " _
+        "browser settings and try again.") %]
+</noscript>
+[% END %]
+<div id="pay_fines_now"[% IF ctx.use_stripe %] class="hide_me"[% END %]>
     [% 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">
@@ -128,7 +135,7 @@ END %]
                 </tr>
                 <tr>
                     <td>[% l('Credit Card #') %]</td>
-                    <td><input type="number" maxlength="16" 
+                    <td><input type="text" maxlength="16"
                     [% IF ctx.use_stripe %]
                     data-stripe="number"
                     [% ELSE %]
@@ -139,7 +146,7 @@ END %]
                 <tr>
                     <td>[% l('Security Code') %]</td>
                     <td>
-                        <input type="number" size="4" maxlength="5" 
+                        <input type="text" size="4" maxlength="5"
                         [% IF ctx.use_stripe %]
                         data-stripe="cvc"
                         [% ELSE %]
diff --git a/Open-ILS/src/templates/opac/parts/js.tt2 b/Open-ILS/src/templates/opac/parts/js.tt2
index 67d09fb263..ed68480402 100644
--- a/Open-ILS/src/templates/opac/parts/js.tt2
+++ b/Open-ILS/src/templates/opac/parts/js.tt2
@@ -1,6 +1,10 @@
 <!-- JS imports, etc.  -->
 <script type="text/javascript" src="[% ctx.media_prefix %]/js/ui/default/opac/simple.js"></script>
 
+[%- IF ctx.use_stripe %]
+<script type="text/javascript">unHideMe($("pay_fines_now"));[%# the DOM is loaded now, right? %]</script>
+[% END -%]
+
 [%- IF ctx.is_staff %]
 <script type="text/javascript" src="[% ctx.media_prefix %]/js/ui/default/opac/staff.js"></script>
     [% IF ctx.page == 'record' %]
-- 
2.11.0