Event::find() - Code Metrics - Inspection of "DDD Milan: Logger events are now correctly inserte..." - FGM/mongodb - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 8.x-2.x ( c6d484...0f1fa4 )

by Frédéric G.

created 2016-06-25 14:41 UTC

Event::find() A

↳ Parent: Event

Complexity

Conditions	1
Paths	1

Size

Total Lines	14
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	10
c	1
b	0
f	0
nc	1
nop	1
dl	0
loc	14
rs	9.4285

<?php

namespace Drupal\mongodb_watchdog;

use MongoDB\BSON\Unserializable;

/**
 * Class Event.
 *
 * @package Drupal\mongodb_watchdog
 */
class Event implements Unserializable {


  // @codingStandardsIgnoreStart
  /**
   * The string representation of a MongoId.
   *
   * @var int
   */
  public $_id;
  // @codingStandardsIgnoreEnd

  /**
   * User id.
   *
   * @var int
   */
  public $uid;

  /**
   * Event type, often a module name.
   *
   * @var string
   */
  public $type;

  /**
   * Event template.
   *
   * @var string
   */
  public $message;

  /**
   * The template parameters.
   *
   * @var array
   */
  public $variables;

  /**
   * A RFC5424 severity level.
   *
   * @var int
   */
  public $severity;

  /**
   * A link provided by the event emitter. Optional.
   *
   * @var string
   */
  public $link;

  /**
   * The absolute URL for the path on which the event was logged.
   *
   * @var string
   */
  public $location;

  /**
   * A HTTP referrer for the path on which the event was logged. Optional.
   *
   * @var string
   */
  public $referrer;

  /**
   * The server host.
   *
   * @var string
   */
  public $hostname;

  /**
   * The timestamp at which the event was logged.
   *
   * @var int
   */
  public $timestamp;

  /**
   * Constructor.
   *
   * @param array $event
   *   The event in array form.
   */
  public function __construct(array $event) {
    $keys = [
      '_id',
      'hostname',
      'link',
      'location',
      'message',
      'referrer',
      'severity',
      'timestamp',
      'type',
      'uid',
      'variables',
    ];
    foreach ($keys as $key) {
      if (isset($event[$key])) {
        $this->$key = $event[$key];
      }
    }
  }

  /**
   * Load a MongoDB watchdog event.
   *
   * @param string $id
/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}
   *   The string representation of a MongoId.
   *
   * @return \Drupal\mongodb_watchdog\Event|bool
   *   FALSE if the event cannot be loaded.
   */
  public function find($template_id) {
    $criteria = ['_id' => new ObjectID($id)];

    $options = [
      'typeMap' => [
        'array' => 'array',
        'document' => 'array',
        'root' => 'Drupal\mongodb_watchdog\Event',
      ],
    ];

    $result = $this->templatesCollection->findOne($criteria, $options);
    var_dump($result);

    return $result;
  }

  function bsonUnserialize(array $data) {

    $event = new static($data);
    return $event;
  }
}



1			<?php
2
3			namespace Drupal\mongodb_watchdog;
4
5			use MongoDB\BSON\Unserializable;
6
7			/**
8			* Class Event.
9			*
10			* @package Drupal\mongodb_watchdog
11			*/
12			class Event implements Unserializable {
			0 ignored issues – show Coding Style introduced 2016-06-25 14:44 UTC by Report Bug Copy Issue Report The property $_id is not named in camelCase. This check marks property names that have not been written in camelCase. In camelCase names are written without any punctuation, the start of each new word being marked by a capital letter. Thus the name database connection string becomes `databaseConnectionString`. Loading history...
13
14			// @codingStandardsIgnoreStart
15			/**
16			* The string representation of a MongoId.
17			*
18			* @var int
19			*/
20			public $_id;
21			// @codingStandardsIgnoreEnd
22
23			/**
24			* User id.
25			*
26			* @var int
27			*/
28			public $uid;
29
30			/**
31			* Event type, often a module name.
32			*
33			* @var string
34			*/
35			public $type;
36
37			/**
38			* Event template.
39			*
40			* @var string
41			*/
42			public $message;
43
44			/**
45			* The template parameters.
46			*
47			* @var array
48			*/
49			public $variables;
50
51			/**
52			* A RFC5424 severity level.
53			*
54			* @var int
55			*/
56			public $severity;
57
58			/**
59			* A link provided by the event emitter. Optional.
60			*
61			* @var string
62			*/
63			public $link;
64
65			/**
66			* The absolute URL for the path on which the event was logged.
67			*
68			* @var string
69			*/
70			public $location;
71
72			/**
73			* A HTTP referrer for the path on which the event was logged. Optional.
74			*
75			* @var string
76			*/
77			public $referrer;
78
79			/**
80			* The server host.
81			*
82			* @var string
83			*/
84			public $hostname;
85
86			/**
87			* The timestamp at which the event was logged.
88			*
89			* @var int
90			*/
91			public $timestamp;
92
93			/**
94			* Constructor.
95			*
96			* @param array $event
97			* The event in array form.
98			*/
99			public function __construct(array $event) {
100			$keys = [
101			'_id',
102			'hostname',
103			'link',
104			'location',
105			'message',
106			'referrer',
107			'severity',
108			'timestamp',
109			'type',
110			'uid',
111			'variables',
112			];
113			foreach ($keys as $key) {
114			if (isset($event[$key])) {
115			$this->$key = $event[$key];
116			}
117			}
118			}
119
120			/**
121			* Load a MongoDB watchdog event.
122			*
123			* @param string $id
			0 ignored issues – show Bug introduced 2016-06-25 14:44 UTC by Report Bug Copy Issue Report There is no parameter named `$id`. Was it maybe removed? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. Consider the following example. The parameter `$italy` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $island * @param array $italy */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was removed, but the annotation was not. Loading history... introduced 2016-06-25 14:44 UTC by Report Bug Copy Issue Report Doc comment for parameter $id does not match actual variable name $template_id Loading history...
124			* The string representation of a MongoId.
125			*
126			* @return \Drupal\mongodb_watchdog\Event\|bool
127			* FALSE if the event cannot be loaded.
128			*/
129			public function find($template_id) {
130			$criteria = ['_id' => new ObjectID($id)];
			0 ignored issues – show Bug introduced 2016-06-25 14:44 UTC by Report Bug Copy Issue Report The variable `$id` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history... introduced 2016-06-25 14:44 UTC by Report Bug Copy Issue Report Variable $id is undefined. Loading history...
131			$options = [
132			'typeMap' => [
133			'array' => 'array',
134			'document' => 'array',
135			'root' => 'Drupal\mongodb_watchdog\Event',
136			],
137			];
138
139			$result = $this->templatesCollection->findOne($criteria, $options);
140			var_dump($result);
			0 ignored issues – show Security Debugging Code introduced 2016-06-25 14:44 UTC by Report Bug Copy Issue Report `var_dump($result);` looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data. Loading history...
141			return $result;
142			}
143
144			function bsonUnserialize(array $data) {
			0 ignored issues – show Best Practice introduced 2016-06-25 14:44 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history... introduced 2016-06-25 14:44 UTC by Report Bug Copy Issue Report Missing function doc comment Loading history... Comprehensibility Best Practice introduced 2016-06-25 14:44 UTC by Report Bug Copy Issue Report It is recommend to declare an explicit visibility for `bsonUnserialize`. Generally, we recommend to declare visibility for all methods in your source code. This has the advantage of clearly communication to other developers, and also yourself, how this method should be consumed. If you are not sure which visibility to choose, it is a good idea to start with the most restrictive visibility, and then raise visibility as needed, i.e. start with `private`, and only raise it to `protected` if a sub-class needs to have access, or `public` if an external class needs access. Loading history...
145			$event = new static($data);
146			return $event;
147			}
148			}
			0 ignored issues – show Coding Style introduced 2016-06-24 11:00 UTC by Report Bug Copy Issue Report As per coding style, files should not end with a newline character. This check marks files that end in a newline character, i.e. an empy line. Loading history...
149

FGM / mongodb

Push — 8.x-2.x ( c6d484...0f1fa4 )

Event::find() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like