Skip to content

Commit 8f77210

Browse files
committed
#6626 #6625 minor CS fixes (removed useless assignments)
1 parent f736acc commit 8f77210

2 files changed

Lines changed: 13 additions & 8 deletions

File tree

lib/Doctrine/ORM/Proxy/ProxyFactory.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,9 @@ public function __construct(EntityManagerInterface $em, $proxyDir, $proxyNs, $au
9191
protected function skipClass(ClassMetadata $metadata)
9292
{
9393
/* @var $metadata \Doctrine\ORM\Mapping\ClassMetadataInfo */
94-
return $metadata->isMappedSuperclass || $metadata->isEmbeddedClass || $metadata->getReflectionClass()->isAbstract();
94+
return $metadata->isMappedSuperclass
95+
|| $metadata->isEmbeddedClass
96+
|| $metadata->getReflectionClass()->isAbstract();
9597
}
9698

9799
/**

tests/Doctrine/Tests/ORM/Proxy/ProxyFactoryTest.php

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -71,15 +71,16 @@ public function testReferenceProxyDelegatesLoadingToThePersister()
7171
$proxy->getDescription();
7272
}
7373

74-
7574
public function testSkipMappedSuperClassesOnGeneration(): void
7675
{
7776
$cm = new ClassMetadata(\stdClass::class);
7877
$cm->isMappedSuperclass = true;
7978

80-
$num = $this->proxyFactory->generateProxyClasses([$cm]);
81-
82-
self::assertSame(0, $num, 'No proxies generated.');
79+
self::assertSame(
80+
0,
81+
$this->proxyFactory->generateProxyClasses([$cm]),
82+
'No proxies generated.'
83+
);
8384
}
8485

8586
/**
@@ -90,9 +91,11 @@ public function testSkipEmbeddableClassesOnGeneration(): void
9091
$cm = new ClassMetadata(\stdClass::class);
9192
$cm->isEmbeddedClass = true;
9293

93-
$num = $this->proxyFactory->generateProxyClasses([$cm]);
94-
95-
self::assertSame(0, $num, 'No proxies generated.');
94+
self::assertSame(
95+
0,
96+
$this->proxyFactory->generateProxyClasses([$cm]),
97+
'No proxies generated.'
98+
);
9699
}
97100

98101
/**

0 commit comments

Comments
 (0)