looks like a bug…

  • andorn

    (@andorn)


    18 years, 1 month ago

    … this line in functions-formatting.php:
    elseif (strstr($curl, '<code') || strstr($curl, '<pre') || strstr($curl, '<kbd' || strstr($curl, '<style') || strstr($curl, '<script')))

    is it a typo or i just misunderstand something?

Viewing 4 replies - 1 through 4 (of 4 total)
  • katesgasis

    (@katesgasis)

    18 years, 1 month ago

    It’s not a bug. It’s looking for a match.

    Thread Starter andorn

    (@andorn)

    18 years, 1 month ago

    mmm…
    but why there is no brace after strstr($curl, '<kbd'?
    strstr is (string,string) function, not (string,bool) so i’m surprised

    katesgasis

    (@katesgasis)

    18 years, 1 month ago

    Because it only searches for something that starts with “<kbd” in the variable $curl. If found, the string starting from ‘<kbd to the end of $curl will be returned.

    Thread Starter andorn

    (@andorn)

    18 years, 1 month ago

    As i understand, here we try to catch “<kbd” tag. So our function have to return something eq to true on “<kbd foobar”, is it?

    strstr($curl,'<kbd') works like this, but strstr($curl,'<kbd' || ... is not.

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘looks like a bug…’ is closed to new replies.