Database PHP API Fehler mit Primary Key

Hanashi · December 28, 2021 at 12:16 PM

Hallo,

seit dem neuesten Core Update scheint es Probleme mit der Database PHP API bei Primary Keys zu geben. Insbesondere geht es hier um folgendes Plugin:

WoltLab Cloud

File

Discord API

This plugin serves as an interface between other plugins and your Discord server. This plugin is especially relevant for developers. It can also be used as a base plugin for other plugins.

Hanashi

September 11, 2023 at 9:30 AM

Vor dem Update konnte ich es problemlos installieren. Seitdem Update erhalte ich folgende Fehlermeldung:

Requested URL: POST /acp/index.php?install-package/&t=9ec3b8a191facae151de3b590ab16e60537439f69aa027e14cabd7a0631d01ad-6ksJPWOR0JIrsH75S8Grog==

Referrer: https://******/acp/index.php?package-start-install/&action=install

Error Message: Das Datenbanklayout konnte aufgrund folgender Fehler nicht aktualisiert werden: Der Primärindex wcf1_discord_webhook (webhookID) heißt nicht 'PRIMARY'.

Type: RuntimeException

File (Line): /Users/******/www/wsc54/lib/system/database/table/DatabaseTableChangeProcessor.class.php (1120)

Stacktrace

/Users/******/www/wsc54/acp/install_dev.hanashi.wsc.discord-api.php (95): wcf\system\database\table\DatabaseTableChangeProcessor->process(…)
/Users/******/www/wsc54/lib/system/package/plugin/ScriptPackageInstallationPlugin.class.php (84): include(…)
/Users/******/www/wsc54/lib/system/package/plugin/ScriptPackageInstallationPlugin.class.php (59): wcf\system\package\plugin\ScriptPackageInstallationPlugin->run(…)
/Users/******/www/wsc54/lib/system/package/PackageInstallationDispatcher.class.php (792): wcf\system\package\plugin\ScriptPackageInstallationPlugin->install(…)
/Users/******/www/wsc54/lib/system/package/PackageInstallationDispatcher.class.php (153): wcf\system\package\PackageInstallationDispatcher->executePIP(…)
/Users/******/www/wsc54/lib/acp/action/InstallPackageAction.class.php (82): wcf\system\package\PackageInstallationDispatcher->install(…)
/Users/******/www/wsc54/lib/action/AbstractDialogAction.class.php (73): wcf\acp\action\InstallPackageAction->stepInstall(…)
/Users/******/www/wsc54/lib/action/AbstractAction.class.php (53): wcf\action\AbstractDialogAction->execute(…)
/Users/******/www/wsc54/lib/system/request/Request.class.php (89): wcf\action\AbstractAction->__run(…)
/Users/******/www/wsc54/lib/system/request/RequestHandler.class.php (119): wcf\system\request\Request->execute(…)
/Users/******/www/wsc54/acp/index.php (11): wcf\system\request\RequestHandler->handle(…)

Tue, 28 Dec 2021 11:13:36 +0000
Message: Das Datenbanklayout konnte aufgrund folgender Fehler nicht aktualisiert werden:             Der Primärindex wcf1_discord_webhook (webhookID) heißt nicht 'PRIMARY'.     
PHP version: 8.0.9
WoltLab Suite version: 5.4.10
Request URI: POST /acp/index.php?install-package/&t=9ec3b8a191facae151de3b590ab16e60537439f69aa027e14cabd7a0631d01ad-6ksJPWOR0JIrsH75S8Grog==
Referrer: https://******/acp/index.php?package-start-install/&action=install
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/15.2 Safari/605.1.15
Peak Memory Usage: 3650760/134217728
======
Error Class: RuntimeException
Error Message: Das Datenbanklayout konnte aufgrund folgender Fehler nicht aktualisiert werden:             Der Primärindex wcf1_discord_webhook (webhookID) heißt nicht 'PRIMARY'.     
Error Code: 0
File: /Users/******/www/wsc54/lib/system/database/table/DatabaseTableChangeProcessor.class.php (1120)
Extra Information: -
Stack Trace: [{"file":"\/Users\/******\/www\/wsc54\/acp\/install_dev.hanashi.wsc.discord-api.php","line":95,"function":"process","class":"wcf\\system\\database\\table\\DatabaseTableChangeProcessor","type":"->","args":[]},{"file":"\/Users\/******\/www\/wsc54\/lib\/system\/package\/plugin\/ScriptPackageInstallationPlugin.class.php","line":84,"args":["\/Users\/******\/www\/wsc54\/acp\/install_dev.hanashi.wsc.discord-api.php"],"function":"include","class":"","type":""},{"file":"\/Users\/******\/www\/wsc54\/lib\/system\/package\/plugin\/ScriptPackageInstallationPlugin.class.php","line":59,"function":"run","class":"wcf\\system\\package\\plugin\\ScriptPackageInstallationPlugin","type":"->","args":["\/Users\/******\/www\/wsc54\/acp\/install_dev.hanashi.wsc.discord-api.php"]},{"file":"\/Users\/******\/www\/wsc54\/lib\/system\/package\/PackageInstallationDispatcher.class.php","line":792,"function":"install","class":"wcf\\system\\package\\plugin\\ScriptPackageInstallationPlugin","type":"->","args":[]},{"file":"\/Users\/******\/www\/wsc54\/lib\/system\/package\/PackageInstallationDispatcher.class.php","line":153,"function":"executePIP","class":"wcf\\system\\package\\PackageInstallationDispatcher","type":"->","args":[{"attributes":"[redacted]","pip":"[redacted]","value":"[redacted]"}]},{"file":"\/Users\/******\/www\/wsc54\/lib\/acp\/action\/InstallPackageAction.class.php","line":82,"function":"install","class":"wcf\\system\\package\\PackageInstallationDispatcher","type":"->","args":["d7da57bc"]},{"file":"\/Users\/******\/www\/wsc54\/lib\/action\/AbstractDialogAction.class.php","line":73,"function":"stepInstall","class":"wcf\\acp\\action\\InstallPackageAction","type":"->","args":[]},{"file":"\/Users\/******\/www\/wsc54\/lib\/action\/AbstractAction.class.php","line":53,"function":"execute","class":"wcf\\action\\AbstractDialogAction","type":"->","args":[]},{"file":"\/Users\/******\/www\/wsc54\/lib\/system\/request\/Request.class.php","line":89,"function":"__run","class":"wcf\\action\\AbstractAction","type":"->","args":[]},{"file":"\/Users\/******\/www\/wsc54\/lib\/system\/request\/RequestHandler.class.php","line":119,"function":"execute","class":"wcf\\system\\request\\Request","type":"->","args":[]},{"file":"\/Users\/******\/www\/wsc54\/acp\/index.php","line":11,"function":"handle","class":"wcf\\system\\request\\RequestHandler","type":"->","args":["wcf",true]}]

So sieht der Teil aus für die Tabelle wcf1_discord_webhook:

PHP: install_dev.hanashi.wsc.discord-api.php

DatabaseTable::create('wcf1_discord_webhook')
    ->columns([
        BigintDatabaseTableColumn::create('webhookID')
            ->length(20)
            ->notNull(),
        BigintDatabaseTableColumn::create('channelID')
            ->length(20)
            ->notNull(),
        NotNullInt10DatabaseTableColumn::create('botID'),
        VarcharDatabaseTableColumn::create('webhookToken')
            ->length(100)
            ->notNull(),
        VarcharDatabaseTableColumn::create('webhookName')
            ->length(50)
            ->notNull(),
        VarcharDatabaseTableColumn::create('webhookTitle')
            ->length(100)
            ->notNull(),
        VarcharDatabaseTableColumn::create('usageBy')
            ->length(100)
            ->notNull(),
        NotNullInt10DatabaseTableColumn::create('webhookTime'),
    ])
    ->indices([
        DatabaseTableIndex::create()
            ->type(DatabaseTableIndex::PRIMARY_TYPE)
            ->columns(['webhookID']),
    ]),

Display More

**Joshua Rüsweg** · December 28, 2021 at 12:41 PM Official Post

Hallo,

es liegt an folgender Änderung:

Check the name of the PRIMARY KEY in PHP DDL · WoltLab/WCF@94a5d04

Resolves #4592

github.com

und dem Issue dazu:

PHP DDL: Validate that a PRIMARY KEY is called PRIMARY · Issue #4592 · WoltLab/WCF

… this can happen if the DatabaseTablePrimaryIndex factory is not used.

github.com

Prinzipiell solltest du auch anstelle des DatabaseTableIndex die Klasse DatabaseTablePrimaryIndex nutzen:

WCF/DatabaseTablePrimaryIndex.class.php at 5.4 · WoltLab/WCF

WoltLab Suite Core (previously WoltLab Community Framework) - WCF/DatabaseTablePrimaryIndex.class.php at 5.4 · WoltLab/WCF

github.com

Diese stellt sicher, dass der korrekte Name für den Key vergeben wird.

Ich schaue intern im Plugin-Store aber mal, wie viele Plugins das von dir verwendete Konstrukt nutzen und schaue, ob wir da einen Work-Around temporär für Version 5.4 einbauen (und dann erst mit 5.5 einen Fehler werfen).

beMananas · December 28, 2021 at 12:47 PM

Hallo

Ich bin nur am Handy und kann es gerade nicht testen. Ich hatte vor einer weile ein Ähnliches Problem und hatte mit WoltLab kontakt. Ggf wurde da ja nun was angepasst. Versuch mal folgendes anstelle dem Primary Teil

Code

DatabaseTablePrimaryIndex::create()
                ->columns(['webhookID']),

Ich glaube alternativ könntest du dein create anpassen:

Code

DatabaseTableIndex::create('PRIMARY')

Edit: zu spät

Gruss

Oliver

**Joshua Rüsweg** · December 28, 2021 at 12:53 PM Official Post

Hanashi Da dein Plugin das einzige im Plugin-Store ist, welches die Tabellen ohne den korrekten Key-Namen anlegt, wird sich ein Work-Around für Version 5.4 nicht lohnen. Wenn du dein Plugin entsprechend anpasst, kümmere ich mich zeitnah darum, dass das im Plugin-Store freigeschaltet wird.

Hanashi · December 28, 2021 at 1:00 PM

Quote from Joshua Rüsweg

Wenn du dein Plugin entsprechend anpasst, kümmere ich mich zeitnah darum, dass das im Plugin-Store freigeschaltet wird.

Danke dir, ist schon eingereicht

Database PHP API Fehler mit Primary Key

Discord API

Joshua Rüsweg December 28, 2021 at 12:51 PM

Alexander Ebert January 5, 2022 at 7:48 PM

Participate now!

Similar Threads

Fehlermeldung bei der Installation

Database PHP API - Foreign Key löschen

The provided API key has an IP address restriction

Fehler im Bereich Profilfelder

Discord API

Joshua Rüsweg December 28, 2021 at 12:51 PM

Alexander Ebert January 5, 2022 at 7:48 PM

Participate now!

Similar Threads

Fehlermeldung bei der Installation

Database PHP API - Foreign Key löschen

The provided API key has an IP address restriction

Fehler im Bereich Profilfelder

Joshua Rüsweg December 28, 2021 at 12:51 PM

Alexander Ebert January 5, 2022 at 7:48 PM