Changeset 12902

Timestamp:

04/21/2021 02:53:57 PM (3 years ago)

Author:

dcavins

Message:

BP Tools: Use radio buttons to choose operation.

Use a radio button group instead of checkboxes to
select which one operation to run from the WP Admin
BP Tools screen.

Fixes #8458.

File:

• trunk/src/bp-core/admin/bp-core-admin-tools.php (modified) (3 diffs)

trunk/src/bp-core/admin/bp-core-admin-tools.php

@@ -29 +29 @@
         <p><?php esc_html_e( 'Use the tools below to manually recalculate these relationships.', 'buddypress' ); ?>
         </p>
-        <p class="attention"><?php esc_html_e( 'Some of these tools create substantial database overhead. Avoid running more than one repair job at a time.', 'buddypress' ); ?></p>
-
-        <h2><?php esc_html_e( 'Activate checkboxe(s) to select the operation(s) to perform', 'buddypress' ); ?></h2>
+
+        <h2><?php esc_html_e( 'Select the operation to perform', 'buddypress' ); ?></h2>
 
         <form class="settings" method="post" action="">
@@ -41 +40 @@
                     <p>
                         <label for="<?php echo esc_attr( str_replace( '_', '-', $item[0] ) ); ?>">
-                            <input type="checkbox" class="checkbox" name="<?php echo esc_attr( $item[0] ) . '" id="' . esc_attr( str_replace( '_', '-', $item[0] ) ); ?>" value="1" /> <?php echo esc_html( $item[1] ); ?>
+                            <input type="radio" class="radio" name="bp-tools-run[]" id="<?php echo esc_attr( str_replace( '_', '-', $item[0] ) ); ?>" value="<?php echo esc_attr( $item[0] ); ?>" /> <?php echo esc_html( $item[1] ); ?>
                         </label>
                     </p>
@@ -82 +81 @@
 
     foreach ( (array) bp_admin_repair_list() as $item ) {
-        if ( isset( $item[2] ) && isset( $_POST[$item[0]] ) && 1 === absint( $_POST[$item[0]] ) && is_callable( $item[2] ) ) {
+        if ( isset( $item[2] ) && isset( $_POST['bp-tools-run'] ) && in_array( $item[0], (array) $_POST['bp-tools-run'], true ) && is_callable( $item[2] ) ) {
             $messages[] = call_user_func( $item[2] );
         }