Skip to:
Content

bbPress.org

Changeset 5440


Ignore:
Timestamp:
07/09/2014 11:58:59 PM (11 years ago)
Author:
johnjamesjacoby
Message:

Brackets and code formatting improvements to admin component.

Location:
trunk/src/includes/admin
Files:
10 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/includes/admin/actions.php

    r5314 r5440  
    106106
    107107    // Bail if plugin is not network activated
    108     if ( ! is_plugin_active_for_network( bbpress()->basename ) )
     108    if ( ! is_plugin_active_for_network( bbpress()->basename ) ) {
    109109        return;
     110    }
    110111
    111112    // Switch to the new blog

  • trunk/src/includes/admin/admin.php

    r5399 r5440  
    132132
    133133        // Bail to prevent interfering with the deactivation process
    134         if ( bbp_is_deactivation() )
     134        if ( bbp_is_deactivation() ) {
    135135            return;
     136        }
    136137
    137138        /** General Actions ***************************************************/
    … …  
    274275
    275276        // Bail if plugin is not network activated
    276         if ( ! is_plugin_active_for_network( bbpress()->basename ) )
     277        if ( ! is_plugin_active_for_network( bbpress()->basename ) ) {
    277278            return;
     279        }
    278280
    279281        add_submenu_page(
    … …  
    296298
    297299        // Bail if plugin is not network activated
    298         if ( ! is_plugin_active_for_network( bbpress()->basename ) )
     300        if ( ! is_plugin_active_for_network( bbpress()->basename ) ) {
    299301            return;
     302        }
    300303
    301304        add_submenu_page(
    … …  
    316319     */
    317320    public static function new_install() {
    318         if ( !bbp_is_install() )
     321        if ( !bbp_is_install() ) {
    319322            return;
     323        }
    320324
    321325        bbp_create_initial_content();
    … …  
    336340        // Bail if no sections available
    337341        $sections = bbp_admin_get_settings_sections();
    338         if ( empty( $sections ) )
     342        if ( empty( $sections ) ) {
    339343            return false;
     344        }
    340345
    341346        // Are we using settings integration?
    … …  
    346351
    347352            // Only proceed if current user can see this section
    348             if ( ! current_user_can( $section_id ) )
     353            if ( ! current_user_can( $section_id ) ) {
    349354                continue;
     355            }
    350356
    351357            // Only add section and fields if section has fields
    352358            $fields = bbp_admin_get_settings_fields_for_section( $section_id );
    353             if ( empty( $fields ) )
     359            if ( empty( $fields ) ) {
    354360                continue;
     361            }
    355362
    356363            // Toggle the section if core integration is on
    … …  
    450457
    451458        // Leave if we're not in the import section
    452         if ( !defined( 'WP_LOAD_IMPORTERS' ) )
     459        if ( !defined( 'WP_LOAD_IMPORTERS' ) ) {
    453460            return;
     461        }
    454462
    455463        // Load Importer API
    … …  
    656664     */
    657665    public function hide_theme_compat_packages( $sections = array() ) {
    658         if ( count( bbpress()->theme_compat->packages ) <= 1 )
     666        if ( count( bbpress()->theme_compat->packages ) <= 1 ) {
    659667            unset( $sections['bbp_settings_theme_compat'] );
     668        }
    660669
    661670        return $sections;
    … …  
    10161025
    10171026                            // Site errored out, no response?
    1018                             if ( is_wp_error( $response ) )
     1027                            if ( is_wp_error( $response ) ) {
    10191028                                wp_die( sprintf( __( 'Warning! Problem updating %1$s. Your server may not be able to connect to sites running on it. Error message: <em>%2$s</em>', 'bbpress' ), $siteurl, $response->get_error_message() ) );
     1029                            }
    10201030
    10211031                            // Switch to the new blog

  • trunk/src/includes/admin/converter.php

    r5428 r5440  
    2828
    2929        // "I wonder where I'll float next."
    30         if ( empty( $_SERVER['REQUEST_METHOD'] ) )
     30        if ( empty( $_SERVER['REQUEST_METHOD'] ) ) {
    3131            return;
     32        }
    3233
    3334        // Bail if request is not correct
    … …  
    3637            // Converter is converting
    3738            case 'POST' :
    38                 if ( ( empty( $_POST['action'] ) || ( 'bbconverter_process' !=  $_POST['action'] ) ) )
     39                if ( ( empty( $_POST['action'] ) || ( 'bbconverter_process' !=  $_POST['action'] ) ) ) {
    3940                    return;
     41                }
    4042
    4143                break;
    … …  
    4345            // Some other admin page
    4446            case 'GET'  :
    45                 if ( ( empty( $_GET['page'] ) || ( 'bbp-converter' !=  $_GET['page'] ) ) )
     47                if ( ( empty( $_GET['page'] ) || ( 'bbp-converter' !=  $_GET['page'] ) ) ) {
    4648                    return;
     49                }
    4750
    4851                break;
    … …  
    216219
    217220            function bbconverter_start() {
    218                 if( false == bbconverter_is_running ) {
     221                if( false === bbconverter_is_running ) {
    219222                    bbconverter_is_running = true;
    220223                    jQuery('#bbp-converter-start').hide();
    … …  
    246249                bbconverter_log(response);
    247250
    248                 if ( response == '<p class="loading"><?php esc_html_e( 'Conversion Complete', 'bbpress' ); ?></p>' || response.indexOf('error') > -1 ) {
     251                if ( response === '<p class="loading"><?php esc_html_e( 'Conversion Complete', 'bbpress' ); ?></p>' || response.indexOf('error') > -1 ) {
    249252                    bbconverter_log('<p>Repair any missing information: <a href="<?php echo admin_url(); ?>tools.php?page=bbp-repair">Continue</a></p>');
    250253                    bbconverter_stop();
    … …  
    259262
    260263            function bbconverter_log(text) {
    261                 if ( jQuery('#bbp-converter-message').css('display') == 'none' ) {
     264                if ( jQuery('#bbp-converter-message').css('display') === 'none' ) {
    262265                    jQuery('#bbp-converter-message').show();
    263266                }
    … …  
    287290        $query  = get_option( '_bbp_converter_query' );
    288291
    289         if ( ! empty( $query ) )
     292        if ( ! empty( $query ) ) {
    290293            $before = '<p class="loading" title="' . esc_attr( $query ) . '">';
     294        }
    291295
    292296        echo $before . $output . $after;
    … …  
    324328        // Bail if platform did not get saved
    325329        $platform = !empty( $_POST['_bbp_converter_platform' ] ) ? $_POST['_bbp_converter_platform' ] : get_option( '_bbp_converter_platform' );
    326         if ( empty( $platform ) )
     330        if ( empty( $platform ) ) {
    327331            return;
     332        }
    328333
    329334        // Include the appropriate converter.
    … …  
    548553
    549554        $table_name = $wpdb->prefix . 'bbp_converter_translator';
    550         if ( ! empty( $drop ) && $wpdb->get_var( "SHOW TABLES LIKE '{$table_name}'" ) == $table_name )
     555        if ( ! empty( $drop ) && $wpdb->get_var( "SHOW TABLES LIKE '{$table_name}'" ) == $table_name ) {
    551556            $wpdb->query( "DROP TABLE {$table_name}" );
     557        }
    552558
    553559        require_once( ABSPATH . '/wp-admin/includes/upgrade.php' );

  • trunk/src/includes/admin/forums.php

    r5353 r5440  
    9292     */
    9393    private function bail() {
    94         if ( !isset( get_current_screen()->post_type ) || ( $this->post_type != get_current_screen()->post_type ) )
     94        if ( !isset( get_current_screen()->post_type ) || ( $this->post_type != get_current_screen()->post_type ) ) {
    9595            return true;
     96        }
    9697
    9798        return false;
    … …  
    118119    public function edit_help() {
    119120
    120         if ( $this->bail() ) return;
     121        if ( $this->bail() ) {
     122            return;
     123        }
    121124
    122125        // Overview
    … …  
    179182    public function new_help() {
    180183
    181         if ( $this->bail() ) return;
     184        if ( $this->bail() ) {
     185            return;
     186        }
    182187
    183188        $customize_display = '<p>' . __( 'The title field and the big forum editing Area are fixed in place, but you can reposition all the other boxes using drag and drop, and can minimize or expand them by clicking the title bar of each box. Use the Screen Options tab to unhide more boxes (Excerpt, Send Trackbacks, Custom Fields, Discussion, Slug, Author) or to choose a 1- or 2-column layout for this screen.', 'bbpress' ) . '</p>';
    … …  
    241246    public function attributes_metabox() {
    242247
    243         if ( $this->bail() ) return;
     248        if ( $this->bail() ) {
     249            return;
     250        }
    244251
    245252        add_meta_box (
    … …  
    279286    public function attributes_metabox_save( $forum_id ) {
    280287
    281         if ( $this->bail() ) return $forum_id;
     288        if ( $this->bail() ) {
     289            return $forum_id;
     290        }
    282291
    283292        // Bail if doing an autosave
    284         if ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE )
     293        if ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE ) {
    285294            return $forum_id;
     295        }
    286296
    287297        // Bail if not a post request
    288         if ( ! bbp_is_post_request() )
     298        if ( ! bbp_is_post_request() ) {
    289299            return $forum_id;
     300        }
    290301
    291302        // Nonce check
    292         if ( empty( $_POST['bbp_forum_metabox'] ) || !wp_verify_nonce( $_POST['bbp_forum_metabox'], 'bbp_forum_metabox_save' ) )
     303        if ( empty( $_POST['bbp_forum_metabox'] ) || !wp_verify_nonce( $_POST['bbp_forum_metabox'], 'bbp_forum_metabox_save' ) ) {
    293304            return $forum_id;
     305        }
    294306
    295307        // Only save for forum post-types
    296         if ( ! bbp_is_forum( $forum_id ) )
     308        if ( ! bbp_is_forum( $forum_id ) ) {
    297309            return $forum_id;
     310        }
    298311
    299312        // Bail if current user cannot edit this forum
    300         if ( !current_user_can( 'edit_forum', $forum_id ) )
     313        if ( !current_user_can( 'edit_forum', $forum_id ) ) {
    301314            return $forum_id;
     315        }
    302316
    303317        // Parent ID
    … …  
    328342    public function admin_head() {
    329343
    330         if ( $this->bail() ) return;
    331 
    332         ?>
     344        if ( $this->bail() ) {
     345            return;
     346        } ?>
    333347
    334348        <style type="text/css" media="screen">
    … …  
    445459
    446460                    $is_open = bbp_is_forum_open( $forum_id );
    447                     $message = true === $is_open ? 'closed' : 'opened';
    448                     $success = true === $is_open ? bbp_close_forum( $forum_id ) : bbp_open_forum( $forum_id );
     461                    $message = ( true === $is_open )
     462                        ? 'closed'
     463                        : 'opened';
     464                    $success = ( true === $is_open )
     465                        ? bbp_close_forum( $forum_id )
     466                        : bbp_open_forum( $forum_id );
    449467
    450468                    break;
    … …  
    510528            switch ( $notice ) {
    511529                case 'opened' :
    512                     if ( $message = $is_failure ) {
    513                         $message = sprintf( __( 'There was a problem opening the forum "%1$s".', 'bbpress' ), $forum_title );
    514                     } else {
    515                         $message = sprintf( __( 'Forum "%1$s" successfully opened.', 'bbpress' ), $forum_title );
    516                     }
     530                    $message = ( $is_failure === true )
     531                        ? sprintf( __( 'There was a problem opening the forum "%1$s".', 'bbpress' ), $forum_title )
     532                        : sprintf( __( 'Forum "%1$s" successfully opened.',             'bbpress' ), $forum_title );
    517533                    break;
    518534
    519535                case 'closed' :
    520                     if ( $message = $is_failure ) {
    521                         $message = sprintf( __( 'There was a problem closing the forum "%1$s".', 'bbpress' ), $forum_title );
    522                     } else {
    523                         $message = sprintf( __( 'Forum "%1$s" successfully closed.', 'bbpress' ), $forum_title );
    524                     }
     536                    $message = ( $is_failure === true )
     537                        ? sprintf( __( 'There was a problem closing the forum "%1$s".', 'bbpress' ), $forum_title )
     538                        : sprintf( __( 'Forum "%1$s" successfully closed.',             'bbpress' ), $forum_title );
    525539                    break;
    526540            }
    … …  
    551565    public function column_headers( $columns ) {
    552566
    553         if ( $this->bail() ) return $columns;
     567        if ( $this->bail() ) {
     568            return $columns;
     569        }
    554570
    555571        $columns = array (
    … …  
    585601    public function column_data( $column, $forum_id ) {
    586602
    587         if ( $this->bail() ) return;
     603        if ( $this->bail() ) {
     604            return;
     605        }
    588606
    589607        switch ( $column ) {
    … …  
    606624            case 'bbp_forum_freshness' :
    607625                $last_active = bbp_get_forum_last_active_time( $forum_id, false );
    608                 if ( !empty( $last_active ) )
     626                if ( !empty( $last_active ) ) {
    609627                    echo esc_html( $last_active );
    610                 else
     628                } else {
    611629                    esc_html_e( 'No Topics', 'bbpress' );
     630                }
    612631
    613632                break;
    … …  
    684703        global $post_ID;
    685704
    686         if ( $this->bail() ) return $messages;
     705        if ( $this->bail() ) {
     706            return $messages;
     707        }
    687708
    688709        // URL for the current forum

  • trunk/src/includes/admin/functions.php

    r5401 r5440  
    5454 */
    5555function bbp_admin_custom_menu_order( $menu_order = false ) {
    56     if ( false === bbpress()->admin->show_separator )
     56    if ( false === bbpress()->admin->show_separator ) {
    5757        return $menu_order;
     58    }
    5859
    5960    return true;
    … …  
    7273
    7374    // Bail if user cannot see any top level bbPress menus
    74     if ( empty( $menu_order ) || ( false === bbpress()->admin->show_separator ) )
     75    if ( empty( $menu_order ) || ( false === bbpress()->admin->show_separator ) ) {
    7576        return $menu_order;
     77    }
    7678
    7779    // Initialize our custom order array
    … …  
    131133
    132134    // Bail if not on an admin page and not getting a sample permalink
    133     if ( !empty( $sample ) && is_admin() && bbp_is_custom_post_type() )
     135    if ( !empty( $sample ) && is_admin() && bbp_is_custom_post_type() ) {
    134136        return urldecode( $post_link );
     137    }
    135138
    136139    // Return post link
    … …  
    175178 */
    176179function bbp_do_uninstall( $site_id = 0 ) {
    177     if ( empty( $site_id ) )
     180    if ( empty( $site_id ) ) {
    178181        $site_id = get_current_blog_id();
     182    }
    179183
    180184    switch_to_blog( $site_id );
    … …  
    241245
    242246    // This tweaks the Tools subnav menu to only show one bbPress menu item
    243     if ( ! in_array( $plugin_page, array( 'bbp-settings' ) ) )
     247    if ( ! in_array( $plugin_page, array( 'bbp-settings' ) ) ) {
    244248        $submenu_file = 'bbp-repair';
     249    }
    245250}
    246251

  • trunk/src/includes/admin/replies.php

    r5300 r5440  
    9999     */
    100100    private function bail() {
    101         if ( !isset( get_current_screen()->post_type ) || ( $this->post_type !== get_current_screen()->post_type ) )
     101        if ( !isset( get_current_screen()->post_type ) || ( $this->post_type !== get_current_screen()->post_type ) ) {
    102102            return true;
     103        }
    103104
    104105        return false;
    … …  
    125126    public function edit_help() {
    126127
    127         if ( $this->bail() ) return;
     128        if ( $this->bail() ) {
     129            return;
     130        }
    128131
    129132        // Overview
    … …  
    189192    public function new_help() {
    190193
    191         if ( $this->bail() ) return;
     194        if ( $this->bail() ) {
     195            return;
     196        }
    192197
    193198        $customize_display = '<p>' . __( 'The title field and the big reply editing Area are fixed in place, but you can reposition all the other boxes using drag and drop, and can minimize or expand them by clicking the title bar of each box. Use the Screen Options tab to unhide more boxes (Excerpt, Send Trackbacks, Custom Fields, Discussion, Slug, Author) or to choose a 1- or 2-column layout for this screen.', 'bbpress' ) . '</p>';
    … …  
    249254    public function attributes_metabox() {
    250255
    251         if ( $this->bail() ) return;
     256        if ( $this->bail() ) {
     257            return;
     258        }
    252259
    253260        add_meta_box (
    … …  
    277284    public function attributes_metabox_save( $reply_id ) {
    278285
    279         if ( $this->bail() ) return $reply_id;
     286        if ( $this->bail() ) {
     287            return $reply_id;
     288        }
    280289
    281290        // Bail if doing an autosave
    282         if ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE )
     291        if ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE ) {
    283292            return $reply_id;
     293        }
    284294
    285295        // Bail if not a post request
    286         if ( ! bbp_is_post_request() )
     296        if ( ! bbp_is_post_request() ) {
    287297            return $reply_id;
     298        }
    288299
    289300        // Check action exists
    290         if ( empty( $_POST['action'] ) )
     301        if ( empty( $_POST['action'] ) ) {
    291302            return $reply_id;
     303        }
    292304
    293305        // Nonce check
    294         if ( empty( $_POST['bbp_reply_metabox'] ) || !wp_verify_nonce( $_POST['bbp_reply_metabox'], 'bbp_reply_metabox_save' ) )
     306        if ( empty( $_POST['bbp_reply_metabox'] ) || !wp_verify_nonce( $_POST['bbp_reply_metabox'], 'bbp_reply_metabox_save' ) ) {
    295307            return $reply_id;
     308        }
    296309
    297310        // Current user cannot edit this reply
    298         if ( !current_user_can( 'edit_reply', $reply_id ) )
     311        if ( !current_user_can( 'edit_reply', $reply_id ) ) {
    299312            return $reply_id;
     313        }
    300314
    301315        // Get the reply meta post values
    … …  
    336350    public function author_metabox() {
    337351
    338         if ( $this->bail() ) return;
     352        if ( $this->bail() ) {
     353            return;
     354        }
    339355
    340356        // Bail if post_type is not a reply
    341         if ( empty( $_GET['action'] ) || ( 'edit' !== $_GET['action'] ) )
     357        if ( empty( $_GET['action'] ) || ( 'edit' !== $_GET['action'] ) ) {
    342358            return;
     359        }
    343360
    344361        // Add the metabox
    … …  
    368385    public function admin_head() {
    369386
    370         if ( $this->bail() ) return;
    371 
    372         ?>
     387        if ( $this->bail() ) {
     388            return;
     389        } ?>
    373390
    374391        <style type="text/css" media="screen">
    … …  
    447464    public function toggle_reply() {
    448465
    449         if ( $this->bail() ) return;
     466        if ( $this->bail() ) {
     467            return;
     468        }
    450469
    451470        // Only proceed if GET is a reply toggle action
    … …  
    458477            // Get reply and die if empty
    459478            $reply = bbp_get_reply( $reply_id );
    460             if ( empty( $reply ) ) // Which reply?
     479            if ( empty( $reply ) ) {
    461480                wp_die( __( 'The reply was not found!', 'bbpress' ) );
    462 
    463             if ( !current_user_can( 'moderate', $reply->ID ) ) // What is the user doing here?
     481            }
     482
     483            // What is the user doing here?
     484            if ( !current_user_can( 'moderate', $reply->ID ) ) {
    464485                wp_die( __( 'You do not have the permission to do that!', 'bbpress' ) );
     486            }
    465487
    466488            switch ( $action ) {
    … …  
    478500            $message = array( 'bbp_reply_toggle_notice' => $message, 'reply_id' => $reply->ID );
    479501
    480             if ( false === $success || is_wp_error( $success ) )
     502            if ( false === $success || is_wp_error( $success ) ) {
    481503                $message['failed'] = '1';
     504            }
    482505
    483506            // Do additional reply toggle actions (admin side)
    … …  
    518541
    519542            // Empty? No reply?
    520             if ( empty( $notice ) || empty( $reply_id ) )
     543            if ( empty( $notice ) || empty( $reply_id ) ) {
    521544                return;
     545            }
    522546
    523547            // Get reply and bail if empty
    524548            $reply = bbp_get_reply( $reply_id );
    525             if ( empty( $reply ) )
     549            if ( empty( $reply ) ) {
    526550                return;
     551            }
    527552
    528553            $reply_title = bbp_get_reply_title( $reply->ID );
    … …  
    530555            switch ( $notice ) {
    531556                case 'spammed' :
    532                     $message = $is_failure === true ? sprintf( __( 'There was a problem marking the reply "%1$s" as spam.', 'bbpress' ), $reply_title ) : sprintf( __( 'Reply "%1$s" successfully marked as spam.', 'bbpress' ), $reply_title );
     557                    $message = ( $is_failure === true )
     558                        ? sprintf( __( 'There was a problem marking the reply "%1$s" as spam.', 'bbpress' ), $reply_title )
     559                        : sprintf( __( 'Reply "%1$s" successfully marked as spam.',             'bbpress' ), $reply_title );
    533560                    break;
    534561
    535562                case 'unspammed' :
    536                     $message = $is_failure === true ? sprintf( __( 'There was a problem unmarking the reply "%1$s" as spam.', 'bbpress' ), $reply_title ) : sprintf( __( 'Reply "%1$s" successfully unmarked as spam.', 'bbpress' ), $reply_title );
     563                    $message = ( $is_failure === true )
     564                        ? sprintf( __( 'There was a problem unmarking the reply "%1$s" as spam.', 'bbpress' ), $reply_title )
     565                        : sprintf( __( 'Reply "%1$s" successfully unmarked as spam.',             'bbpress' ), $reply_title );
    537566                    break;
    538567            }
    … …  
    563592    public function column_headers( $columns ) {
    564593
    565         if ( $this->bail() ) return $columns;
     594        if ( $this->bail() ) {
     595            return $columns;
     596        }
    566597
    567598        $columns = array(
    … …  
    607638    public function column_data( $column, $reply_id ) {
    608639
    609         if ( $this->bail() ) return;
     640        if ( $this->bail() ) {
     641            return;
     642        }
    610643
    611644        // Get topic ID
    … …  
    719752    public function row_actions( $actions, $reply ) {
    720753
    721         if ( $this->bail() ) return $actions;
     754        if ( $this->bail() ) {
     755            return $actions;
     756        }
    722757
    723758        unset( $actions['inline hide-if-no-js'] );
    … …  
    727762
    728763        // User cannot view replies in trash
    729         if ( ( bbp_get_trash_status_id() === $reply->post_status ) && !current_user_can( 'view_trash' ) )
     764        if ( ( bbp_get_trash_status_id() === $reply->post_status ) && !current_user_can( 'view_trash' ) ) {
    730765            unset( $actions['view'] );
     766        }
    731767
    732768        // Only show the actions if the user is capable of viewing them
    … …  
    773809    public function filter_dropdown() {
    774810
    775         if ( $this->bail() ) return;
     811        if ( $this->bail() ) {
     812            return;
     813        }
    776814
    777815        // Add Empty Spam button
    … …  
    805843    public function filter_post_rows( $query_vars ) {
    806844
    807         if ( $this->bail() ) return $query_vars;
     845        if ( $this->bail() ) {
     846            return $query_vars;
     847        }
    808848
    809849        // Add post_parent query_var if one is present
    … …  
    836876        global $post_ID;
    837877
    838         if ( $this->bail() ) return $messages;
     878        if ( $this->bail() ) {
     879            return $messages;
     880        }
    839881
    840882        // URL for the current topic

  • trunk/src/includes/admin/settings.php

    r5365 r5440  
    429429
    430430    // Bail if section is empty
    431     if ( empty( $section_id ) )
     431    if ( empty( $section_id ) ) {
    432432        return false;
     433    }
    433434
    434435    $fields = bbp_admin_get_settings_fields();
    … …  
    855856
    856857    // Flush rewrite rules when this section is saved
    857     if ( isset( $_GET['settings-updated'] ) && isset( $_GET['page'] ) )
    858         flush_rewrite_rules(); ?>
     858    if ( isset( $_GET['settings-updated'] ) && isset( $_GET['page'] ) ) {
     859        flush_rewrite_rules();
     860    } ?>
    859861
    860862    <p><?php esc_html_e( 'Customize your Forums root. Partner with a WordPress Page and use Shortcodes for more flexibility.', 'bbpress' ); ?></p>
    … …  
    873875?>
    874876
    875         <input name="_bbp_root_slug" id="_bbp_root_slug" type="text" class="regular-text code" value="<?php bbp_form_option( '_bbp_root_slug', 'forums', true ); ?>"<?php bbp_maybe_admin_setting_disabled( '_bbp_root_slug' ); ?> />
     877    <input name="_bbp_root_slug" id="_bbp_root_slug" type="text" class="regular-text code" value="<?php bbp_form_option( '_bbp_root_slug', 'forums', true ); ?>"<?php bbp_maybe_admin_setting_disabled( '_bbp_root_slug' ); ?> />
    876878
    877879<?php
    … …  
    12971299
    12981300    // Bail if no directory was found (how did this happen?)
    1299     if ( empty( $curdir ) )
     1301    if ( empty( $curdir ) ) {
    13001302        return;
     1303    }
    13011304
    13021305    // Loop through files in the converters folder and assemble some options
    … …  
    15351538
    15361539    // Bail if current screen could not be found
    1537     if ( empty( $current_screen ) )
     1540    if ( empty( $current_screen ) ) {
    15381541        return;
     1542    }
    15391543
    15401544    // Overview
    … …  
    16471651
    16481652        // Fallback to default
    1649         if ( empty( $value ) )
     1653        if ( empty( $value ) ) {
    16501654            $value = $default;
     1655        }
    16511656
    16521657        // Allow plugins to further filter the output

  • trunk/src/includes/admin/tools.php

    r5429 r5440  
    7777function bbp_admin_repair_handler() {
    7878
    79     if ( ! bbp_is_post_request() )
     79    if ( ! bbp_is_post_request() ) {
    8080        return;
     81    }
    8182
    8283    check_admin_referer( 'bbpress-do-counts' );
    … …  
    249250            case 0:
    250251                return false;
    251                 break;
    252252
    253253            case 1:
    … …  
    382382
    383383    $sql_delete = "DELETE FROM `{$wpdb->postmeta}` WHERE `meta_key` = '_bbp_voice_count';";
    384     if ( is_wp_error( $wpdb->query( $sql_delete ) ) )
     384    if ( is_wp_error( $wpdb->query( $sql_delete ) ) ) {
    385385        return array( 1, sprintf( $statement, $result ) );
     386    }
    386387
    387388    // Post types and status
    … …  
    402403                GROUP BY `postmeta`.`meta_value`);";
    403404
    404     if ( is_wp_error( $wpdb->query( $sql ) ) )
     405    if ( is_wp_error( $wpdb->query( $sql ) ) ) {
    405406        return array( 2, sprintf( $statement, $result ) );
     407    }
    406408
    407409    return array( 0, sprintf( $statement, __( 'Complete!', 'bbpress' ) ) );
    … …  
    427429
    428430    $sql_delete = "DELETE FROM `{$wpdb->postmeta}` WHERE `meta_key` = '_bbp_reply_count_hidden';";
    429     if ( is_wp_error( $wpdb->query( $sql_delete ) ) )
     431    if ( is_wp_error( $wpdb->query( $sql_delete ) ) ) {
    430432        return array( 1, sprintf( $statement, $result ) );
     433    }
    431434
    432435    // Post types and status
    … …  
    436439
    437440    $sql = "INSERT INTO `{$wpdb->postmeta}` (`post_id`, `meta_key`, `meta_value`) (SELECT `post_parent`, '_bbp_reply_count_hidden', COUNT(`post_status`) as `meta_value` FROM `{$wpdb->posts}` WHERE `post_type` = '{$rpt}' AND `post_status` IN ( '{$tps}', '{$sps}' ) GROUP BY `post_parent`);";
    438     if ( is_wp_error( $wpdb->query( $sql ) ) )
     441    if ( is_wp_error( $wpdb->query( $sql ) ) ) {
    439442        return array( 2, sprintf( $statement, $result ) );
     443    }
    440444
    441445    return array( 0, sprintf( $statement, __( 'Complete!', 'bbpress' ) ) );
    … …  
    474478
    475479    // Bail if forum IDs returned an error
    476     if ( is_wp_error( $forum_ids ) || empty( $wpdb->last_result ) )
     480    if ( is_wp_error( $forum_ids ) || empty( $wpdb->last_result ) ) {
    477481        return array( 2, sprintf( $statement, __( 'Failed!', 'bbpress' ) ) );
     482    }
    478483
    479484    // Stash the last results
    … …  
    484489
    485490        // Only update if is a converted forum
    486         if ( ! isset( $group_forums->meta_value ) )
     491        if ( ! isset( $group_forums->meta_value ) ) {
    487492            continue;
     493        }
    488494
    489495        // Attempt to update group meta
    … …  
    596602
    597603    $sql_delete = "DELETE FROM {$wpdb->postmeta} WHERE meta_key IN ( '_bbp_topic_count', '_bbp_total_topic_count' );";
    598     if ( is_wp_error( $wpdb->query( $sql_delete ) ) )
     604    if ( is_wp_error( $wpdb->query( $sql_delete ) ) ) {
    599605        return array( 1, sprintf( $statement, $result ) );
     606    }
    600607
    601608    $forums = get_posts( array( 'post_type' => bbp_get_forum_post_type(), 'numberposts' => -1 ) );
    … …  
    675682    $insert_rows = $wpdb->get_results( $sql_select );
    676683
    677     if ( is_wp_error( $insert_rows ) )
     684    if ( is_wp_error( $insert_rows ) ) {
    678685        return array( 1, sprintf( $statement, $result ) );
     686    }
    679687
    680688    $key           = $wpdb->prefix . '_bbp_topic_count';
    681689    $insert_values = array();
    682     foreach ( $insert_rows as $insert_row )
     690    foreach ( $insert_rows as $insert_row ) {
    683691        $insert_values[] = "('{$insert_row->post_author}', '{$key}', '{$insert_row->_count}')";
    684 
    685     if ( !count( $insert_values ) )
     692    }
     693
     694    if ( !count( $insert_values ) ) {
    686695        return array( 2, sprintf( $statement, $result ) );
     696    }
    687697
    688698    $sql_delete = "DELETE FROM `{$wpdb->usermeta}` WHERE `meta_key` = '{$key}';";
    689     if ( is_wp_error( $wpdb->query( $sql_delete ) ) )
     699    if ( is_wp_error( $wpdb->query( $sql_delete ) ) ) {
    690700        return array( 3, sprintf( $statement, $result ) );
     701    }
    691702
    692703    foreach ( array_chunk( $insert_values, 10000 ) as $chunk ) {
    … …  
    721732    $insert_rows = $wpdb->get_results( $sql_select );
    722733
    723     if ( is_wp_error( $insert_rows ) )
     734    if ( is_wp_error( $insert_rows ) ) {
    724735        return array( 1, sprintf( $statement, $result ) );
     736    }
    725737
    726738    $key           = $wpdb->prefix . '_bbp_reply_count';
    727739    $insert_values = array();
    728     foreach ( $insert_rows as $insert_row )
     740    foreach ( $insert_rows as $insert_row ) {
    729741        $insert_values[] = "('{$insert_row->post_author}', '{$key}', '{$insert_row->_count}')";
    730 
    731     if ( !count( $insert_values ) )
     742    }
     743
     744    if ( !count( $insert_values ) ) {
    732745        return array( 2, sprintf( $statement, $result ) );
     746    }
    733747
    734748    $sql_delete = "DELETE FROM `{$wpdb->usermeta}` WHERE `meta_key` = '{$key}';";
    735     if ( is_wp_error( $wpdb->query( $sql_delete ) ) )
     749    if ( is_wp_error( $wpdb->query( $sql_delete ) ) ) {
    736750        return array( 3, sprintf( $statement, $result ) );
     751    }
    737752
    738753    foreach ( array_chunk( $insert_values, 10000 ) as $chunk ) {
    … …  
    767782    $users     = $wpdb->get_results( "SELECT `user_id`, `meta_value` AS `favorites` FROM `{$wpdb->usermeta}` WHERE `meta_key` = '{$key}';" );
    768783
    769     if ( is_wp_error( $users ) )
     784    if ( is_wp_error( $users ) ) {
    770785        return array( 1, sprintf( $statement, $result ) );
     786    }
    771787
    772788    $topics = $wpdb->get_col( "SELECT `ID` FROM `{$wpdb->posts}` WHERE `post_type` = '" . bbp_get_topic_post_type() . "' AND `post_status` = '" . bbp_get_public_status_id() . "';" );
    773789
    774     if ( is_wp_error( $topics ) )
     790    if ( is_wp_error( $topics ) ) {
    775791        return array( 2, sprintf( $statement, $result ) );
     792    }
    776793
    777794    $values = array();
    778795    foreach ( $users as $user ) {
    779         if ( empty( $user->favorites ) || !is_string( $user->favorites ) )
     796        if ( empty( $user->favorites ) || !is_string( $user->favorites ) ) {
    780797            continue;
     798        }
    781799
    782800        $favorites = array_intersect( $topics, explode( ',', $user->favorites ) );
    783         if ( empty( $favorites ) || !is_array( $favorites ) )
     801        if ( empty( $favorites ) || !is_array( $favorites ) ) {
    784802            continue;
     803        }
    785804
    786805        $favorites_joined = implode( ',', $favorites );
    … …  
    797816
    798817    $sql_delete = "DELETE FROM `{$wpdb->usermeta}` WHERE `meta_key` = '{$key}';";
    799     if ( is_wp_error( $wpdb->query( $sql_delete ) ) )
     818    if ( is_wp_error( $wpdb->query( $sql_delete ) ) ) {
    800819        return array( 4, sprintf( $statement, $result ) );
     820    }
    801821
    802822    foreach ( array_chunk( $values, 10000 ) as $chunk ) {
    … …  
    830850    $users     = $wpdb->get_results( "SELECT `user_id`, `meta_value` AS `subscriptions` FROM `{$wpdb->usermeta}` WHERE `meta_key` = '{$key}';" );
    831851
    832     if ( is_wp_error( $users ) )
     852    if ( is_wp_error( $users ) ) {
    833853        return array( 1, sprintf( $statement, $result ) );
     854    }
    834855
    835856    $topics = $wpdb->get_col( "SELECT `ID` FROM `{$wpdb->posts}` WHERE `post_type` = '" . bbp_get_topic_post_type() . "' AND `post_status` = '" . bbp_get_public_status_id() . "';" );
    836     if ( is_wp_error( $topics ) )
     857    if ( is_wp_error( $topics ) ) {
    837858        return array( 2, sprintf( $statement, $result ) );
     859    }
    838860
    839861    $values = array();
    840862    foreach ( $users as $user ) {
    841         if ( empty( $user->subscriptions ) || !is_string( $user->subscriptions ) )
     863        if ( empty( $user->subscriptions ) || !is_string( $user->subscriptions ) ) {
    842864            continue;
     865        }
    843866
    844867        $subscriptions = array_intersect( $topics, explode( ',', $user->subscriptions ) );
    845         if ( empty( $subscriptions ) || !is_array( $subscriptions ) )
     868        if ( empty( $subscriptions ) || !is_array( $subscriptions ) ) {
    846869            continue;
     870        }
    847871
    848872        $subscriptions_joined = implode( ',', $subscriptions );
    … …  
    859883
    860884    $sql_delete = "DELETE FROM `{$wpdb->usermeta}` WHERE `meta_key` = '{$key}';";
    861     if ( is_wp_error( $wpdb->query( $sql_delete ) ) )
     885    if ( is_wp_error( $wpdb->query( $sql_delete ) ) ) {
    862886        return array( 4, sprintf( $statement, $result ) );
     887    }
    863888
    864889    foreach ( array_chunk( $values, 10000 ) as $chunk ) {
    … …  
    892917    $users     = $wpdb->get_results( "SELECT `user_id`, `meta_value` AS `subscriptions` FROM `{$wpdb->usermeta}` WHERE `meta_key` = '{$key}';" );
    893918
    894     if ( is_wp_error( $users ) )
     919    if ( is_wp_error( $users ) ) {
    895920        return array( 1, sprintf( $statement, $result ) );
     921    }
    896922
    897923    $forums = $wpdb->get_col( "SELECT `ID` FROM `{$wpdb->posts}` WHERE `post_type` = '" . bbp_get_forum_post_type() . "' AND `post_status` = '" . bbp_get_public_status_id() . "';" );
    898     if ( is_wp_error( $forums ) )
     924    if ( is_wp_error( $forums ) ) {
    899925        return array( 2, sprintf( $statement, $result ) );
     926    }
    900927
    901928    $values = array();
    902929    foreach ( $users as $user ) {
    903         if ( empty( $user->subscriptions ) || !is_string( $user->subscriptions ) )
     930        if ( empty( $user->subscriptions ) || !is_string( $user->subscriptions ) ) {
    904931            continue;
     932        }
    905933
    906934        $subscriptions = array_intersect( $forums, explode( ',', $user->subscriptions ) );
    907         if ( empty( $subscriptions ) || !is_array( $subscriptions ) )
     935        if ( empty( $subscriptions ) || !is_array( $subscriptions ) ) {
    908936            continue;
     937        }
    909938
    910939        $subscriptions_joined = implode( ',', $subscriptions );
    … …  
    921950
    922951    $sql_delete = "DELETE FROM `{$wpdb->usermeta}` WHERE `meta_key` = '{$key}';";
    923     if ( is_wp_error( $wpdb->query( $sql_delete ) ) )
     952    if ( is_wp_error( $wpdb->query( $sql_delete ) ) ) {
    924953        return array( 4, sprintf( $statement, $result ) );
     954    }
    925955
    926956    foreach ( array_chunk( $values, 10000 ) as $chunk ) {
    … …  
    956986
    957987    // Bail if no role map exists
    958     if ( empty( $role_map ) )
     988    if ( empty( $role_map ) ) {
    959989        return array( 1, sprintf( $statement, __( 'Failed!', 'bbpress' ) ) );
     990    }
    960991
    961992    // Iterate through each role...
    … …  
    10141045
    10151046    // First, delete everything.
    1016     if ( is_wp_error( $wpdb->query( "DELETE FROM `$wpdb->postmeta` WHERE `meta_key` IN ( '_bbp_last_reply_id', '_bbp_last_topic_id', '_bbp_last_active_id', '_bbp_last_active_time' );" ) ) )
     1047    if ( is_wp_error( $wpdb->query( "DELETE FROM `$wpdb->postmeta` WHERE `meta_key` IN ( '_bbp_last_reply_id', '_bbp_last_topic_id', '_bbp_last_active_id', '_bbp_last_active_time' );" ) ) ) {
    10171048        return array( 1, sprintf( $statement, $result ) );
     1049    }
    10181050
    10191051    // Post types and status
    … …  
    10281060            FROM `$wpdb->posts` AS `topic` INNER JOIN `$wpdb->posts` AS `reply` ON `topic`.`ID` = `reply`.`post_parent`
    10291061            WHERE `reply`.`post_status` = '{$pps}' AND `topic`.`post_type` = '{$tpt}' AND `reply`.`post_type` = '{$rpt}'
    1030             GROUP BY `topic`.`ID` );" ) ) )
     1062            GROUP BY `topic`.`ID` );" ) ) ) {
    10311063        return array( 2, sprintf( $statement, $result ) );
     1064    }
    10321065
    10331066    // For any remaining topics, give a reply ID of 0.
    … …  
    10361069            FROM `$wpdb->posts` AS `topic` LEFT JOIN `$wpdb->postmeta` AS `reply`
    10371070            ON `topic`.`ID` = `reply`.`post_id` AND `reply`.`meta_key` = '_bbp_last_reply_id'
    1038             WHERE `reply`.`meta_id` IS NULL AND `topic`.`post_type` = '{$tpt}' );" ) ) )
     1071            WHERE `reply`.`meta_id` IS NULL AND `topic`.`post_type` = '{$tpt}' );" ) ) ) {
    10391072        return array( 3, sprintf( $statement, $result ) );
     1073    }
    10401074
    10411075    // Now we give all the forums with topics the ID their last topic.
    … …  
    10441078            FROM `$wpdb->posts` AS `forum` INNER JOIN `$wpdb->posts` AS `topic` ON `forum`.`ID` = `topic`.`post_parent`
    10451079            WHERE `topic`.`post_status` = '{$pps}' AND `forum`.`post_type` = '{$fpt}' AND `topic`.`post_type` = '{$tpt}'
    1046             GROUP BY `forum`.`ID` );" ) ) )
     1080            GROUP BY `forum`.`ID` );" ) ) ) {
    10471081        return array( 4, sprintf( $statement, $result ) );
     1082    }
    10481083
    10491084    // For any remaining forums, give a topic ID of 0.
    … …  
    10521087            FROM `$wpdb->posts` AS `forum` LEFT JOIN `$wpdb->postmeta` AS `topic`
    10531088            ON `forum`.`ID` = `topic`.`post_id` AND `topic`.`meta_key` = '_bbp_last_topic_id'
    1054             WHERE `topic`.`meta_id` IS NULL AND `forum`.`post_type` = '{$fpt}' );" ) ) )
     1089            WHERE `topic`.`meta_id` IS NULL AND `forum`.`post_type` = '{$fpt}' );" ) ) ) {
    10551090        return array( 5, sprintf( $statement, $result ) );
     1091    }
    10561092
    10571093    // After that, we give all the topics with replies the ID their last reply (again, this time for a different reason).
    … …  
    10601096            FROM `$wpdb->posts` AS `topic` INNER JOIN `$wpdb->posts` AS `reply` ON `topic`.`ID` = `reply`.`post_parent`
    10611097            WHERE `reply`.`post_status` = '{$pps}' AND `topic`.`post_type` = '{$tpt}' AND `reply`.`post_type` = '{$rpt}'
    1062             GROUP BY `topic`.`ID` );" ) ) )
     1098            GROUP BY `topic`.`ID` );" ) ) ) {
    10631099        return array( 6, sprintf( $statement, $result ) );
     1100    }
    10641101
    10651102    // For any remaining topics, give a reply ID of themself.
    … …  
    10681105            FROM `$wpdb->posts` AS `topic` LEFT JOIN `$wpdb->postmeta` AS `reply`
    10691106            ON `topic`.`ID` = `reply`.`post_id` AND `reply`.`meta_key` = '_bbp_last_active_id'
    1070             WHERE `reply`.`meta_id` IS NULL AND `topic`.`post_type` = '{$tpt}' );" ) ) )
     1107            WHERE `reply`.`meta_id` IS NULL AND `topic`.`post_type` = '{$tpt}' );" ) ) ) {
    10711108        return array( 7, sprintf( $statement, $result ) );
     1109    }
    10721110
    10731111    // Give topics with replies their last update time.
    … …  
    10761114            FROM `$wpdb->posts` AS `topic` INNER JOIN `$wpdb->posts` AS `reply` ON `topic`.`ID` = `reply`.`post_parent`
    10771115            WHERE `reply`.`post_status` = '{$pps}' AND `topic`.`post_type` = '{$tpt}' AND `reply`.`post_type` = '{$rpt}'
    1078             GROUP BY `topic`.`ID` );" ) ) )
     1116            GROUP BY `topic`.`ID` );" ) ) ) {
    10791117        return array( 8, sprintf( $statement, $result ) );
     1118    }
    10801119
    10811120    // Give topics without replies their last update time.
    … …  
    10841123            FROM `$wpdb->posts` AS `topic` LEFT JOIN `$wpdb->postmeta` AS `reply`
    10851124            ON `topic`.`ID` = `reply`.`post_id` AND `reply`.`meta_key` = '_bbp_last_active_time'
    1086             WHERE `reply`.`meta_id` IS NULL AND `topic`.`post_type` = '{$tpt}' );" ) ) )
     1125            WHERE `reply`.`meta_id` IS NULL AND `topic`.`post_type` = '{$tpt}' );" ) ) ) {
    10871126        return array( 9, sprintf( $statement, $result ) );
     1127    }
    10881128
    10891129    // Forums need to know what their last active item is as well. Now it gets a bit more complex to do in the database.
    10901130    $forums = $wpdb->get_col( "SELECT `ID` FROM `$wpdb->posts` WHERE `post_type` = '{$fpt}' and `post_status` != 'auto-draft';" );
    1091     if ( is_wp_error( $forums ) )
     1131    if ( is_wp_error( $forums ) ) {
    10921132        return array( 10, sprintf( $statement, $result ) );
     1133    }
    10931134
    10941135    // Loop through forums
    … …  
    11321173
    11331174    // Bail if no forums found
    1134     if ( empty( $forums ) || is_wp_error( $forums ) )
     1175    if ( empty( $forums ) || is_wp_error( $forums ) ) {
    11351176        return array( 1, sprintf( $statement, $result ) );
     1177    }
    11361178
    11371179    // Loop through forums and get their sticky topics
    … …  
    12121254
    12131255    // First, delete everything.
    1214     if ( is_wp_error( $wpdb->query( "DELETE FROM `$wpdb->postmeta` WHERE `meta_key` = '_bbp_forum_id';" ) ) )
     1256    if ( is_wp_error( $wpdb->query( "DELETE FROM `$wpdb->postmeta` WHERE `meta_key` = '_bbp_forum_id';" ) ) ) {
    12151257        return array( 1, sprintf( $statement, $result ) );
     1258    }
    12161259
    12171260    // Post types and status
    … …  
    12261269                AS `forum`
    12271270            WHERE `forum`.`post_type` = '{$fpt}'
    1228             GROUP BY `forum`.`ID` );" ) ) )
     1271            GROUP BY `forum`.`ID` );" ) ) ) {
    12291272        return array( 2, sprintf( $statement, $result ) );
     1273    }
    12301274
    12311275    // Next, give all the topics with replies the ID their last reply.
    … …  
    12351279                AS `topic`
    12361280            WHERE `topic`.`post_type` = '{$tpt}'
    1237             GROUP BY `topic`.`ID` );" ) ) )
     1281            GROUP BY `topic`.`ID` );" ) ) ) {
    12381282        return array( 3, sprintf( $statement, $result ) );
     1283    }
    12391284
    12401285    // Next, give all the topics with replies the ID their last reply.
    … …  
    12481293            WHERE `topic`.`post_type` = '{$tpt}'
    12491294                AND `reply`.`post_type` = '{$rpt}'
    1250             GROUP BY `reply`.`ID` );" ) ) )
     1295            GROUP BY `reply`.`ID` );" ) ) ) {
    12511296        return array( 4, sprintf( $statement, $result ) );
     1297    }
    12521298
    12531299    // Complete results
    … …  
    12731319
    12741320    // First, delete everything.
    1275     if ( is_wp_error( $wpdb->query( "DELETE FROM `$wpdb->postmeta` WHERE `meta_key` = '_bbp_topic_id';" ) ) )
     1321    if ( is_wp_error( $wpdb->query( "DELETE FROM `$wpdb->postmeta` WHERE `meta_key` = '_bbp_topic_id';" ) ) ) {
    12761322        return array( 1, sprintf( $statement, $result ) );
     1323    }
    12771324
    12781325    // Post types and status
    … …  
    12861333                AS `topic`
    12871334            WHERE `topic`.`post_type` = '{$tpt}'
    1288             GROUP BY `topic`.`ID` );" ) ) )
     1335            GROUP BY `topic`.`ID` );" ) ) ) {
    12891336        return array( 3, sprintf( $statement, $result ) );
     1337    }
    12901338
    12911339    // Next, give all the topics with replies the ID their last reply.
    … …  
    12991347            WHERE `topic`.`post_type` = '{$tpt}'
    13001348                AND `reply`.`post_type` = '{$rpt}'
    1301             GROUP BY `reply`.`ID` );" ) ) )
     1349            GROUP BY `reply`.`ID` );" ) ) ) {
    13021350        return array( 4, sprintf( $statement, $result ) );
     1351    }
    13031352
    13041353    // Complete results
    … …  
    14461495
    14471496    // Bail if not resetting
    1448     if ( ! bbp_is_post_request() || empty( $_POST['bbpress-are-you-sure'] ) )
     1497    if ( ! bbp_is_post_request() || empty( $_POST['bbpress-are-you-sure'] ) ) {
    14491498        return;
     1499    }
    14501500
    14511501    // Only keymasters can proceed
    1452     if ( ! bbp_is_user_keymaster() )
     1502    if ( ! bbp_is_user_keymaster() ) {
    14531503        return;
     1504    }
    14541505
    14551506    check_admin_referer( 'bbpress-reset' );

  • trunk/src/includes/admin/topics.php

    r5300 r5440  
    9999     */
    100100    private function bail() {
    101         if ( !isset( get_current_screen()->post_type ) || ( $this->post_type !== get_current_screen()->post_type ) )
     101        if ( !isset( get_current_screen()->post_type ) || ( $this->post_type !== get_current_screen()->post_type ) ) {
    102102            return true;
     103        }
    103104
    104105        return false;
    … …  
    125126    public function edit_help() {
    126127
    127         if ( $this->bail() ) return;
     128        if ( $this->bail() ) {
     129            return;
     130        }
    128131
    129132        // Overview
    … …  
    190193    public function new_help() {
    191194
    192         if ( $this->bail() ) return;
     195        if ( $this->bail() ) {
     196            return;
     197        }
    193198
    194199        $customize_display = '<p>' . __( 'The title field and the big topic editing Area are fixed in place, but you can reposition all the other boxes using drag and drop, and can minimize or expand them by clicking the title bar of each box. Use the Screen Options tab to unhide more boxes (Excerpt, Send Trackbacks, Custom Fields, Discussion, Slug, Author) or to choose a 1- or 2-column layout for this screen.', 'bbpress' ) . '</p>';
    … …  
    249254    public function attributes_metabox() {
    250255
    251         if ( $this->bail() ) return;
     256        if ( $this->bail() ) {
     257            return;
     258        }
    252259
    253260        add_meta_box (
    … …  
    277284    public function attributes_metabox_save( $topic_id ) {
    278285
    279         if ( $this->bail() ) return $topic_id;
     286        if ( $this->bail() ) {
     287            return $topic_id;
     288        }
    280289
    281290        // Bail if doing an autosave
    282         if ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE )
     291        if ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE ) {
    283292            return $topic_id;
     293        }
    284294
    285295        // Bail if not a post request
    286         if ( ! bbp_is_post_request() )
     296        if ( ! bbp_is_post_request() ) {
    287297            return $topic_id;
     298        }
    288299
    289300        // Nonce check
    290         if ( empty( $_POST['bbp_topic_metabox'] ) || !wp_verify_nonce( $_POST['bbp_topic_metabox'], 'bbp_topic_metabox_save' ) )
     301        if ( empty( $_POST['bbp_topic_metabox'] ) || !wp_verify_nonce( $_POST['bbp_topic_metabox'], 'bbp_topic_metabox_save' ) ) {
    291302            return $topic_id;
     303        }
    292304
    293305        // Bail if current user cannot edit this topic
    294         if ( !current_user_can( 'edit_topic', $topic_id ) )
     306        if ( !current_user_can( 'edit_topic', $topic_id ) ) {
    295307            return $topic_id;
     308        }
    296309
    297310        // Get the forum ID
    … …  
    352365    public function author_metabox() {
    353366
    354         if ( $this->bail() ) return;
     367        if ( $this->bail() ) {
     368            return;
     369        }
    355370
    356371        // Bail if post_type is not a topic
    357         if ( empty( $_GET['action'] ) || ( 'edit' !== $_GET['action'] ) )
     372        if ( empty( $_GET['action'] ) || ( 'edit' !== $_GET['action'] ) ) {
    358373            return;
     374        }
    359375
    360376        // Add the metabox
    … …  
    384400    public function admin_head() {
    385401
    386         if ( $this->bail() ) return;
    387 
    388         ?>
     402        if ( $this->bail() ) {
     403            return;
     404        } ?>
    389405
    390406        <style type="text/css" media="screen">
    … …  
    471487    public function toggle_topic() {
    472488
    473         if ( $this->bail() ) return;
     489        if ( $this->bail() ) {
     490            return;
     491        }
    474492
    475493        // Only proceed if GET is a topic toggle action
    … …  
    482500
    483501            // Bail if topic is missing
    484             if ( empty( $topic ) )
     502            if ( empty( $topic ) ) {
    485503                wp_die( __( 'The topic was not found!', 'bbpress' ) );
    486 
    487             if ( !current_user_can( 'moderate', $topic->ID ) ) // What is the user doing here?
     504            }
     505
     506            // What is the user doing here?
     507            if ( !current_user_can( 'moderate', $topic->ID ) ) {
    488508                wp_die( __( 'You do not have the permission to do that!', 'bbpress' ) );
     509            }
    489510
    490511            switch ( $action ) {
    … …  
    493514
    494515                    $is_open = bbp_is_topic_open( $topic_id );
    495                     $message = true === $is_open ? 'closed' : 'opened';
    496                     $success = true === $is_open ? bbp_close_topic( $topic_id ) : bbp_open_topic( $topic_id );
     516                    $message = ( true === $is_open )
     517                        ? 'closed'
     518                        : 'opened';
     519                    $success = ( true === $is_open )
     520                        ? bbp_close_topic( $topic_id )
     521                        : bbp_open_topic( $topic_id );
    497522
    498523                    break;
    … …  
    502527
    503528                    $is_sticky = bbp_is_topic_sticky( $topic_id );
    504                     $is_super  = false === $is_sticky && !empty( $_GET['super'] ) && ( "1" === $_GET['super'] ) ? true : false;
    505                     $message   = true  === $is_sticky ? 'unsticked'     : 'sticked';
    506                     $message   = true  === $is_super  ? 'super_sticked' : $message;
    507                     $success   = true  === $is_sticky ? bbp_unstick_topic( $topic_id ) : bbp_stick_topic( $topic_id, $is_super );
     529                    $is_super  = ( false === $is_sticky ) && !empty( $_GET['super'] ) && ( "1" === $_GET['super'] )
     530                        ? true
     531                        : false;
     532                    $message   = ( true  === $is_sticky )
     533                        ? 'unsticked'
     534                        : 'sticked';
     535                    $message   = ( true  === $is_super )
     536                        ? 'super_sticked'
     537                        : $message;
     538                    $success   = ( true  === $is_sticky )
     539                        ? bbp_unstick_topic( $topic_id )
     540                        : bbp_stick_topic( $topic_id, $is_super );
    508541
    509542                    break;
    … …  
    513546
    514547                    $is_spam = bbp_is_topic_spam( $topic_id );
    515                     $message = true === $is_spam ? 'unspammed' : 'spammed';
    516                     $success = true === $is_spam ? bbp_unspam_topic( $topic_id ) : bbp_spam_topic( $topic_id );
     548                    $message = ( true === $is_spam )
     549                        ? 'unspammed'
     550                        : 'spammed';
     551                    $success = ( true === $is_spam )
     552                        ? bbp_unspam_topic( $topic_id )
     553                        : bbp_spam_topic( $topic_id );
    517554
    518555                    break;
    … …  
    521558            $message = array( 'bbp_topic_toggle_notice' => $message, 'topic_id' => $topic->ID );
    522559
    523             if ( false === $success || is_wp_error( $success ) )
     560            if ( false === $success || is_wp_error( $success ) ) {
    524561                $message['failed'] = '1';
     562            }
    525563
    526564            // Do additional topic toggle actions (admin side)
    … …  
    552590    public function toggle_topic_notice() {
    553591
    554         if ( $this->bail() ) return;
     592        if ( $this->bail() ) {
     593            return;
     594        }
    555595
    556596        // Only proceed if GET is a topic toggle action
    … …  
    561601
    562602            // Bais if no topic_id or notice
    563             if ( empty( $notice ) || empty( $topic_id ) )
     603            if ( empty( $notice ) || empty( $topic_id ) ) {
    564604                return;
     605            }
    565606
    566607            // Bail if topic is missing
    567608            $topic = bbp_get_topic( $topic_id );
    568             if ( empty( $topic ) )
     609            if ( empty( $topic ) ) {
    569610                return;
     611            }
    570612
    571613            $topic_title = bbp_get_topic_title( $topic->ID );
    … …  
    573615            switch ( $notice ) {
    574616                case 'opened'    :
    575                     $message = $is_failure === true ? sprintf( __( 'There was a problem opening the topic "%1$s".',           'bbpress' ), $topic_title ) : sprintf( __( 'Topic "%1$s" successfully opened.',           'bbpress' ), $topic_title );
     617                    $message = ( $is_failure === true )
     618                        ? sprintf( __( 'There was a problem opening the topic "%1$s".', 'bbpress' ), $topic_title )
     619                        : sprintf( __( 'Topic "%1$s" successfully opened.',             'bbpress' ), $topic_title );
    576620                    break;
    577621
    578622                case 'closed'    :
    579                     $message = $is_failure === true ? sprintf( __( 'There was a problem closing the topic "%1$s".',           'bbpress' ), $topic_title ) : sprintf( __( 'Topic "%1$s" successfully closed.',           'bbpress' ), $topic_title );
     623                    $message = ( $is_failure === true )
     624                        ? sprintf( __( 'There was a problem closing the topic "%1$s".', 'bbpress' ), $topic_title )
     625                        : sprintf( __( 'Topic "%1$s" successfully closed.',             'bbpress' ), $topic_title );
    580626                    break;
    581627
    582628                case 'super_sticked' :
    583                     $message = $is_failure === true ? sprintf( __( 'There was a problem sticking the topic "%1$s" to front.', 'bbpress' ), $topic_title ) : sprintf( __( 'Topic "%1$s" successfully sticked to front.', 'bbpress' ), $topic_title );
     629                    $message = ( $is_failure === true )
     630                        ? sprintf( __( 'There was a problem sticking the topic "%1$s" to front.', 'bbpress' ), $topic_title )
     631                        : sprintf( __( 'Topic "%1$s" successfully sticked to front.',             'bbpress' ), $topic_title );
    584632                    break;
    585633
    586634                case 'sticked'   :
    587                     $message = $is_failure === true ? sprintf( __( 'There was a problem sticking the topic "%1$s".',          'bbpress' ), $topic_title ) : sprintf( __( 'Topic "%1$s" successfully sticked.',          'bbpress' ), $topic_title );
     635                    $message = ( $is_failure === true )
     636                        ? sprintf( __( 'There was a problem sticking the topic "%1$s".', 'bbpress' ), $topic_title )
     637                        : sprintf( __( 'Topic "%1$s" successfully sticked.',             'bbpress' ), $topic_title );
    588638                    break;
    589639
    590640                case 'unsticked' :
    591                     $message = $is_failure === true ? sprintf( __( 'There was a problem unsticking the topic "%1$s".',        'bbpress' ), $topic_title ) : sprintf( __( 'Topic "%1$s" successfully unsticked.',        'bbpress' ), $topic_title );
     641                    $message = ( $is_failure === true )
     642                        ? sprintf( __( 'There was a problem unsticking the topic "%1$s".', 'bbpress' ), $topic_title )
     643                        : sprintf( __( 'Topic "%1$s" successfully unsticked.',             'bbpress' ), $topic_title );
    592644                    break;
    593645
    594646                case 'spammed'   :
    595                     $message = $is_failure === true ? sprintf( __( 'There was a problem marking the topic "%1$s" as spam.',   'bbpress' ), $topic_title ) : sprintf( __( 'Topic "%1$s" successfully marked as spam.',   'bbpress' ), $topic_title );
     647                    $message = ( $is_failure === true )
     648                        ? sprintf( __( 'There was a problem marking the topic "%1$s" as spam.', 'bbpress' ), $topic_title )
     649                        : sprintf( __( 'Topic "%1$s" successfully marked as spam.',             'bbpress' ), $topic_title );
    596650                    break;
    597651
    598652                case 'unspammed' :
    599                     $message = $is_failure === true ? sprintf( __( 'There was a problem unmarking the topic "%1$s" as spam.', 'bbpress' ), $topic_title ) : sprintf( __( 'Topic "%1$s" successfully unmarked as spam.', 'bbpress' ), $topic_title );
     653                    $message = ( $is_failure === true )
     654                        ? sprintf( __( 'There was a problem unmarking the topic "%1$s" as spam.', 'bbpress' ), $topic_title )
     655                        : sprintf( __( 'Topic "%1$s" successfully unmarked as spam.',             'bbpress' ), $topic_title );
    600656                    break;
    601657            }
    … …  
    606662            ?>
    607663
    608             <div id="message" class="<?php echo $is_failure === true ? 'error' : 'updated'; ?> fade">
     664            <div id="message" class="<?php echo ( $is_failure === true ) ? 'error' : 'updated'; ?> fade">
    609665                <p style="line-height: 150%"><?php echo esc_html( $message ); ?></p>
    610666            </div>
    … …  
    626682    public function column_headers( $columns ) {
    627683
    628         if ( $this->bail() ) return $columns;
     684        if ( $this->bail() ) {
     685            return $columns;
     686        }
    629687
    630688        $columns = array(
    … …  
    669727    public function column_data( $column, $topic_id ) {
    670728
    671         if ( $this->bail() ) return;
     729        if ( $this->bail() ) {
     730            return;
     731        }
    672732
    673733        // Get topic forum ID
    … …  
    769829    public function row_actions( $actions, $topic ) {
    770830
    771         if ( $this->bail() ) return $actions;
     831        if ( $this->bail() ) {
     832            return $actions;
     833        }
    772834
    773835        unset( $actions['inline hide-if-no-js'] );
    774836
    775837        // Show view link if it's not set, the topic is trashed and the user can view trashed topics
    776         if ( empty( $actions['view'] ) && ( bbp_get_trash_status_id() === $topic->post_status ) && current_user_can( 'view_trash' ) )
     838        if ( empty( $actions['view'] ) && ( bbp_get_trash_status_id() === $topic->post_status ) && current_user_can( 'view_trash' ) ) {
    777839            $actions['view'] = '<a href="' . esc_url( bbp_get_topic_permalink( $topic->ID ) ) . '" title="' . esc_attr( sprintf( __( 'View &#8220;%s&#8221;', 'bbpress' ), bbp_get_topic_title( $topic->ID ) ) ) . '" rel="permalink">' . esc_html__( 'View', 'bbpress' ) . '</a>';
     840        }
    778841
    779842        // Only show the actions if the user is capable of viewing them :)
    … …  
    784847            if ( in_array( $topic->post_status, array( bbp_get_public_status_id(), bbp_get_closed_status_id() ) ) ) {
    785848                $close_uri = wp_nonce_url( add_query_arg( array( 'topic_id' => $topic->ID, 'action' => 'bbp_toggle_topic_close' ), remove_query_arg( array( 'bbp_topic_toggle_notice', 'topic_id', 'failed', 'super' ) ) ), 'close-topic_' . $topic->ID );
    786                 if ( bbp_is_topic_open( $topic->ID ) )
     849                if ( bbp_is_topic_open( $topic->ID ) ) {
    787850                    $actions['closed'] = '<a href="' . esc_url( $close_uri ) . '" title="' . esc_attr__( 'Close this topic', 'bbpress' ) . '">' . _x( 'Close', 'Close a Topic', 'bbpress' ) . '</a>';
    788                 else
     851                } else {
    789852                    $actions['closed'] = '<a href="' . esc_url( $close_uri ) . '" title="' . esc_attr__( 'Open this topic',  'bbpress' ) . '">' . _x( 'Open',  'Open a Topic',  'bbpress' ) . '</a>';
     853                }
    790854            }
    791855
    … …  
    805869            // Spam
    806870            $spam_uri  = wp_nonce_url( add_query_arg( array( 'topic_id' => $topic->ID, 'action' => 'bbp_toggle_topic_spam' ), remove_query_arg( array( 'bbp_topic_toggle_notice', 'topic_id', 'failed', 'super' ) ) ), 'spam-topic_'  . $topic->ID );
    807             if ( bbp_is_topic_spam( $topic->ID ) )
     871            if ( bbp_is_topic_spam( $topic->ID ) ) {
    808872                $actions['spam'] = '<a href="' . esc_url( $spam_uri ) . '" title="' . esc_attr__( 'Mark the topic as not spam', 'bbpress' ) . '">' . esc_html__( 'Not spam', 'bbpress' ) . '</a>';
    809             else
     873            } else {
    810874                $actions['spam'] = '<a href="' . esc_url( $spam_uri ) . '" title="' . esc_attr__( 'Mark this topic as spam',    'bbpress' ) . '">' . esc_html__( 'Spam',     'bbpress' ) . '</a>';
    811 
     875            }
    812876        }
    813877
    … …  
    843907    public function filter_dropdown() {
    844908
    845         if ( $this->bail() ) return;
     909        if ( $this->bail() ) {
     910            return;
     911        }
    846912
    847913        // Add Empty Spam button
    … …  
    875941    function filter_post_rows( $query_vars ) {
    876942
    877         if ( $this->bail() ) return $query_vars;
     943        if ( $this->bail() ) {
     944            return $query_vars;
     945        }
    878946
    879947        // Add post_parent query_var if one is present
    … …  
    906974        global $post_ID;
    907975
    908         if ( $this->bail() ) return $messages;
     976        if ( $this->bail() ) {
     977            return $messages;
     978        }
    909979
    910980        // URL for the current topic

  • trunk/src/includes/admin/users.php

    r4995 r5440  
    4444
    4545        // Bail if in network admin
    46         if ( is_network_admin() )
    47             return;
     46        if ( is_network_admin() ) {
     47            return;
     48        }
    4849
    4950        // User profile edit/display actions
    … …  
    7071
    7172        // Bail if current user cannot edit users
    72         if ( ! current_user_can( 'edit_user', $profileuser->ID ) )
    73             return;
     73        if ( ! current_user_can( 'edit_user', $profileuser->ID ) ) {
     74            return;
     75        }
    7476
    7577        // Get the roles
    … …  
    7779
    7880        // Only keymasters can set other keymasters
    79         if ( ! bbp_is_user_keymaster() )
    80             unset( $dynamic_roles[ bbp_get_keymaster_role() ] ); ?>
     81        if ( ! bbp_is_user_keymaster() ) {
     82            unset( $dynamic_roles[ bbp_get_keymaster_role() ] );
     83        } ?>
    8184
    8285        <h3><?php esc_html_e( 'Forums', 'bbpress' ); ?></h3>
    … …  
    126129
    127130        // Bail if current user cannot promote users
    128         if ( !current_user_can( 'promote_users' ) )
    129             return;
     131        if ( !current_user_can( 'promote_users' ) ) {
     132            return;
     133        }
    130134
    131135        // Get the roles
    … …  
    133137
    134138        // Only keymasters can set other keymasters
    135         if ( ! bbp_is_user_keymaster() )
    136             unset( $dynamic_roles[ bbp_get_keymaster_role() ] ); ?>
     139        if ( ! bbp_is_user_keymaster() ) {
     140            unset( $dynamic_roles[ bbp_get_keymaster_role() ] );
     141        } ?>
    137142
    138143        <label class="screen-reader-text" for="bbp-new-role"><?php esc_html_e( 'Change forum role to&hellip;', 'bbpress' ) ?></label>
    … …  
    158163
    159164        // Bail if current user cannot promote users
    160         if ( !current_user_can( 'promote_users' ) )
    161             return;
     165        if ( !current_user_can( 'promote_users' ) ) {
     166            return;
     167        }
    162168
    163169        // Bail if no users specified
    164         if ( empty( $_REQUEST['users'] ) )
    165             return;
     170        if ( empty( $_REQUEST['users'] ) ) {
     171            return;
     172        }
    166173
    167174        // Bail if this isn't a bbPress action
    168         if ( empty( $_REQUEST['bbp-new-role'] ) || empty( $_REQUEST['bbp-change-role'] ) )
    169             return;
     175        if ( empty( $_REQUEST['bbp-new-role'] ) || empty( $_REQUEST['bbp-change-role'] ) ) {
     176            return;
     177        }
    170178
    171179        // Check that the new role exists
    172180        $dynamic_roles = bbp_get_dynamic_roles();
    173         if ( empty( $dynamic_roles[ $_REQUEST['bbp-new-role'] ] ) )
    174             return;
     181        if ( empty( $dynamic_roles[ $_REQUEST['bbp-new-role'] ] ) ) {
     182            return;
     183        }
    175184
    176185        // Get the current user ID
    … …  
    182191
    183192            // Don't let a user change their own role
    184             if ( $user_id === $current_user_id )
     193            if ( $user_id === $current_user_id ) {
    185194                continue;
     195            }
    186196
    187197            // Set up user and role data
    … …  
    190200
    191201            // Only keymasters can set other keymasters
    192             if ( in_array( bbp_get_keymaster_role(), array( $user_role, $new_role ) ) && ! bbp_is_user_keymaster() )
     202            if ( in_array( bbp_get_keymaster_role(), array( $user_role, $new_role ) ) && ! bbp_is_user_keymaster() ) {
    193203                continue;
     204            }
    194205
    195206            // Set the new forums role
Note: See TracChangeset for help on using the changeset viewer.