https://bugzilla.wikimedia.org/show_bug.cgi?id=35167

Dan Collins <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
                 CC|                            |[email protected]
            Version|unspecified                 |(wikimedia-deployment)
     Ever Confirmed|0                           |1

--- Comment #1 from Dan Collins <[email protected]> 2012-03-14 19:19:44 UTC 
---
Confirmed on 1.19wmf1. From the parser:

    $tocline = preg_replace(
        array( '#<(?!/?(sup|sub|i|b)(?: [^>]*)?>).*?'.'>#',
'#<(/?(sup|sub|i|b))(?: .*?)?'.'>#' ),
        array( '',                          '<$1>' ),
        $safeHeadline
    );

Those regexen are rather ugly, but let's see if we can't add a very limited
allowance for span:

    $tocline = preg_replace(
        array( '#<(?!/?(sup|sub|i|b|span dir="ltr")(?: [^>]*)?>).*?'.'>#',
'#<(/?(sup|sub|i|b|span dir="ltr"))(?: .*?)?'.'>#' ),
        array( '',                          '<$1>' ),
        $safeHeadline
    );

But I have to ask - someone went through the effort of not having a ?> in that
regex in two different places, and then left a ?> in another place, and I
notice that the world hasn't exploded.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are on the CC list for the bug.

_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to