[ https://issues.apache.org/jira/browse/WW-4489?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14487234#comment-14487234 ]
Jasper Rosenberg edited comment on WW-4489 at 4/9/15 11:58 AM: --------------------------------------------------------------- Well, all I was trying to point out is that the current pattern matches anything before {noformat}\\bclass{noformat} because of the "|.*|" section. I figured that wasn't intentional, but wasn't sure what the intent was. was (Author: perfnorm): Well, all I was trying to point out is that the current pattern matches anything before \\bclass, because of the "|.*|" section. I figured that wasn't intentional, but wasn't sure what the intent was. > Simplify DefaultExcludedPatternsChecker class pattern > ----------------------------------------------------- > > Key: WW-4489 > URL: https://issues.apache.org/jira/browse/WW-4489 > Project: Struts 2 > Issue Type: Bug > Components: Core Interceptors > Affects Versions: 2.3.20 > Reporter: Jasper Rosenberg > Priority: Minor > Fix For: 2.3.24 > > > The current pattern that matches class expressions looks like: > {code:java}"(.*\\.|^|.*|\\[('|\"))\\bclass(\\.|('|\")]|\\[).*"{code} > However, as far as I can tell, this is completely equivalent to: > {code:java}".*\\bclass(\\.|('|\")]|\\[).*"{code} > Not sure if this is a bug, or an improvement. -- This message was sent by Atlassian JIRA (v6.3.4#6332)