plugin pads variables with trailing white space.
Expand 555123 555 here
This seems to break some plugins and would break pages where variables are expected to expand without any white space.
Edit this page with the Raw editor, remove the trailing space from the 123. Then edit the page with wysiwyg, and make a minor change elsewhere on the page and save it. The string 555123555 becomes 555123 555.
- 17 Jan 2008
The actual cause is that Wysiwyg adds a space after any text in bullets. Not just settings.
Besides this bug we also have Item5190
and probably many others not yet reported because of this one.
Crawford. Is this space added as a way to avoid new-line eating or is it just a plain bug? I tried to see where in the code it is added. But it is not obvious to me.
As far as I can see editing and saving in TMCE adds a space to ALL bullets except the last one. Not the last one in a sequence of bullets. The last one in the topic does not get the space.
The first thing to consider is - strip off trailing spaces. But I know of at least ONE setting where the trailing space it intentional in a * Set statement so that is not a good idea either.
We have to step carefully if we fix this. We do not want an "eat newline" bug to re-appear.
- 17 Jan 2008
I'd be concerned about stripping spaces as well as adding them. We do a fair amount of dynamic content generation using variables, inserting text into tables, and other places where an extra space might change the layout or content. I think that in the case of set statements it's pretty critical that the user's literal input is not modified in any way.
I think I'd be more comfortable if the WYSIWYG editor wasn't the default editor on 4.2. At least with the 4.1 skins the user makes a fairly easy decision on whether or not to use the WYSIWYG. Now with "Raw Edit" hidden down on the bottom of the topic, users are going to be surprised, and I expect a lot of topic damage as users get used to the changes.
- 18 Jan 2008
A link to this has also been added to the installation guide for 4.2.0.
If there is a large public demand for a real feature to swap this we may add it in 4.2.1 but not in 4.2.0. Too risky.
But the TWiki:Codev.TWikiRawEditDefault04x02
is a quick and easy hack.
- 18 Jan 2008
Goddam it, how many times have i fixed this bug? At least three. It is soooo
hard to get right!
- 21 Jan 2008
OK, fourth time lucky. I discovered TMCE was adding a space where it didn't used to add a space
The merging from MAIN to T42 branch did not succeed. Something was missing. Another good reason why the developer and not I should do the merging.
Also updating the date on the change history. I will update t.o plugins also.