Hi @diddiausde
welche PHP Version wird genau genutzt? Die letzte stable Version kann sich auf 5.6, 7.0, 7.1 oder 7.2 beziehen, dann würde ich das Problem mal selbst nachstellen, in der Hoffnung den selben Fehler zu erhalten.
In der Sidebar auf der Startseite stehen noch diverse andere notices, mit welchem Shortcode wird diese Ausgabe erzeugt?
Grüße vom Felix
Heyho,
der lokale Server läuft auf einer RasberryPi mit WP 4.94 auf einem Apache 2.4.25
und PHP 7.0.27-0+deb9u1. In der Seitenleiste auf der Startseite wird nur folgender Shortcode angewendet:
[sishandball marked=”Dessau-Rosslauer HV 06″ hide_cols=”4,5,6,8″ type=”standings” limit=”5″ league=”001518000000000000000000000000000002000″]
Dieser erzeugt alleine diese Latte an Fehler auf der Startseite.
grüßchen
Heyho,
gibt’s etwas neues? 🙂 Ich will bei einem kostenlosen und wahnsinnig hilfreichen Plugin nicht nerven. Muss ansonsten einfach mal selbst mit meinen bescheidenen php-fähigkeiten nach dem Problem suchen.
lg
Stephan
Hi @diddiausde
die PHP notices sollten im Livesystem am besten deaktiviert werden, bzgl. der non-static calls setze ich mich an ein Update, dieser Fehler ist klar wird aber in der Regel auch von PHP ignoriert.
Grüße vom Felix
Natürlich kannst du gerne mal selbst in den Code gehen und versuchen die Fehler zu korrigieren, wenn du da weiter kommst würde ich dein Debugging mit ins Update einfließen lassen.
Heyho,
also ich hab es gelöst indem ich die meisten functions in der class-sis-handball-admin.php u. class-sis-handball-admin-viewhelpers.php auf static gesetzt hab.
die Php-Notices kamen aus der php.ini … waren also eh zu vernachlässigen, wie du schon sagtest
Merci erstmal 😀
Hi @diddiausde
sehr cool, ich werde zusehen die Funktionen im gesamten Plugin dies hingehend anzupassen und das Plugin zu updaten, damit auch andere dieses Problem nicht mehr haben.
Speichere dir auf jeden Fall mal deine komplett angepasste Version lokal ab, dann kannst du das Update (sobald verfügbar) einspielen, in der Hoffnung das die Seite dann auch mit der “offiziellen” Version funktioniert (sonst kannst du keine zukünftigen Updates mehr machen, was ziemlich blöd wäre).
Schön das es klappt ;D und ein schönes Wochenende
Grüße vom Felix
Das Problem wurde mit Version 1.0.31 gelöst
Heyho,
lief alles zur vollsten Zufriedenheit. Habe leider bereits das Nächste festgestellt. Oh man sorry. Da fehlen mir jetzt die JS-Kenntnisse, aber vielleicht bekomme ich das auch noch hin 😀
Das Problem besteht dann, wenn man mehrere Ansetzungsfelder auf einer Seite hat:
Ich habe einmal “absolvierte Spiele” und “kommende spiele”. wenn man die anzeige der spiele limitiert hat, dann reagiert der “alle anzeigen”-link auf alle Ansetzungsfelder.
Das kommt wohl aus der “sis-handball-public.js”.
$(this).parents().find(‘.sis-limit-hidden’).show();
das ist wohl zu allgemein und wirkt sich dann eben auf alle Felder aus.
@diddiausde
🤪wie dämlich, ich fixe das.
Grüße vom Felix
Tatsächlich nur eine Zeile, damit sollte es nun klappen.
https://plugins.trac.wordpress.org/changeset/1841476
schön läuft 🙂 Herzlichen Dank 😉
Mir ist noch was rein praktisches aufgefallen. Ich muss um meine Tabellen bzw. Listen noch extra Div’s ziehen, um sie stylen zu können. (aufsteiger, absteiger, relegation etc.) Du gibst zwar ‘ne class mit, aber die ist halt leider viel zu genau. und der parent-selector ist wieder zu ungenau.
also bei einer tabelle ist das jetzt so:
.sis-league-id-001518000000000000000000000000000002000
Diese Klasse ändert sich in der nächsten Saison, nicht aber das styling.
würde man einer tabelle eine klasse selbst mitgeben können, könnte man auch saisonübergreifend die css beibehalten. oder tabellen mit gleicher mannschaftsanzahl bzw auf- absteigerzahl gleich stylen können…
etwas umständlich formuliert. vielleicht verstehst du was ich meine 😀
… das hat aber alles Zeit. Vielleicht kommst du irgendwann mal wieder dazu.
Bis dahin erstmal vielen Dank und weiterhin gutes Gelingen.
Stephan 🙂
@diddiausde
Gute Idee, einfach im Shortcode eine Klasse mitgeben zu können macht Sinn. Ich implementiere das nächste Woche, das ist kein großer Akt.
Schönes Wochenende,
Grüße vom Felix
-
This reply was modified 6 years, 1 month ago by
Felix W..
@diddiausde
Ich habe in Version 1.0.33 solch eine Option eingebaut, das brauchte ich sowieso schon mal und mit der aktuellen Umsetzung macht es denke ich Sinn solche Feinheiten (die vor allem auch nicht jeder nutzt) in einer “Advanced Option” auszulagern.
Der Shortcode kann um additional_params erweitert werden, dort kann die extra Klasse mit class:'my-class' angegeben werden, siehe:
[sishandball type="xxx" league="xxx" additional_params="foo:'bar'|foo:'bar'|foo:'bar'"]
Alle additional_params optionen findest du auch in den FAQs, schaus dir einfach mal an, ich denke damit sollte deine Problematik umzusetzen sein.
Grüße vom Felix
