Merge pull request #70 from creative-commoners/pulls/2.0/remove-todo

GuySartorelli · web-flow · commit 1ce60392d043 · 2023-10-31T10:10:05.000+13:00
FIX Method CopyMethod::copy as deprecated
diff --git a/src/Methods/CopyMethod.php b/src/Methods/CopyMethod.php
@@ -30,18 +30,18 @@ public function exposeDirectory($source, $target)
         $this->filesystem->removeDirectory($target);
 
         // Copy to destination
-        if (!$this->copy($source, $target)) {
+        if (!$this->filesystem->copy($source, $target)) {
             throw new RuntimeException("Could not write to directory $target");
         }
     }
 
     /**
      * Copies a file or directory from $source to $target.
      *
-     * @todo Replace with `$this->filesystem->copy() once composer 1.6.0 is released
      *
      * @param string $source
      * @param string $target
+     * @deprecated 5.2 Use Filesystem::copy instead
      * @return bool
      */
     public function copy($source, $target)

Original file line number	Diff line number	Diff line change
`@@ -30,18 +30,18 @@ public function exposeDirectory($source, $target)`
`30`	`30`	`$this->filesystem->removeDirectory($target);`
`31`	`31`
`32`	`32`	`// Copy to destination`
`33`		`- if (!$this->copy($source, $target)) {`
	`33`	`+ if (!$this->filesystem->copy($source, $target)) {`
`34`	`34`	`throw new RuntimeException("Could not write to directory $target");`
`35`	`35`	`}`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`/**`
`39`	`39`	`* Copies a file or directory from $source to $target.`
`40`	`40`	`*`
`41`		- * @todo Replace with `$this->filesystem->copy() once composer 1.6.0 is released
`42`	`41`	`*`
`43`	`42`	`* @param string $source`
`44`	`43`	`* @param string $target`
	`44`	`+ * @deprecated 5.2 Use Filesystem::copy instead`
`45`	`45`	`* @return bool`
`46`	`46`	`*/`
`47`	`47`	`public function copy($source, $target)`