Details
-
Improvement
-
Status: Closed
-
Major
-
Resolution: Fixed
-
None
-
None
Description
Currently, the mojos use a parameter map. Setting this up is a little tedious and can be error prone.
The parameters can be populated as fields. This will make them simpler to develop and less error prone. It should also facilitate easier use of complex configuration like lists and nested objects using the existing plexus configuration technique.
It may be appropriate to move the qdox tags to the parameters themselves, reducing the need for as much configuration, but that is a secondary task - to be opened later after more thought.
If possible the request should be removed (the log can be made a part of AbstractPlugin - need to check anything else in the request that might not suit this). The reponse can already be removed and made into a return type as it only has one object within it. THis greatly simplifies the interface to:
public PluginExecutionResult execute()
throws PluginExecutionException;