Maven 2 & 3

Resource needs a toString() method to properly print in debug mode

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.0.8
  • Fix Version/s: 2.0.9, 3.0-alpha-1
  • Component/s: None
  • Labels:
    None
  • Patch Submitted:
    Yes
  • Number of attachments :
    1

Description

It would be nice if Resouce had a toString() method so it would print something intelligent rather than simply a memory location in debug mode. (People have complained about this on the Users list, and I've noticed it myself – its annoying.)

Before this change, it prints like:
Resource@8f4fb3

With this patch, it prints like (assuming you accept my FileSet and PatternSet patches):
Resource {targetPath: /target/resource-test, filtering: true, FileSet {directory: null, PatternSet [includes: {}, excludes: {}]}}

Issue Links

is duplicated by

Improvement - An improvement or enhancement to an existing feature or task. MNG-2053 Implements a better toString method in the maven objects exemple in : o.a.m.p.MavenProject

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
relates to

Improvement - An improvement or enhancement to an existing feature or task. MNG-3467 PatternSet needs a toString() method to properly print in debug mode

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Improvement - An improvement or enhancement to an existing feature or task. MNG-3468 FileSet needs a toString() method to properly print in debug mode

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Hide
Permalink
Wayne Fay added a comment -

You can accept one patch without the other, but to get rid of all the @340sd9 stuff when printing Resources in debug mode, we need all 3.

Show
Wayne Fay added a comment - You can accept one patch without the other, but to get rid of all the @340sd9 stuff when printing Resources in debug mode, we need all 3.

People

  • Assignee:
    Brian Fox
    Reporter:
    Wayne Fay
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: