install.sql; ADD CONSTRAINT schlägt fehl.

veneanar · Jan 9th 2017

Hallo,

ich brauche eine Tabelle in der Feeds gespeichert werden. Ein Backgroundjob soll regelmäßig nach Updates schauen, ich bekomme die Daten mit einen Timestamp sowie einer Benutzerzuordnung. Beide Daten zusammen sind einzigartig, daher möchte ich diese als UNIQUE CONSTRAINT verwenden um bei den Updates der Feed bereits ausgelesene mittels INSERT ... ON DUPLICATE KEY UPDATE ...zu überspringen.

So wird das ganze definiert: ALTER TABLE wcf1_gman_feedlist ADD CONSTRAINT charId_feedTime_UQ UNIQUE (charID, feedTime);

Das WCF versteht diese Anweisung aber falsch und will eine Column an die tabelle tackern: wcf\system\package\PackageInstallationSQLParser->executeAddColumnStatement('wcf1_gman_feedlist', 'CONSTRAINT', [ type => , notNull => , default => , autoIncrement => , key => ])

Das Resultat ist diese Fehlermeldung:

Fehlermeldung

Ein Fehler ist aufgetreten

Interner Fehlercode: 04c75206f8f49961a95867dac5fe8d1fd445f5eb

Was ist passiert?

Leider ist es bei der Verarbeitung zu einem Fehler gekommen und die Ausführung wurde abgebrochen. Falls möglich, leiten Sie bitte den oben stehenden Fehlercode an den Administrator weiter.

Administratoren können die vollständige Fehlermeldung mit Hilfe dieses Codes in der Administrationsoberfläche unter „Protokoll » Fehler“ einsehen. Zusätzlich wurden die Informationen in die Protokolldatei */log/2016-12-30.txt geschrieben und können beispielsweise mit Hilfe eines FTP-Programms abgerufen werden.

Hinweis: Der Fehlercode wird zufällig generiert, erlaubt keinen Rückschluss auf die Ursache und ist daher für Dritte nutzlos.

System Information

PHP Version:

7.0.13-1~dotdeb+8.1
WoltLab Suite Core:

3.0.0 RC 4
Peak Memory Usage:

4.364/128 MiB
Request URI:

/com.sylvansagarde/acp/index.php?install-package/&t=948d42b4ef3c3cc6d98bfcc4e7bca90d61a4a39f
Referrer:

http://192.168.0.14/com.sylvan…ation-confirm/&queueID=54
User Agent:

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

Error

Error Type:

wcf\system\exception\SystemException
Error Message:

Cannot add column 'CONSTRAINT' to table 'wcf1_gman_feedlist'.
File:

*/lib/system/package/PackageInstallationSQLParser.class.php (281)
Stack Trace:
- #0 */lib/system/database/util/SQLParser.class.php (155):
- wcf\system\package\PackageInstallationSQLParser->executeAddColumnStatement('wcf1_gman_feedlist', 'CONSTRAINT', [ type => , notNull => , default => , autoIncrement => , key => ])
- #1 */lib/system/database/util/SQLParser.class.php (47):
- wcf\system\database\util\SQLParser->executeStatement('ALTER TABLE', 'ALTER TABLE wcf1_gman_feedlist ADD CONSTRAINT charId_feedTime_UQ UNIQUE (charID, feedTime)')
- #2 */lib/system/package/PackageInstallationSQLParser.class.php (103):
- wcf\system\database\util\SQLParser->execute()
- #3 */lib/system/package/plugin/SQLPackageInstallationPlugin.class.php (45):
- wcf\system\package\PackageInstallationSQLParser->test()
- #4 */lib/system/package/PackageInstallationDispatcher.class.php (595):
- wcf\system\package\plugin\SQLPackageInstallationPlugin->install()
- #5 */lib/system/package/PackageInstallationDispatcher.class.php (134):
- wcf\system\package\PackageInstallationDispatcher->executePIP([ attributes => , pip => , value => ])
- #6 */lib/acp/action/InstallPackageAction.class.php (72):
- wcf\system\package\PackageInstallationDispatcher->install('cfdb4fd8')
- #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 (94):
- wcf\system\request\Request->execute()
- #11 */acp/index.php (9):
- wcf\system\request\RequestHandler->handle('wcf', true)

Zur besseren Übersicht die komplette install.sql (In Zeile 222 ist besagtes ALTER TABLE zu finden):

SQL: install.sql

DROP TABLE IF EXISTS wcf1_gman_group;
CREATE TABLE wcf1_gman_group (
  groupID INT(10) NOT NULL AUTO_INCREMENT PRIMARY KEY,
  title VARCHAR(50)  NOT NULL,
  teaser VARCHAR(250) NOT NULL DEFAULT '',
  wcfGroupID INT(10) NOT NULL,
  showCalender TINYINT(1) NOT NULL DEFAULT 0,
  calendarTitle VARCHAR(50) NOT NULL,
  calendartext TEXT NULL,
  fetchCalendar TINYINT(1) NOT NULL DEFAULT 0,
  calendarQuerry VARCHAR(25) NOT NULL DEFAULT '',
  gameTitle VARCHAR(50)  NOT NULL DEFAULT '',
  gameRank TINYINT(1) NOT NULL DEFAULT 0,
  showRoaster TINYINT(1) NOT NULL DEFAULT 1,
  articIeID INT(10) NULL,
  threadID INT(10) NULL,
  boardID INT(10) NULL,
  mediaID INT(10) NULL,
  isRaidgruop INT(10) NOT NULL,
  fetchWCL TINYINT(1) NOT NULL DEFAULT 0,
  wclQuerry VARCHAR(100) NOT NULL DEFAULT '',
  orderNo SMALLINT(4) NOT NULL,
  lastUpdate INT(10) NOT NULL
  KEY (wcfGroupID, articIeID, threadID, boardID, mediaID);
) ;

ALTER TABLE wcf1_gman_group ADD FOREIGN KEY (articIeID) REFERENCES wcf1_article (articleID) ON DELETE SET NULL;
ALTER TABLE wcf1_gman_group ADD FOREIGN KEY (threadID) REFERENCES wbb1_thread (threadID) ON DELETE SET NULL;
ALTER TABLE wcf1_gman_group ADD FOREIGN KEY (boardID) REFERENCES wbb1_board (boardID) ON DELETE SET NULL;
ALTER TABLE wcf1_gman_group ADD FOREIGN KEY (mediaID) REFERENCES wcf1_media (mediaID) ON DELETE SET NULL;

DROP TABLE IF EXISTS wcf1_gman_wow_realm;
CREATE TABLE wcf1_gman_wow_realm (
  name VARCHAR(30) NOT NULL PRIMARY KEY,
  type VARCHAR(20)  NOT NULL DEFAULT '',
  population VARCHAR(10)  NOT NULL DEFAULT '',
  queue TINYINT(1) NULL DEFAULT 0,
  status TINYINT(2) NOT NULL DEFAULT 0,
  slug VARCHAR(30)  NOT NULL,
  battlegroup VARCHAR(50)  NOT NULL DEFAULT '',
  timezone VARCHAR(25)  NOT NULL DEFAULT '',
  connected_realms TEXT NULL,
  locale VARCHAR(6)  NOT NULL DEFAULT '',
  lastUpdate INT(10) NOT NULL

);


DROP TABLE IF EXISTS wcf1_gman_wow_character;
CREATE TABLE wcf1_gman_wow_character (
  charID VARCHAR(35) NOT NULL PRIMARY KEY,
  userID INT(10) NULL,
  isMain TINYINT(1) NOT NULL DEFAULT 0,
  inGuild  TINYINT(1) NOT NULL DEFAULT 0,
  realmID VARCHAR(30) NOT NULL ,
  bnetData TEXT NULL,
  primaryGroup INT(10) NULL,
  groups VARCHAR(100) NOT NULL,
  bnetUpdate INT(10) NOT NULL,
  firstSeen INT(10) NOT NULL,
  guildRank TINYINT(2) NULL DEFAULT 11, 
  averageItemLevel smallint(5) NULL,
  averageItemLevelEquipped smallint(5) NULL,
  head TEXT NULL,
  neck TEXT NULL,
  shoulder TEXT NULL,
  back TEXT NULL,
  chest TEXT NULL,
  shirt TEXT NULL,
  wrist TEXT NULL,
  hands TEXT NULL,
  waist TEXT NULL,
  legs TEXT NULL,
  feet TEXT  NULL,
  finger1 TEXT NULL,
  finger2 TEXT NULL,
  trinket1 TEXT NULL,
  trinket2 TEXT NULL,
  mainHand TEXT NULL,
  offHand TEXT NULL,
  KEY(userID, realmID, primaryGroup)
) ;

ALTER TABLE wcf1_gman_wow_character ADD FOREIGN KEY (userID) REFERENCES wcf1_user (userID) ON DELETE SET NULL;
ALTER TABLE wcf1_gman_wow_character ADD FOREIGN KEY (primaryGroup) REFERENCES wcf1_gman_group (groupID) ON DELETE SET NULL;

DROP TABLE IF EXISTS wcf1_gman_guild;
CREATE TABLE wcf1_gman_guild (
 guildID SMALLINT(4) NOT NULL PRIMARY KEY,
 articleID INT(10) NULL,
 pageID INT(10) NULL,
 leaderID VARCHAR(35) NULL,
 birthday INT(10) NOT NULL,
 logoID INT(10) NULL,
 bnetUpdate INT(10) NOT NULL,
 bnetData TEXT NULL,
 KEY(leaderID, articleID, pageID)
);

ALTER TABLE wcf1_gman_guild ADD FOREIGN KEY (articleID) REFERENCES wcf1_article (articleID) ON DELETE SET NULL;
ALTER TABLE wcf1_gman_guild ADD FOREIGN KEY (pageID) REFERENCES wcf1_page (pageID) ON DELETE SET NULL;
ALTER TABLE wcf1_gman_guild ADD FOREIGN KEY (leaderID) REFERENCES wcf1_gman_wow_character (charID) ON DELETE RESTRICT;


DROP TABLE IF EXISTS wcf1_gman_application;
CREATE TABLE wcf1_gman_application (
  appID INT(10) NOT NULL AUTO_INCREMENT PRIMARY KEY,
  threadID INT(10) NULL ,
  pollID INT(10) DEFAULT NULL ,
  charID VARCHAR(35) NULL ,
  name VARCHAR(50) NOT NULL,
  autoClose TINYINT(10) NOT NULL DEFAULT 0,
  pollEnd INT(10) NOT NULL,
  assignedOfficerID INT(10) NULL,
  interviewDate INT(10) NOT NULL ,
  appState TINYINT(1) NOT NULL DEFAULT 0,
  openDate INT(10) NOT NULL,
  KEY (assignedOfficerID, threadID, pollID)
) ;
ALTER TABLE wcf1_gman_application ADD FOREIGN KEY (assignedOfficerID) REFERENCES wcf1_user (userID) ON DELETE SET NULL;
ALTER TABLE wcf1_gman_application ADD FOREIGN KEY (threadID) REFERENCES wbb1_thread (threadID) ON DELETE SET NULL;

DROP TABLE IF EXISTS wcf1_gman_pointtype;
CREATE TABLE wcf1_gman_pointtype (
  typeID SMALLINT(4) NOT NULL AUTO_INCREMENT PRIMARY KEY,
  groupID INT(10) NOT NULL,
  content VARCHAR(250) NOT NULL DEFAULT ''
) ;

DROP TABLE IF EXISTS wcf1_gman_pointtrans;
CREATE TABLE wcf1_gman_pointtrans (
  transID INT(10) NOT NULL AUTO_INCREMENT PRIMARY KEY,
  charID VARCHAR(35) NULL,
  groupID INT(10) NULL,
  eventID INT(10) NULL,
  amount SMALLINT(4) NOT NULL,
  typeID SMALLINT(4) NULL,
  itemID INT(10) NOT NULL DEFAULT 0,
  comment VARCHAR(250)  NOT NULL DEFAULT '' ,
  issuerID INT(10) NULL,
  transDate INT(10) NOT NULL,
  autoDelete TINYINT(1) NOT NULL DEFAULT 0,
  deleteDate INT(10) NULL

  KEY (deleteDate, autoDelete, charID, groupID, typeID)
) ;

ALTER TABLE wcf1_gman_pointtrans ADD FOREIGN KEY (issuerID) REFERENCES wcf1_user (userID) ON DELETE SET NULL;
ALTER TABLE wcf1_gman_pointtrans ADD FOREIGN KEY (eventID) REFERENCES calendar1_event (eventID) ON DELETE SET NULL;
ALTER TABLE wcf1_gman_pointtrans ADD FOREIGN KEY (charID) REFERENCES wcf1_gman_wow_character (charID) ON DELETE CASCADE;
ALTER TABLE wcf1_gman_pointtrans ADD FOREIGN KEY (typeID) REFERENCES wcf1_gman_pointtype (typeID) ON DELETE RESTRICT;

DROP TABLE IF EXISTS wcf1_gman_wow_classes;
CREATE TABLE wcf1_gman_wow_classes (
  wclassID INT(10) NOT NULL PRIMARY KEY,
  mask INT(10) NOT NULL,
  powerType VARCHAR(15)  NOT NULL ,
  name VARCHAR(20)  NOT NULL ,
  color VARCHAR(12)  NOT NULL 
) ;
INSERT INTO wcf1_gman_wow_classes (wclassID, mask, powerType, name, color) VALUES ('1', '1', 'rage', 'Krieger', '#C79C6E'), ('2', '2', 'mana', 'Paladin', '#F58CBA'), ('3', '4', 'focus', 'Jäger', '#ABD473'), ('4', '8', 'energy', 'Schurke', '#FFF569'), ('5', '16', 'mana', 'Priester', '#FFFFFF'), ('6', '32', 'runic-power', 'Todesritter', '#C41F3B'), ('7', '64', 'mana', 'Schamane', '#0070DE'), ('8', '128', 'mana', 'Magier', '#69CCF0'), ('9', '256', 'mana', 'Hexenmeister', '#9482C9'), ('10', '512', 'energy', 'Mönch', '#00FF96'), ('11', '1024', 'mana', 'Druide', '#FF7D0A'), ('12', '2048', 'fury', 'Dämonenjäger', '#A330C9')

DROP TABLE IF EXISTS wcf1_gman_wow_races;
CREATE TABLE wcf1_gman_wow_races (
  wraceID INT(10) NOT NULL PRIMARY KEY,
  mask INT(10) NOT NULL,
  side VARCHAR(20)  NOT NULL,
  sideID TINYINT(4) NOT NULL,
  name VARCHAR(50)  NOT NULL
) ;
INSERT INTO wcf1_gman_wow_races (wraceID, mask, side, sideID, name) VALUES ('1', '1', 'alliance', '0', 'Mensch'), ('2', '2', 'horde', '1', 'Orc'), ('3', '4', 'alliance', '0', 'Zwerg'), ('4', '8', 'alliance', '0', 'Nachtelf'), ('5', '16', 'horde', '1', 'Untoter'), ('6', '32', 'horde', '1', 'Tauren'), ('7', '64', 'alliance', '0', 'Gnom'), ('8', '128', 'horde', '1', 'Troll'), ('9', '256', 'horde', '1', 'Goblin'), ('10', '512', 'horde', '1', 'Blutelf'), ('11', '1024', 'alliance', '0', 'Draenei'), ('22', '2097152', 'alliance', '0', 'Worgen'), ('24', '8388608', 'neutral', '2', 'Pandaren'), ('25', '16777216', 'alliance', '0', 'Pandaren'), ('26', '33554432', 'horde', '1', 'Pandaren');

DROP TABLE IF EXISTS wcf1_gman_wow_acms;
CREATE TABLE wcf1_gman_wow_acms (
  acmID INT(10) NOT NULL PRIMARY KEY,
  bnetData TEXT NULL ,
  bnetUpdate INT(10) NOT NULL 
) ;

DROP TABLE IF EXISTS wcf1_gman_wow_gacms;
  CREATE TABLE wcf1_gman_wow_gacms (
  gacmID INT(10) NOT NULL PRIMARY KEY,
  bnetData TEXT NULL,
  bnetUpdate INT(10) NOT NULL
) ;

DROP TABLE IF EXISTS wcf1_gman_wow_items;
CREATE TABLE wcf1_gman_wow_items (
  itemID INT(10) NOT NULL PRIMARY KEY,
  bnetData TEXT  NULL,
  bnetUpdate INT(10) NOT NULL
) ;

DROP TABLE IF EXISTS wcf1_gman_wow_itemclass;
CREATE TABLE wcf1_gman_wow_itemclass (
  itemclassID INT(10) NOT NULL PRIMARY KEY,
  bnetData TEXT NULL ,
  bnetUpdate INT(10) NOT NULL 
) ;

DROP TABLE IF EXISTS wcf1_gman_guild_acm;
CREATE TABLE wcf1_gman_guild_acm (
  guildAcmID int(5) NOT NULL PRIMARY KEY,
  gacmID int(5) NOT NULL,
  gacmTime int(10) NOT NULL,
  articelID int(10) NULL
)

DROP TABLE IF EXISTS wcf1_gman_feedlist;
CREATE TABLE wcf1_gman_feedlist (
  type enum('ACHIEVEMENT','CRITERIA','LOOT','BOSSKILL')  NOT NULL ,
  charID VARCHAR(35) NULL,
  itemID INT(10) NULL,
  acmID INT(10) NULL,
  quantity INT(10) NOT NULL,
  bonusLists TEXT NULL,
  context TEXT NULL,
  feedTime INT(10) NOT NULL,
  inGuild TINYINT(1) NOT NULL DEFAULT 1,
  KEY (charID, feedTime)
) ;
ALTER TABLE wcf1_gman_feedlist ADD CONSTRAINT charId_feedTime_UQ  UNIQUE (charID, feedTime);
ALTER TABLE wcf1_gman_feedlist ADD FOREIGN KEY (charID) REFERENCES wcf1_gman_wow_character (charID) ON DELETE CASCADE;

Display More

Vielleicht hat ja jemand den entschiedenen Tipp

Fighter456 · Jan 9th 2017

SQL

DROP TABLE IF EXISTS wcf1_gman_feedlist;
CREATE TABLE wcf1_gman_feedlist (
  type enum('ACHIEVEMENT','CRITERIA','LOOT','BOSSKILL')  NOT NULL ,
  charID VARCHAR(35) NULL,
  itemID INT(10) NULL,
  acmID INT(10) NULL,
  quantity INT(10) NOT NULL,
  bonusLists TEXT NULL,
  context TEXT NULL,
  feedTime INT(10) NOT NULL,
  inGuild TINYINT(1) NOT NULL DEFAULT 1,
  KEY (charID, feedTime),
  UNIQUE KEY(charID, feedTime)
);

Display More

**Alexander Ebert** · Jan 9th 2017

Diese Syntax wird nicht unterstützt, der UNIQUE-Key erhält automatisch einen Namen auf Basis der abdeckten Spalten. Dies funktioniert auch beim Entfernen des Keys.

Im Übrigen nutzen wir das Anlegen von Tabellen um derartige Keys direkt festzulegen, so befindet sich abseits der Foreign Keys alles an einem Ort.

veneanar · Jan 9th 2017

Vielen Dank für die Antworten!

Alexander Ebert Wie heißt dann der erstellte UNIQUE Key? Kann ich diesen im Datenbank Objekt verwenden? Beispiel: der erstellte Key heißt charID_feedTime

PHP

class FeedList extends DatabaseObject {
    /**
     * @inheritDoc
     */
    protected static $databaseTableName = 'gman_feedlist';
    /**
     * @inheritDoc
     */
    protected static $databaseTableIndexName = 'charID_feedTime';

}

Display More

**Alexander Ebert** · Jan 9th 2017

Ich glaube, hier besteht ein Missverständnis: \wcf\data\DatabaseObject::$databaseTableIndexName bezieht sich auf den Namen der Spalte die als primärer Index dient, bei Themen zum Beispiel threadID. Der Name des UNIQUE-Keys ist an der Stelle weder gefragt, noch gültig, auch wenn die Bezeichnung möglicherweise etwas irreführend ist.

Idealerweise solltest du weder Tabellen-Name, noch Index direkt festlegen sondern diese automatisch erzeugen lassen. Dies setzt entsprechende Anpassungen des Namespace und Klassennamens voraus, erlaubt dann aber eine einwandfreie Zuordnung. Wenn du allerdings abweichende Bezeichnungen nutzen willst, dann musst du sie selbst angeben. Die Funktionsweise wird hier erläutert: https://docs.woltlab.com/php_database-objects.html

veneanar · Jan 9th 2017

Ah prima, ich wusste gar nicht, dass ich mir $databaseTableIndexNamesparen kann, wenn ich die Indexes wie den Tabellennamen benenne. Da ich mir alle meine DBOs direkt aus der Datenbank generieren lasse, war mir die zusätzlich "Schreibarbeit" immer egal.

Ich habe aber nun ein weiteres Problem. Ich muss bestimmte columns einen anderen Charset zuordenen. Nur bekomme ich das nicht durch den installer, die zusätzlichen Angaben werde immer ignoriert. Bisher versucht habe ich:

[...] charID VARCHAR(35) NOT NULL COLLATE utf8mb4_bin, [...]

oder

ALTER TABLE wcf1_gman_feedlist CHANGE charID charID VARCHAR(35) NOT NULL COLLATE utf8mb4_bin;

Ein Blick in den Parser verrät auch warum.

Daher habe ich mir den Parser lokal gepatcht weil ich gerne an meinem Plugin weiterentwickeln möchte:

PHP: SQLParsrer.class.php

else if (preg_match("~^ALTER\s+TABLE\s+(\w+)\s+(?:(ADD)\s+(?:COLUMN\s+)?|(CHANGE)\s+(?:COLUMN\s+)?(\w+)\s+)(\w+)\s+(\w+)(?:\s*\((\s*(?:\d+(?:\s*,\s*\d+)?|'[^']*'(?:\s*,\s*'[^']*')*))\s*\))?(?:\s+UNSIGNED)?(?:\s+(NOT NULL|NULL))?(?:\s+DEFAULT\s+(-?\d+.\d+|-?\d+|NULL|'[^'\\\\]*(?:\\\\.[^'\\\\]*)*'))?(?:\s+(AUTO_INCREMENT))?(?:\s+(UNIQUE|PRIMARY)(?: KEY)?)?(?:\s(COLLATE\s+\w+))~is", $query, $match)) {

PHP: SQLParsrer.class.php

if (!empty($match[12])) {
                        if (preg_match('~^COLLATE\s+(\w+)$~', StringUtil::trim($match[12]), $match2)) {
                            $columnData['collation'] = $match2[1];
                        }
                        else {
                                throw new SystemException("Unsupported SQL statement '".$query."'");
                        }
                    }

PHP: MySQLDatabaseEditor.class.php

// coallition
        if (isset($columnData['collation'])) $definition .= " COLLATE ".$columnData['collation'];

Das ist natürlich reichlich kompliziert. Gibt es eine andere Lösung die Kollation zu ändern?

**Alexander Ebert** · Jan 9th 2017

Quote from veneanar

Nur bekomme ich das nicht durch den installer, die zusätzlichen Angaben werde immer ignoriert.

Wozu willst du auf case-sensitive wechseln? Was genau steht denn in charID drin, damit du darauf angewiesen bist, basierend auf dem Tabellennamen würde ich auf den Charakternamen bei WoW tippen, diese ist aber (zu mindestens im Original) case-insensitive.

veneanar · Jan 9th 2017

Quote from veneanar

Ein bekanntes Problem ist aber, dass utf8mb4_unicode_ci keine eindeutige Vergleiche zulässt. Spätestens, wenn die gespeicherten Werte Umlaute oder Akzente verwenden kann SQL die Schlüssel nicht mehr eindeutig identifizieren.

In der Spalte werden Charakternamen gespeichert. Zum Beispiel: Veneanar und Veneanár. Ich will einen neuen Datensatz für Veneanár anlegen, Veneanar ist schon vorhanden. Die Daten aber nicht neu angelegt, da SQL "so was ähnliches" schon findet.

**Tim Düsterhus** · Jan 9th 2017

Hallo,

ich nehme an, dass das direkt aus irgendeiner API raus fällt? Dann nimm VARBINARY, dann konvertiert MySQL nicht daran rum.

veneanar · Jan 9th 2017

Ja die Daten kommen aus der battle.net API im JSON Format und werden per BULK UPDATE/ INSERT importiert.

PHP

 static public function updateGuildMemberList() {
        $url = static::buildURL('guild', 'wow', null, ['members']);
        $request = new HTTPRequest($url);
        $request->execute();
        $reply = $request->getReply();
        if ($reply['statusCode'] != 200) {
            throw new LoggedException('Cannot connect to battle.net: '.$url.' returns: HTTP: ' . $reply['statusCode']);
        }

        $guildmember = JSON::decode($reply['body'], true)['members'];
        $sql = "INSERT INTO  wcf".WCF_N."_gman_wow_character
                            (charID, isMain, inGuild, realmID, bnetData, groups, bnetUpdate, firstSeen, guildRank)
                VALUES      (?,0,1,?,?,0,?,?,?)
                ON DUPLICATE KEY UPDATE
                            inGuild = VALUES(inGuild),
                            bnetData = VALUES(bnetData),
                            bnetUpdate = VALUES(bnetUpdate)
            ";
        $statement = WCF::getDB()->prepareStatement($sql);

        WCF::getDB()->beginTransaction();

        foreach ($guildmember as $member) {
            $member["character"]['lastModified'] = $member["character"]['lastModified'] / 1000;
            $member["character"]["guild"] = null;

            $statement->execute([
                $member["character"]["name"] . "-". $member["character"]["realm"],
                $member["character"]["realm"],
                JSON::encode($member["character"]),
                TIME_NOW,
                TIME_NOW,
                $member["rank"]
           ]);
        }

        WCF::getDB()->commitTransaction();
    }

Display More

An VARBINARYhabe ich gar nicht gedacht, werde ich mal testen. Danke für den Hinweiß. Ich bin schon halb verrückt geworden weil ich gar nicht wusste Umlaute etc. ignoriert werden und auf Stackoverflow wurde die Verwendung von utf8mb4_bin empfohlen.

Ich habe alles auf VARBINARY umgestellt und wie es aussieht funktioniert alles prima. Ein Abend ist zwar drauf gegangen, dafür habe ich

meine REGEX Kenntnisse aufgefrischt
einen besseren Einblick in die API bekommen
und jede Menge Leute verrückt gemacht

Danke an alle für die schnelle Hilfe.

**Tim Düsterhus** · Jan 10th 2017

Hallo,

a) Denke daran, dass du bei VARBINARY die maximale Länge des Feldes vervierfachen musst, um UTF-8 zu speichern.

b) Zeile 6 ist über, wenn die Antwort kein 2xx-Statuscode ist, dann wird ohnehin eine Exception geworfen.

veneanar · Jan 12th 2017

Ich hab die Status Code Prüfung ersetzt:

Code

static public function updateCharacter(array $updateList) {
        $charDataList = [];
        foreach($updateList as $update) {
            $data = explode("-", $update['charID'], 2);
            $url = static::buildURL('character', 'wow', ['char' => $data[0], 'realm' => $data[1]], ['guild', 'items']);
            $request = new HTTPRequest($url);
            try {
                $request->execute();
            }
            catch (\Exception $e) {
                file_put_contents(WCF_DIR . 'log/bnet.log', '*** ERROR *** '. $url . PHP_EOL, FILE_APPEND);
                continue;
            }
            $reply = $request->getReply();
            $charData=JSON::decode($reply['body'], true);
            if ($update['bnetUpdate'] < ($charData['lastModified'] / 1000)) {
                    $charData['charID'] = $update['charID'];
                    $charDataList[] = $charData;
                    static::updatePictures($charData['thumbnail']);
             }
        }
...
}

Display More

Damit die cronjobs nicht deaktiviert werden weil ein Spieler z.B. den Char gelöscht hat oder das battle.net mal ein paar Minuten nicht erreichbar ist. Vor allem will ich in den Logs sehen wo es scheitert, da HTTPRequest im Fehlerfall nicht die URL herausrückt. Bei 400-500 Updates suche ich mir ja 'n Wolf.

Morik · Jan 13th 2017

Wie kommst du darauf dass man die URL nicht bekommt ?

Musst den Fehler nur richtig loggen, dann kann man es sich ganz normal im ACP anschauen:

Code

catch (LoggedException $e) {
            $e->getExceptionID(); // log error
            
            // do something like counting the error char up until it counts as deleted etc
        }

veneanar · Jan 13th 2017

Ja, das war auch mein erster Gedanke, aber:

Quote

Ein Fehler ist aufgetreten

Interner Fehlercode: f28093e2fadbeaae4334f40beb4f1e089b878fcb

Was ist passiert?

Leider ist es bei der Verarbeitung zu einem Fehler gekommen und die Ausführung wurde abgebrochen. Falls möglich, leiten Sie bitte den oben stehenden Fehlercode an den Administrator weiter.

Administratoren können die vollständige Fehlermeldung mit Hilfe dieses Codes in der Administrationsoberfläche unter „Protokoll » Fehler“ einsehen. Zusätzlich wurden die Informationen in die Protokolldatei */log/2017-01-12.txt geschrieben und können beispielsweise mit Hilfe eines FTP-Programms abgerufen werden.

Hinweis: Der Fehlercode wird zufällig generiert, erlaubt keinen Rückschluss auf die Ursache und ist daher für Dritte nutzlos.

System Information

PHP Version:

7.0.14-1~dotdeb+8.1

WoltLab Suite Core:

3.0.0

Peak Memory Usage:

7.339/-0 MiB

Request URI:

Referrer:

User Agent:

Final Error

Error Type:

wcf\system\exception\HTTPNotFoundException

Error Message:

Received status code '404' from server

File:

*/lib/util/HTTPRequest.class.php (504)

Stack Trace:

#0 */lib/util/HTTPRequest.class.php (389):

wcf\util\HTTPRequest->parseReply()

#1 */lib/system/wow/bnetAPI.class.php (122):

wcf\util\HTTPRequest->execute()

#2 */lib/system/cronjob/GuildCharUpdateCronjob.class.php (20):

wcf\system\wow\bnetAPI::updateCharacter([ 432 items ])

#3 */lib/system/cronjob/GuildCharUpdateCronjob.class.php (30):

wcf\system\cronjob\GuildCharUpdateCronjob::doaction()

#4 */../../../root/guildCharUpdate.sh (4):

wcf\system\cronjob\GuildCharUpdateCronjob::directExecute()

Original Error

Error Type:

wcf\util\exception\HTTPException

Error Message:

Received status code '404' from server

Error Code:

404

File:

*/lib/util/HTTPRequest.class.php (504)

Body:

{"status":"nok", "reason": "Character not found."}

Status Code:

404

Stack Trace:

#0 */lib/util/HTTPRequest.class.php (389):

wcf\util\HTTPRequest->parseReply()

#1 */lib/system/wow/bnetAPI.class.php (122):

wcf\util\HTTPRequest->execute()

#2 */lib/system/cronjob/GuildCharUpdateCronjob.class.php (20):

wcf\system\wow\bnetAPI::updateCharacter([ 432 items ])

#3 */lib/system/cronjob/GuildCharUpdateCronjob.class.php (30):

wcf\system\cronjob\GuildCharUpdateCronjob::doaction()

#4 */../../../root/guildCharUpdate.sh (4):

wcf\system\cronjob\GuildCharUpdateCronjob::directExecute()

Display More

Fighter456 · Jan 13th 2017

Dann fange die HTTPNotFoundException.

veneanar · Jan 13th 2017

Danke für den Hinweiß, ich hab es eingebaut. Ich wollte gestern das erste Mal das Plugin packen und richtig testen. Nur gibt es schon wieder ein Problem mit der install.sql:

Während der Installation sollen zwei Tabellen befüllt werden: wcf1_gman_wow_classes und wcf1_gman_wow_races. Er füllt aber nur die *_races, die *_classes bleibt leer:

Code

DROP TABLE IF EXISTS wcf1_gman_wow_classes;
CREATE TABLE wcf1_gman_wow_classes (
  wclassID INT(10) NOT NULL PRIMARY KEY,
  mask INT(10) NOT NULL,
  powerType VARCHAR(15)  NOT NULL ,
  name VARCHAR(20)  NOT NULL ,
  color VARCHAR(12)  NOT NULL 
) ;
INSERT INTO wcf1_gman_wow_classes (wclassID, mask, powerType, name, color) VALUES ('1', '1', 'rage', 'Krieger', '#C79C6E'), ('2', '2', 'mana', 'Paladin', '#F58CBA'), ('3', '4', 'focus', 'Jäger', '#ABD473'), ('4', '8', 'energy', 'Schurke', '#FFF569'), ('5', '16', 'mana', 'Priester', '#FFFFFF'), ('6', '32', 'runic-power', 'Todesritter', '#C41F3B'), ('7', '64', 'mana', 'Schamane', '#0070DE'), ('8', '128', 'mana', 'Magier', '#69CCF0'), ('9', '256', 'mana', 'Hexenmeister', '#9482C9'), ('10', '512', 'energy', 'Mönch', '#00FF96'), ('11', '1024', 'mana', 'Druide', '#FF7D0A'), ('12', '2048', 'fury', 'Dämonenjäger', '#A330C9')

DROP TABLE IF EXISTS wcf1_gman_wow_races;
CREATE TABLE wcf1_gman_wow_races (
  wraceID INT(10) NOT NULL PRIMARY KEY,
  mask INT(10) NOT NULL,
  side VARCHAR(20)  NOT NULL,
  sideID TINYINT(4) NOT NULL,
  name VARCHAR(50)  NOT NULL
) ;
INSERT INTO wcf1_gman_wow_races (wraceID, mask, side, sideID, name) VALUES ('1', '1', 'alliance', '0', 'Mensch'), ('2', '2', 'horde', '1', 'Orc'), ('3', '4', 'alliance', '0', 'Zwerg'), ('4', '8', 'alliance', '0', 'Nachtelf'), ('5', '16', 'horde', '1', 'Untoter'), ('6', '32', 'horde', '1', 'Tauren'), ('7', '64', 'alliance', '0', 'Gnom'), ('8', '128', 'horde', '1', 'Troll'), ('9', '256', 'horde', '1', 'Goblin'), ('10', '512', 'horde', '1', 'Blutelf'), ('11', '1024', 'alliance', '0', 'Draenei'), ('22', '2097152', 'alliance', '0', 'Worgen'), ('24', '8388608', 'neutral', '2', 'Pandaren'), ('25', '16777216', 'alliance', '0', 'Pandaren'), ('26', '33554432', 'horde', '1', 'Pandaren');

Display More

Während der Installation wird kein Fehler angezeigt. Die komplette install.sql findet man hier: https://github.com/Veneanar/in…master/plugin/install.sql

**Alexander Ebert** · Jan 13th 2017

Das Semikolon am Ende von Zeile 9 fehlt.

install.sql; ADD CONSTRAINT schlägt fehl.

Participate now!

Share