@@ -24,7 +24,7 @@ |
||
24 | 24 | * @param string|null $message |
25 | 25 | * @param Exception|null $previous |
26 | 26 | */ |
27 | - public function __construct( EntityId $entityId, $message = null, ?Exception $previous = null ) { |
|
27 | + public function __construct(EntityId $entityId, $message = null, ?Exception $previous = null) { |
|
28 | 28 | $this->entityId = $entityId; |
29 | 29 | |
30 | 30 | parent::__construct( |
@@ -24,7 +24,7 @@ |
||
24 | 24 | * @param string|null $message |
25 | 25 | * @param Exception|null $previous |
26 | 26 | */ |
27 | - public function __construct( EntityId $entityId, $message = null, ?Exception $previous = null ) { |
|
27 | + public function __construct(EntityId $entityId, $message = null, ?Exception $previous = null) { |
|
28 | 28 | $this->entityId = $entityId; |
29 | 29 | |
30 | 30 | parent::__construct( |
@@ -22,7 +22,7 @@ discard block |
||
22 | 22 | private $entityLookup; |
23 | 23 | private array $propertyIdsInProcess = []; |
24 | 24 | |
25 | - public function __construct( EntityLookup $entityLookup ) { |
|
25 | + public function __construct(EntityLookup $entityLookup) { |
|
26 | 26 | $this->entityLookup = $entityLookup; |
27 | 27 | } |
28 | 28 | |
@@ -34,16 +34,16 @@ discard block |
||
34 | 34 | * @return string |
35 | 35 | * @throws PropertyDataTypeLookupException |
36 | 36 | */ |
37 | - public function getDataTypeIdForProperty( PropertyId $propertyId ) { |
|
38 | - if ( array_key_exists( $propertyId->getSerialization(), $this->propertyIdsInProcess ) ) { |
|
37 | + public function getDataTypeIdForProperty(PropertyId $propertyId) { |
|
38 | + if (array_key_exists($propertyId->getSerialization(), $this->propertyIdsInProcess)) { |
|
39 | 39 | // avoid self-referencing loop for newly created properties (T374230) |
40 | - throw new PropertyDataTypeLookupException( $propertyId, 'loop detected' ); |
|
40 | + throw new PropertyDataTypeLookupException($propertyId, 'loop detected'); |
|
41 | 41 | } |
42 | - $this->propertyIdsInProcess[ $propertyId->getSerialization() ] = true; |
|
42 | + $this->propertyIdsInProcess[$propertyId->getSerialization()] = true; |
|
43 | 43 | try { |
44 | - return $this->getProperty( $propertyId )->getDataTypeId(); |
|
44 | + return $this->getProperty($propertyId)->getDataTypeId(); |
|
45 | 45 | } finally { |
46 | - unset( $this->propertyIdsInProcess[ $propertyId->getSerialization() ] ); |
|
46 | + unset($this->propertyIdsInProcess[$propertyId->getSerialization()]); |
|
47 | 47 | } |
48 | 48 | } |
49 | 49 | |
@@ -53,11 +53,11 @@ discard block |
||
53 | 53 | * @return Property |
54 | 54 | * @throws PropertyDataTypeLookupException |
55 | 55 | */ |
56 | - private function getProperty( PropertyId $propertyId ) { |
|
57 | - $property = $this->entityLookup->getEntity( $propertyId ); |
|
56 | + private function getProperty(PropertyId $propertyId) { |
|
57 | + $property = $this->entityLookup->getEntity($propertyId); |
|
58 | 58 | |
59 | - if ( !( $property instanceof Property ) ) { |
|
60 | - throw new PropertyDataTypeLookupException( $propertyId ); |
|
59 | + if (!($property instanceof Property)) { |
|
60 | + throw new PropertyDataTypeLookupException($propertyId); |
|
61 | 61 | } |
62 | 62 | |
63 | 63 | return $property; |