Skip to content

Commit 8fcf66e

Browse files
dunglasdunglas
committed
Fix CS
1 parent f90aae8 commit 8fcf66e

File tree

5 files changed

+15
-20
lines changed

5 files changed

+15
-20
lines changed

‎src/AnnotationGenerator/AbstractAnnotationGenerator.php‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,6 @@ protected function toPhpType(array $field, bool $adderOrRemover = false): string
157157
$data = '\\'.\DateTimeInterface::class;
158158
break;
159159
case 'Number':
160-
// No break
161160
case 'Float':
162161
$data = 'float';
163162
break;

‎src/AnnotationGenerator/ApiPlatformCoreAnnotationGenerator.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ public function generateUses(string $className): array
5252
$resource = $this->classes[$className]['resource'];
5353

5454
$subClassOf = $resource->get('rdfs:subClassOf');
55-
$typeIsEnum = $subClassOf && $subClassOf->getUri() === TypesGenerator::SCHEMA_ORG_ENUMERATION;
55+
$typeIsEnum = $subClassOf && TypesGenerator::SCHEMA_ORG_ENUMERATION === $subClassOf->getUri();
5656

5757
return $typeIsEnum ? [] : [ApiResource::class, ApiProperty::class];
5858
}

‎src/AnnotationGenerator/DoctrineMongoDBAnnotationGenerator.php‎

Lines changed: 8 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -63,23 +63,20 @@ public function generateFieldAnnotations(string $className, string $fieldName):
6363
$type = 'boolean';
6464
break;
6565
case 'Date':
66-
// No break
6766
case 'DateTime':
6867
$type = 'date';
6968
break;
7069
case 'Time':
7170
$type = 'time';
7271
break;
7372
case 'Number':
74-
// No break
7573
case 'Float':
7674
$type = 'float';
7775
break;
7876
case 'Integer':
7977
$type = 'integer';
8078
break;
8179
case 'Text':
82-
// No break
8380
case 'URL':
8481
$type = 'string';
8582
break;
@@ -97,14 +94,14 @@ public function generateFieldAnnotations(string $className, string $fieldName):
9794

9895
$annotations[] = $annotation;
9996
} else {
100-
if ($field['cardinality'] === CardinalitiesExtractor::CARDINALITY_0_1
101-
|| $field['cardinality'] === CardinalitiesExtractor::CARDINALITY_1_1
102-
|| $field['cardinality'] === CardinalitiesExtractor::CARDINALITY_N_0
103-
|| $field['cardinality'] === CardinalitiesExtractor::CARDINALITY_N_1) {
97+
if (CardinalitiesExtractor::CARDINALITY_0_1 === $field['cardinality']
98+
|| CardinalitiesExtractor::CARDINALITY_1_1 === $field['cardinality']
99+
|| CardinalitiesExtractor::CARDINALITY_N_0 === $field['cardinality']
100+
|| CardinalitiesExtractor::CARDINALITY_N_1 === $field['cardinality']) {
104101
$annotations[] = sprintf('@MongoDB\ReferenceOne(targetDocument="%s", simple=true))', $this->getRelationName($field['range']));
105-
} elseif ($field['cardinality'] === CardinalitiesExtractor::CARDINALITY_0_N
106-
|| $field['cardinality'] === CardinalitiesExtractor::CARDINALITY_1_N
107-
|| $field['cardinality'] === CardinalitiesExtractor::CARDINALITY_N_N) {
102+
} elseif (CardinalitiesExtractor::CARDINALITY_0_N === $field['cardinality']
103+
|| CardinalitiesExtractor::CARDINALITY_1_N === $field['cardinality']
104+
|| CardinalitiesExtractor::CARDINALITY_N_N === $field['cardinality']) {
108105
$annotations[] = sprintf('@MongoDB\ReferenceMany(targetDocument="%s", simple=true)', $this->getRelationName($field['range']));
109106
}
110107
}
@@ -120,7 +117,7 @@ public function generateUses(string $className): array
120117
$resource = $this->classes[$className]['resource'];
121118

122119
$subClassOf = $resource->get('rdfs:subClassOf');
123-
$typeIsEnum = $subClassOf && $subClassOf->getUri() === TypesGenerator::SCHEMA_ORG_ENUMERATION;
120+
$typeIsEnum = $subClassOf && TypesGenerator::SCHEMA_ORG_ENUMERATION === $subClassOf->getUri();
124121

125122
return $typeIsEnum ? [] : ['Doctrine\ODM\MongoDB\Mapping\Annotations as MongoDB'];
126123
}

‎src/AnnotationGenerator/DoctrineOrmAnnotationGenerator.php‎

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ public function generateFieldAnnotations(string $className, string $fieldName):
105105
if ($field['ormColumn']) {
106106
$annotation .= sprintf('(%s)', $field['ormColumn']);
107107
} else {
108-
if ($type !== 'string' || $field['isNullable'] || $field['isUnique']) {
108+
if ('string' !== $type || $field['isNullable'] || $field['isUnique']) {
109109
$isColumnHasProperties = true;
110110
}
111111

@@ -117,11 +117,11 @@ public function generateFieldAnnotations(string $className, string $fieldName):
117117
$annotation .= '(';
118118
}
119119

120-
if ($type !== 'string') {
120+
if ('string' !== $type) {
121121
$annotation .= sprintf('type="%s"', $type);
122122
}
123123

124-
if ($type !== 'string' && $field['isNullable']) {
124+
if ('string' !== $type && $field['isNullable']) {
125125
$annotation .= ', ';
126126
}
127127

@@ -156,7 +156,6 @@ public function generateFieldAnnotations(string $className, string $fieldName):
156156
$annotations[] = '@ORM\JoinColumn(nullable=false)';
157157
break;
158158
case CardinalitiesExtractor::CARDINALITY_UNKNOWN:
159-
// No break
160159
case CardinalitiesExtractor::CARDINALITY_N_0:
161160
$annotations[] = sprintf('@ORM\ManyToOne(targetEntity="%s")', $this->getRelationName($field['range']));
162161
break;
@@ -194,7 +193,7 @@ public function generateUses(string $className): array
194193
$resource = $this->classes[$className]['resource'];
195194

196195
$subClassOf = $resource->get('rdfs:subClassOf');
197-
$typeIsEnum = $subClassOf && $subClassOf->getUri() === TypesGenerator::SCHEMA_ORG_ENUMERATION;
196+
$typeIsEnum = $subClassOf && TypesGenerator::SCHEMA_ORG_ENUMERATION === $subClassOf->getUri();
198197

199198
return $typeIsEnum ? [] : ['Doctrine\ORM\Mapping as ORM'];
200199
}

‎src/TypesGenerator.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -447,7 +447,7 @@ private function isEnum(\EasyRdf_Resource $type): bool
447447
{
448448
$subClassOf = $type->get('rdfs:subClassOf');
449449

450-
return $subClassOf && $subClassOf->getUri() === self::SCHEMA_ORG_ENUMERATION;
450+
return $subClassOf && self::SCHEMA_ORG_ENUMERATION === $subClassOf->getUri();
451451
}
452452

453453
/**
@@ -621,7 +621,7 @@ private function generateField(array $config, array $class, \EasyRdf_Resource $t
621621
}
622622

623623
$cardinality = $propertyConfig['cardinality'] ?? false;
624-
if (!$cardinality || $cardinality === CardinalitiesExtractor::CARDINALITY_UNKNOWN) {
624+
if (!$cardinality || CardinalitiesExtractor::CARDINALITY_UNKNOWN === $cardinality) {
625625
$cardinality = $property ? $this->cardinalities[$propertyName] : CardinalitiesExtractor::CARDINALITY_1_1;
626626
}
627627

0 commit comments

Comments
 (0)