WBCE CMS Forum

colinax

Developer

[Erledigt] [1.2 Beta] Zeitzonenbug

Des Weiteren habe ich ein/zwei Bugs gefunden.

1.
Zeitformat, Datumsformat und Zeichensatz und Zeitzonenauswahl in Grundeinstellungen bieten dir die Option "Bitte auswählen" an, was micht sein sollte, oder geht's da nicht anders?

Wenn Zeitzone, Zeitformat, Datumsformat und Zeichensatz mit der Auswahl "Bitte auswählen" gespeichert werden fallen die auf default zurück, soweit so gut nur Zeichensatz behält  "Bitte auswählen" und arbeitet ohne weiter.

Das  fehlende Charset ist jetzt nicht so schlimm, spätestens mit 1.3 und min. php 5.4 wird's behoben.

Woher das "Bitte auswählen" kommt, weis ich nicht, bei Sprache und Co. ist es nicht vorhanden (wie es sein sollte).

Edit: zu 1. Existiert schon in 1.1.11

2.
Die Zeitzonenauswahl in Meine Daten markiert gleich 3 Werte mit selected="selected" und zeigt daher den default "GMT" aus der timezones.php an.

Edit: Wie es aussieht ist das dreifache selected="selected" die Folge davon dass value="system_default" für die Standardeinstellung und  value="0" für GMT nicht  vergeben werden können, um das zu kompensieren wird stattdessen value="1" verwendet, da value="1" für die Zeitzone +1 steht, werden entsprechend alle drei value="1" Werte mit selected="selected" gekennzeichnet.

Der Bug ist Theme unabhängig und tritt sowohl in php 5.6 und 7.0 auf.

Beitrag geändert von colinax (06.05.2017 23:54:07)

colinax

Developer

Re: [Erledigt] [1.2 Beta] Zeitzonenbug

Edit zu 2.:

Ich habe den Verursacher des Bugs gefunden gefunden.

Sobald man in der preferences/index.php die Zeilen 69 - 71 auskommentiert/rausnimmt ist der Bug nicht mehr vorhanden.

Ich vermute mal das sich der Code mit irgendwas in account/details.php oder account/preferences_form.php nicht verträgt.

cwsoft

Mitglied

Re: [Erledigt] [1.2 Beta] Zeitzonenbug

@Colinax: Der Zeitzonenbug in Preferences ist mir in der alten 1.2.x und PHP 7.1 schonmal aufgefallen und sollte dort gefixt sein: https://github.com/WBCE/WBCE_CMS/commit … 7544dd2713

Beitrag geändert von cwsoft (07.05.2017 14:13:57)

---

Account inactive since 2018/11/17.

colinax

Developer

Re: [Erledigt] [1.2 Beta] Zeitzonenbug

@Colinax: Der Zeitzonenbug in Preferences ist mir in der alten 1.2.x und PHP 7.1 schonmal aufgefallen und sollte dort gefixt sein: https://github.com/WBCE/WBCE_CMS/commit … 7544dd2713

Dein Fix für die Preferences sorgt dafür das der Bug unter 5.6 und 7.0 auftritt, ob dieser auch (wieder) unter 7.1 vorhanden ist weis ich nicht, meine Testumgebung kann nur 5.6 und 7.0.

cwsoft

Mitglied

Re: [Erledigt] [1.2 Beta] Zeitzonenbug

Hatte mit PHP 7.1, 7.0 und PHP 5.6 getestet und damals keinen Fehler mehr festgestellt. Kann mir das nächste Woche nochmal anschauen. Habe gerade nur ein Smartphone zur Hand.

Gruß

---

Account inactive since 2018/11/17.

colinax

Developer

Re: [Erledigt] [1.2 Beta] Zeitzonenbug

Hab jetzt noch mal rumprobiert, wenn der Bug in Preferences sichtbar ist und den Fix auskommentiert, danach die Seite aufruft ist der Bug weg und die value Werte stimmen auch.

Macht man die Auskommentierung rückgängig, ist der Bug optisch verschunden, da keine Option mehr selected="selected" besitzt, die Value Werte stimmen trotzdem nicht.

cwsoft

Mitglied

Re: [Erledigt] [1.2 Beta] Zeitzonenbug

Dann spiel Deinen Fix für PHP 5.6/7.0 ein. Ich teste dann nochmal mit PHP 7.1.

---

Account inactive since 2018/11/17.

colinax

Developer

Re: [Erledigt] [1.2 Beta] Zeitzonenbug

So hatte gerade Zeit und habe mir noch xampp mit php 7.1.2 installiert.

Leider tritt der Fehler auch dort auf, hab den Fix wieder entfernt.

florian

Administrator

Re: [Erledigt] [1.2 Beta] Zeitzonenbug

bump

---

Wir Benötigen: Cents, Euros... jetzt spenden!

colinax

Developer

Re: [Erledigt] [1.2 Beta] Zeitzonenbug

So hab das Problem nochmal getestet.

Ohne fix passt unter 5.6 und 7.0 alles nur unter 7.1 bekomme ich folgenden Warning: A non-numeric value encountered in C:\xampp\htdocs\admin\preferences\index.php on line 71

Mit dem Fix passt alles unter 7.1 leider haben 5.6 und 7.0 ein select Problem.

Getestet mit:
php 7.1 läuft auf XAMPP
php 5.6 und 7.0 laufen auf MAMP und diversen Live-Server

colinax

Developer

Re: [Erledigt] [1.2 Beta] Zeitzonenbug

Die Lösung ist zwar Quick and Dirty,

aber wenn wer der value="system_default" durch value="20" ersetzt wird, ist der Bug behoben.

cwsoft

Mitglied

Re: [Erledigt] [1.2 Beta] Zeitzonenbug

Ich schau es mir heute Nacht mal an und such ne passende Lösung.

---

Account inactive since 2018/11/17.

colinax

Developer

Re: [Erledigt] [1.2 Beta] Zeitzonenbug

Ich schau es mir heute Nacht mal an und such ne passende Lösung.

Danke, ich hab aber trotzdem schon mal den Hotfix hochgeladen.

cwsoft

Mitglied

Re: [Erledigt] [1.2 Beta] Zeitzonenbug

Ah ok. Na dann lassen wir den Hotfix erstmal drin.

---

Account inactive since 2018/11/17.