Pouzor / mongoDBBundle

Repository/Repository.php

Unused Use Statements -2 removed lines

@@ -5,8 +5,6 @@
 
 use MongoDB\BulkWriteResult;
 use MongoDB\Collection;
-use MongoDB\Operation\InsertOne;
-use Psr\Log\LoggerInterface;
 use Psr\Log\NullLogger;
 use Pouzor\MongoDBBundle\Constants\DriverClasses;
 use Pouzor\MongoDBBundle\Constants\Query;

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

Services/ArrayAccessor.php

Spacing +5 added lines, -5 removed lines

@@ -56,7 +56,7 @@ 
         $keys = explode('.', $path);
         foreach ($keys as $k) {
             if (isset($data[$k])) {
-                $data =& $data[$k];
+                $data = & $data[$k];
             } else {
                 return $default;
             }
@@ -94,10 +94,10 @@ 
         $last = array_pop($keys);
         foreach ($keys as $k) {
             if (isset($data[$k]) && is_array($data[$k])) {
-                $data =& $data[$k];
+                $data = & $data[$k];
             } else {
                 $data[$k] = array();
-                $data =& $data[$k];
+                $data = & $data[$k];
             }
         }
         $data[$last] = $value;
@@ -132,7 +132,7 @@ 
         $last = array_pop($keys);
         foreach ($keys as $k) {
             if (isset($data[$k]) && is_array($data[$k])) {
-                $data =& $data[$k];
+                $data = & $data[$k];
             } else {
                 return null;
             }
@@ -179,7 +179,7 @@ 
         $last = array_pop($keys);
         foreach ($keys as $k) {
             if (isset($data[$k]) && is_array($data[$k])) {
-                $data =& $data[$k];
+                $data = & $data[$k];
             } else {
                 return;
             }

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;
 

Command/BuildIndexesCommand.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
 
         $manager = $this->getContainer()->get('document.manager');
 
-        $callback = function ($name) use ($style) {
+        $callback = function($name) use ($style) {
             $style->comment(sprintf('%s : Ok', $name));
         };
 

DependencyInjection/Configuration.php

Spacing +4 added lines, -4 removed lines

@@ -70,12 +70,12 @@ 
                                     ->beforeNormalization()
                                     // Handle readPreferenceTag XML nodes
                                     ->ifTrue(
-                                        function ($v) {
+                                        function($v) {
                                             return isset($v['readPreferenceTag']);
                                         }
                                     )
                                     ->then(
-                                        function ($v) {
+                                        function($v) {
                                             // Equivalent of fixXmlConfig() for inner node
                                             if (isset($v['readPreferenceTag']['name'])) {
                                                 $v['readPreferenceTag'] = array($v['readPreferenceTag']);
@@ -97,12 +97,12 @@ 
                     ->end()
                     ->validate()
                     ->ifTrue(
-                        function ($v) {
+                        function($v) {
                             return count($v['readPreferenceTags']) === 0;
                         }
                     )
                     ->then(
-                        function ($v) {
+                        function($v) {
                             unset($v['readPreferenceTags']);
 
                             return $v;

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']);
         }
     }
 }