Fix empty() bug on DBPrefix

MGatner · web-flow · commit 3201d8089a9a · 2019-09-09T17:00:54.000-04:00
diff --git a/system/Database/Forge.php b/system/Database/Forge.php
@@ -605,7 +605,7 @@ public function dropTable(string $tableName, bool $ifExists = false, bool $casca
 		}
 
 		// If the prefix is already starting the table name, remove it...
-		if (! empty($this->db->DBPrefix) && strpos($tableName, $this->db->DBPrefix) === 0)
+		if ($this->db->DBPrefix && strpos($tableName, $this->db->DBPrefix) === 0)
 		{
 			$tableName = substr($tableName, strlen($this->db->DBPrefix));
 		}

Original file line number	Diff line number	Diff line change
`@@ -605,7 +605,7 @@ public function dropTable(string $tableName, bool $ifExists = false, bool $casca`
`605`	`605`	`}`
`606`	`606`
`607`	`607`	`// If the prefix is already starting the table name, remove it...`
`608`		`- if (! empty($this->db->DBPrefix) && strpos($tableName, $this->db->DBPrefix) === 0)`
	`608`	`+ if ($this->db->DBPrefix && strpos($tableName, $this->db->DBPrefix) === 0)`
`609`	`609`	`{`
`610`	`610`	`$tableName = substr($tableName, strlen($this->db->DBPrefix));`
`611`	`611`	`}`