On Jan 26, 2009, at 11:12 AM, asiri (SVN) wrote:

Author: asiri Date: 2009-01-26 11:12:27 +0100 (Mon, 26 Jan 2009) New Revision: 15733

Modified: platform/web/trunk/wysiwyg/src/main/java/com/xpn/xwiki/wysiwyg/ client/plugin/importer/ImporterPlugin.java Log: Addressing review comments: Checkstyle fix.

Modified: platform/web/trunk/wysiwyg/src/main/java/com/xpn/xwiki/ wysiwyg/client/plugin/importer/ImporterPlugin.java =================================================================== --- platform/web/trunk/wysiwyg/src/main/java/com/xpn/xwiki/wysiwyg/ client/plugin/importer/ImporterPlugin.java 2009-01-26 09:07:00 UTC (rev 15732) +++ platform/web/trunk/wysiwyg/src/main/java/com/xpn/xwiki/wysiwyg/ client/plugin/importer/ImporterPlugin.java 2009-01-26 10:12:27 UTC (rev 15733) @@ -132,7 +132,8 @@ if (importerDialog.isFilterStyles()) { params.put("filterStyles", "strict"); } - params.put("namespacesAware", "false"); // For Office2007 + // For Office2007

I think explaining why it is needed for office 2007 would help.

+ params.put("namespacesAware", "false");

WysiwygService.Singleton.getInstance().cleanOfficeHTML(inputHtml, "wysiwyg", params, this); } }

Thanks -Vincent http://xwiki.com http://xwiki.org http://massol.net