Removed Translate strategy and accompanying test.

Renamed Copy* strategy invoke parameters data -> record.
ScriptFUSION · Jun 12, 2017 · fb31788 · fb31788
1 parent b0a28a0
commit fb31788
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 84 deletions.
diff --git a/src/Strategy/Copy.php b/src/Strategy/Copy.php
@@ -11,21 +11,21 @@ class Copy extends Delegate
     const PATH_SEPARATOR = '->';
 
     /**
-     * @param mixed $data
+     * @param mixed $record
      * @param mixed $context
      *
      * @return mixed
      */
-    public function __invoke($data, $context = null)
+    public function __invoke($record, $context = null)
     {
-        if (!is_array($data)) {
+        if (!is_array($record)) {
             return null;
         }
 
-        if (!is_array($path = parent::__invoke($data, $context))) {
+        if (!is_array($path = parent::__invoke($record, $context))) {
             $path = explode(self::PATH_SEPARATOR, $path);
         }
 
-        return $path ? ArrayWalker::walk($data, $path) : null;
+        return $path ? ArrayWalker::walk($record, $path) : null;
     }
 }
diff --git a/src/Strategy/CopyContext.php b/src/Strategy/CopyContext.php
@@ -23,7 +23,7 @@ public function __construct($path = null)
         $this->walk = (bool)$path;
     }
 
-    public function __invoke($data, $context = null)
+    public function __invoke($record, $context = null)
     {
         if ($this->walk) {
             return parent::__invoke($context);

diff --git a/src/Strategy/CopyKey.php b/src/Strategy/CopyKey.php
@@ -11,7 +11,7 @@ class CopyKey implements Strategy, KeyAware
 {
     use KeyAwareTrait;
 
-    public function __invoke($data, $context = null)
+    public function __invoke($record, $context = null)
     {
         return $this->key;
     }

diff --git a/src/Strategy/Translate.php b/src/Strategy/Translate.php
diff --git a/test/Functional/DocumentationTest.php b/test/Functional/DocumentationTest.php
@@ -312,20 +312,6 @@ public function testToList()
         );
     }
 
-    public function testTranslate()
-    {
-        self::assertSame(
-            'bar',
-            (new Mapper)->map(
-                ['foo' => 'foo'],
-                new Translate(
-                    new Copy('foo'),
-                    ['foo' => 'bar']
-                )
-            )
-        );
-    }
-
     public function testTryCatch()
     {
         self::assertSame(

diff --git a/test/Integration/Mapper/Strategy/TranslateTest.php b/test/Integration/Mapper/Strategy/TranslateTest.php