Opened 17 years ago
Closed 17 years ago
#1901 closed Bug (duplicate)
List item mangling due to extra whitespaces
Reported by: | Greg Miller | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | Project : MediaWiki+FCKeditor | Version: | FCKeditor 2.5.1 |
Keywords: | Cc: |
Description
Once again, I apologize for cross-posting this item; I forgot to set the ticket properties when I submitted #1900 and there does not appear to be any way to edit a ticket after it is submitted.
We first discovered this issue locally in IE7 (FF seems unaffected), but this issue was confirmed in IE7 in the MediaWiki+FCKEditor sandbox. http://mediawiki.fckeditor.net/index.php/Sandbox
The problem is FCKeditor is mis-behaving when dealing with extra white spaces in list items. For example, if you take the following wikitext list
* a bullet list * anchors * links
and put three blank spaces immediately before and after the word "bullet" in the first list item then click 'Show preview' or view the Wikitext and go back to the WYSIWYG display. The list gets mangled to look like the following (again in wikitext).
* * a * bullet list * anchors * links
Sometimes I also get a Javascript error:
Line: 88 Error: 'null' is null or not an object
From the following file -- (I had to fat-finger this in, so it may not be completely accurate) extensions/FCKeditor/fckeditor/editor/js/fckeditorcode_ie.js
This is the offending line of code (line 88):
var FCKEvents=function(A){this.Owner=A;this._RegisteredEvents={};};FCKEvents.prototype.AttachEvent=function(A,B){var C;if (!(C=this._RegisteredEvents[A])) this._RegisteredEvents[A]=[B];else C.push(B);};FCKEvents.prototype.FireEvent=function(A,B){var C=true;var D=this._RegisteredEvents[A];if (D){for (var i=0;i<D.length;i++){try{C=(D[i](this.Owner,B)&&C);}catch(e){if (e.number!=-2146823277) throw e;}}};return C;};
Hi!
Don't worry about the ticket properties so much. It has been changed already to MW+FCKeditor so I will close this one and we will investigate ticket #1900.
Best regards!!
DUP of #1900