as3io / modlr-persister-mongodb

src/Persister.php

Doc Comments +1 added lines, -1 removed lines

@@ -445,8 +445,8 @@
     /**
      * Gets standard database retrieval criteria for an inverse relationship.
      *
-     * @param   EntityMetadata  $metadata       The entity to retrieve database records for.
      * @param   string|array    $identifiers    The IDs to query.
+     * @param string $inverseField
      * @return  array
      */
     protected function getInverseCriteria(EntityMetadata $owner, EntityMetadata $related, $identifiers, $inverseField)

Spacing +61 added lines, -61 removed lines

@@ -69,7 +69,7 @@ 
      * {@inheritDoc}
      * @todo    Implement sorting and pagination (limit/skip).
      */
-    public function all(EntityMetadata $metadata, Store $store, array $identifiers = [])
+    public function all(EntityMetadata $metadata, Store $store, array $identifiers = [ ])
     {
         $criteria = $this->getRetrieveCritiera($metadata, $identifiers);
         $cursor = $this->findFromDatabase($metadata, $criteria);
@@ -106,32 +106,32 @@ 
     public function create(Model $model)
     {
         $metadata = $model->getMetadata();
-        $insert[$this->getIdentifierKey()] = $this->convertId($model->getId());
+        $insert[ $this->getIdentifierKey() ] = $this->convertId($model->getId());
         if (true === $metadata->isChildEntity()) {
-            $insert[$this->getPolymorphicKey()] = $metadata->type;
+            $insert[ $this->getPolymorphicKey() ] = $metadata->type;
         }
 
         $changeset = $model->getChangeSet();
-        foreach ($changeset['attributes'] as $key => $values) {
-            $value = $this->prepareAttribute($metadata->getAttribute($key), $values['new']);
+        foreach ($changeset[ 'attributes' ] as $key => $values) {
+            $value = $this->prepareAttribute($metadata->getAttribute($key), $values[ 'new' ]);
             if (null === $value) {
                 continue;
             }
-            $insert[$key] = $value;
+            $insert[ $key ] = $value;
         }
-        foreach ($changeset['hasOne'] as $key => $values) {
-            $value = $this->prepareHasOne($metadata->getRelationship($key), $values['new']);
+        foreach ($changeset[ 'hasOne' ] as $key => $values) {
+            $value = $this->prepareHasOne($metadata->getRelationship($key), $values[ 'new' ]);
             if (null === $value) {
                 continue;
             }
-            $insert[$key] = $value;
+            $insert[ $key ] = $value;
         }
-        foreach ($changeset['hasMany'] as $key => $values) {
-            $value = $this->prepareHasMany($metadata->getRelationship($key), $values['new']);
+        foreach ($changeset[ 'hasMany' ] as $key => $values) {
+            $value = $this->prepareHasMany($metadata->getRelationship($key), $values[ 'new' ]);
             if (null === $value) {
                 continue;
             }
-            $insert[$key] = $value;
+            $insert[ $key ] = $value;
         }
         $this->createQueryBuilder($metadata)
             ->insert()
@@ -185,9 +185,9 @@ 
         if (null === $models || true === $relMeta->isInverse) {
             return null;
         }
-        $references = [];
+        $references = [ ];
         foreach ($models as $model) {
-            $references[] = $this->createReference($relMeta, $model);
+            $references[ ] = $this->createReference($relMeta, $model);
         }
         return empty($references) ? null : $references;
     }
@@ -202,8 +202,8 @@ 
     protected function createReference(RelationshipMetadata $relMeta, Model $model)
     {
         if (true === $relMeta->isPolymorphic()) {
-            $reference[$this->getIdentifierKey()] = $this->convertId($model->getId());
-            $reference[$this->getPolymorphicKey()] = $model->getType();
+            $reference[ $this->getIdentifierKey() ] = $this->convertId($model->getId());
+            $reference[ $this->getPolymorphicKey() ] = $model->getType();
             return $reference;
         }
         return $this->convertId($model->getId());
@@ -219,39 +219,39 @@ 
         $criteria = $this->getRetrieveCritiera($metadata, $model->getId());
         $changeset = $model->getChangeSet();
 
-        $update = [];
-        foreach ($changeset['attributes'] as $key => $values) {
-            if (null === $values['new']) {
+        $update = [ ];
+        foreach ($changeset[ 'attributes' ] as $key => $values) {
+            if (null === $values[ 'new' ]) {
                 $op = '$unset';
                 $value = 1;
             } else {
                 $op = '$set';
-                $value = $this->prepareAttribute($metadata->getAttribute($key), $values['new']);
+                $value = $this->prepareAttribute($metadata->getAttribute($key), $values[ 'new' ]);
             }
-            $update[$op][$key] = $value;
+            $update[ $op ][ $key ] = $value;
         }
 
         // @todo Must prevent inverse relationships from persisting
-        foreach ($changeset['hasOne'] as $key => $values) {
-            if (null === $values['new']) {
+        foreach ($changeset[ 'hasOne' ] as $key => $values) {
+            if (null === $values[ 'new' ]) {
                 $op = '$unset';
                 $value = 1;
             } else {
                 $op = '$set';
-                $value = $this->prepareHasOne($metadata->getRelationship($key), $values['new']);
+                $value = $this->prepareHasOne($metadata->getRelationship($key), $values[ 'new' ]);
             }
-            $update[$op][$key] = $value;
+            $update[ $op ][ $key ] = $value;
         }
 
-        foreach ($changeset['hasMany'] as $key => $values) {
-            if (null === $values['new']) {
+        foreach ($changeset[ 'hasMany' ] as $key => $values) {
+            if (null === $values[ 'new' ]) {
                 $op = '$unset';
                 $value = 1;
             } else {
                 $op = '$set';
-                $value = $this->prepareHasMany($metadata->getRelationship($key), $values['new']);
+                $value = $this->prepareHasMany($metadata->getRelationship($key), $values[ 'new' ]);
             }
-            $update[$op][$key] = $value;
+            $update[ $op ][ $key ] = $value;
         }
 
         if (empty($update)) {
@@ -334,10 +334,10 @@ 
         if (false === $metadata->isPolymorphic()) {
             return $metadata->type;
         }
-        if (!isset($data[$this->getPolymorphicKey()])) {
+        if (!isset($data[ $this->getPolymorphicKey() ])) {
             throw PersisterException::badRequest(sprintf('Unable to extract polymorphic type. The "%s" key was not found.', $this->getPolymorphicKey()));
         }
-        return $data[$this->getPolymorphicKey()];
+        return $data[ $this->getPolymorphicKey() ];
     }
 
     /**
@@ -367,9 +367,9 @@ 
      */
     protected function hydrateRecords(EntityMetadata $metadata, array $results, Store $store)
     {
-        $records = [];
+        $records = [ ];
         foreach ($results as $data) {
-            $records[] = $this->hydrateRecord($metadata, $data, $store);
+            $records[ ] = $this->hydrateRecord($metadata, $data, $store);
         }
         return $records;
     }
@@ -384,27 +384,27 @@ 
      */
     protected function hydrateRecord(EntityMetadata $metadata, array $data, Store $store)
     {
-        $identifier = $data[$this->getIdentifierKey()];
-        unset($data[$this->getIdentifierKey()]);
+        $identifier = $data[ $this->getIdentifierKey() ];
+        unset($data[ $this->getIdentifierKey() ]);
 
         $type = $this->extractType($metadata, $data);
-        unset($data[$this->getPolymorphicKey()]);
+        unset($data[ $this->getPolymorphicKey() ]);
 
         $metadata = $store->getMetadataForType($type);
         foreach ($metadata->getRelationships() as $key => $relMeta) {
-            if (!isset($data[$key])) {
+            if (!isset($data[ $key ])) {
                 continue;
             }
-            if (true === $relMeta->isMany() && !is_array($data[$key])) {
-                throw PersisterException::badRequest(sprintf('Relationship key "%s" is a reference many. Expected record data type of array, "%s" found on model "%s" for identifier "%s"', $key, gettype($data[$key]), $type, $identifier));
+            if (true === $relMeta->isMany() && !is_array($data[ $key ])) {
+                throw PersisterException::badRequest(sprintf('Relationship key "%s" is a reference many. Expected record data type of array, "%s" found on model "%s" for identifier "%s"', $key, gettype($data[ $key ]), $type, $identifier));
             }
-            $references = $relMeta->isOne() ? [$data[$key]] : $data[$key];
+            $references = $relMeta->isOne() ? [ $data[ $key ] ] : $data[ $key ];
 
-            $extracted = [];
+            $extracted = [ ];
             foreach ($references as $reference) {
-                $extracted[] =  $this->extractRelationship($relMeta, $reference);
+                $extracted[ ] = $this->extractRelationship($relMeta, $reference);
             }
-            $data[$key] = $relMeta->isOne() ? reset($extracted) : $extracted;
+            $data[ $key ] = $relMeta->isOne() ? reset($extracted) : $extracted;
         }
         return new Record($type, $identifier, $data);
     }
@@ -422,9 +422,9 @@ 
         $simple = false === $relMeta->isPolymorphic();
         $idKey = $this->getIdentifierKey();
         $typeKey = $this->getPolymorphicKey();
-        if (true === $simple && is_array($reference) && isset($reference[$idKey])) {
+        if (true === $simple && is_array($reference) && isset($reference[ $idKey ])) {
             return [
-                'id'    => $reference[$idKey],
+                'id'    => $reference[ $idKey ],
                 'type'  => $relMeta->getEntityType(),
             ];
         } elseif (true === $simple && !is_array($reference)) {
@@ -432,10 +432,10 @@ 
                 'id'    => $reference,
                 'type'  => $relMeta->getEntityType(),
             ];
-        } elseif (false === $simple && is_array($reference) && isset($reference[$idKey]) && isset($reference[$typeKey])) {
+        } elseif (false === $simple && is_array($reference) && isset($reference[ $idKey ]) && isset($reference[ $typeKey ])) {
             return [
-                'id'    => $reference[$idKey],
-                'type'  => $reference[$typeKey],
+                'id'    => $reference[ $idKey ],
+                'type'  => $reference[ $typeKey ],
             ];
         } else {
             throw new RuntimeException('Unable to extract a reference id.');
@@ -451,17 +451,17 @@ 
      */
     protected function getInverseCriteria(EntityMetadata $owner, EntityMetadata $related, $identifiers, $inverseField)
     {
-        $criteria[$inverseField] = $this->getIdentifierCriteria($identifiers);
+        $criteria[ $inverseField ] = $this->getIdentifierCriteria($identifiers);
         if (true === $owner->isChildEntity()) {
             // The owner is owned by a polymorphic model. Must include the type with the inverse field criteria.
-            $criteria[$inverseField] = [
-                $this->getIdentifierKey()   => $criteria[$inverseField],
+            $criteria[ $inverseField ] = [
+                $this->getIdentifierKey()   => $criteria[ $inverseField ],
                 $this->getPolymorphicKey()  => $owner->type,
             ];
         }
         if (true === $related->isChildEntity()) {
             // The relationship is owned by a polymorphic model. Must include the type in the root criteria.
-            $criteria[$this->getPolymorphicKey()] = $related->type;
+            $criteria[ $this->getPolymorphicKey() ] = $related->type;
         }
         return $criteria;
     }
@@ -476,12 +476,12 @@ 
     protected function getRetrieveCritiera(EntityMetadata $metadata, $identifiers)
     {
         $idKey = $this->getIdentifierKey();
-        $criteria[$idKey] = $this->getIdentifierCriteria($identifiers);
-        if (empty($criteria[$idKey])) {
-            unset($criteria[$idKey]);
+        $criteria[ $idKey ] = $this->getIdentifierCriteria($identifiers);
+        if (empty($criteria[ $idKey ])) {
+            unset($criteria[ $idKey ]);
         }
         if (true === $metadata->isChildEntity()) {
-            $criteria[$this->getPolymorphicKey()] = $metadata->type;
+            $criteria[ $this->getPolymorphicKey() ] = $metadata->type;
         }
         return $criteria;
     }
@@ -494,16 +494,16 @@ 
      */
     protected function getIdentifierCriteria($identifiers)
     {
-        $criteria = [];
+        $criteria = [ ];
         if (is_array($identifiers)) {
-            $ids = [];
+            $ids = [ ];
             foreach ($identifiers as $id) {
-                $ids[] = $this->convertId($id);
+                $ids[ ] = $this->convertId($id);
             }
             if (1 === count($ids)) {
-                $criteria = $ids[0];
+                $criteria = $ids[ 0 ];
             } elseif (!empty($ids)) {
-                $criteria = ['$in' => $ids];
+                $criteria = [ '$in' => $ids ];
             }
         } else {
             $criteria = $this->convertId($identifiers);

src/StorageMetadataFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
     /**
      * Constructor.
      *
-     * @param   EntityUtility   $entityUtl
+     * @param   EntityUtility   $entityUtil
      */
     public function __construct(EntityUtility $entityUtil)
     {

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
     {
         $persistence = $metadata->persistence;
 
-        $validIdStrategies = ['object'];
+        $validIdStrategies = [ 'object' ];
         if (!in_array($persistence->idStrategy, $validIdStrategies)) {
             throw MetadataException::invalidMetadata($metadata->type, sprintf('The persistence id strategy "%s" is invalid. Valid types are "%s"', $persistence->idStrategy, implode('", "', $validIdStrategies)));
         }