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