Pouzor / mongoDBBundle

Repository/Repository.php

Doc Comments +3 added lines, -3 removed lines

@@ -52,7 +52,7 @@ 
     /**
      * Repository constructor.
      * @param $name
-     * @param $manager
+     * @param \Pouzor\MongoDBBundle\DocumentManager\DocumentManager $manager
      */
     public function __construct($name, $manager)
     {
@@ -545,7 +545,7 @@ 
     /**
      * Return the max value for a field in a collection
      *
-     * @param $field
+     * @param string $field
      * @param array $query
      * @param array $options
      * @return mixed
@@ -582,7 +582,7 @@ 
     /**
      * Return the min value for a field in a collection
      *
-     * @param $field
+     * @param string $field
      * @param array $query
      * @param array $options
      * @return \Traversable

Spacing +10 added lines, -10 removed lines

@@ -92,10 +92,10 @@ 
             if (preg_match($regex, $re->getMessage(), $output)) {
                 $name = $output['name'];
 
-                $this->logger->info('Dropping index ' . $name);
+                $this->logger->info('Dropping index '.$name);
                 $result = $this->collection->dropIndex($name);
 
-                $this->logger->info('Result from dropping index ' . $name, $result);
+                $this->logger->info('Result from dropping index '.$name, $result);
 
                 $this->ensureIndex($fields, $options, $callback);
             }
@@ -519,7 +519,7 @@ 
         $query = [$field => ['$gt' => $min, '$lte' => $max]];
 
         $this->logger->info(
-            'Counting  ' . $field,
+            'Counting  '.$field,
             [
                 'filter' => [
                     $field => ['$gt' => $min, '$lte' => $max]
@@ -575,7 +575,7 @@ 
             ]
         )->toArray();
 
-        return ArrayAccessor::dget($result, '0.' . $field);
+        return ArrayAccessor::dget($result, '0.'.$field);
 
     }
 
@@ -612,7 +612,7 @@ 
             ]
         )->toArray();
 
-        return ArrayAccessor::dget($result, '0.' . $field);
+        return ArrayAccessor::dget($result, '0.'.$field);
     }
 
     /**
@@ -625,7 +625,7 @@ 
     {
 
         $this->logger->info(
-            'Distinct over ' . $field,
+            'Distinct over '.$field,
             [
                 'filter' => $filter,
                 'options' => $options,
@@ -667,28 +667,28 @@ 
          * Sort
          */
         if (isset($options[Query::SORT]) && !is_array($options[Query::SORT])) {
-            throw new \InvalidArgumentException(Query::SORT . ' option must be an array');
+            throw new \InvalidArgumentException(Query::SORT.' option must be an array');
         }
 
         /**
          * Proj
          */
         if (isset($options[Query::PROJECTION]) && !is_array($options[Query::PROJECTION])) {
-            throw new \InvalidArgumentException(Query::PROJECTION . ' option must be an array');
+            throw new \InvalidArgumentException(Query::PROJECTION.' option must be an array');
         }
 
         /**
          *  limit
          */
         if (isset($options[Query::LIMIT]) && !is_integer($options[Query::LIMIT])) {
-            throw new \InvalidArgumentException(Query::LIMIT . " option must be an integer");
+            throw new \InvalidArgumentException(Query::LIMIT." option must be an integer");
         }
 
         /**
          *  offset
          */
         if (isset($options[Query::OFFSET]) && !is_integer($options[Query::OFFSET])) {
-            throw new \InvalidArgumentException(Query::OFFSET . ' option must be an array');
+            throw new \InvalidArgumentException(Query::OFFSET.' option must be an array');
         }
 
         // TODO validate other options

DependencyInjection/MongoDBExtension.php

Spacing +4 added lines, -4 removed lines

@@ -21,7 +21,7 @@ 
     // note that this method is called loadInternal and not load
     protected function loadInternal(array $mergedConfig, ContainerBuilder $container)
     {
-        $loader = new YamlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
+        $loader = new YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
         $loader->load('services.yml');
 
         foreach ($mergedConfig['connections'] as $name => $con) {
@@ -30,15 +30,15 @@ 
                 file_get_contents($con['schema'])
             );
 
-            $def = $container->register('document.manager.' . $name, DocumentManager::class);
+            $def = $container->register('document.manager.'.$name, DocumentManager::class);
             $def->addTag('document.manager', ['name' => $name]);
             $def->setArguments([$con, new Reference('logger')]);
             $def->addTag('monolog.logger', ['channel' => 'odm.mongo']);
         }
 
         if (isset($mergedConfig['default_connection']) && $container->hasDefinition(
-                'document.manager.' . $mergedConfig['default_connection'])) {
-            $container->setAlias('document.manager', 'document.manager.' . $mergedConfig['default_connection']);
+                'document.manager.'.$mergedConfig['default_connection'])) {
+            $container->setAlias('document.manager', 'document.manager.'.$mergedConfig['default_connection']);
         }
     }
 }

Services/ArrayAccessor.php

Braces +3 added lines, -2 removed lines

@@ -195,8 +195,9 @@
      */
     public static function get_key_exist($array, $key, $default = null) {
 
-        if (isset($array[$key]))
-            return $array[$key];
+        if (isset($array[$key])) {
+                    return $array[$key];
+        }
 
         return $default;