WBCE CMS – Way Better Content Editing.
Du bist nicht angemeldet.
Ich habe gerade vorhin (halbe Stunde) die Version auf Github heruntergeladen und auf site-check.cc eingespielt.
Ich sehe da immer noch ziemlich viel rot...
Ja stimmt, da die Position der var noch nicht stimmt, sie muss vor der file kommen.
Offline
Zur Info: Ich hatte das vorhin mit den Templates artfish_slide, aeromstig und daflagga gecheckt und da waren keine Probleme aufgetreten. Das hatte mich zu meiner Aussage "getestet, funktioniert" verleitet.
Code allein macht nicht glücklich. Jetzt spenden!
Offline
Ich habe gerade vorhin (halbe Stunde) die Version auf Github heruntergeladen und auf site-check.cc eingespielt.
Ich sehe da immer noch ziemlich viel rot...
Wird die simplepagehead funktion auf site-check.cc verwendet?
Die Version von GitHub oder eine andere?
Christian
“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale
Offline
Dürfte schon stimmen, vermute eher dass das meta property="og:image" an der falschen Stelle steht, da der Core sich ja nach dem letzten Meta ausrichtet.
Ich würde daher sagen die Probleme haben wir gefixed nur ganz ohne Template Anpassung geht es nicht.
Beitrag geändert von colinax (17.07.2019 10:22:12)
Offline
florian
Dürfete schon stimmen vermute eher dass das meta property="og:image" an der falschen Stelle steht, da der Core sich ja nach dem letzten Meta ausrichtet.
Ich würde daher sagen die Probleme haben wir gefixed nur ganz ohne Template Anpassung geht es nicht.
Nicht unbedingt.
Denn, wenn das neueste simplepagehead drauf ist, setzt es die Platzhalter und es werden keine mehr (nach dem letzten META) gesetzt.
“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale
Offline
Woran erkenne ich denn, ob das jetzt wirklich die letzte Version war?
Schau bitte in das modules/simplepagehead/include.php
ob die Datei diese Zeilen enthält:
https://github.com/WBCE/WBCE_CMS/blob/d … #L190-L193
“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale
Offline
Nein, ist nicht drin.
ICh schau später, was es da hat. Bin beim Kochen...
Die Änderung wurde gestern erst von Martin@mrbaseman eingespielt.
Also ist noch nicht im RC1 drin.
Vielleicht sollten wir einen RC2 nochmal rausjagen? @florian, @mrbaseman, @colinax
Beitrag geändert von stefanek (17.07.2019 11:16:53)
“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale
Offline
Naja RC1 ist geade 4 Tage alt und wir wissen ja nicht welche Version Chio verwendet hat.
Beitrag geändert von colinax (17.07.2019 22:06:36)
Offline
Die Änderung wurde gestern erst von Martin@mrbaseman eingespielt.
Also ist noch nicht im RC1 drin.
Ahja - dann ist das noch die alte Version, unverändert. Dann ist ja klar.
hm ...
ausser einen "hack" - einer Abfrage "vorweg" fällt mir auf die Schnelle auch nichts ein
vor Zeile 51[== PHP ==] $sCurrent_MYSQL_Version = $database->get_one("SELECT VERSION()"); $bMYSQL_57 = version_compare( $sCurrent_MYSQL_Version, "5.7.0", ">=");
und dann bei $sQuery (Zeile
[== PHP ==] $sQuery = "SELECT " . (( $bMYSQL_57 === true) ? "ANY_VALUE(s.`module`)" : "s.`module`") . ", MAX(s.`publ_start` + s.`publ_end`) published, p.`link`,
und weiter unten in der "while" schleife am Anfang
[== PHP ==] if(isset($page["ANY_VALUE(s.`module`)"])) { $page["module"] = $page["ANY_VALUE(s.`module`)"]; }
hm . sprich "größer gleich 5.7 dann mit "any_value" ansonsten ohne .."
Gruß
Kant
Hallo Kant,
leider ruft diese Abfrage vermehrt Fehler hervor.
Die Funktion ANY_VALUE bei MySQL größer gleich 5.7 anzuwenden scheint das Problem zu sein.
Vielleicht ist es ein MAMP spezifisches Problem, das wir anders ausfiltern könnten/sollten?
Gruß,
Christian
“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale
Offline
leider ruft diese Abfrage vermehrt Fehler hervor.
Die Funktion ANY_VALUE bei MySQL größer gleich 5.7 anzuwenden scheint das Problem zu sein.
Vielleicht ist es ein MAMP spezifisches Problem, das wir anders ausfiltern könnten/sollten?
Ich glaube schon zu wissen wo der Fehler herkommt.
Es wird nur die Versionsnummer der DB abgefragt und nicht die DB (z.B. also mysql oder mariaDB)
Die aktuellen Versionen von mysql sind 5.6, 5.7 und 8.0 bei mariaDB 5.5 und >= 10.1
Kurz um, der fix greift auch bei mariaDB was aber da nicht erwünscht ist.
Zum testen einfach in der functions.php Zeile 54 die Bedingung $bMySql_57 auf false setzen dann geht’s wieder.
Offline
@stefanek
was hältst du von dieser Lösung:
if(version_compare($sRunningMySqlVersion, "10.0", "<" )) {
$bMySql_57 = version_compare( $sRunningMySqlVersion, "5.7.0", ">="); }
Offline
Ich bin kein typischer Tester. Bei mir ist dieser Fehler nie aufgetaucht.
Ich bin momentan etwas kurz an Zeit, wenn Du magst, pm Kant, vielleicht mag er da ein Auge drauf werfen. Er kam schon mal mit einer Lösung - die hat eben nicht vorgesehen, dass Leute auch auf mariaDB arbeiten.
Ich denke er kann helfen.
Gruß,
Christian
“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale
Offline
Ich bin kein typischer Tester. Bei mir ist dieser Fehler nie aufgetaucht.
Ich bin momentan etwas kurz an Zeit, wenn Du magst, pm Kant, vielleicht mag er da ein Auge drauf werfen. Er kam schon mal mit einer Lösung - die hat eben nicht vorgesehen, dass Leute auch auf mariaDB arbeiten.
Ich denke er kann helfen.
Ok, werde ihn anschreiben. Testen hätte ich auch nicht verlangt (bin ja mehrfach von den Fehler selbst betroffen).
Die Syntax ist aber soweit Fehlerfei oder?
Beitrag geändert von colinax (29.10.2019 00:10:33)
Offline
Habe mit Kant geschrieben und die Lösung aus #90 jetzt auf GitHub übertragen.
Offline
Super.
Wenn's so funktioniert.
Danke an Euch beide.
“Success is the progressive realization of a worthy ideal.” ― Earl Nightingale
Offline
colinax