Carriage return / linefeed duplication

[anonymous]

When writing a Wiki page on Windows (in FireFox) and then re-opening it for editing (or hitting preview), the number of newlines is multiplied every time. After one edit a single empty line turns into 3. If that page is previewed, those 3 turn into 15 empty lines, which at the next save or preview turn into about 30, etc. It is a severe usability problem, but probably easy to fix.

[daniel]

Sounds like we need to shave off some carriage returns or linefeeds there…

Thanks very much for the report! It's very helpful!

[daniel]

I've been unable to duplicate this. Has anyone else?

[daniel]

Marking this as worksforme for now, since we've not heard anyone else having this problem.

[anonymous]

It doesn't happen on this site for example. Perhaps my installation is somewhat unique in that a) it's on Windows and b) it's not running on Apache, but on the Abyss Webserver. The browser is not the source of the problem, as IE displays the same behavior as FireFox.

[anonymous]

I've looked at the code and the CR's are multiplying every time preview is hit. Here's a solution which works for me. In wiki.py, go to the code around line 673 which says: {{{ if self.args.has_key('text'):

self.page.set_content (self.argstext)}}}

and replace all '\r' with ('\n' seems to be automatically expanded to '\r\n'):

{{{ if self.args.has_key('text'):

self.argstext = self.argstext.replace('\r', ) self.page.set_content (self.argstext)}}}

[anonymous]

That code doesn't look quite as I intended it:

        if self.args.has_key('text'):
            self.args['text'] = self.args['text'].replace('\r', '')
            self.page.set_content (self.args['text'])

Funny thing is that Page.text does not contain extra CR's, just argstext does. Perhaps something funny is going on there.

[david.larsson@…]

I have the same problem. The server is apache on windows. It doesn't matter if I use IE or FireFox…

/David

[daniel]

I've managed to duplicate this now.. My bad for closing it early…

Thanks for reporting it, David.

/D

[daniel]

Should be fixed in [317].