Deprecate ProjectLayout.configurableFiles() as it is replaced by `O…

…bjectFactory.fileCollection()`. Also replace usages of internal `FileOperations.configurableFiles()` method with the public API.

subprojects/code-quality/src/main/groovy/org/gradle/api/plugins/quality/PmdExtension.java

@@ -167,7 +167,7 @@ public FileCollection getRuleSetFiles() {
      * If you want to only use custom rule sets, you must clear {@code ruleSets}.
      */
     public void setRuleSetFiles(FileCollection ruleSetFiles) {
-        this.ruleSetFiles = project.getLayout().configurableFiles(ruleSetFiles);
+        this.ruleSetFiles = project.getObjects().fileCollection().from(ruleSetFiles);
     }
 
     /**

subprojects/core-api/src/main/java/org/gradle/api/file/ProjectLayout.java

@@ -180,6 +180,8 @@ public interface ProjectLayout {
      * @param paths The paths to the files. May be empty.
      * @return The file collection. Never returns null.
      * @since 4.8
+     * @deprecated Please use {@link ObjectFactory#fileCollection()} instead.
      */
+    @Deprecated
     ConfigurableFileCollection configurableFiles(Object... paths);
 }

subprojects/core/src/integTest/groovy/org/gradle/api/file/FileCollectionSymlinkIntegrationTest.groovy

@@ -50,16 +50,24 @@ class FileCollectionSymlinkIntegrationTest extends AbstractIntegrationSpec {
         """
 
         when:
-        executer.usingBuildScript(buildScript).run()
+        maybeDeprecated(code)
+        run()
 
         then:
         noExceptionThrown()
 
         where:
-        desc                                  | code
-        "project.files()"                     | "project.files(file, symlink, symlinked)"
-        "project.layout.files()"              | "project.layout.files(file, symlink, symlinked)"
-        "project.layout.configurableFiles()"  | "project.layout.configurableFiles(file, symlink, symlinked)"
-        "project.fileTree()"                  | "project.fileTree(baseDir)"
+        desc                                 | code
+        "project.files()"                    | "project.files(file, symlink, symlinked)"
+        "project.fileTree()"                 | "project.fileTree(baseDir)"
+        "project.layout.files()"             | "project.layout.files(file, symlink, symlinked)"
+        "project.layout.configurableFiles()" | "project.layout.configurableFiles(file, symlink, symlinked)"
+        "project.objects.fileCollection()"   | "project.objects.fileCollection().from(file, symlink, symlinked)"
+    }
+
+    void maybeDeprecated(String expression) {
+        if (expression.contains("configurableFiles")) {
+            executer.expectDeprecationWarning()
+        }
     }
 }

subprojects/core/src/integTest/groovy/org/gradle/api/file/FilePropertiesIntegrationTest.groovy

@@ -847,7 +847,7 @@ class SomeTask extends DefaultTask {
             class ConsumerTask extends DefaultTask {
             
                 @InputFiles
-                ConfigurableFileCollection inputFiles = project.layout.configurableFiles()
+                ConfigurableFileCollection inputFiles = project.objects.fileCollection()
                 
                 @Optional
                 @OutputFile

subprojects/core/src/integTest/groovy/org/gradle/api/file/ProjectLayoutIntegrationTest.groovy

@@ -18,11 +18,10 @@ package org.gradle.api.file
 
 import org.gradle.api.internal.file.collections.ImmutableFileCollection
 import org.gradle.integtests.fixtures.AbstractIntegrationSpec
-import org.gradle.util.TextUtil
 import spock.lang.Unroll
 
 class ProjectLayoutIntegrationTest extends AbstractIntegrationSpec {
-    private static final String STRING_CALLABLE = 'new java.util.concurrent.Callable<String>() { String call() { return "%s/src/resource/file.txt" } }'
+    private static final String STRING_CALLABLE = 'new java.util.concurrent.Callable<String>() { String call() { return "src/resource/file.txt" } }'
 
     def "can access the project dir and build dir"() {
         buildFile << """
@@ -179,6 +178,7 @@ class ProjectLayoutIntegrationTest extends AbstractIntegrationSpec {
         """
 
         when:
+        maybeDeprecated(expression)
         run()
 
         then:
@@ -196,44 +196,45 @@ class ProjectLayoutIntegrationTest extends AbstractIntegrationSpec {
         file('src/resource/file.txt') << "some text"
 
         buildFile << """
-            def fileCollection = ${TextUtil.normaliseFileSeparators(String.format(expressionTemplate, testDirectory.absolutePath))}
+            def fileCollection = $expression
             println("size = \${fileCollection.files.size()}")
         """
 
         when:
+        maybeDeprecated(expression)
         run()
 
         then:
         outputContains('size = 1')
 
         where:
-        collectionType               | content          | expressionTemplate
-        'FileCollection'             | 'String'         | 'project.layout.files("%s/src/resource/file.txt")'
-        'FileCollection'             | 'File'           | 'project.layout.files(new File("%s", "src/resource/file.txt"))'
-        'FileCollection'             | 'Path'           | 'project.layout.files(java.nio.file.Paths.get("%s/src/resource/file.txt"))'
-        'FileCollection'             | 'URI'            | 'project.layout.files(new File("%s", "/src/resource/file.txt").toURI())'
-        'FileCollection'             | 'URL'            | 'project.layout.files(new File("%s", "/src/resource/file.txt").toURI().toURL())'
+        collectionType               | content          | expression
+        'FileCollection'             | 'String'         | 'project.layout.files("src/resource/file.txt")'
+        'FileCollection'             | 'File'           | 'project.layout.files(new File("src/resource/file.txt"))'
+        'FileCollection'             | 'Path'           | 'project.layout.files(java.nio.file.Paths.get("src/resource/file.txt"))'
+        'FileCollection'             | 'URI'            | 'project.layout.files(new File(projectDir, "/src/resource/file.txt").toURI())'
+        'FileCollection'             | 'URL'            | 'project.layout.files(new File(projectDir, "/src/resource/file.txt").toURI().toURL())'
         'FileCollection'             | 'Directory'      | 'project.layout.files(project.layout.projectDirectory)'
         'FileCollection'             | 'RegularFile'    | 'project.layout.files(project.layout.projectDirectory.file("src/resource/file.txt"))'
-        'FileCollection'             | 'Closure'        | 'project.layout.files({ "%s/src/resource/file.txt" })'
-        'FileCollection'             | 'List'           | 'project.layout.files([ "%s/src/resource/file.txt" ])'
-        'FileCollection'             | 'array'          | 'project.layout.files([ "%s/src/resource/file.txt" ] as Object[])'
-        'FileCollection'             | 'FileCollection' | "project.layout.files(${ImmutableFileCollection.name}.of(new File('%s/src/resource/file.txt')))"
+        'FileCollection'             | 'Closure'        | 'project.layout.files({ "src/resource/file.txt" })'
+        'FileCollection'             | 'List'           | 'project.layout.files([ "src/resource/file.txt" ])'
+        'FileCollection'             | 'array'          | 'project.layout.files([ "src/resource/file.txt" ] as Object[])'
+        'FileCollection'             | 'FileCollection' | "project.layout.files(${ImmutableFileCollection.name}.of(new File('src/resource/file.txt')))"
         'FileCollection'             | 'Callable'       | "project.layout.files($STRING_CALLABLE)"
         'FileCollection'             | 'Provider'       | "project.layout.files(provider($STRING_CALLABLE))"
         'FileCollection'             | 'nested objects' | "project.layout.files({[{$STRING_CALLABLE}]})"
 
-        'ConfigurableFileCollection' | 'String'         | 'project.layout.configurableFiles("%s/src/resource/file.txt")'
-        'ConfigurableFileCollection' | 'File'           | 'project.layout.configurableFiles(new File("%s", "src/resource/file.txt"))'
-        'ConfigurableFileCollection' | 'Path'           | 'project.layout.configurableFiles(java.nio.file.Paths.get("%s/src/resource/file.txt"))'
-        'ConfigurableFileCollection' | 'URI'            | 'project.layout.configurableFiles(new File("%s", "/src/resource/file.txt").toURI())'
-        'ConfigurableFileCollection' | 'URL'            | 'project.layout.configurableFiles(new File("%s", "/src/resource/file.txt").toURI().toURL())'
+        'ConfigurableFileCollection' | 'String'         | 'project.layout.configurableFiles("src/resource/file.txt")'
+        'ConfigurableFileCollection' | 'File'           | 'project.layout.configurableFiles(new File("src/resource/file.txt"))'
+        'ConfigurableFileCollection' | 'Path'           | 'project.layout.configurableFiles(java.nio.file.Paths.get("src/resource/file.txt"))'
+        'ConfigurableFileCollection' | 'URI'            | 'project.layout.configurableFiles(new File(projectDir, "/src/resource/file.txt").toURI())'
+        'ConfigurableFileCollection' | 'URL'            | 'project.layout.configurableFiles(new File(projectDir, "/src/resource/file.txt").toURI().toURL())'
         'ConfigurableFileCollection' | 'Directory'      | 'project.layout.configurableFiles(project.layout.projectDirectory)'
         'ConfigurableFileCollection' | 'RegularFile'    | 'project.layout.configurableFiles(project.layout.projectDirectory.file("src/resource/file.txt"))'
-        'ConfigurableFileCollection' | 'Closure'        | 'project.layout.configurableFiles({ "%s/src/resource/file.txt" })'
-        'ConfigurableFileCollection' | 'List'           | 'project.layout.configurableFiles([ "%s/src/resource/file.txt" ])'
-        'ConfigurableFileCollection' | 'array'          | 'project.layout.configurableFiles([ "%s/src/resource/file.txt" ] as Object[])'
-        'ConfigurableFileCollection' | 'FileCollection' | "project.layout.configurableFiles(${ImmutableFileCollection.name}.of(new File('%s/src/resource/file.txt')))"
+        'ConfigurableFileCollection' | 'Closure'        | 'project.layout.configurableFiles({ "src/resource/file.txt" })'
+        'ConfigurableFileCollection' | 'List'           | 'project.layout.configurableFiles([ "src/resource/file.txt" ])'
+        'ConfigurableFileCollection' | 'array'          | 'project.layout.configurableFiles([ "src/resource/file.txt" ] as Object[])'
+        'ConfigurableFileCollection' | 'FileCollection' | "project.layout.configurableFiles(${ImmutableFileCollection.name}.of(new File('src/resource/file.txt')))"
         'ConfigurableFileCollection' | 'Callable'       | "project.layout.configurableFiles($STRING_CALLABLE)"
         'ConfigurableFileCollection' | 'Provider'       | "project.layout.configurableFiles(provider($STRING_CALLABLE))"
         'ConfigurableFileCollection' | 'nested objects' | "project.layout.configurableFiles({[{$STRING_CALLABLE}]})"
@@ -243,7 +244,7 @@ class ProjectLayoutIntegrationTest extends AbstractIntegrationSpec {
     def 'can create #collectionType with #dependencyType dependency'() {
         buildFile << """
             task myTask {
-                def outputFile = new File('${TextUtil.normaliseFileSeparators(testDirectory.absolutePath)}', 'build/resource/file.txt')
+                def outputFile = file('build/resource/file.txt')
                 doLast {
                     outputFile.text = "some text"
                 }
@@ -255,10 +256,11 @@ class ProjectLayoutIntegrationTest extends AbstractIntegrationSpec {
         """
 
         when:
+        maybeDeprecated(expression)
         run('myTask')
 
         then:
-        outputContains("files = [${new File(testDirectory.absolutePath , '/build/resource/file.txt').absolutePath}]")
+        outputContains("files = [${testDirectory.file('/build/resource/file.txt').absolutePath}]")
 
         where:
         collectionType               | dependencyType | expression
@@ -272,15 +274,15 @@ class ProjectLayoutIntegrationTest extends AbstractIntegrationSpec {
     def '#methodName enforces build dependencies when given Task as input'() {
         buildFile << """
             task producer {
-                def outputFile = new File('${TextUtil.normaliseFileSeparators(testDirectory.absolutePath)}', 'build/resource/file.txt')
+                def outputFile = file('build/resource/file.txt')
                 outputs.file outputFile
                 doLast {
                     outputFile.text = "some text"
                 }
             }
 
             task consumer {
-                def fileCollection = project.layout.$methodName(project.tasks.producer)
+                def fileCollection = $expression(project.tasks.producer)
                 inputs.files fileCollection
                 doLast {
                     println("files = \${fileCollection.files}")
@@ -289,14 +291,15 @@ class ProjectLayoutIntegrationTest extends AbstractIntegrationSpec {
         """
 
         when:
+        maybeDeprecated(expression)
         run('consumer')
 
         then:
         executed(':producer', ':consumer')
-        outputContains("files = [${new File(testDirectory.absolutePath, '/build/resource/file.txt').absolutePath}]")
+        outputContains("files = [${testDirectory.file('/build/resource/file.txt').absolutePath}]")
 
         where:
-        methodName << ['files', 'configurableFiles']
+        expression << ['project.layout.files', 'project.layout.configurableFiles']
     }
 
     @Unroll
@@ -308,18 +311,19 @@ class ProjectLayoutIntegrationTest extends AbstractIntegrationSpec {
             }
 
             dependencies {
-                other files("${TextUtil.normaliseFileSeparators(testDirectory.absolutePath)}/src/resource/file.txt")
+                other files("src/resource/file.txt")
             }
 
             def fileCollection = $expression
             println("files = \${fileCollection.files}")
         """
 
         when:
+        maybeDeprecated(expression)
         run()
 
         then:
-        outputContains("files = [${new File(testDirectory.absolutePath, '/src/resource/file.txt').absolutePath}]")
+        outputContains("files = [${testDirectory.file('/src/resource/file.txt').absolutePath}]")
 
         where:
         collectionType               | expression
@@ -335,6 +339,7 @@ class ProjectLayoutIntegrationTest extends AbstractIntegrationSpec {
         """
 
         expect:
+        maybeDeprecated(expression)
         fails('help')
         errorOutput.contains('java.lang.NullPointerException')
 
@@ -344,4 +349,10 @@ class ProjectLayoutIntegrationTest extends AbstractIntegrationSpec {
         'FileCollection (File...)'   | 'project.layout.files((File) null)'
         'ConfigurableFileCollection' | 'project.layout.configurableFiles(null)'
     }
+
+    void maybeDeprecated(String expression) {
+        if (expression.contains("configurableFiles")) {
+            executer.expectDeprecationWarning()
+        }
+    }
 }

subprojects/core/src/main/java/org/gradle/api/internal/file/DefaultFileOperations.java

@@ -113,21 +113,11 @@ public URI uri(Object path) {
         return fileResolver.resolveUri(path);
     }
 
-    @Override
-    public ConfigurableFileCollection configurableFiles() {
-        return new DefaultConfigurableFileCollection(fileResolver, taskResolver);
-    }
-
     @Override
     public ConfigurableFileCollection configurableFiles(Object... paths) {
         return new DefaultConfigurableFileCollection(fileResolver, taskResolver, paths);
     }
 
-    @Override
-    public ConfigurableFileCollection files(Object... paths) {
-        return new DefaultConfigurableFileCollection(fileResolver, taskResolver, paths);
-    }
-
     @Override
     public FileCollection immutableFiles(Object... paths) {
         return ImmutableFileCollection.usingResolver(fileResolver, paths);

subprojects/core/src/main/java/org/gradle/api/internal/file/DefaultProjectLayout.java

@@ -113,6 +113,7 @@ public FileCollection files(Object... paths) {
 
     @Override
     public ConfigurableFileCollection configurableFiles(Object... files) {
+        DeprecationLogger.nagUserOfReplacedMethod("ProjectLayout.configurableFiles()", "ObjectFactory.fileCollection()");
         return new DefaultConfigurableFileCollection(fileResolver, taskResolver, files);
     }
 

subprojects/core/src/main/java/org/gradle/api/internal/file/FileOperations.java

@@ -41,20 +41,6 @@ public interface FileOperations {
 
     String relativePath(Object path);
 
-    /**
-     * @deprecated Use {@link #immutableFiles(Object...)} to create an immutable file collection,
-     * or {@link #configurableFiles(Object...)} or {@link #configurableFiles()}
-     * to create a mutable file collection.
-     */
-    // Used by Kotlin DSL 0.18.1
-    @Deprecated
-    ConfigurableFileCollection files(Object... paths);
-
-    /**
-     * Creates an empty mutable file collection.
-     */
-    ConfigurableFileCollection configurableFiles();
-
     /**
      * Creates a mutable file collection and initializes it with the given paths.
      */

subprojects/core/src/main/java/org/gradle/api/internal/project/DefaultProject.java

@@ -881,7 +881,7 @@ public URI uri(Object path) {
 
     @Override
     public ConfigurableFileCollection files(Object... paths) {
-        return getLayout().configurableFiles(paths);
+        return getObjects().fileCollection().from(paths);
     }
 
     @Override
@@ -1436,12 +1436,6 @@ private void assertMutatingMethodAllowed(String methodName) {
 
     // These are here just so that ProjectInternal can implement FileOperations to work around https://github.com/gradle/gradle/issues/6027
 
-    @Override
-    @Deprecated
-    public ConfigurableFileCollection configurableFiles() {
-        throw new UnsupportedOperationException();
-    }
-
     @Override
     @Deprecated
     public ConfigurableFileCollection configurableFiles(Object... paths) {