Maven Shared Components

Filtering Resources does not match patterns strictly

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Duplicate
  • Affects Version/s: maven-filtering-1.0-beta-1
  • Fix Version/s: None
  • Component/s: maven-filtering
  • Labels:
    None
  • Number of attachments :
    0

Description

When using Maven Filtering, maven-war-plugin, variables within the JSP that do not correspond to configured properties are updated as well ie.

[code]
${users.id}
${user.id}
${adfadfadf.id}
[/code]

All of those patterns are matched to the project id which causes the JSP throw an exception. Having these properties set at build time lets me configure the context.path and links dynamically depending on the environment. The problem appears to be associated with the plexus pattern matching plugin as it is what decides to match any variable ending in .id

I suggest that an option is added to configure which patterns are matched and if the pattern matching is strict. For example, in my JSPs, it would be ideal to only allow @context.path@ and not ${users.id} so the variables within the JSP remain just that and configured properties are updated accordingly.

Issue Links

Activity

Hide
Olivier Lamy added a comment -

Certainly duplicate with MSHARED-60.
Can you try the current war plugin trunk it should be fixed ?

Show
Olivier Lamy added a comment - Certainly duplicate with MSHARED-60. Can you try the current war plugin trunk it should be fixed ?
Hide
Olivier Lamy added a comment -

And simple test/use case ?
Thanks

Show
Olivier Lamy added a comment - And simple test/use case ? Thanks
Hide
Olivier Lamy added a comment -

Duplicate with MSHARED-60.
You can test with the current war plugin trunk.
If you have any trouble please reopen the issue.

Show
Olivier Lamy added a comment - Duplicate with MSHARED-60. You can test with the current war plugin trunk. If you have any trouble please reopen the issue.
Hide
Walter White added a comment -

Yes, I tested this and it appears to work great now. Thanks much!

-Walter

Show
Walter White added a comment - Yes, I tested this and it appears to work great now. Thanks much! -Walter

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: