Changeset 2819

Timestamp:

03/26/2016 12:19:49 AM (5 years ago)

Author:

iandunn

Message:

WordCamp Payments: Check radio buttons to work around Chrome bug.

Location:

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments

Files:

• includes/wordcamp-budgets.php (modified) (1 diff)
• javascript/wordcamp-budgets.js (modified) (2 diffs)

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/includes/wordcamp-budgets.php

@@ -129 +129 @@
             plugins_url( 'javascript/wordcamp-budgets.js', __DIR__ ),
             array( 'jquery', 'jquery-ui-datepicker', 'media-upload', 'media-views' ),
-            2,
+            3,
             true
         );

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/javascript/wordcamp-budgets.js

@@ -37 +37 @@
                 $( inputField ).prop( 'required', false );
             } );
+
+            app.checkRadioButtons();
         },
 
@@ -76 +78 @@
 
             // todo make the transition smoother
+
+            app.checkRadioButtons();
+        },
+
+        /*
+         * Make sure all radio buttons have values in Chrome
+         *
+         * This is only to work around bug #596138
+         *
+         * @see  https://bugs.chromium.org/p/chromium/issues/detail?id=596138
+         * @todo remove this after Chrome 51 is adopted by most users
+         */
+        checkRadioButtons : function() {
+            var checkedPaymentMethods = $( 'input[name="payment_method"]:checked' );
+
+            if ( ! window.hasOwnProperty( 'chrome' ) ) {
+                return;
+            }
+
+            if ( 'Direct Deposit' != checkedPaymentMethods.val() ) {
+                if ( 0 === $( 'input[name="ach_account_type"]:checked' ).length ) {
+                    $( '#ach_account_type_company' ).prop( 'checked', true );
+                }
+            }
+
+            if ( 0 === checkedPaymentMethods.length ) {
+                $( '#payment_method_direct_deposit' ).prop( 'checked', true );
+            }
         },