Merge pull request #41 from CESNET/dev

vyskocilpavel · web-flow · commit aca488b5e158 · 2020-07-14T19:00:27.000+02:00
Fix error when user identifier is null or ''
diff --git a/lib/DatabaseCommand.php b/lib/DatabaseCommand.php
@@ -199,6 +199,9 @@ private static function addWhereId($where, &$query, &$params)
 
     private function writeLogin($date, $ids, $user)
     {
+        if (empty($user)) {
+            return false;
+        }
         $params = array_merge($ids, [
             'day' => $date->format('Y-m-d'),
             'logins' => 1,

Original file line number	Diff line number	Diff line change
`@@ -199,6 +199,9 @@ private static function addWhereId($where, &$query, &$params)`
`199`	`199`
`200`	`200`	`private function writeLogin($date, $ids, $user)`
`201`	`201`	`{`
	`202`	`+ if (empty($user)) {`
	`203`	`+ return false;`
	`204`	`+ }`
`202`	`205`	`$params = array_merge($ids, [`
`203`	`206`	`'day' => $date->format('Y-m-d'),`
`204`	`207`	`'logins' => 1,`