Uploaded image for project: 'Maven'
  1. Maven
  2. MNG-217

change mojos to per-lookup - use fields instead of req parms

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 2.0-alpha-1
    • Plugins and Lifecycle
    • 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;

      Attachments

        Activity

          People

            brett Brett Porter
            brett Brett Porter
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - 6h Original Estimate - 6h
                6h
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 8.75h
                8.75h