Changeset 3143

Timestamp:

05/14/2016 12:57:54 PM (9 years ago)

Author:

ocean90

Message:

Trac: Disable submit only if the security question isn't already checked.

Props jorbin.
Fixes #851.

File:

• sites/trunk/wordpress.org/public_html/style/trac/trac-security.js (modified) (3 diffs)

sites/trunk/wordpress.org/public_html/style/trac/trac-security.js

@@ -17 +17 @@
             'escape', 'unescape', 'escaped', 'unescaped', 'escapes', 'escaping', 'unescaping', 'esc_',
             'sanitize', 'unsanitize', 'sanitizes', 'unsanitizes', 'sanitized', 'unsanitized', 'sanitization',
-            'valid', 'invalid', 'validate', 'validates', 'validation','hack', 'vulnerable', 'attack', 
-            'compromise', 'escalation', 'injection', 'forgery', 'password', 'passwords', 'cross-site' , 
-            'secure' 
+            'valid', 'invalid', 'validate', 'validates', 'validation','hack', 'vulnerable', 'attack',
+            'compromise', 'escalation', 'injection', 'forgery', 'password', 'passwords', 'cross-site' ,
+            'secure'
         ],
 
@@ -37 +37 @@
 
     function show_box() {
-        // We have a potential problem here
-        submit.prop( 'disabled', true );
+        // Disable submit only if the box isn't already checked.
+        if ( false === $( '#security-question input' ).is( ':checked' ) ) {
+            submit.prop( 'disabled', true );
+        }
+
         if ( $( '#security-question' ).length !== 0 ) {
             // We've already created the checkbox
@@ -58 +61 @@
     jQuery( '#field-summary, #field-description, #field-keywords' ).on( 'keyup', function() {
         var entry = $(this).val();
-        
+
         if ( wp.trac_security.has_overlap( entry, wp.trac_security.badwords ) ) {
             show_box();