Internal: moving most of method to a private method, no changes

Author: weaverryan

src/Configurator/AddLinesConfigurator.php

@@ -35,30 +35,11 @@ public function configure(Recipe $recipe, $config, Lock $lock, array $options =
 
     public function unconfigure(Recipe $recipe, $config, Lock $lock): void
     {
-        foreach ($config as $patch) {
-            if (!isset($patch['file'])) {
-                $this->write(sprintf('The "file" key is required for the "add-lines" configurator for recipe "%s". Skipping', $recipe->getName()));
-
-                continue;
-            }
-
-            // Ignore "requires": the target packages may have just become uninstalled.
-            // Checking for a "content" match is enough.
-
-            $file = $this->path->concatenate([$this->options->get('root-dir'), $patch['file']]);
-            if (!is_file($file)) {
-                continue;
-            }
+        $changes = $this->getUnconfigureFileChanges($recipe, $config);
 
-            if (!isset($patch['content'])) {
-                $this->write(sprintf('The "content" key is required for the "add-lines" configurator for recipe "%s". Skipping', $recipe->getName()));
-
-                continue;
-            }
-            $value = $patch['content'];
-
-            $newContents = $this->getUnPatchedContents($file, $value);
-            file_put_contents($file, $newContents);
+        foreach ($changes as $file => $change) {
+            $this->write(sprintf('[add-lines] Reverting file "%s"', $file));
+            file_put_contents($file, $change);
         }
     }
 
@@ -158,6 +139,38 @@ public function getConfigureFileChanges(Recipe $recipe, $config): array
         return $changes;
     }
 
+    public function getUnconfigureFileChanges(Recipe $recipe, $config): array
+    {
+        $changes = [];
+        foreach ($config as $patch) {
+            if (!isset($patch['file'])) {
+                $this->write(sprintf('The "file" key is required for the "add-lines" configurator for recipe "%s". Skipping', $recipe->getName()));
+
+                continue;
+            }
+
+            // Ignore "requires": the target packages may have just become uninstalled.
+            // Checking for a "content" match is enough.
+
+            $file = $this->path->concatenate([$this->options->get('root-dir'), $patch['file']]);
+            if (!is_file($file)) {
+                continue;
+            }
+
+            if (!isset($patch['content'])) {
+                $this->write(sprintf('The "content" key is required for the "add-lines" configurator for recipe "%s". Skipping', $recipe->getName()));
+
+                continue;
+            }
+            $value = $patch['content'];
+
+            $newContents = $this->getUnPatchedContents($file, $value);
+            $changes[$file] = $newContents;
+        }
+
+        return $changes;
+    }
+
     private function getPatchedContents(string $file, string $value, string $position, ?string $target, bool $warnIfMissing): string
     {
         $fileContents = file_get_contents($file);