/*
 * Copyright 2016 the original author or authors.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *      http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

package org.gradle.api.internal.changedetection.rules;

import com.google.common.collect.ImmutableSet;
import com.google.common.collect.ImmutableSortedMap;
import org.gradle.api.internal.TaskInternal;
import org.gradle.api.internal.changedetection.state.ImplementationSnapshot;
import org.gradle.api.internal.changedetection.state.TaskExecution;
import org.gradle.api.internal.changedetection.state.ValueSnapshot;

import java.util.List;
import java.util.Map;
import java.util.Set;

class InputPropertyValueTaskStateChanges extends SimpleTaskStateChanges {
    private final TaskInternal task;
    private final Set<String> valueChanges;
    private final Set<String> implementationChanges;

    public InputPropertyValueTaskStateChanges(TaskExecution previousExecution, TaskExecution currentExecution, TaskInternal task) {
        ImmutableSortedMap<String, ValueSnapshot> previousInputProperties = previousExecution.getInputProperties();
        ImmutableSet.Builder<String> valueChangesBuilder = ImmutableSet.builder();
        ImmutableSet.Builder<String> implementationChangesBuilder = ImmutableSet.builder();
        ImmutableSortedMap<String, ValueSnapshot> currentInputProperties = currentExecution.getInputProperties();
        for (Map.Entry<String, ValueSnapshot> entry : currentInputProperties.entrySet()) {
            String propertyName = entry.getKey();
            ValueSnapshot currentSnapshot = entry.getValue();
            ValueSnapshot previousSnapshot = previousInputProperties.get(propertyName);
            if (previousSnapshot != null) {
                if (!currentSnapshot.equals(previousSnapshot)) {
                    if (currentSnapshot instanceof ImplementationSnapshot) {
                        implementationChangesBuilder.add(propertyName);
                    } else {
                        valueChangesBuilder.add(propertyName);
                    }
                }
            }
        }
        valueChanges = valueChangesBuilder.build();
        implementationChanges = implementationChangesBuilder.build();
        this.task = task;
    }

    @Override
    protected void addAllChanges(final List<TaskStateChange> changes) {
        for (String propertyName : implementationChanges) {
            changes.add(new DescriptiveChange("Implementation of input property '%s' has changed for %s", propertyName, task));
        }
        for (String propertyName : valueChanges) {
            changes.add(new DescriptiveChange("Value of input property '%s' has changed for %s", propertyName, task));
        }
    }
}

@@ -708,19 +708,19 @@
 mmutable
-Set
+Map
 ;%0Aimport
@@ -1073,30 +1073,8 @@
 Map;
-%0Aimport java.util.Set;
 %0A%0Acl
@@ -1204,85 +1204,44 @@
 nal 
-Set%3CString%3E valueChanges;%0A    private final Set%3CString%3E implementationC
+ImmutableMap%3CString, String%3E c
 hange
-s
+d
 ;%0A%0A 
@@ -1496,35 +1496,35 @@
        Immutable
-Set
+Map
 .Builder%3CString%3E
@@ -1526,114 +1526,25 @@
 ring
-%3E valueChangesBuilder = ImmutableSet.builder();%0A        ImmutableSet.Builder%3CString%3E implementationC
+, String%3E c
 hange
-s
+d
 Buil
@@ -1558,19 +1558,19 @@
 mmutable
-Set
+Map
 .builder
@@ -2111,398 +2111,243 @@
     
-if (currentSnapshot instanceof ImplementationSnapshot) %7B%0A                        implementationChangesBuilder.add(propertyName);%0A                    %7D else %7B%0A                        valueChangesBuilder.add(propertyName);%0A                    %7D%0A                %7D%0A            %7D%0A        %7D%0A        valueChanges = valueChangesBuilder.build();%0A        implementationC
+changedBuilder.put(%0A                        propertyName,%0A                        currentSnapshot instanceof ImplementationSnapshot ? %22Implementation%22 : %22Value%22);%0A                %7D%0A            %7D%0A        %7D%0A        this.c
 hange
-s = implementationC
+d = c
 hange
-s
+d
 Buil
@@ -2495,180 +2495,111 @@
 or (
-String propertyName : implementationChanges) %7B%0A            changes.add(new DescriptiveChange(%22Implementation of input property '%25s' has changed for %25s%22, propertyName, task)
+Map.Entry%3CString, String%3E entry : changed.entrySet()) %7B%0A            String propertyName = entry.getKey(
 );%0A 
@@ -2609,60 +2609,49 @@
     
-%7D%0A
     
-    for (String propertyName : valueChanges) %7B
+String changeType = entry.getValue();
 %0A   
@@ -2694,21 +2694,18 @@
 Change(%22
-Value
+%25s
  of inpu
@@ -2740,16 +2740,28 @@
 for %25s%22,
+ changeType,
  propert

Use map for tracking type of property change