Update mit MySQL-Statement schlägt fehl

MDMAN · July 19, 2017 at 8:00 AM

Ich habe eine Update-Anweisung zu einer Tabellespalte geschrieben. Diese wird in MyPHPAdmin ohne Probleme ausgeführt, doch im WCF kommt folgende Fehlermeldung:

Code

Fatal error: Unsupported SQL statement 'ALTER TABLE `wcf1_user` CHANGE `userIPLogComment` `userIPLogComment` MEDIUMTEXT'
Information:
id: db650d3e89d01594ed8ce7af51fcb4be32e0b89f
error message: Unsupported SQL statement 'ALTER TABLE `wcf1_user` CHANGE `userIPLogComment` `userIPLogComment` MEDIUMTEXT'
error code: 0
file: /var/www/vhosts/mdman.de/dev.mdman.de/wcf/lib/system/database/util/SQLParser.class.php (179)
php version: 7.1.7
wcf version: 2.1.16 (Typhoon)
date: Wed, 19 Jul 2017 05:56:09 +0000
request: /acp/index.php?install-package/&t=24efd77c19f999d7182a0b8d58c5d0db52174c45&s=e9d78903b23f128f6b4da0e9b7496f45127c1f18
referer: http://dev.mdman.de/acp/index.php?package-installation-confirm/&queueID=39&s=e9d78903b23f128f6b4da0e9b7496f45127c1f18

Display More

Die Tabelle bzw. die Spalte sieht vor dem Update so aus bzw. wurde so angelegt:

ALTER TABLE wcf1_user ADD userIPLogComment TEXT ;

und mit dem Update der Paket-Version wollte ich dieses ausführen, dann kommt aber der besagte Fehler...

ALTER TABLE `wcf1_user` CHANGE `userIPLogComment` `userIPLogComment` MEDIUMTEXT ;

Warum macht er denn eine Fehlermeldung?

Barungar · July 19, 2017 at 8:06 AM

Also ich hätte das so geschrieben: ALTER TABLE wcf1_user MODIFY COLUMN userIPLogComment MEDIUMTEXT; den "CHANGE" kenne ich eigentlich nur, wenn man eine Spalte umbenennen will.

**Alexander Ebert** · July 19, 2017 at 9:30 AM Official Post

MDMAN Der Parser sucht nach CHANGE COLUMN.

SQL

ALTER TABLE wcf1_user CHANGE COLUMN userIPLogComment userIPLogComment MEDIUMTEXT;

Barungar MODIFY COLUMN wird von uns nicht unterstützt, da es redundant zu CHANGE COLUMN ist, bzw. sich nur durch die optionale Umbenennung unterscheidet.

MDMAN · July 19, 2017 at 1:59 PM

Alexander Ebert : Ah... ok... danke... das Teste ich jetzt mal...

EDIT sagt: Hat funktioniert! Danke!

MDMAN · July 20, 2017 at 12:02 PM

Au Man... so langsam drehe ich durch...

Jetzt das nächste Problem.

Wenn ich diese Anweisung:

ALTER TABLE wcf1_user_iplog_all_user CHANGE COLUMN IPv4 IPv4 VARCHAR(39) NULL DEFAULT NULL, CHANGE COLUMN IPv6 IPv6 VARCHAR(39) NULL DEFAULT NULL, CHANGE COLUMN userID userID INT(10) NULL;

ausführen möchte, bekomme ich den MySQL-Fehler: "userID can't be NULL" oder so ungefähr.

Aber wenn ich diese Anweisung dann direkt in PHPmyAdmin eingebe, geht es ohne Probleme.

Was ist denn nun schon wieder hier falsch?

Es gibt natürlich einige Änderungen an der Datenbank und an den vorhandenen Daten, darum wurde auch ein PIP geschrieben, der es dann entsprechend bearbeiten soll, aber bevor das PIP ausgeführt wird soll durch eine SQL-Anweisung die Datenbank unter anderen wie oben geändert werden. Doch da hakt es schon... Einer eine Idee oder kann mir da einer mal gerade helfen?

**Alexander Ebert** · July 20, 2017 at 12:24 PM Official Post

Mache es bitte als einzelne Abfragen, diese verketteten Anweisungen mit Komma kann der Parser nicht verarbeiten.

MDMAN · July 20, 2017 at 12:39 PM

Ok... Ich versuche es mal....

MDMAN · July 20, 2017 at 1:15 PM

So langsam drehe ich hier echt am Rad...

Jetzt kommt folgender Fehler:

Code

Datum: 20. Juli 2017, 12:43
Datei (Zeile): /var/www/vhosts/mdman.de/dev.mdman.de/wcf/lib/system/database/statement/PreparedStatement.class.php (97)
Aufgerufene URL: /acp/index.php?install-package/&t=bfe2159a1c39510e0daf2395ca97b12a970b404b&s=841a764c5412bb937df49e616f443bc4d0663f9f
Referrer: http://dev.mdman.de/acp/index.php?package-installation-confirm/&queueID=40&s=841a764c5412bb937df49e616f443bc4d0663f9f
Browser: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3071.115 Safari/537.36
Zusatzinformationen: sql type: wcf\system\database\MySQLDatabase
sql error: Cannot add or update a child row: a foreign key constraint fails ("mdman_dev_mdman_de"."#sql-541_72da9c", CONSTRAINT "281606e1897873b788c05ad9ebc20e6a_fk" FOREIGN KEY ("doubleUserID") REFERENCES "wcf1_user" ("userID") ON DELETE SET NULL)
sql error number: 23000
sql version: 5.5.55-0ubuntu0.14.04.1
sql query: ALTER TABLE `wcf1_user_iplog_double_accounts` ADD CONSTRAINT `281606e1897873b788c05ad9ebc20e6a_fk` FOREIGN KEY (`doubleUserID`) REFERENCES `wcf1_user` (`userID`) ON DELETE SET NULL

Ich geb es gleich auf mit diesen blöden ForeignKeys...

**Alexander Ebert** · July 20, 2017 at 1:30 PM Official Post

Bei FOREIGN KEY müssen beide Spalten miteinander kompatibel sein, d.h. eine identische Definition besitzen. wcf1_user.userID darf nicht NULL sein, die Spalte in deiner Tabelle jedoch schon. Entweder gleichst du die Definition an, oder du kannst den FOREIGN KEY nicht verwenden.

In deiner Tabelle stehen Werte für userID drin, die es in wcf1_user nicht gibt. Unmittelbar bevor du den FOREIGN KEY anlegst, musst du die Werte einmalig abgleichen lassen:

SQL

UPDATE wcf1_user_iplog_double_accounts SET doubleUserID = NULL WHERE doubleUserID NOT IN (SELECT userID FROM wcf_user);

MDMAN · July 20, 2017 at 9:22 PM

Und dann ist es auch egal, welche Struktur die Datenbanken haben? Also ob bei einer NULL stehen darf und bei der anderen Nicht`?

**Alexander Ebert** · July 20, 2017 at 9:38 PM Official Post

Quote from MDMAN

Und dann ist es auch egal, welche Struktur die Datenbanken haben? Also ob bei einer NULL stehen darf und bei der anderen Nicht`?

Ja, die Spalten müssen nur hinsichtlich ihrer Definition von Typ und Breite übereinstimmen. Konkret für die userID-Spalte bedeutet das: Beide Spalten müssen als INT(10) definiert sein. NULL spielt hier keine Rolle, ich hatte mich bei meiner ursprünglichen Aussage vertan.

MDMAN · July 20, 2017 at 9:39 PM

Ok... dann versuche ich jetzt mal das Ganze erneut... Hoffen wir mal es klappt...

MDMAN · July 21, 2017 at 9:34 AM

Sieht gut aus... es hat funktioniert...

MDMAN · October 28, 2019 at 11:32 AM

mmm...

Jetzt habe ich schon wieder ein Problem mit einer SQL-Anweisung... siehe mal hier:

ALTER TABLE `XXXXXXX_product` CHANGE COLUMN `title` `title` VARCHAR(191) CHARACTER SET utf8mb4 COLLATE utf8mb4_unicode_ci NOT NULL DEFAULT '';

Ich dachte es muss jetzt CHANGE COLUMN heißen? Aber egal was ich da eingebe, dieser es kommt immer wieder der Fehler

"unsupported SQL Statement"...

Kann mir da noch einer helfen uns sagen, wo hier das Problem liegt?!?

**Alexander Ebert** · October 28, 2019 at 11:43 AM Official Post

SQL

ALTER TABLE xxxxx_product CHANGE COLUMN title title VARCHAR(191) NOT NULL DEFAULT '';

Das reicht vollkommen aus, das Charset bzw. die Collation wird direkt von der Tabelle genommen.

MDMAN · October 28, 2019 at 11:49 AM

Selbst das habe ich genauso probiert mit dem gleichen Fehler...

**Alexander Ebert** · October 28, 2019 at 11:51 AM Official Post

Sicher? Wenn ich die Abfrage von mir durch den SQLParser jage, wird $match[7] korrekt als 191 erkannt und erfolgreich verarbeitet.

MDMAN · October 28, 2019 at 11:55 AM

Sicher:

Code

PHP Version:

7.3.10

WoltLab Suite Core:

3.1.11

Peak Memory Usage:

36.118/128 MiB

Request URI:

/acp/index.php?install-package/&t=96f26da76884185d04767a3d9c36c4a91d9b00be

Referrer:

https://wsc31.mdman.de/acp/index.php?package-installation-confirm/&queueID=38

User Agent:

Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/76.0.3809.87 Safari/537.36

Error

Error Type:

wcf\system\exception\SystemException

Error Message:

Unsupported SQL statement 'ALTER TABLE `XXXXXXXproduct` CHANGE COLUMN `title` `title` VARCHAR(191) NOT NULL DEFAULT '''

File:

*/lib/system/database/util/SQLParser.class.php (178)

Stack Trace:

#0 */lib/system/database/util/SQLParser.class.php (47):
wcf\system\database\util\SQLParser->executeStatement('ALTER TABLE', 'ALTER TABLE `XXXXXXX_product` CHANGE COLUMN `title` `title` VARCHAR(191) NOT NULL DEFAULT \'\'')
#1 */lib/system/package/PackageInstallationSQLParser.class.php (103):
wcf\system\database\util\SQLParser->execute()
#2 */lib/system/package/plugin/SQLPackageInstallationPlugin.class.php (45):
wcf\system\package\PackageInstallationSQLParser->test()
#3 */lib/system/package/plugin/AbstractPackageInstallationPlugin.class.php (70):
wcf\system\package\plugin\SQLPackageInstallationPlugin->install()
#4 */lib/system/package/PackageInstallationDispatcher.class.php (673):
wcf\system\package\plugin\AbstractPackageInstallationPlugin->update()
#5 */lib/system/package/PackageInstallationDispatcher.class.php (144):
wcf\system\package\PackageInstallationDispatcher->executePIP([ attributes => , pip => , value => ])
#6 */lib/acp/action/InstallPackageAction.class.php (73):
wcf\system\package\PackageInstallationDispatcher->install('654165f4')
#7 */lib/action/AbstractDialogAction.class.php (68):
wcf\acp\action\InstallPackageAction->stepInstall()
#8 */lib/action/AbstractAction.class.php (47):
wcf\action\AbstractDialogAction->execute()
#9 */lib/system/request/Request.class.php (83):
wcf\action\AbstractAction->__run()
#10 */lib/system/request/RequestHandler.class.php (96):
wcf\system\request\Request->execute()
#11 */acp/index.php (9):
wcf\system\request\RequestHandler->handle('wcf', true)

Display More

**Alexander Ebert** · October 28, 2019 at 12:21 PM Official Post

Also, ohne die Backticks passt der Regex, lass die Dinger doch bitte einfach weg.

MDMAN · October 28, 2019 at 12:42 PM

Ok...

Ich versuche es mal...

EDIT sagt: mmm ohne die "Backticks" hat es doch glatt funktioniert... Man das muss man aber auch wissen...

Participate now!