Skip to content

Commit 92bfb9c

Browse files
committed
Codesniffer: fixes
1 parent af692b1 commit 92bfb9c

File tree

12 files changed

+18
-8
lines changed

12 files changed

+18
-8
lines changed

src/Analyser/Database/DatabaseAnalyser.php

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -70,11 +70,11 @@ protected function analyseColumns(Table $table): void
7070
$column->setNullable($col['nullable']);
7171
$column->setType(Helpers::columnType($col['nativetype']));
7272
$column->setDefault($col['default']);
73-
$column->setOnUpdate(Strings::contains(isset($col['vendor']['extra']) ? $col['vendor']['extra'] : $col['vendor']['Extra'], 'on update'));
73+
$column->setOnUpdate(Strings::contains($col['vendor']['extra'] ?? $col['vendor']['Extra'], 'on update'));
7474

7575
// Analyse ENUM
7676
if ($col['nativetype'] === ColumnTypes::NATIVE_TYPE_ENUM) {
77-
$enum = Strings::matchAll(isset($col['vendor']['type']) ? $col['vendor']['type'] : $col['vendor']['Type'], ColumnTypes::NATIVE_REGEX_ENUM, PREG_PATTERN_ORDER);
77+
$enum = Strings::matchAll($col['vendor']['type'] ?? $col['vendor']['Type'], ColumnTypes::NATIVE_REGEX_ENUM, PREG_PATTERN_ORDER);
7878
if ($enum) {
7979
$column->setEnum($enum[1]);
8080
$column->setType(ColumnTypes::TYPE_ENUM);
@@ -95,15 +95,13 @@ protected function analyseIndexes(Table $table): void
9595
$keys = $this->driver->getForeignKeys($tableName);
9696

9797
foreach ($indexes as $index) {
98-
9998
foreach ($index['columns'] as $col) {
10099
$column = $table->getColumn($col);
101100

102101
$column->setPrimary($index['primary']);
103102
$column->setUnique($index['unique']);
104103
$column->setIndex(true);
105104
}
106-
107105
}
108106

109107
foreach ($keys as $key) {

src/Entity/Table.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,7 @@ public function getPrimaryKeyColumns(): array
5959
$primary[] = $column;
6060
}
6161
}
62+
6263
return $primary;
6364
}
6465

src/Generator.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,15 +97,19 @@ public function generate(): void
9797
if ($this->config->get('generator.generate.entities')) {
9898
$this->entityGenerator->generate($database);
9999
}
100+
100101
if ($this->config->get('generator.generate.repositories')) {
101102
$this->repositoryGenerator->generate($database);
102103
}
104+
103105
if ($this->config->get('generator.generate.mappers')) {
104106
$this->mapperGenerator->generate($database);
105107
}
108+
106109
if ($this->config->get('generator.generate.facades')) {
107110
$this->facadeGenerator->generate($database);
108111
}
112+
109113
if ($this->config->get('generator.generate.model')) {
110114
$this->modelGenerator->generate($database);
111115
}

src/Generator/Entity/Decorator/ColumnConstantGenerator.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ public function doDecorate(Column $column, ClassType $class, PhpNamespace $names
2626
if (!$this->config->get('entity.generate.column.constant')) {
2727
return;
2828
}
29+
2930
$name = Strings::upper($this->config->get('entity.column.constants.prefix') . $column->getName());
3031
$class->addConstant($name, Helpers::camelCase($column->getName()));
3132
}

src/Generator/Entity/Decorator/ColumnDocumentor.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ public function doDecorate(Column $column, ClassType $class, PhpNamespace $names
4747
}
4848

4949
// Enum
50-
if (!empty($enum = $column->getEnum())) {
50+
if (!empty($column->getEnum())) {
5151
$doc->setEnum(Strings::upper($column->getName()));
5252
}
5353

src/Generator/Entity/EntityGenerator.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,6 @@ public function generate(Database $database): void
5151

5252
// Add table columns
5353
foreach ($table->getColumns() as $column) {
54-
5554
if ($this->config->get('generator.entity.exclude.primary')) {
5655
if ($column->isPrimary()) continue;
5756
}

src/Generator/Model/ModelGenerator.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ public function generate(Database $database): void
4747
$doc->setVariable(lcfirst($this->entityResolver->resolveEntityName($table)));
4848
$class->addComment((string) $doc);
4949
}
50+
5051
$this->generateFile($this->modelResolver->resolveModelFilename(), (string) $namespace);
5152
}
5253

src/Generator/Repository/RepositoryGenerator.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@ public function generate(Database $database): void
3939
$namespace->addUse($extends);
4040
$class->setExtends($extends);
4141
}
42+
4243
$namespace->addUse($this->entityResolver->resolveEntityNamespace($table) . '\\' . $this->entityResolver->resolveEntityName($table));
4344
$entityName = $this->entityResolver->resolveEntityName($table);
4445
$class->addMethod('getEntityClassNames')

src/Resolver/Impl/SimpleResolver.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ protected function table(Table $table, bool $singularize = false): string
4747
if ($singularize) {
4848
$name = Inflector::singularize($name);
4949
}
50+
5051
return $name;
5152
}
5253

src/Resolver/Impl/SimpleSeparateResolver.php

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,7 @@ protected function resolveFilenameFor(string $type, Table $table): string
7878
if ($this->config->get($type . '.name.singularize')) {
7979
$name = Inflector::singularize($name);
8080
}
81+
8182
$name .= $this->config->get($type . '.filename.suffix');
8283
return $this->config->get($type . '.folder') . DIRECTORY_SEPARATOR . $name . '.' . IFilenameResolver::PHP_EXT;
8384
}
@@ -88,8 +89,8 @@ protected function resolveNameFor(string $type, Table $table): string
8889
if ($this->config->get($type . '.name.singularize')) {
8990
$name = Inflector::singularize($name);
9091
}
91-
$name .= $this->config->get($type . '.name.suffix');
92-
return $name;
92+
93+
return $name . $this->config->get($type . '.name.suffix');
9394
}
9495

9596
}

0 commit comments

Comments
 (0)