fix configuration cache problem with reports (#54)

Signed-off-by: Andreas Schmid <service@aaschmid.de>
aaschmid · Mar 27, 2021 · ee5d3b1 · ee5d3b1
1 parent 76c419e
commit ee5d3b1
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 7 deletions.
diff --git a/src/main/java/de/aaschmid/gradle/plugins/cpd/Cpd.java b/src/main/java/de/aaschmid/gradle/plugins/cpd/Cpd.java
@@ -14,7 +14,7 @@
 import org.gradle.api.InvalidUserDataException;
 import org.gradle.api.file.FileCollection;
 import org.gradle.api.file.FileTree;
-import org.gradle.api.internal.CollectionCallbackActionDecorator;
+import org.gradle.api.model.ObjectFactory;
 import org.gradle.api.reporting.Reporting;
 import org.gradle.api.reporting.SingleFileReport;
 import org.gradle.api.tasks.CacheableTask;
@@ -28,7 +28,6 @@
 import org.gradle.api.tasks.SourceTask;
 import org.gradle.api.tasks.TaskAction;
 import org.gradle.api.tasks.VerificationTask;
-import org.gradle.internal.reflect.Instantiator;
 import org.gradle.workers.WorkerExecutor;
 
 
@@ -88,8 +87,8 @@ public class Cpd extends SourceTask implements VerificationTask, Reporting<CpdRe
     private String skipBlocksPattern;
 
     @Inject
-    public Cpd(CollectionCallbackActionDecorator callbackActionDecorator, Instantiator instantiator, WorkerExecutor workerExecutor) {
-        this.reports = instantiator.newInstance(CpdReportsImpl.class, this, callbackActionDecorator);
+    public Cpd(ObjectFactory objectFactory, WorkerExecutor workerExecutor) {
+        this.reports = objectFactory.newInstance(CpdReportsImpl.class, this);
         this.workerExecutor = workerExecutor;
     }
 

diff --git a/src/main/java/de/aaschmid/gradle/plugins/cpd/CpdPlugin.java b/src/main/java/de/aaschmid/gradle/plugins/cpd/CpdPlugin.java
@@ -119,9 +119,9 @@ private void setupTaskDefaults(Project project, CpdExtension extension) {
             extensionMapping.map("reportsDir", () -> project.getExtensions().getByType(ReportingExtension.class).file("cpd"));
 
             task.getReports().all(report -> {
-                ConventionMapping reportMapping = ((IConventionAware) report).getConventionMapping();
-                reportMapping.map("enabled", () -> "xml".equals(report.getName()));
-                reportMapping.map("destination", () -> new File(extension.getReportsDir(), task.getName() + "." + report.getName()));
+                report.getRequired().convention("xml".equals(report.getName()));
+                report.getOutputLocation().convention(project.getLayout().getProjectDirectory().file(project.provider(() ->
+                    new File(extension.getReportsDir(), task.getName() + "." + report.getName()).getAbsolutePath())));
             });
         });
     }

diff --git a/src/main/java/de/aaschmid/gradle/plugins/cpd/internal/CpdReportsImpl.java b/src/main/java/de/aaschmid/gradle/plugins/cpd/internal/CpdReportsImpl.java
@@ -1,5 +1,7 @@
 package de.aaschmid.gradle.plugins.cpd.internal;
 
+import javax.inject.Inject;
+
 import de.aaschmid.gradle.plugins.cpd.Cpd;
 import de.aaschmid.gradle.plugins.cpd.CpdCsvFileReport;
 import de.aaschmid.gradle.plugins.cpd.CpdReports;
@@ -12,6 +14,7 @@
 
 public class CpdReportsImpl extends TaskReportContainer<SingleFileReport> implements CpdReports {
 
+    @Inject
     public CpdReportsImpl(Cpd task, CollectionCallbackActionDecorator callbackActionDecorator) {
         super(SingleFileReport.class, task, callbackActionDecorator);