volkszaehler.org
volkszaehler.org copied to clipboard
volkszaehler
Ignore cookie when using the uuid[] URL parameter
Mit dem Optionen muss man glaube ich unterscheiden was drin sein kann und was der Nutzer modifizieren kann und will. Im Cookie sind alle daten vom Typ String, Number, und Bool von https://github.com/volkszaehler/volkszaehler.org/blob/dc89a176bd9a644da0348941dac952e0d9115521/htdocs/js/options.js#L26 gespeichert.
Der "normale" Nutzer kann über das Frontend nur refresh Modifizieren. Advanced User könnten die Cookies modifizieren aber ich glaub das macht keiner. Somit stehen im Cookie wenn überhaupt nur die default Werte.
Über die URL können aber auch die options sowohl implizit mit den Parametern from und to als auch explizit mit options gesetzt werden.
https://github.com/volkszaehler/volkszaehler.org/blob/7225eac0a6582b4b8ae25a60d5cc506310fabddb/htdocs/js/functions.js#L225
Mein Vorschlag wäre entweder man lässt es so wie im PR. Oder macht folgendes daraus:
if (!(params.hasOwnProperty('uuid') || params.uuid)) {
// only load cookies when no uuid present in url params
vz.entities.loadCookie(); // load uuids from cookie
}
if (!(params.hasOwnProperty('options') || params.options)) {
// only load cookies when no options present in url params
vz.options.loadCookies(); // load options from cookie
}
Was meinst du?
Mein Vorschlag wäre entweder man lässt es so wie im PR. Oder macht folgendes daraus:
if (!(params.hasOwnProperty('uuid') || params.uuid)) {
...
if (!(params.hasOwnProperty('options') || params.options)) {
Sieht für mich sinnvoll aus. Spricht was dagegen? Könntest du das entsprechend ändern und den PR aktualisieren? Dann würde ich das mergen...
Hallo @jahir, Ich habe den PR entsprechend aktualisiert. Viele Grüße Oliver
Es gibt scheinbar ein Problem mit der Konfiguration der Pipelines:
ja, hab ich gestern auch gesehen, scheint aber mit #937 behoben zu sein, wobei das Entscheidende der Wechsel von ubuntu 18.04 (das von github als deprecated markiert war) auf 22.04. Könnte Zufall sein, aber es lief damit durch und ohne die Änderung hängt es weiterhin... wenn du ein rebase machst, müßte es tun.