Skip to content

Commit 85f7831

Browse files
phansysfranmomu
authored andcommitted
Remove unused variables
1 parent 23316d6 commit 85f7831

37 files changed

+45
-76
lines changed

src/Loggable/Document/Repository/LogEntryRepository.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -163,8 +163,8 @@ protected function fillDocument($document, array $data)
163163
private function getLoggableListener(): LoggableListener
164164
{
165165
if (null === $this->listener) {
166-
foreach ($this->dm->getEventManager()->getAllListeners() as $event => $listeners) {
167-
foreach ($listeners as $hash => $listener) {
166+
foreach ($this->dm->getEventManager()->getAllListeners() as $listeners) {
167+
foreach ($listeners as $listener) {
168168
if ($listener instanceof LoggableListener) {
169169
$this->listener = $listener;
170170

src/Loggable/Entity/Repository/LogEntryRepository.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -180,8 +180,8 @@ protected function mapValue(ClassMetadata $objectMeta, $field, &$value)
180180
private function getLoggableListener(): LoggableListener
181181
{
182182
if (null === $this->listener) {
183-
foreach ($this->_em->getEventManager()->getAllListeners() as $event => $listeners) {
184-
foreach ($listeners as $hash => $listener) {
183+
foreach ($this->_em->getEventManager()->getAllListeners() as $listeners) {
184+
foreach ($listeners as $listener) {
185185
if ($listener instanceof LoggableListener) {
186186
$this->listener = $listener;
187187

src/Sortable/Entity/Repository/SortableRepository.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,8 +45,8 @@ public function __construct(EntityManagerInterface $em, ClassMetadata $class)
4545
{
4646
parent::__construct($em, $class);
4747
$sortableListener = null;
48-
foreach ($em->getEventManager()->getAllListeners() as $event => $listeners) {
49-
foreach ($listeners as $hash => $listener) {
48+
foreach ($em->getEventManager()->getAllListeners() as $listeners) {
49+
foreach ($listeners as $listener) {
5050
if ($listener instanceof SortableListener) {
5151
$sortableListener = $listener;
5252

src/Sortable/SortableListener.php

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -325,8 +325,6 @@ public function postFlush(EventArgs $args)
325325
*/
326326
protected function processInsert(SortableAdapter $ea, array $config, $meta, $object)
327327
{
328-
$em = $ea->getObjectManager();
329-
330328
$old = $meta->getReflectionProperty($config['position'])->getValue($object);
331329
$newPosition = $meta->getReflectionProperty($config['position'])->getValue($object);
332330

src/Translatable/Document/Repository/TranslationRepository.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -230,8 +230,8 @@ public function findTranslationsByObjectId($id)
230230
private function getTranslatableListener(): TranslatableListener
231231
{
232232
if (null === $this->listener) {
233-
foreach ($this->dm->getEventManager()->getAllListeners() as $event => $listeners) {
234-
foreach ($listeners as $hash => $listener) {
233+
foreach ($this->dm->getEventManager()->getAllListeners() as $listeners) {
234+
foreach ($listeners as $listener) {
235235
if ($listener instanceof TranslatableListener) {
236236
return $this->listener = $listener;
237237
}

src/Translatable/Entity/Repository/TranslationRepository.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -237,8 +237,8 @@ public function findTranslationsByObjectId($id)
237237
private function getTranslatableListener(): TranslatableListener
238238
{
239239
if (null === $this->listener) {
240-
foreach ($this->_em->getEventManager()->getAllListeners() as $event => $listeners) {
241-
foreach ($listeners as $hash => $listener) {
240+
foreach ($this->_em->getEventManager()->getAllListeners() as $listeners) {
241+
foreach ($listeners as $listener) {
242242
if ($listener instanceof TranslatableListener) {
243243
return $this->listener = $listener;
244244
}

src/Translatable/Hydrator/ORM/ObjectHydrator.php

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -65,21 +65,14 @@ protected function cleanup()
6565
*/
6666
protected function getTranslatableListener()
6767
{
68-
$translatableListener = null;
69-
foreach ($this->_em->getEventManager()->getAllListeners() as $event => $listeners) {
70-
foreach ($listeners as $hash => $listener) {
68+
foreach ($this->_em->getEventManager()->getAllListeners() as $listeners) {
69+
foreach ($listeners as $listener) {
7170
if ($listener instanceof TranslatableListener) {
72-
$translatableListener = $listener;
73-
74-
break 2;
71+
return $listener;
7572
}
7673
}
7774
}
7875

79-
if (null === $translatableListener) {
80-
throw new RuntimeException('The translation listener could not be found');
81-
}
82-
83-
return $translatableListener;
76+
throw new RuntimeException('The translation listener could not be found');
8477
}
8578
}

src/Translatable/Hydrator/ORM/SimpleObjectHydrator.php

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -65,21 +65,14 @@ protected function cleanup()
6565
*/
6666
protected function getTranslatableListener()
6767
{
68-
$translatableListener = null;
69-
foreach ($this->_em->getEventManager()->getAllListeners() as $event => $listeners) {
70-
foreach ($listeners as $hash => $listener) {
68+
foreach ($this->_em->getEventManager()->getAllListeners() as $listeners) {
69+
foreach ($listeners as $listener) {
7170
if ($listener instanceof TranslatableListener) {
72-
$translatableListener = $listener;
73-
74-
break 2;
71+
return $listener;
7572
}
7673
}
7774
}
7875

79-
if (null === $translatableListener) {
80-
throw new RuntimeException('The translation listener could not be found');
81-
}
82-
83-
return $translatableListener;
76+
throw new RuntimeException('The translation listener could not be found');
8477
}
8578
}

src/Translatable/Query/TreeWalker/TranslationWalker.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -420,8 +420,8 @@ private function extractTranslatedComponents(array $queryComponents): void
420420
private function getTranslatableListener(): TranslatableListener
421421
{
422422
$em = $this->getEntityManager();
423-
foreach ($em->getEventManager()->getAllListeners() as $event => $listeners) {
424-
foreach ($listeners as $hash => $listener) {
423+
foreach ($em->getEventManager()->getAllListeners() as $listeners) {
424+
foreach ($listeners as $listener) {
425425
if ($listener instanceof TranslatableListener) {
426426
return $listener;
427427
}

src/Translatable/TranslatableListener.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -362,7 +362,7 @@ public function getTranslatableLocale($object, $meta, $om = null)
362362
$locale = $value;
363363
}
364364
} elseif ($om instanceof DocumentManager) {
365-
[$mapping, $parentObject] = $om->getUnitOfWork()->getParentAssociation($object);
365+
[ , $parentObject] = $om->getUnitOfWork()->getParentAssociation($object);
366366
if (null !== $parentObject) {
367367
$parentMeta = $om->getClassMetadata(get_class($parentObject));
368368
$locale = $this->getTranslatableLocale($parentObject, $parentMeta, $om);

0 commit comments

Comments
 (0)