Use strict comparison.

cakephp · Oct 16, 2014 · 9eb1150 · 9eb1150
1 parent 749b7f6
commit 9eb1150
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 8 deletions.
diff --git a/src/Database/Schema/SqliteSchema.php b/src/Database/Schema/SqliteSchema.php
@@ -117,12 +117,12 @@ public function convertColumnDescription(Table $table, $row) {
 			'null' => !$row['notnull'],
 			'default' => $row['dflt_value'] === null ? null : trim($row['dflt_value'], "'"),
 		];
-		if ($row['pk'] == true) {
+		if ($row['pk'] === true) {
 			$field['null'] = false;
 			$field['autoIncrement'] = true;
 		}
 		$table->addColumn($row['name'], $field);
-		if ($row['pk'] == true) {
+		if ($row['pk'] === true) {
 			$constraint = (array)$table->constraint('primary') + [
 				'type' => Table::CONSTRAINT_PRIMARY,
 				'columns' => []

diff --git a/src/Event/EventManagerTrait.php b/src/Event/EventManagerTrait.php
@@ -45,7 +45,7 @@ trait EventManagerTrait {
  * @return \Cake\Event\EventManager
  */
 	public function eventManager(EventManager $eventManager = null) {
-		if ($eventManager != null) {
+		if ($eventManager !== null) {
 			$this->_eventManager = $eventManager;
 		} elseif (empty($this->_eventManager)) {
 			$this->_eventManager = new EventManager();
@@ -54,15 +54,15 @@ public function eventManager(EventManager $eventManager = null) {
 	}
 
 /**
- * Wrapper for creating and dispatching events. 
+ * Wrapper for creating and dispatching events.
  *
  * Returns a dispatched event.
  *
  * @param string $name Name of the event.
  * @param array $data Any value you wish to be transported with this event to
  * it can be read by listeners.
  *
- * @param object $subject The object that this event applies to 
+ * @param object $subject The object that this event applies to
  * ($this by default).
  *
  * @return \Cake\Event\Event

diff --git a/src/Routing/RouteBuilder.php b/src/Routing/RouteBuilder.php
@@ -120,7 +120,7 @@ public function __construct($collection, $path, array $params = [], array $optio
  * @return string|void
  */
 	public function routeClass($routeClass = null) {
-		if ($routeClass == null) {
+		if ($routeClass === null) {
 			return $this->_routeClass;
 		}
 		$this->_routeClass = $routeClass;

diff --git a/src/Routing/Router.php b/src/Routing/Router.php
@@ -154,7 +154,7 @@ class Router {
  * @return string|void
  */
 	public static function defaultRouteClass($routeClass = null) {
-		if ($routeClass == null) {
+		if ($routeClass === null) {
 			return static::$_defaultRouteClass;
 		}
 		static::$_defaultRouteClass = $routeClass;

diff --git a/src/View/ViewVarsTrait.php b/src/View/ViewVarsTrait.php
@@ -120,7 +120,7 @@ public function viewOptions($options = null, $merge = true) {
 			$this->_validViewOptions = [];
 		}
 
-		if ($options == null) {
+		if ($options === null) {
 			return $this->_validViewOptions;
 		}