Menu

#6 AJAX and FCKEditor

open
None
5
2007-01-12
2006-12-15
johnpinto75
No

FCKFaces and ajax currently do not work.

There are a couple of things I changed to enable this.

1)If a custom resource path is specified then we use this to serve the JScript and do not go through the servlet. (This is only necessary due to the latest FCKeditor nightly having fixed the issue, but it is an improvement never the less. Better off not having the servlet serving JScript if the custom resource exists. Plus it does not make sense to import JS from jar and use custom path.)

2)Cannot inline the javascript import statement. I have created a flag to disbable the import. Please note that you must then import the FCKEditor/editor.js script manually.

3)Removed all JScript after
oFCKeditor.ReplaceTextarea();

I'm not sure why that was there originally, but it fails when using AJAX? Editor works fine without it.

Two new web.xml configuration params:

<context-param>
<param-name>customResourcePath</param-name>
<param-value>/</param-value>
<description>Allows to specify the path of the FCKeditor resources</description>
</context-param>

<context-param>
<param-name>org.fckfaces.DISABLE_JSCRIPT_IMPORT</param-name>
<param-value>true</param-value>
<description>Disbale inline script import statement</description>
</context-param>

Please note that originally customResourcePath was a servlet init variable, now it is a context-param due to scope where it is used. Somewhat confusing to have it in two places though.

I have included FCKFaceEditor with all the modifications. Hopefully you will add this to the next release.

Thanks,
John

Discussion

  • johnpinto75

    johnpinto75 - 2006-12-15

    Modified for Ajax use.

     
  • Shawn Recinto

    Shawn Recinto - 2007-01-12
    • assigned_to: nobody --> srecinto
     
  • Shawn Recinto

    Shawn Recinto - 2007-01-12

    Logged In: YES
    user_id=1376251
    Originator: NO

    Thanks John we will look this over and work it into the next release comming up in the next few weeks.

    Forgive us for the delay, we have been busy on a release of another project.

     

Log in to post a comment.

MongoDB Logo MongoDB