WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (7 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/tests/phpunit/tests/pomo/pluralForms.php

    r41730 r42343  
    1313     * @return string the expression with parentheses added
    1414     */
    15     protected static function parenthesize_plural_expression($expression) {
     15    protected static function parenthesize_plural_expression( $expression ) {
    1616        $expression .= ';';
    17         $res = '';
    18         $depth = 0;
    19         for ($i = 0; $i < strlen($expression); ++$i) {
    20             $char = $expression[$i];
    21             switch ($char) {
     17        $res         = '';
     18        $depth       = 0;
     19        for ( $i = 0; $i < strlen( $expression ); ++$i ) {
     20            $char = $expression[ $i ];
     21            switch ( $char ) {
    2222                case '?':
    2323                    $res .= ' ? (';
    … …  
    2828                    break;
    2929                case ';':
    30                     $res .= str_repeat(')', $depth) . ';';
    31                     $depth= 0;
     30                    $res  .= str_repeat( ')', $depth ) . ';';
     31                    $depth = 0;
    3232                    break;
    3333                default:
    … …  
    3535            }
    3636        }
    37         return rtrim($res, ';');
     37        return rtrim( $res, ';' );
    3838    }
    3939
    … …  
    5454                return array();
    5555            }
    56             require_once $filename;         
    57         }
    58 
    59         $locales = GP_Locales::locales();
     56            require_once $filename;
     57        }
     58
     59        $locales            = GP_Locales::locales();
    6060        $plural_expressions = array();
    6161        foreach ( $locales as $slug => $locale ) {
    … …  
    8282
    8383        $parenthesized = self::parenthesize_plural_expression( $expression );
    84         $old_style = tests_make_plural_form_function( $nplurals, $parenthesized );
    85         $pluralForms = new Plural_Forms( $expression );
     84        $old_style     = tests_make_plural_form_function( $nplurals, $parenthesized );
     85        $pluralForms   = new Plural_Forms( $expression );
    8686
    8787        $generated_old = array();
    … …  
    103103                array(
    104104                    -1 => 1,
    105                     0 => 1,
    106                     1 => 0,
    107                     2 => 1,
    108                     5 => 1,
     105                    0  => 1,
     106                    1  => 0,
     107                    2  => 1,
     108                    5  => 1,
    109109                    10 => 1,
    110110                ),
    … …  
    115115                array(
    116116                    -1 => 1,
    117                     0 => 2,
    118                     1 => 1,
    119                     2 => 1,
     117                    0  => 2,
     118                    1  => 1,
     119                    2  => 1,
    120120                ),
    121121            ),
    … …  
    126126                    -2 => 2,
    127127                    -1 => 2,
    128                     0 => 2,
    129                     1 => 2,
    130                     2 => 1,
    131                     3 => 1,
     128                    0  => 2,
     129                    1  => 2,
     130                    2  => 1,
     131                    3  => 1,
    132132                ),
    133133            ),
    … …  
    137137                    -2 => 3,
    138138                    -1 => 3,
    139                     0 => 3,
    140                     1 => 3,
    141                     2 => 2,
    142                     3 => 1,
    143                     4 => 1,
     139                    0  => 3,
     140                    1  => 3,
     141                    2  => 2,
     142                    3  => 1,
     143                    4  => 1,
    144144                ),
    145145            ),
    … …  
    153153    public function test_simple( $expression, $expected ) {
    154154        $pluralForms = new Plural_Forms( $expression );
    155         $actual = array();
     155        $actual      = array();
    156156        foreach ( array_keys( $expected ) as $num ) {
    157157            $actual[ $num ] = $pluralForms->get( $num );
    … …  
    214214        try {
    215215            $pluralForms = new Plural_Forms( $expression );
    216             if( $call_get ) {
     216            if ( $call_get ) {
    217217                $pluralForms->get( 1 );
    218218            }
    … …  
    230230    public function test_cache() {
    231231        $mock = $this->getMockBuilder( 'Plural_Forms' )
    232             ->setMethods(array('execute'))
    233             ->setConstructorArgs(array('n != 1'))
     232            ->setMethods( array( 'execute' ) )
     233            ->setConstructorArgs( array( 'n != 1' ) )
    234234            ->getMock();
    235235
    236         $mock->expects($this->once())
    237             ->method('execute')
    238             ->with($this->identicalTo(2))
    239             ->will($this->returnValue(1));
    240 
    241         $first = $mock->get( 2 );
     236        $mock->expects( $this->once() )
     237            ->method( 'execute' )
     238            ->with( $this->identicalTo( 2 ) )
     239            ->will( $this->returnValue( 1 ) );
     240
     241        $first  = $mock->get( 2 );
    242242        $second = $mock->get( 2 );
    243243        $this->assertEquals( $first, $second );
Note: See TracChangeset for help on using the changeset viewer.