Item9509: Rendering differences between 1.0.x and 1.1

See also Item9432 and Item9492

For skin=text;conttentype=text/plain, Trunk leaves <nop> and <noautolink> tags in place. Release removes them.

-- GeorgeClark - 20 Aug 2010

The problem appears to be a change in the _prepare routine of Foswiki/UI/View.pm. The line: $text =~ s/( ?) *<\/?(nop|noautolink)\/?>\n?/$1/gois; was removed.

http://trac.foswiki.org/changeset/4065/trunk/core/lib/Foswiki/UI/View.pm

-- GeorgeClark - 20 Aug 2010

Micha, can you comment on why you removed the code that stripped noautolink and nop from View::_prepare ? That seems to be the root cause of the JQuery problem in Item9492. I've got a unit test to check in that will verify the issue but need feedback on how to fix. Just put back the removed code, or is there some other issue that triggered it's removal?

-- GeorgeClark - 20 Aug 2010

Well compare http://translate.foswiki.org/System/JQueryAjaxHelper?section=user;skin=text&q=m&limit=10&timestamp=1282306571573&contenttype=text/plain with http://translate.foswiki.org/System/JQueryAjaxHelper?section=user;skin=text&q=m&limit=10&timestamp=1282306571573

Removing the contenttype url param was the quickest possible fix as the content type isn't relevant for further processing the result in this case.

I am not sure who or why nop and noautolink are disabled when is contenttype=text/plain ... nor can I possibly think of any advantage of leaving them in. So that's possibly an error.

-- MichaelDaum - 20 Aug 2010

Commenting is disabled while not logged in