FGM / mongodb

modules/mongodb/mongodb_drush.php

Doc Comments +1 added lines

@@ -115,6 +115,7 @@
 
 /**
  * Returns the basic shell command string.
+ * @param string $alias
  */
 function _drush_mongodb_connect($alias) {
   $connections = variable_get('mongodb_connections', array());

Spacing +2 added lines, -2 removed lines

@@ -90,9 +90,9 @@
   $db = $connection['db'];
 
   $query = $host;
-  $query .= '/' . $db;
+  $query .= '/'.$db;
 
-  $command = 'mongo ' . $query;
+  $command = 'mongo '.$query;
   return $command;
 }
 

modules/mongodb/mongodb_module.php

Doc Comments +1 added lines

@@ -319,6 +319,7 @@
 
 /**
  * Return the next id in a sequence.
+ * @param string $name
  */
 function mongodb_next_id($name, $existing_id = 0) {
   // Atomitcaly get the next id in the sequence.

Spacing +2 added lines, -2 removed lines

@@ -13,7 +13,7 @@ 
 function mongodb_help($route_name, RouteMatchInterface $route_match) {
   switch ($route_name) {
     case 'help.page.mongodb':
-      return '<p>' . t('The Drupal <a href="!project">MongoDB</a> project implements a generic interface to the <a href="!mongo">MongoDB</a> database server.', [
+      return '<p>'.t('The Drupal <a href="!project">MongoDB</a> project implements a generic interface to the <a href="!mongo">MongoDB</a> database server.', [
         '!project' => 'http://drupal.org/project/mongodb',
         '!mongo' => 'http://www.mongodb.org/',
       ]);
@@ -274,7 +274,7 @@ 
   if (!empty($GLOBALS['drupal_test_info']['test_run_id'])) {
     $simpletest_prefix = $GLOBALS['drupal_test_info']['test_run_id'];
   }
-  return $simpletest_prefix . $name;
+  return $simpletest_prefix.$name;
 }
 
 /**

modules/mongodb_queue/mongodb_queue.inc

Doc Comments +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
   /**
    * Claim an item in the queue for processing.
    *
-   * @param string $lease_time
+   * @param integer $lease_time
    *   How long the processing is expected to take in seconds,
    *
    * @return object/boolean

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
    */
   public function __construct($name) {
     // @todo: make sure that $name is a valid collection name?
-    $this->collection = 'queue.' . $name;
+    $this->collection = 'queue.'.$name;
   }
 
   /**

modules/mongodb_session/mongodb_session.inc

Doc Comments +3 added lines, -2 removed lines

@@ -141,7 +141,7 @@ 
  * @param array $value
  *   Serialized array of the session data.
  *
- * @return boolean
+ * @return null|boolean
  *   This function will always return TRUE.
  */
 function _drupal_session_write($sid, $value) {
@@ -300,6 +300,7 @@ 
 
 /**
  * Return whether a session has been started.
+ * @param boolean $set
  */
 function drupal_session_started($set = NULL) {
   static $session_started = FALSE;
@@ -386,7 +387,7 @@ 
 /**
  * Deletes the session cookie.
  *
- * @param sting $name
+ * @param string $name
  *   Name of session cookie to delete.
  * @param boolean $secure
  *   Force the secure value of the cookie.

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 if (!function_exists('_drupal_session_open')) {
-  include_once dirname(__FILE__) . '/../mongodb_module.php';
+  include_once dirname(__FILE__).'/../mongodb_module.php';
 
 /**
  * @file
@@ -108,7 +108,7 @@ 
     $user = db_query("SELECT * FROM {users} u WHERE uid = :uid", array(':uid' => $session['uid']))->fetchObject();
     // Add roles element to $user and insert an entry into fields_current.user
     // so next time we get it from Mongo.
-    include_once dirname(__FILE__) . '/mongodb_session.module';
+    include_once dirname(__FILE__).'/mongodb_session.module';
     $user->roles = array();
     $user->roles = mongodb_session_user_insert(array(), $user);
   }
@@ -322,7 +322,7 @@ 
       $old_insecure_session_id = $_COOKIE[$insecure_session_name];
     }
     $params = session_get_cookie_params();
-    $session_id = drupal_hash_base64(uniqid(mt_rand(), TRUE) . drupal_random_bytes(55));
+    $session_id = drupal_hash_base64(uniqid(mt_rand(), TRUE).drupal_random_bytes(55));
     // If a session cookie lifetime is set, the session will expire
     // $params['lifetime'] seconds from the current request. If it is not set,
     // it will expire when the browser is closed.
@@ -379,7 +379,7 @@ 
     _drupal_session_delete_cookie(substr(session_name(), 1), FALSE);
   }
   elseif (variable_get('https', FALSE)) {
-    _drupal_session_delete_cookie('S' . session_name(), TRUE);
+    _drupal_session_delete_cookie('S'.session_name(), TRUE);
   }
 }
 

modules/mongodb_watchdog/mongodb_watchdog.admin.inc

Doc Comments +2 added lines

@@ -214,6 +214,8 @@
 
 /**
  * Initialize the global pager variables for use in a mongodb_watchdog event list.
+ * @param integer $element
+ * @param integer $limit
  */
 function mongodb_watchdog_pager_init($element, $limit, $total) {
   global $pager_page_array, $pager_total, $pager_total_items;

Spacing +8 added lines, -8 removed lines

@@ -64,7 +64,7 @@ 
   foreach ($cursor as $id => $value) {
     if ($value['type'] == 'php' && $value['message'] == '%type: %message in %function (line %line of %file).') {
       $collection = mongodb_collection(variable_get('mongodb_watchdog', 'watchdog'));
-      $collection = $collection->db->selectCollection('watchdog_event_' . $value['_id']);
+      $collection = $collection->db->selectCollection('watchdog_event_'.$value['_id']);
       if ($result = $collection->find()->sort(array('$natural' => -1))->limit(1)->getNext()) {
         $value['file'] = basename($result['variables']['%file']);
         $value['line'] = $result['variables']['%line'];
@@ -78,7 +78,7 @@ 
       isset($value['count']) && $value['count'] > 1 ? $value['count'] : '',
       t($value['type']),
       empty($value['timestamp']) ? '' : format_date($value['timestamp'], 'short'),
-      empty($value['file']) ? '' : truncate_utf8(basename($value['file']), 30) . (empty($value['line']) ? '' : ('+' . $value['line'])),
+      empty($value['file']) ? '' : truncate_utf8(basename($value['file']), 30).(empty($value['line']) ? '' : ('+'.$value['line'])),
       l($message, "admin/reports/mongodb/$id"),
     );
   }
@@ -96,7 +96,7 @@ 
       ->find(mongodb_watchdog_build_filter_query())
       ->count();
     $pager_total[0] = ceil($pager_total_items[0] / $per_page);
-    $pager_page_array[0] = max(0, min((int) $pager_page_array[0], ((int)$pager_total[0]) - 1));
+    $pager_page_array[0] = max(0, min((int) $pager_page_array[0], ((int) $pager_total[0]) - 1));
     $pager_limits[0] = $per_page;
     $build['pager'] = array(
       '#theme' => 'pager',
@@ -151,7 +151,7 @@ 
   // @todo: the count is unreliable, so just get the actual number of entries.
 //$total = min($dblog['count'], variable_get('mongodb_watchdog_items', 10000));
   $collection = mongodb_collection(variable_get('mongodb_watchdog', 'watchdog'));
-  $collection = $collection->db->selectCollection('watchdog_event_' . $dblog['_id']);
+  $collection = $collection->db->selectCollection('watchdog_event_'.$dblog['_id']);
   $total = $collection->count();
   $limit = 20;
   $pagenumber = mongodb_watchdog_pager_init(0, $limit, $total);
@@ -191,7 +191,7 @@ 
     }
     $rows[] = array(
       format_date($event['timestamp'], 'short'),
-      l($account['name'], 'user/' . $account['uid']),
+      l($account['name'], 'user/'.$account['uid']),
       $request_uri ? l(truncate_utf8(basename(($request_uri)), 20), $request_uri) : '',
       $referer ? l(truncate_utf8(basename(($referer)), 20), $referer) : '',
       check_plain($ip),
@@ -226,7 +226,7 @@ 
   }
   $pager_total_items[$element] = $total;
   $pager_total[$element] = ceil($pager_total_items[$element] / $limit);
-  $pager_page_array[$element] = max(0, min((int)$pager_page_array[$element], ((int)$pager_total[$element]) - 1));
+  $pager_page_array[$element] = max(0, min((int) $pager_page_array[$element], ((int) $pager_total[$element]) - 1));
   return isset($pager_page_array[$element]) ? $pager_page_array[$element] : 0;
 }
 
@@ -291,7 +291,7 @@ 
     '#collapsed' => empty($_SESSION),
     '#attached' => array(
       'css' => array(
-        drupal_get_path('module', 'mongodb_watchdog') . '/mongodb_watchdog.css',
+        drupal_get_path('module', 'mongodb_watchdog').'/mongodb_watchdog.css',
     )),
   );
 
@@ -489,7 +489,7 @@ 
 
   // Find occurrences of error type.
   $key = $template['_id'];
-  $event_collection = mongodb_collection('watchdog_event_' . $key);
+  $event_collection = mongodb_collection('watchdog_event_'.$key);
   $reduce = <<<EOT
 function (doc, accumulator) {
   accumulator.count++;

modules/mongodb_watchdog/src/Controller/AdminController.php

Unused Use Statements -2 removed lines

@@ -10,9 +10,7 @@
 use Drupal\Component\Utility\Unicode;
 use Drupal\Core\DependencyInjection\ContainerInjectionInterface;
 use Drupal\Core\Url;
-use Drupal\mongodb\Connection;
 use Drupal\mongodb_watchdog\Logger;
-use Psr\Log\LoggerInterface;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 
 /**

Spacing +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
         isset($value['count']) && $value['count'] > 1 ? intval($value['count']) : 0,
         t($value['type']),
         empty($value['timestamp']) ? '' : format_date($value['timestamp'], 'short'),
-        empty($value['file']) ? '' : Unicode::truncate(basename($value['file']), 30) . (empty($value['line']) ? '' : ('+' . $value['line'])),
+        empty($value['file']) ? '' : Unicode::truncate(basename($value['file']), 30).(empty($value['line']) ? '' : ('+'.$value['line'])),
         \Drupal::l($message, Url::fromRoute('mongodb_watchdog.detail', ['id' => $id])),
       );
     }

modules/mongodb_watchdog/src/Logger.php

Unused Use Statements -2 removed lines

@@ -10,9 +10,7 @@
 
 use Drupal\Component\Utility\Unicode;
 use Drupal\Core\Logger\LogMessageParserInterface;
-use Drupal\Core\Logger\RfcLoggerTrait;
 use Psr\Log\AbstractLogger;
-use Psr\Log\LoggerInterface;
 
 /**
  * Class Logger is a PSR/3 Logger using a MongoDB data store.

Spacing +1 added lines, -1 removed lines

@@ -117,7 +117,7 @@
    *   A collection object for the specified template id.
    */
   public function eventCollection($template_id) {
-    $collection_name = static::EVENT_COLLECTION_PREFIX . $template_id;
+    $collection_name = static::EVENT_COLLECTION_PREFIX.$template_id;
     assert('preg_match(static::EVENT_COLLECTIONS_PATTERN, $collection_name)');
     return $this->database->selectCollection($collection_name);
   }

example.settings.local.php

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
  */
 
 if (!function_exists('configure_mongodb')) {
-  function configure_mongodb(\Drupal\Core\Site\Settings &$settings) {
+  function configure_mongodb(\Drupal\Core\Site\Settings & $settings) {
     $settings['mongodb'] = [
       'clients' => [
         // Client alias => constructor parameters.

modules/mongodb_cache/mongodb_cache.inc

Spacing +12 added lines, -12 removed lines

@@ -5,7 +5,7 @@ 
  * Replace your cache-backend file with this file. See README.txt for details.
  */
 
-include_once dirname(__FILE__) . '/../mongodb_module.php';
+include_once dirname(__FILE__).'/../mongodb_module.php';
 
 /**
  * MonogoDB cache implementation.
@@ -24,7 +24,7 @@ 
     $collection = mongodb_collection($this->bin);
     // Garbage collection necessary when enforcing a minimum cache lifetime.
     $this->garbageCollection($this->bin);
-    $cache = $collection->findOne(array('_id' => (string)$cid));
+    $cache = $collection->findOne(array('_id' => (string) $cid));
     return $this->prepareItem($cache);
   }
 
@@ -57,10 +57,10 @@ 
     $collection = mongodb_collection($this->bin);
 
     // Garbage collection necessary when enforcing a minimum cache lifetime.
-    $cache_flush = variable_get('cache_flush_' . $this->bin, 0);
+    $cache_flush = variable_get('cache_flush_'.$this->bin, 0);
     if ($cache_flush && ($cache_flush + variable_get('cache_lifetime', 0) <= REQUEST_TIME)) {
       // Reset the variable immediately to prevent a meltdown in heavy load situations.
-      variable_set('cache_flush_' . $this->bin, 0);
+      variable_set('cache_flush_'.$this->bin, 0);
       // Time to flush old cache data
 
 
@@ -88,7 +88,7 @@ 
       return FALSE;
     }
     unset($cache['_id']);
-    $cache = (object)$cache;
+    $cache = (object) $cache;
     // If the data is permanent or we are not enforcing a minimum cache lifetime
     // always return the cached data.
     if ($cache->expire == CACHE_PERMANENT || !variable_get('cache_lifetime', 0)) {
@@ -116,8 +116,8 @@ 
   function set($cid, $data, $expire = CACHE_PERMANENT) {
     $scalar = is_scalar($data);
     $entry = array(
-      '_id' => (string)$cid,
-      'cid' => (string)$cid,
+      '_id' => (string) $cid,
+      'cid' => (string) $cid,
       'created' => REQUEST_TIME,
       'expire' => $expire,
       'serialized' => !$scalar,
@@ -151,17 +151,17 @@ 
         // cached data that was cached before the timestamp.
         $user->cache = REQUEST_TIME;
 
-        $cache_flush = variable_get('cache_flush_' . $this->bin, 0);
+        $cache_flush = variable_get('cache_flush_'.$this->bin, 0);
         if ($cache_flush == 0) {
           // This is the first request to clear the cache, start a timer.
-          variable_set('cache_flush_' . $this->bin, REQUEST_TIME);
+          variable_set('cache_flush_'.$this->bin, REQUEST_TIME);
         }
         elseif (REQUEST_TIME > ($cache_flush + variable_get('cache_lifetime', 0))) {
           // Clear the cache for everyone, cache_lifetime seconds have
           // passed since the first request to clear the cache.
 
           $collection->remove(array('expire' => array('$ne' => CACHE_PERMANENT, '$lte' => REQUEST_TIME)), mongodb_default_write_options(FALSE));
-          variable_set('cache_flush_' . $this->bin, 0);
+          variable_set('cache_flush_'.$this->bin, 0);
         }
       }
       else {
@@ -175,7 +175,7 @@ 
           $collection->remove();
         }
         else {
-          $collection->remove(array('cid' => new MongoRegex('/'. preg_quote($cid) .'.*/')), mongodb_default_write_options(FALSE));
+          $collection->remove(array('cid' => new MongoRegex('/'.preg_quote($cid).'.*/')), mongodb_default_write_options(FALSE));
         }
       }
       elseif (is_array($cid)) {
@@ -187,7 +187,7 @@ 
         while (count($cid));
       }
       else {
-        $collection->remove(array('_id' => (string)$cid), mongodb_default_write_options(FALSE));
+        $collection->remove(array('_id' => (string) $cid), mongodb_default_write_options(FALSE));
       }
     }
   }