Skip to content

Commit 171675c

Browse files
joelwurtzjoelwurtz
committed
feat(identifier): use a hash system to avoid double loop
1 parent d33b20f commit 171675c

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

47 files changed

+346
-360
lines changed

‎src/Attribute/MapFrom.php‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
* @param string[]|null $groups The groups to map the property
2121
* @param string|null $dateTimeFormat The date-time format to use when transforming this property
2222
* @param bool|null $extractTypesFromGetter If true, the types will be extracted from the getter method
23+
* @param bool|null $identifier If true, the property will be used as an identifier
2324
*/
2425
public function __construct(
2526
public string|array|null $source = null,
@@ -32,6 +33,7 @@ public function __construct(
3233
public int $priority = 0,
3334
public ?string $dateTimeFormat = null,
3435
public ?bool $extractTypesFromGetter = null,
36+
public ?bool $identifier = null,
3537
) {
3638
}
3739
}

‎src/Attribute/MapIdentifier.php‎

Lines changed: 0 additions & 10 deletions
This file was deleted.

‎src/Attribute/MapTo.php‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
* @param string[]|null $groups The groups to map the property
2121
* @param string|null $dateTimeFormat The date-time format to use when transforming this property
2222
* @param bool|null $extractTypesFromGetter If true, the types will be extracted from the getter method
23+
* @param bool|null $identifier If true, the property will be used as an identifier
2324
*/
2425
public function __construct(
2526
public string|array|null $target = null,
@@ -32,6 +33,7 @@ public function __construct(
3233
public int $priority = 0,
3334
public ?string $dateTimeFormat = null,
3435
public ?bool $extractTypesFromGetter = null,
36+
public ?bool $identifier = null,
3537
) {
3638
}
3739
}

‎src/EventListener/MapFromListener.php‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,7 @@ private function addPropertyFromTarget(GenerateMapperEvent $event, MapFrom $mapF
8484
groups: $mapFrom->groups,
8585
priority: $mapFrom->priority,
8686
extractTypesFromGetter: $mapFrom->extractTypesFromGetter,
87+
identifier: $mapFrom->identifier,
8788
);
8889

8990
if (\array_key_exists($propertyMetadata->target->property, $event->properties) && $event->properties[$propertyMetadata->target->property]->priority >= $propertyMetadata->priority) {

‎src/EventListener/MapToListener.php‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,7 @@ private function addPropertyFromSource(GenerateMapperEvent $event, MapTo $mapTo,
8585
groups: $mapTo->groups,
8686
priority: $mapTo->priority,
8787
extractTypesFromGetter: $mapTo->extractTypesFromGetter,
88+
identifier: $mapTo->identifier,
8889
);
8990

9091
if (\array_key_exists($propertyMetadata->target->property, $event->properties) && $event->properties[$propertyMetadata->target->property]->priority >= $propertyMetadata->priority) {

‎src/Extractor/MappingExtractor.php‎

Lines changed: 0 additions & 70 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,8 @@
44

55
namespace AutoMapper\Extractor;
66

7-
use AutoMapper\Attribute\MapIdentifier;
87
use AutoMapper\Configuration;
98
use AutoMapper\Event\PropertyMetadataEvent;
10-
use AutoMapper\Metadata\TargetPropertyMetadata;
119
use Symfony\Component\PropertyInfo\PropertyInfoExtractorInterface;
1210
use Symfony\Component\PropertyInfo\PropertyReadInfo;
1311
use Symfony\Component\PropertyInfo\PropertyReadInfoExtractorInterface;
@@ -189,72 +187,4 @@ public function getDateTimeFormat(PropertyMetadataEvent $propertyMetadataEvent):
189187

190188
return $this->configuration->dateTimeFormat;
191189
}
192-
193-
public function isIdentifier(?\ReflectionClass $targetReflectionClass, TargetPropertyMetadata $target): bool
194-
{
195-
if ($targetReflectionClass === null) {
196-
return false;
197-
}
198-
199-
// check on reflection property
200-
if ($targetReflectionClass->hasProperty($target->property)) {
201-
$reflectionProperty = $targetReflectionClass->getProperty($target->property);
202-
203-
if (\count($reflectionProperty->getAttributes(MapIdentifier::class)) > 0) {
204-
return true;
205-
}
206-
}
207-
208-
// check on the getter to read, it may not be defined yet so we cannot skip this everytime
209-
if ($target->readAccessor !== null) {
210-
// check with read accessor on property
211-
if ($target->readAccessor->type === ReadAccessor::TYPE_PROPERTY && $targetReflectionClass->hasProperty(
212-
$target->readAccessor->property
213-
)) {
214-
$reflectionProperty = $targetReflectionClass->getProperty(
215-
$target->readAccessor->property
216-
);
217-
218-
if (\count($reflectionProperty->getAttributes(MapIdentifier::class)) > 0) {
219-
return true;
220-
}
221-
}
222-
223-
// check with read accessor on method
224-
if ($target->readAccessor->type === ReadAccessor::TYPE_METHOD && $targetReflectionClass->hasMethod(
225-
$target->readAccessor->property
226-
)) {
227-
$reflectionMethod = $targetReflectionClass->getMethod(
228-
$target->readAccessor->property
229-
);
230-
231-
if (\count($reflectionMethod->getAttributes(MapIdentifier::class)) > 0) {
232-
return true;
233-
}
234-
}
235-
}
236-
237-
// same for the write mutator, we cannot globally skip this check as it may be defined latter
238-
if ($target->writeMutator !== null) {
239-
// check for property
240-
if ($target->writeMutator->type === WriteMutator::TYPE_PROPERTY && $targetReflectionClass->hasProperty($target->writeMutator->property)) {
241-
$reflectionProperty = $targetReflectionClass->getProperty(
242-
$target->writeMutator->property
243-
);
244-
245-
if (\count($reflectionProperty->getAttributes(MapIdentifier::class)) > 0) {
246-
return true;
247-
}
248-
}
249-
250-
// check for parameter in constructor
251-
if ($target->writeMutator->type === WriteMutator::TYPE_CONSTRUCTOR && $target->writeMutator->parameter !== null) {
252-
if (\count($target->writeMutator->parameter->getAttributes(MapIdentifier::class)) > 0) {
253-
return true;
254-
}
255-
}
256-
}
257-
258-
return false;
259-
}
260190
}

‎src/Extractor/MappingExtractorInterface.php‎

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -49,9 +49,4 @@ public function getReadAccessor(string $class, string $property): ?ReadAccessor;
4949
* @param array<string, mixed> $context
5050
*/
5151
public function getWriteMutator(string $source, string $target, string $property, array $context = []): ?WriteMutator;
52-
53-
/**
54-
* Check if a property is an identifier.
55-
*/
56-
public function isIdentifier(?\ReflectionClass $targetReflectionClass, TargetPropertyMetadata $target): bool;
5752
}

‎src/Extractor/ReadAccessor.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ public function __construct(
6161
*
6262
* @throws CompileException
6363
*/
64-
public function getExpression(Expr\Variable $input, bool $target = false): Expr
64+
public function getExpression(Expr $input, bool $target = false): Expr
6565
{
6666
if (self::TYPE_METHOD === $this->type) {
6767
$methodCallArguments = [];

‎src/GeneratedMapper.php‎

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,14 @@ public function registerMappers(AutoMapperRegistryInterface $registry): void
3636
{
3737
}
3838

39-
public function areIdentifiersEquals(mixed $source, mixed $target): bool
39+
public function getSourceHash(mixed $value): ?string
4040
{
41-
return false;
41+
return null;
42+
}
43+
44+
public function getTargetHash(mixed $value): ?string
45+
{
46+
return null;
4247
}
4348

4449
/** @var array<string, MapperInterface<object, object>|MapperInterface<object, array<mixed>>|MapperInterface<array<mixed>, object>> */

‎src/Generator/IdentifierEqualGenerator.php‎

Lines changed: 0 additions & 83 deletions
This file was deleted.

0 commit comments

Comments
 (0)