search

SlicewOw / webSPELL_Cups

Cup add-on for webSPELL NOR
GNU General Public License v3.0
4 stars 2 forks source link

Gameaccount module: Game is not saved #75

Closed renegade1337 closed 5 years ago

renegade1337 commented 5 years ago

Wenn ich versuche unter index.php?site=gameaccount einen CS:GO Gameaccount einzutragen dann wird zwar der Wert gespreichert allerdings das Game nicht! unbenannt-3

SlicewOw commented 5 years ago

Funktioniert es nur mit CS:GO nicht oder mit allen Spielen?

renegade1337 commented 5 years ago

Bei LOL ging es, bei CS:GO nicht!

renegade1337 commented 5 years ago

Ich Teste das mit einer schon vorhandenen Datenbank bevor das Cupaddon installiert wurde. Das Problem besteht wohl darin das du das Games Module umgeschrieben hast. Und zwar nicht auf Basis von der aktuellen webSPELL NOR Version. Dort wurden nämlich die Game kürzel verlängert damit auch soetwas wie cod:bo4 oder auch cs:go möglich ist. Außerdem wurden die Gameicons soweit verändert das auch png´s zulässig sind.

SlicewOw commented 5 years ago

Demnach hast du bei dir in der Datenbank für CS:GO "csgo" als Tag eingetragen? Dann ist die Lösung relativ simpel.

renegade1337 commented 5 years ago

cs:go steht drin. Und die Gameicons sind alle png's

SlicewOw commented 5 years ago

Bitte pflege die Änderungen in der install-cup-addon.php bei dir nach. Dann sollte das Problem behoben sein - magst du das einmal validieren?

Sehe gerade, dass die Einrückungen die Änderungen überschatten:

Folgende Tabellen müssen angepasst werden:

Jeweils den Wert von 3 auf 10 erhöhen.

renegade1337 commented 5 years ago

Das hinzufügen von einem Gameaccount hat jetzt geklappt! Im Admincenter werden allerdings die Icons im Game Module noch nicht richtig dargestellt. unbenannt-4 Das csgo icon liegt mit folgender bennenung im images/games/ Ordner: cs_go.png

SlicewOw commented 5 years ago

Das Problem mit dem Icon hatte ich in diesem Commit https://github.com/SlicewOw/webSPELL_Cups/commit/07fa6e86e823be484931253c7f264554b2f5c9c6 behoben. Hattest du den schon ausgechecked?

renegade1337 commented 5 years ago

Habe ich so schon drin, die anderen Icons auf dem Screnshot sind auch png´s diese werden ausgegeben nur die nicht wo ein noppelpunkt im gametag sind. Gespeichert sind die Bilder aber mit einem untersprich, also cs_go.png.

SlicewOw commented 5 years ago

Bin aktuell noch nicht weitergekommen - Stand heute: Es wird eine Datei ohne Endung hochgeladen mit dem Namen, der vor dem ":" steht. Am Beispiel "cs:go" wird eine Datei erstellt mit dem Namen "cs" (ohne Dateityp). Warum? Keine Ahnung.

renegade1337 commented 5 years ago

Hast du dir das ganze mal im Original webSPELL NOR 1.2.5 angeschaut? Dort wurde es ja so umgesetzt.

SlicewOw commented 5 years ago

Siehe Commit: https://github.com/SlicewOw/webSPELL_Cups/commit/e1471919dfd360337e7ae21ecd349445d92b231b

Ich habe es nicht hinbekommen und da ich bei meinem Add-On diese Spalte anders verwende und einen "Anzeige"-Tag hinzugefügt habe (z.B. "CS:GO"), belasse ich es erstmal dabei. Mit dem Commit ist ein Error-Handling hinzugekommen, wenn also ein ":" im Tag ist, wird ein Fehler angezeigt.