Liver64/LoxBerry-Sonos

ENH: playmode case insensitive

Woersty opened this issue · 7 comments

Hallo,

schlage vor, den playmode caseinsensitive zu machen.

Upps. Das ist ja noch die 1.0.1 hier.

schau ich mir mal an, ändere prinzipiell seeeehr ungerne die Syntax weil man sich sonst komplett durch seine MS Konfiguration wühlen muss.

Die Syntax ist mit der Änderung case-insensitive.
Bei dir war sie es nicht.
Also löst siehe eher das Problem, dass man sich durch die MS Konfig wühlen muss.
Er nimmt den Parameter an wie er kommt und wandelt ihn fix in Grossschreibung für die PHP Klasse

habe mal dein "preg_replace" rausgenommen, dann haben auch wieder alle playmodes in kleinschreibung funktioniert.

Warum? Wovon redest du?
Das preg_replace ist schon ok an der Stelle.
Aber du musst den _ noch hinzufügen. Ich hatte die Kombimodes nicht beachtet.
Mit der Schreibweise hat das nix zu tun.
Also preg_replace("/[^a-zA-Z0-9_]+/", "", strtoupper($_GET['playmode']));

jetzt passts :-)

ok, funzt jetzt