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 _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
