Possibly expensive SQL Query on determining files to process #144

Fix cast for MySQL (cherry picked from commit 7d8ae58)
GDATASoftwareAG · Oct 23, 2024 · db353f9 · db353f9
1 parent 66c672f
commit db353f9
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/lib/Db/DbFileMapper.php b/lib/Db/DbFileMapper.php
@@ -95,8 +95,11 @@ public function getFileIdsWithTags(array $includedTagIds, int $limit, int $offse
 	 * @throws Exception if the database platform is not supported
 	 */
 	private function getStringTypeDeclarationSQL(): string {
-		$platform = $this->db->getDatabasePlatform()->getName();
-		if ($platform === 'mysql' || $platform === 'sqlite' || $platform === 'postgres') {
+		$platform = $this->db->getDatabaseProvider();
+		if ($platform === "mysql") {
+			$stringType = 'CHAR(64)';
+		}
+		else if ($platform === 'sqlite' || $platform === 'postgres') {
 			$stringType = 'VARCHAR(64)';
 		} else {
 			throw new Exception('Unsupported database platform: ' . $platform);