Wedge

Public area => Bug reports => The Pub => Archived fixes => Topic started by: Arantor on March 20th, 2012, 08:56 PM

Title: Thoughts don't handle entity content properly
Post by: Arantor on March 20th, 2012, 08:56 PM

See live's thought, it had a double space in it, wedit::preparsecode converted that to an nbsp, but that's displayed rather than converted.

Title: Re: Thoughts don't handle entity content properly
Post by: Nao on March 20th, 2012, 08:59 PM

Hmm.

Thoughts are still a tad buggy, most notably in the sidebar with Ajax or something. Didn't know about that specific one though. Iirc Wedge and SMF add a nbsp and a space every time it finds two consecutive spaces, right?

Title: Re: Thoughts don't handle entity content properly
Post by: Arantor on March 20th, 2012, 09:19 PM

Specifically the preparser does that, switches two spaces for a nbsp then a space.

Title: Re: Thoughts don't handle entity content properly
Post by: Nao on March 20th, 2012, 11:40 PM

So I guess all we have to do is a str_replace?

Title: Re: Thoughts don't handle entity content properly
Post by: Arantor on March 20th, 2012, 11:42 PM

Probably, need to sit and play with it some more in case there's other odd side effects of over-processing. I'm thinking of ', " and & in particular...

Title: Re: Thoughts don't handle entity content properly
Post by: Nao on March 20th, 2012, 11:46 PM

I hate preparsing. There. I said it.

Title: Re: Thoughts don't handle entity content properly
Post by: Arantor on March 20th, 2012, 11:50 PM

You and me both :P

Title: Re: Thoughts don't handle entity content properly
Post by: Nao on March 20th, 2012, 11:59 PM

Isn't there an un_preparsecode() function or something somewhere? :whistle:

Title: Re: Thoughts don't handle entity content properly
Post by: Arantor on March 21st, 2012, 12:04 AM

There is, there's also an un_htmlspecialchars function and both have their place. But their place is rather specific and complicated.

Title: Re: Thoughts don't handle entity content properly
Post by: Arantor on February 20th, 2013, 12:02 AM

Can't reproduce now.