WordPress.org

Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (4 years ago)
Author:
pento
Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/class-wp-ajax-response.php

    r42201 r42343  
    2424     */
    2525    public function __construct( $args = '' ) {
    26         if ( !empty($args) )
    27             $this->add($args);
     26        if ( ! empty( $args ) ) {
     27            $this->add( $args );
     28        }
    2829    }
    2930
     
    6465    public function add( $args = '' ) {
    6566        $defaults = array(
    66             'what' => 'object', 'action' => false,
    67             'id' => '0', 'old_id' => false,
    68             'position' => 1,
    69             'data' => '', 'supplemental' => array()
     67            'what'         => 'object',
     68            'action'       => false,
     69            'id'           => '0',
     70            'old_id'       => false,
     71            'position'     => 1,
     72            'data'         => '',
     73            'supplemental' => array(),
    7074        );
    7175
     
    7377
    7478        $position = preg_replace( '/[^a-z0-9:_-]/i', '', $r['position'] );
    75         $id = $r['id'];
    76         $what = $r['what'];
    77         $action = $r['action'];
    78         $old_id = $r['old_id'];
    79         $data = $r['data'];
     79        $id       = $r['id'];
     80        $what     = $r['what'];
     81        $action   = $r['action'];
     82        $old_id   = $r['old_id'];
     83        $data     = $r['data'];
    8084
    8185        if ( is_wp_error( $id ) ) {
    8286            $data = $id;
    83             $id = 0;
     87            $id   = 0;
    8488        }
    8589
     
    8791        if ( is_wp_error( $data ) ) {
    8892            foreach ( (array) $data->get_error_codes() as $code ) {
    89                 $response .= "<wp_error code='$code'><![CDATA[" . $data->get_error_message( $code ) . "]]></wp_error>";
     93                $response .= "<wp_error code='$code'><![CDATA[" . $data->get_error_message( $code ) . ']]></wp_error>';
    9094                if ( ! $error_data = $data->get_error_data( $code ) ) {
    9195                    continue;
     
    9397                $class = '';
    9498                if ( is_object( $error_data ) ) {
    95                     $class = ' class="' . get_class( $error_data ) . '"';
     99                    $class      = ' class="' . get_class( $error_data ) . '"';
    96100                    $error_data = get_object_vars( $error_data );
    97101                }
     
    107111                }
    108112
    109                 $response .= "</wp_error_data>";
     113                $response .= '</wp_error_data>';
    110114            }
    111115        } else {
     
    124128            $action = $_POST['action'];
    125129        }
    126         $x = '';
     130        $x  = '';
    127131        $x .= "<response action='{$action}_$id'>"; // The action attribute in the xml output is formatted like a nonce action
    128         $x .=   "<$what id='$id' " . ( false === $old_id ? '' : "old_id='$old_id' " ) . "position='$position'>";
    129         $x .=       $response;
    130         $x .=       $s;
    131         $x .=   "</$what>";
    132         $x .= "</response>";
     132        $x .= "<$what id='$id' " . ( false === $old_id ? '' : "old_id='$old_id' " ) . "position='$position'>";
     133        $x .= $response;
     134        $x .= $s;
     135        $x .= "</$what>";
     136        $x .= '</response>';
    133137
    134138        $this->responses[] = $x;
     
    146150        header( 'Content-Type: text/xml; charset=' . get_option( 'blog_charset' ) );
    147151        echo "<?xml version='1.0' encoding='" . get_option( 'blog_charset' ) . "' standalone='yes'?><wp_ajax>";
    148         foreach ( (array) $this->responses as $response )
     152        foreach ( (array) $this->responses as $response ) {
    149153            echo $response;
     154        }
    150155        echo '</wp_ajax>';
    151         if ( wp_doing_ajax() )
     156        if ( wp_doing_ajax() ) {
    152157            wp_die();
    153         else
     158        } else {
    154159            die();
     160        }
    155161    }
    156162}
Note: See TracChangeset for help on using the changeset viewer.