Merge pull request #429 from ryantse/fix-property-cache

Fix property caching when using injectAllReactorProjects.

Author: TheSnoozer

src/main/java/pl/project13/maven/git/GitCommitIdMojo.java

@@ -532,9 +532,7 @@ private void handlePluginFailure(Exception e) throws GitCommitIdExecutionExcepti
 
   private void appendPropertiesToReactorProjects() {
     for (MavenProject mavenProject : reactorProjects) {
-
-      // TODO check message
-      log.info("{}] project {}", mavenProject.getName(), mavenProject.getName());
+      log.info("Adding properties to project: {}", mavenProject.getName());
 
       publishPropertiesInto(mavenProject.getProperties());
       mavenProject.setContextValue(CONTEXT_KEY, properties);

src/main/java/pl/project13/maven/git/GitDataProvider.java

@@ -248,7 +248,7 @@ protected SimpleDateFormat getSimpleDateFormatWithTimeZone() {
   protected void maybePut(@Nonnull Properties properties, String key, SupplierEx<String> value)
           throws GitCommitIdExecutionException {
     String keyWithPrefix = prefixDot + key;
-    if (properties.contains(keyWithPrefix)) {
+    if (properties.containsKey(keyWithPrefix)) {
       String propertyValue = properties.getProperty(keyWithPrefix);
       log.info("Using cached {} with value {}", keyWithPrefix, propertyValue);
     } else if (PropertiesFilterer.isIncluded(keyWithPrefix, includeOnlyProperties, excludeProperties)) {

src/main/java/pl/project13/maven/git/build/BuildServerDataProvider.java

@@ -166,7 +166,7 @@ protected void put(@Nonnull Properties properties, @Nonnull String key, String v
 
   protected void maybePut(@Nonnull Properties properties, @Nonnull String key, Supplier<String> supplier) {
     String keyWithPrefix = prefixDot + key;
-    if (properties.contains(keyWithPrefix)) {
+    if (properties.containsKey(keyWithPrefix)) {
       String propertyValue = properties.getProperty(keyWithPrefix);
       log.info("Using cached {} with value {}", keyWithPrefix, propertyValue);
     } else if (PropertiesFilterer.isIncluded(keyWithPrefix, includeOnlyProperties, excludeProperties)) {