Firefox/Mozilla XHTML Toolbar and PHP issues

Dear all!

If you operate Mozilla/Firefox using XHTML with a proper HTTP type, you might come to several problems:

1) fckeditor.php produces improper XHTML code. This can be fixed using the following patch (apply it to v.2, RC3)

diff -u -r1.1 fckeditor.php
--- fckeditor.php 9 Mar 2005 10:20:12 -0000 1.1
+++ fckeditor.php 9 Mar 2005 12:23:07 -0000
@@ -59,13 +59,13 @@
$Link = "{$this->BasePath}editor/fckeditor.html?InstanceName={$this->InstanceName}" ;

if ( $this->ToolbarSet != '' )
- $Link .= "&Toolbar={$this->ToolbarSet}" ;
+ $Link .= "&Toolbar={$this->ToolbarSet}" ;

// Render the linked hidden field.
- $Html .= "<input type=\"hidden\" id=\"{$this->InstanceName}\" name=\"{$this->InstanceName}\" value=\"{$HtmlValue}\">" ;
+ $Html .= "<input type=\"hidden\" id=\"{$this->InstanceName}\" name=\"{$this->InstanceName}\" value=\"{$HtmlValue}\" />" ;

// Render the configurations hidden field.
- $Html .= "<input type=\"hidden\" id=\"{$this->InstanceName}___Config\" value=\"" . $this->GetConfigFieldString() . "\">" ;
+ $Html .= "<input type=\"hidden\" id=\"{$this->InstanceName}___Config\" value=\"" . $this->GetConfigFieldString() . "\" />" ;

// Render the editor IFRAME.
$Html .= "<iframe id=\"{$this->InstanceName}___Frame\" src=\"{$Link}\" width=\"{$this->Width}\" height=\"{$this->Height}\" frameborder=\"no\" scrolling=\"no\"></iframe>" ;
@@ -116,7 +116,7 @@
foreach ( $this->Config as $sKey => $sValue )
{
if ( $bFirst == false )
- $sParams .= '&' ;
+ $sParams .= '&amp;' ;
else
$bFirst = false ;

2) After starting opening a XHTML page Mozilla/Firefox does not display the toolbar. The following patch fixes an typo in the editor/_source/classes/fckpanel_gecko.js

This patch can also be directly fixed in the editor/js/fckeditorcode_gecko_2.js by end-users who does not like to re-package the tool.

diff -u -r1.1 fckpanel_gecko.js
--- fckpanel_gecko.js 9 Mar 2005 12:25:43 -0000 1.1
+++ fckpanel_gecko.js 9 Mar 2005 12:26:37 -0000
@@ -48,7 +48,7 @@

FCKPanel.prototype.Create = function()
{
- this._IFrame = this.Window.document.body.appendChild( this.Window.document.createElement('IFRAME') ) ;
+ this._IFrame = this.Window.document.body.appendChild( this.Window.document.createElement('iframe') ) ;
this._IFrame.src = 'about:blank' ;
this._IFrame.frameBorder = '0';
this._IFrame.scrolling = 'no' ;