Returns NULL instead of FALSE for non-exists entities

Author: janpecha

src/TQueryableEntity.php

@@ -33,7 +33,7 @@ public function find($field, Query $query)
 
 		/**
 		 * Fetchs one entity by Query object
-		 * @return Entity|FALSE
+		 * @return Entity|NULL
 		 */
 		public function findOne($field, Query $query)
 		{
@@ -42,7 +42,7 @@ public function findOne($field, Query $query)
 			if ($entities) {
 				return $entities[0];
 			}
-			return FALSE;
+			return NULL;
 		}
 
 

src/TQueryableRepository.php

@@ -31,15 +31,15 @@ public function find(Query $query)
 
 		/**
 		 * Fetchs one entity by Query object
-		 * @return Entity|FALSE
+		 * @return Entity|NULL
 		 */
 		public function findOne(Query $query)
 		{
 			$row = $this->applyQuery($query)
 				->removeClause('limit')
 				->removeClause('offset')
 				->fetch();
-			return $row ? $this->createEntity($row) : FALSE;
+			return $row ? $this->createEntity($row) : NULL;
 		}
 
 

src/TRepository.php

@@ -20,7 +20,7 @@ public function createNewEntity()
 
 		/**
 		 * Fetchs entity by primary key
-		 * @return Entity|FALSE
+		 * @return Entity|NULL
 		 */
 		public function get($id)
 		{
@@ -30,7 +30,7 @@ public function get($id)
 
 		/**
 		 * Fetchs entity by column value
-		 * @return Entity|FALSE
+		 * @return Entity|NULL
 		 */
 		protected function getByColumn($column, $value)
 		{
@@ -39,9 +39,6 @@ protected function getByColumn($column, $value)
 				->where('%n = ?', $column, $value)
 				->fetch();
 
-			if ($row === FALSE) {
-				return $row;
-			}
-			return $this->createEntity($row);
+			return $row ? $this->createEntity($row) : NULL;
 		}
 	}

tests/InlmModel/TQueryableEntity.phpt

@@ -76,7 +76,7 @@ test(function () use ($authorRepository, $sql) {
 		->where('@id > 9999')
 		->findOne();
 
-	Assert::false($book);
+	Assert::null($book);
 });
 
 

tests/InlmModel/TQueryableRepository.phpt

@@ -67,7 +67,7 @@ test(function () use ($bookRepository, $sql) {
 		->where('@id > 9999')
 		->findOne();
 
-	Assert::false($book);
+	Assert::null($book);
 });
 
 

tests/InlmModel/TRepository.phpt

@@ -49,5 +49,5 @@ test(function () use ($bookRepository) {
 
 test(function () use ($bookRepository) {
 	$book = $bookRepository->get(9999);
-	Assert::false($book);
+	Assert::null($book);
 });