* REPO-4639: Split tika engine_config.json into separate transformers.
* WIP: REPO-4639 Content conversion failed using Tika
The Tika T-Engine "transform" option does not exist when called via the Transform Service or Local transforms, which resulted in no transforms taking place. However this value is really not be needed as the T-Engine should be able to read its own engine_config.xml to work out which sub transform to use. Transforms only worked via Legacy transforms, which used a T-Engine.
This code is based on tried and tested ACS repository code. It has been further simplified.
TODO:
- replace the ConfigFileFinder class just added with something that uses Spring to read the JSON. i.e. simplify it.
- replace the CombinedConfig class just added with something that does not need the InLineTransformer. i.e. simplify it.
- create tests based on the repo tests
- remove the source and target mimetype checks in Tika as a check against engine_config.xml is cleaner.
- repeat the process for the Misc T-Engine as it has similar code checking source and target mimetypes.
- remove the transform option passed by the legacy transforms.
* Removed CombindConfig and ConfigFileFnder classes.
* Extracted AbstractTransformRegistry so that it may be used in the ACS repository too.
TODO AbstractTransformRegistry and AbstractTransformRegistry need to be moved to the alfresco-transform-model pakage
* tidy up only
* REPO-4639: Add priority to duplicate transforms.
* REPO-4639: Refactor TikaTransformationIT to use the new Tika /transform specifications
Changes AbstractTransformerControllerTest as the engine_config is now loaded in TransformRegistryImpl instead of AbstractTransformerController
* Rename to TransformServiceRegistry, so we don't have to change the repo code.
* Added the baseUrl parameter to the register method and fixed the missed rename in the last commit.
* Javadoc change only
* Moved common classes (with repo) AbbstractTransformRegistry and TransformServiceRegistry to alfresco-transform-model
* Replace (simplify) all the isTransformable calls with a check against the JSON.
- Tests now only pass targetEncoding to the 'string' transformer.
* Fix failing tests.
* Revert port change
* REPO-4639 : Add priorities to misc engine_config
* REPO-4639 : Add priorities to pdf-renderer and imagemagick engine_config
* Remove test that is @Ignored
* Pick up alfresco-transformer-model 1.0.2.7-REPO-4639-1
* REPO-4639 : Add priorities to libreoffice engine_config
* REPO-4639 : Add priorities to tika engine_config
* REPO-4639 : Remove all priorities with value equal to 50 (default) from engine_config
* Switch over to using TransformServiceRegistry in org.alfresco.transform.client.registry
Reintroduce the noExtensionSourceFilenameTest having removed @Ignore.
* New whitesource issue on commons-compress 1.18. Upgrading to 1.19.
* Removed the text/javascript -> text/plain test as this is not supported
* Modifications as a result of changes to method names in alfresco-transform-model
* Pick up alfresco-transform-model 1.0.2.7-ATS545-2
* Remove unused imports
- move startup message from controllers to the Application classes (SpringBoot configuration beans)
- added static imports for most static variables and static methods
- simplified a few nested *if*s
- replaced Arrays.asList() with explicit immutable collections
- fixed a few IntelliJ code inspection warnings
- code refactoring:
if successful -> return the object ( Optional.ofNullable(request) )
if not successful -> throw an exception (instead of sending an error message on the queue)
- when calling 'convert()' method, catch this exception and send a T-Reply with error message on the queue
- add correlationID to error logs
- upgraded to Spring boot 2.0.5
- added dependency to spring-boot-starter-web (before it was brought in by thymeleaf 1.5.15)
- removed try-catch since UnsupportedEncodingException is not thrown anymore by HierarchicalUriComponents.encodeUriComponent(..)
- updated WebApplicationConfig to implement WebMvcConfigurer (WebMvcConfigurerAdapter is deprecated)
- updated import of LocalServerPort
- updated tests to use new method from InvocationOnMock
