[SOLR-9300] fix replace expression in GetMavenDependenciesTask's dependencyToArtifactId method.

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 5.6, 6.2, 7.0
• Assignee: Christine Poerschke
• Reporter: Christine Poerschke
• Created: 13/Jul/16 11:29
• Updated: 08/Jun/19 15:37
• Resolved: 13/Jul/16 16:19

Description

Fix replace expression in GetMavenDependenciesTask's dependencyToArtifactId method. (Christine Poerschke, Daniel Collins)

One line replace vs. replaceAll in GetMavenDependenciesTask.java since String replace takes the CharSequence (or char) literally and String replaceAll (or replaceFirst) takes a regular expression such a "(?<!solr-)analysis-".

(As an aside, in case anyone else is also wondering about the meaning of (?<!solr-) in the expression, the Pattern javadocs list a special construct (?<!X) as

X, via zero-width negative lookbehind

and the Lookahead and Lookbehind Zero-Length Assertions tutorial helped us understand the meaning of that list entry.)

Issue Links

• relates to: SOLR-9306 give solr/contrib/analysis-extras's test classes access to lucene/analysis's test classes