WP error converting URL for a domaini containing "1X"

[delluva]

Hello,

we've been experiencing the following problem while using WP 2.0 (and 2.0.1) on one of our projects.
The domain we're working on is "www.1x100.net".
WP converts the domain name into "www.1×100.net", and, obviously, stops working (the theme is not loaded correctly).

We've fixed the error by simply commenting line 36 in file "wp-includes/functions-formatting.php":

$curl = preg_replace('/(\d+)x(\d+)/', "$1×$2", $curl);

Actually we do ignore if any other problems may occour by commenting this line of code.
The problem appear to be present even if the "1x100" is the name of a folder in the complete URL.
Hope that this helps you.

Thanks for your time and your wonderful product!

lorenzo dell'Uva

[davidhouse]

I don't think we should be passing URLs through wp_texturise...

× is a multiplication sign in case anyone's interested.

[davidhouse]

Could you load wp-admin/options.php in your browser, find the option 'siteurl' and paste it here please? I'm wondering whether the value in the database is already incorrect or whether it's getting screwed up on the fly.

[markjaquith]

X multiplication fix

[markjaquith]

This is the second time that that thing has caused problems. See #1890

The uploaded patch should fix it in the 'home' value. anything with "url" or "directory" was already excluded.

I'm actually of the opinion that this functionality should be banished, for all the trouble it has caused. If you're a math blogger, you'll likely know how to code a multiplication symbol.

[delluva]

Hello,

we've applied the fix but it doesn't work correctly. Still need to comment the line 36.
By the way the site url is displayed correctly in the wp-admin/options.php even when the system gives the error.

[davidhouse]

You probably need siteurl as well. Try this second patch.

[markjaquith]

david,
it is alread testing strstr($info, 'url'), so strstr($info, 'siteurl') is redundant (it'll always match 'url' first).

I still think this thing should be banished altogether.

[davidhouse]

You're not wrong. Strange your patch didn't work then...

delluva, could you tell us exactly what the error says? (Particularily the line number will be helpful)

Mark: I'm not sure we should give up on this quite yet. If we don't find the culprit here, removing one line from wp_texturise won't solve all the problems: it could screw up on some other regex, and we'll get further tickets in the future. Best to find where wp_texturise is being called and remove it.

[davidhouse]

According to MattRead, "...the bloginfo filter passes all bloginfo calls through wptexturize".

I'll have a look at this tonight.

[davidhouse]

No idea why this took us so long to solve ;)

[markjaquith]

1

[davidhouse]

*steals bug back*

[davidhouse]

This should work now.

/me kicks trac

[ryan]

(In [3681]) Don't texturize home. Props Mark Jaquith and David House. fixes #2381

[ryan]

Reopen for inclusion in 2.0.3.

[ryan]

(In [3746]) Don't texturize home. Props Mark Jaquith and David House. fixes #2381

[(none)]

Milestone 2.0.3 deleted