Fixed wrong typehint

strictify · Aug 5, 2022 · ac8650c · ac8650c
1 parent d1265fa
commit ac8650c
Showing 1 changed file with 1 addition and 16 deletions.
diff --git a/src/Accessor/CollectionMapper.php b/src/Accessor/CollectionMapper.php
@@ -41,14 +41,10 @@ public function update(array $options, &$data, FormInterface $form): void
         }
 
         $removeEntry = $options['entry_options']['remove_entry'] ?? null;
-        $removalCallback = $this->extractRemoveEntryFromCollection($form);
         foreach ($toRemove as $item) {
             if ($removeEntry) {
                 $removeEntry($item);
             }
-            if ($removalCallback) {
-                $removalCallback($item);
-            }
             $this->submit($data, $item, $removerReflection);
         }
     }
@@ -113,7 +109,7 @@ private function getExtraValues(callable $compare, array $originalValues, array
         return $extraValues;
     }
 
-    private function search(callable $compare, int|string $key, object|array $value, object|array $originalValues): false|int|string
+    private function search(callable $compare, int|string $key, mixed $value, object|array $originalValues): false|int|string
     {
         foreach ($originalValues as $originalKey => $originalValue) {
             if ($value && $originalValue && $compare($value, $originalValue, $key, $originalKey) === true) {
@@ -123,15 +119,4 @@ private function search(callable $compare, int|string $key, object|array $value,
 
         return false;
     }
-
-    private function extractRemoveEntryFromCollection(FormInterface $form): ?Closure
-    {
-        foreach ($form as $child) {
-            $options = $child->getConfig()->getOptions();
-
-            return $options['remove_entry'] ?? null;
-        }
-
-        return null;
-    }
 }