Make WordPress Core


Ignore:
Timestamp:
08/17/2015 05:35:58 PM (10 years ago)
Author:
azaozz
Message:

Fix creating of extra <br /> tags in both PHP and JS variants of wpautop(). Add PHP tests to catch similar problems in the future.
Props valendesigns, azaozz. Fixes #33377.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/tests/phpunit/tests/formatting/Autop.php

    r33469 r33624  
    445445        );
    446446    }
    447    
     447
     448    /**
     449     * wpautop() should not convert line breaks after <br /> tags
     450     *
     451     * @ticket 33377
     452     */
     453    function test_that_wpautop_skips_line_breaks_after_br() {
     454        $content = '
     455line 1<br>
     456line 2<br/>
     457line 3<br />
     458line 4
     459line 5
     460';
     461
     462        $expected = '<p>line 1<br />
     463line 2<br />
     464line 3<br />
     465line 4<br />
     466line 5</p>';
     467
     468        $this->assertEquals( $expected, trim( wpautop( $content ) ) );
     469    }
     470
     471    /**
     472     * wpautop() should convert multiple line breaks into a paragraph regarless of <br /> format
     473     *
     474     * @ticket 33377
     475     */
     476    function test_that_wpautop_adds_a_paragraph_after_multiple_br() {
     477        $content = '
     478line 1<br>
     479<br/>
     480line 2<br/>
     481<br />
     482';
     483
     484        $expected = '<p>line 1</p>
     485<p>line 2</p>';
     486
     487        $this->assertEquals( $expected, trim( wpautop( $content ) ) );
     488    }
     489
    448490}
Note: See TracChangeset for help on using the changeset viewer.