Maven JXR

JXR has no option to include / exclude source files

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 2.1
  • Component/s: jxr
  • Labels:
    None
  • Testcase included:
    yes
  • Number of attachments :
    1

Description

While compiler plugin can exclude/include files jxr generates the sources for all of them

Activity

Ascending order - Click to sort in descending order
Carlos Sanchez made changes -
Field Original Value New Value
Type Bug [ 1 ] New Feature [ 2 ]
Brett Porter made changes -
Workflow Maven [ 46513 ] Maven New [ 49837 ]
Brett Porter made changes -
Key MOJO-258 MJXR-7
Project Mojo [ 11062 ] Maven 2.x JXR Plugin [ 11194 ]
Component/s jxr [ 11741 ]
Affects Version/s 2.0-beta-1 [ 12116 ]
Brett Porter made changes -
Fix Version/s 2.0 [ 12480 ]
Brett Porter made changes -
Project Maven 2.x JXR Plugin [ 11194 ] Maven JXR [ 11085 ]
Key MJXR-7 JXR-17
Fix Version/s 1.1 [ 12533 ]
Fix Version/s 2.0 [ 12480 ]
Complexity Intermediate
Hide
Permalink
Dennis Lundberg added a comment -

Here is a proposed patch to add include/exclude functionality. As this is my first venture into JXR I would appreciate if someone could review this patch. If it is considered good I can apply it myself.

Please note the @todo on line 108 in JXR.java. I see that there is some extra handling in the JXR-version of DirectoryScanner, but I don't think subclassing is the most elegant solution. As it is now the includes/excludes must be added twice. Once for PackageManager and once for DirectoryScanner.

Show
Dennis Lundberg added a comment - Here is a proposed patch to add include/exclude functionality. As this is my first venture into JXR I would appreciate if someone could review this patch. If it is considered good I can apply it myself. Please note the @todo on line 108 in JXR.java. I see that there is some extra handling in the JXR-version of DirectoryScanner, but I don't think subclassing is the most elegant solution. As it is now the includes/excludes must be added twice. Once for PackageManager and once for DirectoryScanner.
Dennis Lundberg made changes -
Attachment JXR-17.patch [ 25771 ]
Dennis Lundberg made changes -
Patch attached Yes
Assignee Dennis Lundberg [ dennislundberg ]
Testcase included yes
Hide
Permalink
Dennis Lundberg added a comment -

I have applied the patch, after Brett had a look at it.

Show
Dennis Lundberg added a comment - I have applied the patch, after Brett had a look at it.
Dennis Lundberg made changes -
Status Open [ 1 ] Closed [ 6 ]
Resolution Fixed [ 1 ]
Dennis Lundberg made changes -
Component/s jxr [ 12710 ]

People

  • Assignee:
    Dennis Lundberg
    Reporter:
    Carlos Sanchez
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: