siriusphp / orm

src/CastingManager.php

Spacing +2 added lines, -2 removed lines

@@ -9,7 +9,7 @@ 
 
     public function register($name, callable $func)
     {
-        if (! $name) {
+        if (!$name) {
             return; // ignore
         }
         $this->casts[$name] = $func;
@@ -36,7 +36,7 @@ 
 
     public function bool($value)
     {
-        return ! ! $value;
+        return !!$value;
     }
 
     public function int($value)

src/Entity/GenericEntity.php

Spacing +4 added lines, -4 removed lines

@@ -98,7 +98,7 @@ 
         }
 
         $value = $this->castAttribute($attribute, $value);
-        if (! isset($this->attributes[$attribute]) || $value != $this->attributes[$attribute]) {
+        if (!isset($this->attributes[$attribute]) || $value != $this->attributes[$attribute]) {
             $this->changed[$attribute] = true;
             $this->state               = StateEnum::CHANGED;
         }
@@ -109,7 +109,7 @@ 
 
     public function get($attribute)
     {
-        if (! $attribute) {
+        if (!$attribute) {
             return null;
         }
 
@@ -120,7 +120,7 @@ 
 
     public function getPersistenceState()
     {
-        if (! empty($this->changed)) {
+        if (!empty($this->changed)) {
         }
 
         return $this->state;
@@ -150,7 +150,7 @@ 
         $changes = $this->changed;
         foreach ($this->attributes as $name => $value) {
             if (is_object($value) && method_exists($value, 'getChanges')) {
-                if (! empty($value->getChanges())) {
+                if (!empty($value->getChanges())) {
                     $changes[$name] = true;
                 }
             }

src/Relation/ManyToMany.php

Indentation +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
             ->newQuery();
 
         $query = $this->joinWithThroughTable($query)
-                      ->where($this->options[RelationConfig::THROUGH_NATIVE_COLUMN], $nativePks);
+                        ->where($this->options[RelationConfig::THROUGH_NATIVE_COLUMN], $nativePks);
 
         if ($this->getOption(RelationConfig::QUERY_CALLBACK) &&
             is_callable($this->getOption(RelationConfig::QUERY_CALLBACK))) {

Spacing +9 added lines, -9 removed lines

@@ -19,28 +19,28 @@ 
         $this->setOptionIfMissing(RelationConfig::THROUGH_COLUMNS_PREFIX, 'pivot_');
 
         $foreignKey = $this->foreignMapper->getPrimaryKey();
-        if (! isset($this->options[RelationConfig::FOREIGN_KEY])) {
+        if (!isset($this->options[RelationConfig::FOREIGN_KEY])) {
             $this->options[RelationConfig::FOREIGN_KEY] = $foreignKey;
         }
 
         $nativeKey = $this->foreignMapper->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->getTable(), $this->nativeMapper->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->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->getTableAlias(true));
 
             $this->options[RelationConfig::THROUGH_FOREIGN_COLUMN] = $this->getKeyColumn($prefix, $foreignKey);
@@ -100,7 +100,7 @@ 
         $throughAlias = $this->getOption(RelationConfig::THROUGH_TABLE_ALIAS);
         $throughName  = $throughAlias ?? $through;
 
-        if (! empty($throughColumns)) {
+        if (!empty($throughColumns)) {
             $prefix = $this->getOption(RelationConfig::THROUGH_COLUMNS_PREFIX);
             foreach ($throughColumns as $col) {
                 $query->columns("{$throughName}.{$col} AS {$prefix}{$col}");
@@ -186,7 +186,7 @@ 
         $remainingRelations = $this->getRemainingRelations($action->getOption('relations'));
 
         // no cascade delete? treat as save so we can process the changes
-        if (! $this->isCascade()) {
+        if (!$this->isCascade()) {
             $this->addActionOnSave($action);
         } else {
             // retrieve them again from the DB since the related collection might not have everything
@@ -208,7 +208,7 @@ 
 
         /** @var Collection $foreignEntities */
         $foreignEntities = $this->nativeMapper->getEntityAttribute($action->getEntity(), $this->name);
-        if (! $foreignEntities) {
+        if (!$foreignEntities) {
             return;
         }
 
@@ -216,7 +216,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

src/Relation/ManyToOne.php

Spacing +3 added lines, -3 removed lines

@@ -14,11 +14,11 @@ 
         parent::applyDefaults();
 
         $foreignKey = $this->foreignMapper->getPrimaryKey();
-        if (! isset($this->options[RelationConfig::FOREIGN_KEY])) {
+        if (!isset($this->options[RelationConfig::FOREIGN_KEY])) {
             $this->options[RelationConfig::FOREIGN_KEY] = $foreignKey;
         }
 
-        if (! isset($this->options[RelationConfig::NATIVE_KEY])) {
+        if (!isset($this->options[RelationConfig::NATIVE_KEY])) {
             $nativeKey                                 = $this->getKeyColumn($this->name, $foreignKey);
             $this->options[RelationConfig::NATIVE_KEY] = $nativeKey;
         }
@@ -90,7 +90,7 @@ 
     protected function addActionOnDelete(BaseAction $action)
     {
         // no cascade delete? treat it as a save
-        if (! $this->isCascade()) {
+        if (!$this->isCascade()) {
             $this->addActionOnSave($action);
         } else {
             $nativeEntity  = $action->getEntity();

src/Relation/OneToMany.php

Spacing +5 added lines, -5 removed lines

@@ -14,11 +14,11 @@ 
     protected function applyDefaults(): void
     {
         $nativeKey = $this->nativeMapper->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::FOREIGN_KEY])) {
+        if (!isset($this->options[RelationConfig::FOREIGN_KEY])) {
             $prefix                                     = Inflector::singularize($this->nativeMapper->getTable());
             $this->options[RelationConfig::FOREIGN_KEY] = $this->getKeyColumn($prefix, $nativeKey);
         }
@@ -69,7 +69,7 @@ 
     public function attachEntities(EntityInterface $nativeEntity, EntityInterface $foreignEntity)
     {
         foreach ($this->keyPairs as $nativeCol => $foreignCol) {
-            $nativeKeyValue  = $this->nativeMapper->getEntityAttribute($nativeEntity, $nativeCol);
+            $nativeKeyValue = $this->nativeMapper->getEntityAttribute($nativeEntity, $nativeCol);
             $this->foreignMapper->setEntityAttribute($foreignEntity, $foreignCol, $nativeKeyValue);
         }
     }
@@ -91,7 +91,7 @@ 
         $remainingRelations = $this->getRemainingRelations($action->getOption('relations'));
 
         // no cascade delete? treat as save so we can process the changes
-        if (! $this->isCascade()) {
+        if (!$this->isCascade()) {
             $this->addActionOnSave($action);
         } else {
             // retrieve them again from the DB since the related collection might not have everything
@@ -123,7 +123,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]);
                 $saveAction->addColumns($this->getExtraColumnsForAction());

src/Action/Update.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
 
     public function revert()
     {
-        if (! $this->hasRun) {
+        if (!$this->hasRun) {
             return;
         }
         $this->entity->setPersistenceState($this->entityState);

src/Action/AttachEntities.php

Spacing +4 added lines, -4 removed lines

@@ -51,10 +51,10 @@
         $conn = $this->relation->getNativeMapper()->getWriteConnection();
         $throughTable = (string)$this->relation->getOption(RelationConfig::THROUGH_TABLE);
 
-        $throughNativeColumns = (array) $this->relation->getOption(RelationConfig::THROUGH_NATIVE_COLUMN);
-        $throughForeignColumns = (array) $this->relation->getOption(RelationConfig::THROUGH_FOREIGN_COLUMN);
-        $nativeKey = (array) $this->nativeEntity->getPk();
-        $foreignKey = (array) $this->foreignEntity->getPk();
+        $throughNativeColumns = (array)$this->relation->getOption(RelationConfig::THROUGH_NATIVE_COLUMN);
+        $throughForeignColumns = (array)$this->relation->getOption(RelationConfig::THROUGH_FOREIGN_COLUMN);
+        $nativeKey = (array)$this->nativeEntity->getPk();
+        $foreignKey = (array)$this->foreignEntity->getPk();
 
         $delete = new \Sirius\Sql\Delete($conn);
         $delete->from($throughTable);

src/Orm.php

Spacing +5 added lines, -5 removed lines

@@ -29,7 +29,7 @@ 
     public function __construct(ConnectionLocator $connectionLocator, CastingManager $castingManager = null)
     {
         $this->connectionLocator = $connectionLocator;
-        if (! $castingManager) {
+        if (!$castingManager) {
             $castingManager = new CastingManager();
         }
         $this->castingManager = $castingManager;
@@ -63,7 +63,7 @@ 
             unset($this->lazyMappers[$mapperName]);
         }
 
-        if (! isset($this->mappers[$mapperName]) || ! $this->mappers[$mapperName]) {
+        if (!isset($this->mappers[$mapperName]) || !$this->mappers[$mapperName]) {
             throw new InvalidArgumentException(sprintf('Mapper named %s is not registered', $mapperName));
         }
 
@@ -94,14 +94,14 @@ 
     {
         $foreignMapper = $options[RelationConfig::FOREIGN_MAPPER];
         if ($this->has($foreignMapper)) {
-            if (! $foreignMapper instanceof Mapper) {
+            if (!$foreignMapper instanceof Mapper) {
                 $foreignMapper = $this->get($foreignMapper);
             }
         }
         $type          = $options[RelationConfig::TYPE];
         $relationClass = __NAMESPACE__ . '\\Relation\\' . Str::className($type);
 
-        if (! class_exists($relationClass)) {
+        if (!class_exists($relationClass)) {
             throw new InvalidArgumentException("{$relationClass} does not exist");
         }
 
@@ -115,7 +115,7 @@ 
         }
 
         $mapper = $mapperConfigOrFactory($this);
-        if (! $mapper instanceof Mapper) {
+        if (!$mapper instanceof Mapper) {
             throw new InvalidArgumentException(
                 'The mapper generated from the factory is not a valid `Mapper` instance'
             );

src/Mapper.php

Spacing +14 added lines, -14 removed lines

@@ -137,7 +137,7 @@ 
             $mapper->entityClass = $mapperConfig->entityClass;
         }
 
-        if ($mapperConfig->behaviours && ! empty($mapperConfig->behaviours)) {
+        if ($mapperConfig->behaviours && !empty($mapperConfig->behaviours)) {
             $mapper->use(...$mapperConfig->behaviours);
         }
 
@@ -147,10 +147,10 @@ 
     public function __construct(Orm $orm, HydratorInterface $entityHydrator = null, QueryBuilder $queryBuilder = null)
     {
         $this->orm = $orm;
-        if (! $entityHydrator) {
+        if (!$entityHydrator) {
             $entityHydrator = new GenericEntityHydrator($orm, $this);
         }
-        if (! $queryBuilder) {
+        if (!$queryBuilder) {
             $this->queryBuilder = QueryBuilder::getInstance();
         }
         $this->entityHydrator = $entityHydrator;
@@ -221,7 +221,7 @@ 
         $mapper = $this;
 
         $singular = Inflector::singularize($this->getTableAlias(true));
-        $castingManager->register($singular, function ($value) use ($mapper, $castingManager) {
+        $castingManager->register($singular, function($value) use ($mapper, $castingManager) {
             if ($value instanceof $this->entityClass) {
                 return $value;
             }
@@ -230,7 +230,7 @@ 
         });
 
         $plural = $this->getTableAlias(true);
-        $castingManager->register($plural, function ($values) use ($mapper, $castingManager) {
+        $castingManager->register($plural, function($values) use ($mapper, $castingManager) {
             if ($values instanceof Collection) {
                 return $values;
             }
@@ -264,7 +264,7 @@ 
      */
     public function getTableAlias($returnTableIfNull = false)
     {
-        return (! $this->tableAlias && $returnTableIfNull) ? $this->table : $this->tableAlias;
+        return (!$this->tableAlias && $returnTableIfNull) ? $this->table : $this->tableAlias;
     }
 
     public function getTableReference()
@@ -329,8 +329,8 @@ 
             return null;
         }
 
-        $receivedTracker = ! ! $tracker;
-        if (! $tracker) {
+        $receivedTracker = !!$tracker;
+        if (!$tracker) {
             $receivedTracker = false;
             $tracker         = new Tracker($this, [$data]);
         }
@@ -339,7 +339,7 @@ 
         $this->injectRelations($entity, $tracker, $load);
         $entity->setPersistenceState(StateEnum::SYNCHRONIZED);
 
-        if (! $receivedTracker) {
+        if (!$receivedTracker) {
             $tracker->replaceRows([$entity]);
             if ($tracker->isDisposable()) {
                 unset($tracker);
@@ -394,7 +394,7 @@ 
             $queryCallback = $eagerLoad[$name] ?? null;
             $nextLoad      = Arr::getChildren($eagerLoad, $name);
 
-            if (! $tracker->hasRelation($name)) {
+            if (!$tracker->hasRelation($name)) {
                 $tracker->setRelation($name, $relation, $queryCallback);
             }
 
@@ -431,7 +431,7 @@ 
 
     public function getRelation($name): Relation
     {
-        if (! $this->hasRelation($name)) {
+        if (!$this->hasRelation($name)) {
             throw new \InvalidArgumentException("Relation named {$name} is not registered for this mapper");
         }
 
@@ -439,7 +439,7 @@ 
             $this->relations[$name] = $this->orm->createRelation($this, $name, $this->relations[$name]);
         }
         $relation = $this->relations[$name];
-        if (! $relation instanceof Relation) {
+        if (!$relation instanceof Relation) {
             throw new \InvalidArgumentException("Relation named {$name} is not a proper Relation instance");
         }
 
@@ -492,7 +492,7 @@ 
 
     public function newSaveAction(EntityInterface $entity, $options): BaseAction
     {
-        if (! $entity->getPk()) {
+        if (!$entity->getPk()) {
             $action = new Insert($this, $entity, $options);
         } else {
             $action = new Update($this, $entity, $options);
@@ -529,7 +529,7 @@ 
 
     protected function assertCanPersistEntity($entity)
     {
-        if (! $entity || ! $entity instanceof $this->entityClass) {
+        if (!$entity || !$entity instanceof $this->entityClass) {
             throw new \InvalidArgumentException(sprintf(
                 'Mapper %s can only persist entity of class %s. %s class provided',
                 __CLASS__,