Maven 2 & 3

Plugin Goal report should indicate which fields are read only. (Patch Attached)

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Won't Fix
  • Affects Version/s: None
  • Fix Version/s: 2.0.5
  • Component/s: Plugin API
  • Labels:
    None
  • Complexity:
    Novice
  • Number of attachments :
    2

Description

The report shows which fields are discovered and optional, but not which are read only. This can lead to some newby confusion where they think every field needs to be defined.

Issue Links

This issue is duplicated by:
MNG-2375 Generate docs for mojo goals should list whehter or not each parameter is read-only Minor Closed

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Brian Fox added a comment -

The first patch decorates with (Read Only) similar to (Discovered) and (Optional). A less confusing option is to not even show read only parameters. 2nd patch created for this option.

Show
Brian Fox added a comment - The first patch decorates with (Read Only) similar to (Discovered) and (Optional). A less confusing option is to not even show read only parameters. 2nd patch created for this option.
Hide
Permalink
Dan Tran added a comment -

+1 for NOT show at all

Show
Dan Tran added a comment - +1 for NOT show at all
Hide
Permalink
John Allen added a comment -

+1 for not showing, readonly == private impl.

Show
John Allen added a comment - +1 for not showing, readonly == private impl.
Hide
Permalink
fabrizio giustina added a comment -

another +1 from me for not showing readonly parameters, this will really make the goal parameters page easier to read for users.

Patch is good, can't commit it direclty since plugin tools are in the maven core area... can anybody with enough karma commit it?

Show
fabrizio giustina added a comment - another +1 from me for not showing readonly parameters, this will really make the goal parameters page easier to read for users. Patch is good, can't commit it direclty since plugin tools are in the maven core area... can anybody with enough karma commit it?
Hide
Permalink
Brian Fox added a comment -

Bueller?

Show
Brian Fox added a comment - Bueller?
Hide
Permalink
Dennis Lundberg added a comment -

With the changes to plugin-plugin in MPLUGIN-7, read-only parameters are no longer shown. So there is now no need for this patch.

Show
Dennis Lundberg added a comment - With the changes to plugin-plugin in MPLUGIN-7, read-only parameters are no longer shown. So there is now no need for this patch.

People

Vote (3)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Codehaus. Try JIRA - bug tracking software for your team.