search

renao / sams-wordpress-plugin

WordPress plugin to integrate DVV (SAMS) data into our clubs website.
Other
3 stars 2 forks source link

SAMS Schnittstelle liefert fehlerhafte Daten und Abruf der Bundesliga Daten funktioniert nicht #8

Open exile64 opened 1 month ago

exile64 commented 1 month ago

Auf diesem Weg mal ein dickes Danke für das Plugin. Ich habe ewig nach solch einer Lösung gesucht seit dem das web-splugin mehr oder weniger eingestellt wurde. DANKE

Kleines Problem habe ich. Bei unserer BVV Liga Abfrage steht bei allen Tabellenplätzen "Rang 1". Ist das ein SAMS bzw Abfragefehler oder ähnliches oder liegts evtl. am Plugin?

BG

renao commented 1 month ago

Hi, danke für den Hinweis.

Ich habe mal unsere Webseite gecheckt und dort tritt aktuell auch dieser Fehler auf.

Daraufhin habe ich bei Abruf der Rohdaten entdeckt, dass die Schnittstelle für alle Teams einen Eintrag "1" liefert.

Ich versuche mal herauszufinden, wo ich einen Bugreport einstellen kann und melde mich sonst nochmal zurück, wenn ich was neues erfahre.

exile64 commented 1 month ago

Falls du auch weißt wieso ich keine Daten der 2. BL bekomme nehme ich gerne Tipps an. Für die regionalen Verbände klappt es super, nur VBL will irgendwie nicht :-(

PS: Um evtl. die Frage vorweg zu greifen, Ja, habe 2 verschiedene API Keys für den Einsatz :D

renao commented 1 month ago

Ich hatte gesehen, dass sich mittlerweile auch ein paar der Basis-URLs geändert haben.

Wenn es die Zeit hergibt mache ich diese Konfiguration anpassbar, denn mittlerweile wird auch bei uns auf Landesebene das SAMS-System wohl flächendeckend eingesetzt.

Allerdings habe ich gerade bei meinen Tests gegen die Basis-URL dvv-ligen.de/xml/... regelmäßig die Rückmeldung bekommen, dass das Server-Zertifikat nicht gültig sei. Vielleicht macht das auch gerade noch ungewollte Probleme.

Ich denke ich werde da morgen nochmal genauer reinsehen können 👍

exile64 commented 1 month ago

Das wäre spitze! Es ist wirklich sehr hilfreich das Plugin um einfach sich die Daten über die API zu ziehen. Klasse Arbeit! Danke nochmals an dieser Stelle 👍

renao commented 1 month ago

Hi @exile64

wie bisher immer gibt es keine Rückmeldungen von dem "Support" des Herstellers - ich denke wir müssen wohl oder übel erstmal davon ausgehen, dass die Platzierungen ggf. nicht in nächste Zeit gefixt werden. Sollte ich was hören - wie bereits schon gesagt - geb ich das bekannt.

Ansonsten plane ich gerade das Plugin nochmal "fit für 2024/2025" zu machen - sollte alles wie gewünscht funktionieren, würde ich auch zusehen, dass das Plugin in der WordPress Plugin Bibliothek aufgenommen wird und somit der ganze Aufwand zur Installation nicht mehr so weit vom WordPress Standard abweicht.

Ich mach mir mal ein paar Gedanken und aktualisiere dann hier nach und nach das Repository/Projekt auf GitHub 👍

exile64 commented 1 month ago

hey @renao ja, leider kenne ich das Problem teilweise gut mit der IT dort :-) Aber ja, ich warte gerne, und wenn das in die WP Plugin Bibliothek integriert wird, wäre das super und würde das etwas einfacher/schneller machen. Top!

Hast du evtl. noch einen Hinweis für mich oder Tipp wieso ich die VBL Daten nicht abrufen kann, den regionalen Verband aber schon?

Habe einmal den API-Key vom BVV sowie einen der VBL.

renao commented 1 month ago

Kannst du mir die konkreten Ligen einmal nennen? Dann kann ich das entsprechend einmal selbst ausprobieren.

exile64 commented 1 month ago

funktioniert bei: Brandenburgischer Volleyballverband (BVV)

funktioniert nicht bei:

  1. Bundesliga Nord (VBL)

Ich schaue parrallel auch nochmal.

EDIT/Ergänzung Shortcode: [samstable apikey="**" matchseriesid="770581327"] (sollte eigentlich die passende ID sein)

renao commented 1 month ago

Ja, klingt in der Verwendung der Schnittstelle erstmal gut.

Aus irgendwelchen Gründen hat der Hersteller scheinbar unterschiedliche Basis-Adressen je nach Verband/Liga angelegt. Dafür muss ich mir mal eine gute Lösung ausdenken - könntest du sonst zwischenzeitlich mal folgende URL aufrufen:

https://www.volleyball-bundesliga.de/xml/rankings.xhtml?matchSeriesId=770581327&apiKey=**** Natürlich mit deinem API Key :-)

Kriegst du da eine XML-Datei zurück oder eine Fehlermeldung?

Nachtrag: hier auch nochmal die Stelle aus der Dokumentation zur Schnittstelle, die ich dafür nutze: http://wiki.sams-server.de/wiki/XML-Schnittstelle#Abfragen

exile64 commented 1 month ago

@renao Ja, die xml Abfrage funktioniert. Bekomme die rankings via XML ausgegeben.

renao commented 1 month ago

Alles klar, dann haben wir das Problem schonmal erfasst.

Ich denke mir eine praktikable Lösung aus - vielleicht auch eine, bei der ich nicht von Hand die ganzen URLs pflegen muss oder man jedes mal angeben muss, von welcher Basisadresse die Daten geholt werden sollen.

Der Hersteller macht es mir gerade nicht sehr einfach 😬

exile64 commented 1 month ago

Kann ich mir gut vorstellen. Verstehe eh nicht wieso es für 3. so "unfreundlich" gemacht wird die Tabellen etc. einbinden zu können. Ich denke auch das eine manuelle Eingabe der erforderlichen Liga der beste Weg ist. Das sollte ja auch für den Nutzer des Plugins noch machbar sein diese herauszufinden ^^ Vielleicht dann durch verschiedene Shortcodes realisieren die je nach Konfiguration (also Ligaadresse) dann erstellt werden. So könnte ich mir das als Laie vorstellen :D

renao commented 1 month ago

Also beispielhaft dann: [bvv-ranking ...] , [vbl-ranking ...] usw.? 🤔

exile64 commented 1 month ago

Dieser Verein hier hat auch meines Erachtens nach selbst ein Plugin geschrieben oder schreiben lassen. Vielleicht hilft dir das in irgendeiner Art weiter.

https://www.vc-stralsund.de/wildcats/spielplan/

exile64 commented 1 month ago

Also beispielhaft dann: [bvv-ranking ...] , [vbl-ranking ...] usw.? 🤔

genau. Wäre auch für dich (bis man evtl. etwas anderes machen kann) eine machbare Lösung die ich mir vorstellen könnte.

renao commented 1 month ago

Guck ich mir mal an.

Finde die Idee mit den eigenen Shortcodes für die Verbände/Ligen ganz gut. Erspart man dem Nutzer technischen Schnickschnack - finde das jetzt schon relativ "nutzerunfreundlich" diese blöden IDs rauszusuchen und anzugeben.

Habt ihr für diese matchSeriesIdseigentlich eine aktuelle Übersicht? Beim WVV interessiert das leider niemanden und auch dort laufen meine Nachfragen ins Nirvana.

Wenn ich die Daten nämlich irgendwo einfach "einsammeln" könnte, dann könnte hoffentlich auch irgendwann einfach diese blöde ID rausfliegen - aber das sind Schritt 4 und 5 aus heutiger Sicht.

Ich schaue mal was einen Fix bzgl. der URL anbelangt.

P.S.: VC Stralsund? Ich glaub meine Frau hat gegen die damals sogar gespielt 😁

exile64 commented 1 month ago

Habt ihr für diese matchSeriesIdseigentlich eine aktuelle Übersicht? Beim WVV interessiert das leider niemanden und auch dort laufen meine Nachfragen ins Nirvana.

Ich zieh mir die auch nur über die XML Datei oder im besten Falle einfach über die URL der jeweiligen Liga in der Tabelle bzw Mannschaftsseite (oder aus der ical url 😆).

Eine alternative wäre auch https://volleyball-ergebnisdienst.de/main/index.html Aber finde deine via Plugin deutlich angenehmer.