Hi Wang, On Jul 14, 2008, at 6:13 PM, daning (SVN) wrote:

Author: daning Date: 2008-07-14 18:13:15 +0200 (Mon, 14 Jul 2008) New Revision: 323

Added: xwiki-plugin-officeimporter/src/main/java/com/xpn/xwiki/plugin/ officeimporter/OfficeConverter.java xwiki-plugin-officeimporter/src/main/java/com/xpn/xwiki/plugin/ officeimporter/OfficeConverterDefaultImp.java xwiki-plugin-officeimporter/src/main/java/com/xpn/xwiki/plugin/ officeimporter/OfficeConverterFactory.java xwiki-plugin-officeimporter/src/main/java/com/xpn/xwiki/plugin/ officeimporter/OfficeDocumentType.java xwiki-plugin-officeimporter/src/main/java/com/xpn/xwiki/plugin/ officeimporter/OfficeServerConnection.java Removed: xwiki-plugin-officeimporter/src/main/java/com/xpn/xwiki/plugin/ officeimporter/ImporterServerConnectionFactory.java xwiki-plugin-officeimporter/src/main/java/com/xpn/xwiki/plugin/ officeimporter/OfficeImporter.java xwiki-plugin-officeimporter/src/main/java/com/xpn/xwiki/plugin/ officeimporter/OfficeImporterFactory.java xwiki-plugin-officeimporter/src/main/java/com/xpn/xwiki/plugin/ officeimporter/OfficeImporterImp.java xwiki-plugin-officeimporter/src/main/java/com/xpn/xwiki/plugin/ officeimporter/OfficeImporterWebServiceImp.java Modified: xwiki-plugin-officeimporter/src/main/java/com/xpn/xwiki/plugin/ officeimporter/OfficeImporterPlugin.java Log: XSANDBOX-32: 1. rename importer to converter 2. add OfficeConverterException 3. add enum OfficeDocumentType 4. rename OfficeImporterImp to OfficeConverterDefaultImp

FWIW I personally prefer to have: - OfficeConverter (interface) - DefaultOfficeConverter (implementation) WDYT? Thanks -Vincent