Tyler Johnson added a comment - 15/Mar/13 12:00 AM

PartialSubmit on the ice:inputRichText isn't firing in IF 3.2. Using saveOnSubmit="false" and partialSubmit="true". Attaching test case intended for deployment on Tomcat 7.

yip.ng added a comment - 18/Mar/13 12:34 PM - edited

Changes were applied to icefaces2 only. (Original target was 2.0.1) They were carried over to icefaces3 when icefaces3 was created. (Exact rev. 24137 shows up in icefaces3 svn history.) However, the file fckeditor_ext.js is now gone. Global search of some code snippets turned up nothing. Where did the original code get transferred to?

yip.ng added a comment - 19/Mar/13 10:01 AM - edited

Found a similarly-named file ckeditor_ext.js, but the code is totally different. The Java renderer class is also very different now. So it's not just a simple matter of re-applying the changes at rev. 24137.
Also, the original partial submit feature added was for when clicking the save button. See JIRA description and screenshot-02.png. However, the new test case doesn't even have a save button in it and it seems to want a partial submit on blur. So which should we do?

We need to restore the original partialSubmit functionality in a backwards compatible way only, no changes.

Done, but there seems to be something strange about the partial submit. It seems to be doing a full submit. See screenshot-03.png.

M: C:\svn\ossrepo\icefaces3\trunk\icefaces\compat\components\src\main\resources\META-INF\resources\inputrichtext\ckeditor_ext.js#33993
M: C:\svn\ossrepo\icefaces3\trunk\icefaces\compat\components\src\main\java\com\icesoft\faces\component\inputrichtext\InputRichTextRenderer.java#33993

The test case should be updated to use an ice:outputText to output the markup output instead of an input component. Then the test should work correctly with this change.

Done. Zip file of modified test case: ICE-6656.zip. Video of testing modified test case: http://screencast.com/t/qxaG6j7J.