Changeset 42217 for trunk/src/wp-signup.php

Timestamp:

11/23/2017 04:08:42 AM (7 years ago)

Author:

pento

Message:

General: Reformat inline if () statements inside HTML tags.

This pattern occurs a handful of times across the codebase:

<div class="foo<?php if ( $bar ) { echo ' baz'; } ?>">

Unfortunately, it doesn't really play nicely with phpcbf, so all instances need to be removed in preperation for auto code formatting.

See #41057.

File:

• trunk/src/wp-signup.php (modified) (2 diffs)

trunk/src/wp-signup.php

@@ -167 +167 @@
             ?>
         </p>
-    <?php endif; // Languages. ?>
+    <?php
+        endif; // Languages.
+
+        $blog_public_on_checked = $blog_public_off_checked = '';
+        if ( isset( $_POST['blog_public'] ) && '0' == $_POST['blog_public']  ) {
+            $blog_public_off_checked = 'checked="checked"';
+        } else {
+            $blog_public_on_checked = 'checked="checked"';
+        }
+    ?>
 
     <div id="privacy">
@@ -175 +184 @@
             <br style="clear:both" />
             <label class="checkbox" for="blog_public_on">
-                <input type="radio" id="blog_public_on" name="blog_public" value="1" <?php if ( !isset( $_POST['blog_public'] ) || $_POST['blog_public'] == '1' ) { ?>checked="checked"<?php } ?> />
+                <input type="radio" id="blog_public_on" name="blog_public" value="1" <?php echo $blog_public_on_checked; ?> />
                 <strong><?php _e( 'Yes' ); ?></strong>
             </label>
             <label class="checkbox" for="blog_public_off">
-                <input type="radio" id="blog_public_off" name="blog_public" value="0" <?php if ( isset( $_POST['blog_public'] ) && $_POST['blog_public'] == '0' ) { ?>checked="checked"<?php } ?> />
+                <input type="radio" id="blog_public_off" name="blog_public" value="0" <?php echo $blog_public_off_checked; ?> />
                 <strong><?php _e( 'No' ); ?></strong>
             </label>