siriusphp / orm

src/Relation/OneToOne.php

Spacing +5 added lines, -5 removed lines

@@ -26,10 +26,10 @@ 
 
     protected function addActionOnDelete(BaseAction $action)
     {
-        $relations          = $action->getOption('relations');
+        $relations = $action->getOption('relations');
 
         // no cascade delete? treat it as a save
-        if (! $this->isCascade()) {
+        if (!$this->isCascade()) {
             $this->addActionOnSave($action);
         } elseif ($relations === true || in_array($this->name, (array)$relations)) {
             $nativeEntity       = $action->getEntity();
@@ -41,7 +41,7 @@ 
                                     ->first();
 
             if ($foreignEntity) {
-                $deleteAction       = $this->foreignMapper
+                $deleteAction = $this->foreignMapper
                     ->newDeleteAction($foreignEntity, ['relations' => $remainingRelations]);
                 $action->prepend($deleteAction);
                 $action->append($this->newSyncAction($action->getEntity(), $foreignEntity, 'delete'));
@@ -61,11 +61,11 @@ 
 
     protected function addActionOnSave(BaseAction $action)
     {
-        if (! $this->relationWasChanged($action->getEntity())) {
+        if (!$this->relationWasChanged($action->getEntity())) {
             return;
         }
 
-        if (! $action->includesRelation($this->name)) {
+        if (!$action->includesRelation($this->name)) {
             return;
         }
 

src/Relation/Relation.php

Spacing +5 added lines, -5 removed lines

@@ -76,7 +76,7 @@ 
 
     protected function setOptionIfMissing($name, $value)
     {
-        if (! isset($this->options[$name])) {
+        if (!isset($this->options[$name])) {
             $this->options[$name] = $value;
         }
     }
@@ -115,7 +115,7 @@ 
 
     protected function getKeyColumn($name, $column)
     {
-        if (! is_array($column)) {
+        if (!is_array($column)) {
             return $name . '_' . $column;
         }
 
@@ -176,7 +176,7 @@ 
 
         foreach ($entities as $entity) {
             $entityId = $this->getEntityId($this->foreignMapper, $entity, array_values($this->keyPairs));
-            if (! isset($result[$entityId])) {
+            if (!isset($result[$entityId])) {
                 $result[$entityId] = [];
             }
             $result[$entityId][] = $entity;
@@ -223,7 +223,7 @@ 
      */
     protected function getRemainingRelations($relations)
     {
-        if (! is_array($relations)) {
+        if (!is_array($relations)) {
             return $relations;
         }
 
@@ -244,7 +244,7 @@ 
         if (is_array($guards)) {
             foreach ($guards as $col => $val) {
                 // guards that are strings (eg: 'deleted_at is null') can't be used as extra columns
-                if (! is_int($col)) {
+                if (!is_int($col)) {
                     $cols[$col] = $val;
                 }
             }

src/Relation/HasAggregates.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 
         $aggregates     = [];
         $aggregatesList = $this->getOption(RelationConfig::AGGREGATES);
-        if (! is_array($aggregatesList) || empty($aggregatesList)) {
+        if (!is_array($aggregatesList) || empty($aggregatesList)) {
             $this->aggregates = $aggregates;
 
             return;

src/Blueprint/Orm.php

Spacing +8 added lines, -8 removed lines

@@ -39,27 +39,27 @@
     {
         $errors = [];
 
-        if (! $this->entityNamespace) {
+        if (!$this->entityNamespace) {
             $errors[] = 'Missing entity namespace property';
         }
 
-        if (! $this->entityDestination) {
+        if (!$this->entityDestination) {
             $errors[] = 'Missing entity destination property';
-        } elseif (! is_dir($this->entityDestination)) {
+        } elseif (!is_dir($this->entityDestination)) {
             $errors[] = sprintf('%s is not a valid directory', $this->entityDestination);
-        } elseif (! is_writable($this->entityDestination)) {
+        } elseif (!is_writable($this->entityDestination)) {
             $errors[] = sprintf('%s is not writable', $this->entityDestination);
         }
 
-        if (! $this->mapperNamespace) {
+        if (!$this->mapperNamespace) {
             $errors[] = 'Missing mapper namespace property';
         }
 
-        if (! $this->mapperDestination) {
+        if (!$this->mapperDestination) {
             $errors[] = 'Missing entity destination property';
-        } elseif (! is_dir($this->mapperDestination)) {
+        } elseif (!is_dir($this->mapperDestination)) {
             $errors[] = sprintf('%s is not a valid directory', $this->mapperDestination);
-        } elseif (! is_writable($this->mapperDestination)) {
+        } elseif (!is_writable($this->mapperDestination)) {
             $errors[] = sprintf('%s is not writable', $this->mapperDestination);
         }
 

src/Blueprint/Relation.php

Spacing +7 added lines, -7 removed lines

@@ -49,28 +49,28 @@ 
     {
         $errors = [];
 
-        if (! $this->name) {
+        if (!$this->name) {
             $errors[] = "Unknown relation name";
         }
 
-        if (! $this->type) {
+        if (!$this->type) {
             $errors[] = "Unknown relation type";
         }
 
-        if (! $this->nativeKey) {
+        if (!$this->nativeKey) {
             $errors[] = "Missing native key column";
         }
 
-        if (! $this->foreignMapper) {
+        if (!$this->foreignMapper) {
             $errors[] = "Missing foreign mapper name";
         }
 
-        if (! $this->foreignKey) {
+        if (!$this->foreignKey) {
             $errors[] = "Missing foreign key";
         }
 
         $strategies = [RelationConfig::LOAD_LAZY, RelationConfig::LOAD_EAGER, RelationConfig::LOAD_NONE];
-        if (! in_array($this->loadStrategy, $strategies)) {
+        if (!in_array($this->loadStrategy, $strategies)) {
             $errors[] = sprintf("Relation loading strategy is not valid (allowed values: %s)", implode(', ', $strategies));
         }
 
@@ -205,7 +205,7 @@ 
                 $value !== 0 &&
                 $value !== '' &&
                 $value !== [] &&
-                ! is_object($value)) {
+                !is_object($value)) {
                 $result[Str::underscore($prop)] = $value;
             }
             if (is_object($value) && is_callable($value)) {

src/Blueprint/ClassMethod.php

Spacing +2 added lines, -2 removed lines

@@ -32,11 +32,11 @@
     {
         $errors = [];
 
-        if (! $this->name) {
+        if (!$this->name) {
             $errors[] = 'Method requires a name';
         }
 
-        if (! in_array($this->visibility, [ClassType::VISIBILITY_PUBLIC, ClassType::VISIBILITY_PROTECTED])) {
+        if (!in_array($this->visibility, [ClassType::VISIBILITY_PUBLIC, ClassType::VISIBILITY_PROTECTED])) {
             $errors[] = 'Wrong method visilibity type. Only `public` and `protected` are allowed.';
         }
 

src/Blueprint/Relation/ManyToMany.php

Spacing +4 added lines, -4 removed lines

@@ -58,11 +58,11 @@ 
 
     protected function maybeSetAdditionalProperties()
     {
-        if (! $this->mapper || ! $this->foreignMapper) {
+        if (!$this->mapper || !$this->foreignMapper) {
             return;
         }
 
-        if (! $this->throughTable) {
+        if (!$this->throughTable) {
             $tablePrefix = $this->mapper->getTableAlias() ?
                 str_replace($this->mapper->getTable(), '', $this->mapper->getTableAlias())
                 : '';
@@ -77,11 +77,11 @@ 
             }
         }
 
-        if (! $this->throughNativeColumn) {
+        if (!$this->throughNativeColumn) {
             $this->throughNativeColumn = Inflector::singularize($this->mapper->getName()) . '_id';
         }
 
-        if (! $this->throughForeignColumn) {
+        if (!$this->throughForeignColumn) {
             $this->throughForeignColumn = Inflector::singularize($this->foreignMapper) . '_id';
         }
     }

src/Query.php

Spacing +7 added lines, -7 removed lines

@@ -138,7 +138,7 @@ 
             return $this;
         }
 
-        if (! $this->mapper->hasRelation($name)) {
+        if (!$this->mapper->hasRelation($name)) {
             throw new \InvalidArgumentException(
                 sprintf("Relation %s, not defined for %s", $name, $this->mapper->getConfig()->getTable())
             );
@@ -200,8 +200,8 @@ 
             return null;
         }
 
-        $receivedTracker = ! ! $tracker;
-        if (! $tracker) {
+        $receivedTracker = !!$tracker;
+        if (!$tracker) {
             $receivedTracker = false;
             $tracker         = new Tracker([$data]);
         }
@@ -211,7 +211,7 @@ 
         $this->injectAggregates($entity, $tracker, $load);
         $entity->setState(StateEnum::SYNCHRONIZED);
 
-        if (! $receivedTracker) {
+        if (!$receivedTracker) {
             $tracker->replaceRows([$entity]);
         }
 
@@ -262,7 +262,7 @@ 
             $queryCallback = $eagerLoad[$name] ?? null;
             $nextLoad      = Arr::getChildren($eagerLoad, $name);
 
-            if (! $tracker->hasRelation($name)) {
+            if (!$tracker->hasRelation($name)) {
                 $tracker->setRelation($name, $relation, $queryCallback, $nextLoad);
             }
 
@@ -278,7 +278,7 @@ 
     {
         foreach ($this->mapper->getRelations() as $name) {
             $relation = $this->mapper->getRelation($name);
-            if (! method_exists($relation, 'getAggregates')) {
+            if (!method_exists($relation, 'getAggregates')) {
                 continue;
             }
             $aggregates = $relation->getAggregates();
@@ -307,7 +307,7 @@ 
      */
     public function chunk(int $size, callable $callback, int $limit = 100000)
     {
-        if (! $this->orderBy->build()) {
+        if (!$this->orderBy->build()) {
             $this->orderBy(...(array)$this->mapper->getConfig()->getPrimaryKey());
         }
 

src/Relation/ManyToMany.php

Spacing +11 added lines, -11 removed lines

@@ -22,28 +22,28 @@ 
         parent::applyDefaults();
 
         $foreignKey = $this->foreignMapper->getConfig()->getPrimaryKey();
-        if (! isset($this->options[RelationConfig::FOREIGN_KEY])) {
+        if (!isset($this->options[RelationConfig::FOREIGN_KEY])) {
             $this->options[RelationConfig::FOREIGN_KEY] = $foreignKey;
         }
 
         $nativeKey = $this->foreignMapper->getConfig()->getPrimaryKey();
-        if (! isset($this->options[RelationConfig::NATIVE_KEY])) {
+        if (!isset($this->options[RelationConfig::NATIVE_KEY])) {
             $this->options[RelationConfig::NATIVE_KEY] = $nativeKey;
         }
 
-        if (! isset($this->options[RelationConfig::THROUGH_TABLE])) {
+        if (!isset($this->options[RelationConfig::THROUGH_TABLE])) {
             $tables = [$this->foreignMapper->getConfig()->getTable(), $this->nativeMapper->getConfig()->getTable()];
             sort($tables);
             $this->options[RelationConfig::THROUGH_TABLE] = implode('_', $tables);
         }
 
-        if (! isset($this->options[RelationConfig::THROUGH_NATIVE_COLUMN])) {
+        if (!isset($this->options[RelationConfig::THROUGH_NATIVE_COLUMN])) {
             $prefix = Inflector::singularize($this->nativeMapper->getConfig()->getTableAlias(true));
 
             $this->options[RelationConfig::THROUGH_NATIVE_COLUMN] = $this->getKeyColumn($prefix, $nativeKey);
         }
 
-        if (! isset($this->options[RelationConfig::THROUGH_FOREIGN_COLUMN])) {
+        if (!isset($this->options[RelationConfig::THROUGH_FOREIGN_COLUMN])) {
             $prefix = Inflector::singularize($this->foreignMapper->getConfig()->getTableAlias(true));
 
             $this->options[RelationConfig::THROUGH_FOREIGN_COLUMN] = $this->getKeyColumn($prefix, $foreignKey);
@@ -96,7 +96,7 @@ 
         $throughAlias = $this->getOption(RelationConfig::THROUGH_TABLE_ALIAS);
         $throughName  = $throughAlias ?? $through;
 
-        if (! empty($throughColumns)) {
+        if (!empty($throughColumns)) {
             foreach ($throughColumns as $col => $alias) {
                 $query->columns("{$throughName}.{$col} AS {$alias}");
             }
@@ -177,7 +177,7 @@ 
 
     protected function addActionOnDelete(BaseAction $action)
     {
-        $nativeEntity       = $action->getEntity();
+        $nativeEntity = $action->getEntity();
 
         // retrieve them again from the DB since the related collection might not have everything
         // for example due to a relation query callback
@@ -192,11 +192,11 @@ 
 
     protected function addActionOnSave(BaseAction $action)
     {
-        if (! $this->relationWasChanged($action->getEntity())) {
+        if (!$this->relationWasChanged($action->getEntity())) {
             return;
         }
 
-        if (! $action->includesRelation($this->name)) {
+        if (!$action->includesRelation($this->name)) {
             return;
         }
 
@@ -204,7 +204,7 @@ 
 
         /** @var Collection $foreignEntities */
         $foreignEntities = $this->nativeEntityHydrator->get($action->getEntity(), $this->name);
-        if (! $foreignEntities || !$foreignEntities instanceof Collection || $foreignEntities->isEmpty()) {
+        if (!$foreignEntities || !$foreignEntities instanceof Collection || $foreignEntities->isEmpty()) {
             return;
         }
 
@@ -212,7 +212,7 @@ 
 
         // save the entities still in the collection
         foreach ($foreignEntities as $foreignEntity) {
-            if (! empty($foreignEntity->getChanges())) {
+            if (!empty($foreignEntity->getChanges())) {
                 $saveAction = $this->foreignMapper
                     ->newSaveAction($foreignEntity, [
                         'relations' => $remainingRelations

Indentation +2 added lines, -2 removed lines

@@ -62,7 +62,7 @@ 
         $query = $this->foreignMapper->newQuery();
 
         $query = $this->joinWithThroughTable($query)
-                      ->where($this->options[RelationConfig::THROUGH_NATIVE_COLUMN], $nativePks);
+                        ->where($this->options[RelationConfig::THROUGH_NATIVE_COLUMN], $nativePks);
 
         $query = $this->applyQueryCallback($query);
 
@@ -116,7 +116,7 @@ 
     {
         $subselect = $this->foreignMapper->newQuery();
         $subselect = $query->subSelectForJoinWith($this->foreignMapper)
-                           ->as($reference);
+                            ->as($reference);
         #$subselect->resetGuards();
         #$subselect->setGuards($this->foreignMapper->getConfig()->getGuards());