Import Pressebericht erzeugt weiße Seite

[svdoldie]

Wenn man einen Pressebericht hochladen möchte erscheint eine Weiße Seite mit folgender Fehlermeldung:

Notice: Use of undefined constant Spielberichtsnummer - assumed 'Spielberichtsnummer' in /var/www/vhosts/svdoerpum.de/svdoerpum.de/administrator/components/com_sportsmanagement/models/match.php on line 2673

Warning: Creating default object from empty value in /var/www/vhosts/svdoerpum.de/svdoerpum.de/administrator/components/com_sportsmanagement/models/match.php on line 2758

Catchable fatal error: Object of class JDatabaseDriverMysqli could not be converted to string in /var/www/vhosts/svdoerpum.de/svdoerpum.de/administrator/components/com_sportsmanagement/models/match.php on line 2782

[tdittmann]

Habe bei mir auch mal einen Import versucht. Die Meldung "Spielnummer ist identisch" wird angezeigt, darunter eine Tabelle mit den folgenden Spalten:

Spielnummer Vorname Nachname In der Datenbank ? Projektteam zugeordnet ? Projektposition Staff Position Vorname Nachname In der Datenbank ? Projektteam zugeordnet ? Projektposition Spieler Minute R�ckennummer f�r Nummer f�r Spieler Projektposition Spieler Minute Karte R�ckennummer Grund In der Datenbank ? Event

Wenn ich anschließend auf den Save-Button klicke kommt bei mir folgende Fehlermeldung:

Layout „savepressebericht_3“ nicht gefunden!

Hinweis: bei mir fehlen z.B. alle Joomla-Texte, wie "COM_SPORTSMANAGEMENT_ADMIN_PRESSEBERICHT_IMPORT_TABLE_TITLE_1"

[diddipoeler]

Kannst du bitte mal eine Datei mit einem Pressebericht hier anhängen ? Ich arbeite nämlich nicht mit den Presseberichten.

[tdittmann]

Natürlich, habe es als .txt hochgeladen, da csv-Dateien wohl von GitHub nicht unterstützt werden

Pressebericht.txt

[svdoldie]

aktuell tritt folgende Fehlermeldung auf: 1146 Table 'sk13-121152-1_svdoerpumde.#__sportsmanagement' doesn't exist

[diddipoeler]

Kannst Du mir mal bitte eine Datei per Mail zusenden ?

[svdoldie]

Ist unterwegs ;-)

[svdoldie]

ich habe es gerade nochmal versucht. Wenn man den Pressebericht hochlädt, läuft er in einen Time-out Fehler

[diddipoeler]

Da muss ich noch einiges programmieren !

[svdoldie]

Sieht doch schon vielversprechend aus :-)

[diddipoeler]

Da stehen sogar die Torschützen: http://www.kicker.de/news/fussball/amateure/schleswig-holstein-vl-nord-west-fb-1/2017-18/19/4168854/spielschema_sc-weiche-flensburg-08-iii-53216_sv-doerpum-14392.html

[svdoldie]

Das ist richtig. Bei Fussball.de gibt's die auch, aber eben nicht in der Exportdatei

[svdoldie]

Es gibt auch eine Pressebericht.pdf, da stehen auch die Torschützen mit drauf. Wenn man die vorher in eine excel-Datei umwandelt, eignet die sich evtl auch zum import Pressebericht2-040067008.pdf Pressebericht2-040067008.xlsx

[diddipoeler]

Leider kann ich in der Datei des Presseberichtes nicht erkennen, wer in der Startaufstellung steht.

[svdoldie]

in der Exceltabelle?

[appukonrad]

maybe it could be interesting unofficial-fussball-de-api https://github.com/mauricewipf/unofficial-fussball-de-api

[diddipoeler]

Im Pressebericht !

[svdoldie]

Verstehe ich jetzt nicht. Ging vor kurzem noch, sogar mit richtiger Positionszuordnung

[tdittmann]

Wenn ich mir den Pressebericht von mir anschaue, kann man an der Spaltenüberschrift sehen, welcher Spieler in der Startaufstellung ist und welcher Auswechselspieler ist. Eingelesen werden sie ja auch separat, vielleicht einfach ein weiteres Flag zum Spieler hinzufügen, ob dieser in der Startelf ist oder nicht?

Hab hierfür einen PR aufgemacht: #836

Funktioniert bei euch der Import von Auswechslungen oder Karten? Bei mir zumindest nicht und ich finde auch in der savePressebericht nicht die notwendigen Codezeilen... wurde das schon implementiert?

[svdoldie]

Moin. Ich hab mal ein bisschen getestet. Sieht schon super aus :+1: Die Auswechselungen werden allerdings noch nicht gespeichert. Bei den Ereignissen frage ich mich ob man die nicht automatisch zuweisen kann, da Ereignis und die dazugehörige Person schon ausgelesen werden. Zum anderen eignet sich der Import prima um Spieler einer Mannschaft zuzuordnen, wenn sie dieser noch nicht zugehören. Dabei wäre es aber schön, wenn auch die gewählte Position aus der Startaufstellung übernommen wird.

[tdittmann]

Morgen, sehr gut ;) Genau die Auswechslungen hab ich noch nicht gemacht, das schau ich mir als nächstes an. Die Ereignisse automatisch zuordnen? Du meinst, wenn im Spielbericht "Gelbe-Karte" steht und wir ein solches Ereignis auch haben, dann das automatisch auswählen? Könnte man sich anschauen :wink: Die Spieler werden doch aktuell schon einem Team zugeordnet oder? Und die Position ist leider aus dem Pressebericht nicht korrekt auszulesen oder? Außer der erste Spieler ist immer TW, die nächsten 4 Abwehr, die nächsten 4 Mittelfeld und die letzten 2 Stürmer... wobei hier das System mit 4-4-2 dann vorgegeben ist :(

[svdoldie]

Mit den Spielern verhält es sich folgendermaßen: Sind dem Team eines Projektes noch keine Spieler zugeordnet, bzw. im Pressebericht ist einer, der noch nicht dem Team zugeordnet wurde, werden diese bzw. derjenige beim Import dem Team zugeordnet. Sehr gute Sache :+1: Wenn diese Spieler in der Startaufstellung stehen und "neu" sind, könnte man ihnen die gewählte Position beim zuordnen des Team gleich mit übergeben. Ich hoffe nun ist es verständlicher

[svdoldie]

Noch ein kleiner Nachtrag: die Spieler, die nicht in der Startaufstellung stehen werden auch dem Team zugewiesen, sofern sie noch nicht vorhanden waren

[tdittmann]

Ich glaub ich habs verstanden. Ich habe jetzt bspw. einen Spieler noch nicht dem Team zugeordnet, der dann im Pressebericht drinsteht. Wenn ich aktuell den Pressebericht importiere sieht es wie folgt aus:

Zukünftig soll dann die Position drin stehen, die die Person hat, richtig?

PS: PR für Auswechslungen ist da

[svdoldie]

Nein, nicht ganz. Die Person soll die an dieser Stelle ausgewählte Position für das Projekt erhalten

[svdoldie]

Das mit den Auswechslungen funktioniert top 👍 Der Stand mit den Positionen ist folgender: Die Positionen werden im Spielbericht richtig gespeichert. In der Kaderübersicht nur beim Torhüter. Angreifer werden in der Kaderübersicht angelegt, allerdings ohne Position. Sind Spieler bereits im Kader gespeichert, werden sie durch den Presseimport nochmal mit falschen Positionen angelegt. Bei den Torhüter und Angreifern tritt das nicht auf.

[svdoldie]

Staff klappt jetzt auch :+1:

[tdittmann]

Schiedsrichter sollten nun auch funktionieren

[svdoldie]

Schiedsrichter importieren klappt :+1:

[svdoldie]

Nochmal zum Anlegen von Spielern einer Mannschaft durch den Pressebericht. Ich habe nochmal eine neue Testinstallation angelegt, nur das ich diemal vor Import eines Presseberichts die Positionen in der richtigen Reihenfolge gespeichert habe. Keine Spieler +Staff sind dem Team zugeordnet. Die zugeorneten Positionen sind Torhüter, Abwehrspieler, Mittelfeldspieler, Angreifer, Trainer und Co-Trainer. Nach dem ersten Import eines Pressebericht sieht es folgendermaßen aus: Die Personen werden alle dem Projektteam zugeordnet, und die Positionen stimmen auch, bis auf die Abwehrspieler, die sind den Mittelfeldspielern zugordnet. Allerdings muss man dann noch in der Kaderübersicht alle Spieler speichern, damit die Positionen "richtig" zugeordnet werden. Im Spielbericht ist alles richtig gespeichert!!!

[svdoldie]

Importiert man dann den nächsten Pressebericht, werden alle neuen Spieler angelegt. das kann man im Spielbericht(Frontend) überprüfen. Allerdings wird nun keinen Spieler mehr in der Kaderübersicht (Backend) angzeigt, Im Frontend schon. Da aber die Positionen im Backend zwar zugeordnet werden, nun aber nicht mehr gespeichert werden können, sind sie in der Kaderanzeige (Frontend) ohne Position angezeigt. Zudem werden die als Abwehrspieler zugeordneten Spieler aus dem Pressebericht erneut als Mittelfeldspieler angelegt. Beim Staff gibt es diese ganze Problematik auch.

[tdittmann]

Jetzt konnte ich es nachstellen. Ich habe immer mit meinem ersten Projekt getestet, da waren zufällig die Position-IDs identisch mit den Projekt-Position-IDs... ich glaub da hab ich etwas durcheinander gebracht. Das muss ich mir dann nochmal genauer anschauen.

[tdittmann]

Wenn der PR durch ist bitte mit den Schiedsrichtern nochmal testen.

[svdoldie]

Geht einwandfrei mit den Schiedsrichtern :+1:

[tdittmann]

Perfekt, dann schau ich mir die Tage nochmals Spieler + Staff an

[svdoldie]

Hört sich gut an :+1: :-)

[tdittmann]

Spieler + Staff bitte nochmal testen, sobald der PR gemergt wurde. Außerdem wird ab sofort auch der Kapitän gespeichert

[diddipoeler]

PR wurde gemerged.

[svdoldie]

Situation nach dem ersten Import: Alle Spieler sowie Staff werden inkl. Position der Mannschaft richtig zugeordnet !!! 👍 Im Spielbericht wird der Staff allerdings mit den falschen Positionen angezeigt, das führt in meinem Fall dazu, dass der Trainer gar nicht angezeigt wird. Siehe Spielbericht https://svdoldie.stony-sport4u.de/index.php?option=com_sportsmanagement&view=matchreport&cfg_which_database=0&s=1&p=1:verbandsliga-nord-18-19&mid=7:sv-doerpum_tus-collegia-juebek&Itemid=115 und Kader https://svdoldie.stony-sport4u.de/index.php?option=com_sportsmanagement&view=roster&cfg_which_database=0&s=1:18-19&p=1:verbandsliga-nord-18-19&tid=13:sv-doerpum&Itemid=272

Nun werde ich den 2. Pressebericht importieren...

[svdoldie]

Nach dem 2. Import sieht es genauso aus wie nach dem ersten 👍 Kader ist im Backend weiterhin sichtbar und es wird nichts doppelt angelegt. Die Importmaske sieht jetzt auch übersichtlicher aus, aber das ist vielleicht auch Geschmacksache ;-) Wie oben erwähnt ist damit nur noch die Zuordnung des Staffs im Spielbericht noch nicht in Ordnung

[tdittmann]

Werden alle Staffs mit der falschen Position gespeichert? Bei mir konnte ich das Problem bisher nicht nachstellen. Ich habe nur das Problem, dass die Einwechslungen erst beim zweiten Import funktionieren (wenn man eine leere Installation hat)

[svdoldie]

Die Staffs werden nur !!! im Spielbericht falsch angezeigt. In der Anzeige im Backend, sowie in der Kaderanzeige im Frontend ist alles korrekt!!! Siehe obige Links, wobei der Trainer gar nicht angezeigt wird.

Das mit den Einwechselungen hatte ich dann auch. Ich hatte bisher vermutet, ich hatte das beim ersten Spiel (= 1. Import) einfach vergessen

[tdittmann]

Sorry, hatte ich falsch gelesen. Für die Einwechslungen habe ich bereits einen Fix

[svdoldie]

Prima :+1:

Darf ich dich noch an die Ereignisse erinnern? ;-)

[tdittmann]

Wenn du die automatische Zuweisung meinst, dann natürlich gerne 👍

• [x] Spielbericht-View: Staffs werden falsch angezeigt
• [x] Automatische Selektierung eines Ereignisses im Pressebericht

[svdoldie]

die meinte ich ;-) Übrigens tolle Arbeit !!! :+1:

[tdittmann]

Automatische Selektierung sollte nun auch funktionieren. Die Events sollten aktuell so heißen:

• Gelbe Karte
• Gelb-Rote Karte
• Rote Karte

Zusätzlich werden auch Rote Karten eingearbeitet, die hatten noch gefehlt 👍

Bei den Staffs habe ich die falschen Position-IDs verwendet. Jetzt sollte es passen

[svdoldie]

Das mit den den Staffs funktioniert jetzt :+1: Die automatische Selektierung der Ereignisse funktioniert bei mir nicht, siehe Bild

Zudem ist jetzt das Modalfenster des Import ziemlich klein geworden, siehe Bild

[tdittmann]

Wie heißen deine Ereignisse? Das Modal ist mir auch aufgefallen, aber da hab ich eigentlich nichts dran gemacht :(

[svdoldie]

Die Ereignisse haben die Standardnamen von JSM.

[tdittmann]

Ok, dann muss ich mal schauen wie die sind und das nochmal abändern. Ich war mir nicht mehr sicher ob ich sie selbst angelegt habe oder anlegen lassen 👍