Benutzer Diskussion:Christoph Jauera (WMDE)/Gadgets/DeepCat.js

aus Wikipedia, der freien Enzyklopädie
Zur Navigation springen Zur Suche springen

TypeError: localStorage is null — Bug nach Update?

[Quelltext bearbeiten]

@Christoph Fischer (WMDE): ich habe eine Kopie von dem Gadget in meinem BNR laufen, um das Limit von 70 auf 200 Kategorien zu erhöhen. Nach dem Update diese Woche habe ich den Quelltext von hier übernommen, jetzt gibts einen Javascript-Fehler. Die Firefox-Konsole meldet: "TypeError: localStorage is null" in der Funktion checkShouldHideHints(). Die deepcat-Suche funktioniert jetzt auch nicht mehr, und weitere Javascripts werden ebenso nicht mehr korrekt ausgeführt. Kannst Du den Fehler nachvollziehen? Danke und Viele Grüße, MisterSynergy (Diskussion) 22:37, 10. Okt. 2016 (CEST)Beantworten

Mitlesend:
Ich empfehle Wikipedia:Technik/Skin/JS/mw#.storage.
Auf phab:T142838 schrieb ich bereits:
There is a mw.storage package available which asserts availability and access permission on client side.
In solchen Fällen schlicht nichts merken.
@MisterSynergy: Das Problem ist, dass dein Browser oder deine Konfiguration einen winzigkleinen Speicher nicht ermöglicht; deaktiviert oder randvoll oder uralter Browser.
VG --PerfektesChaos 23:01, 10. Okt. 2016 (CEST)Beantworten
Danke für die prompte Antwort. Tatsächlich ist in meinem aktuellen Firefox 49 localStorage deaktiviert (und deshalb leer; per about:preferences, dort dom.storage.enabled=false). Das möchte ich gern auch weiterhin so beibehalten. —MisterSynergy (Diskussion) 06:49, 11. Okt. 2016 (CEST)Beantworten
@MisterSynergy: ich habe das Gadget entsprechend verbessert und auch die Empfehlung von PerfektesChaos umgesetzt. Siehe JS Datei. Viele Grüße und viel Spaß damit ;-) -- Christoph Fischer (WMDE) (Diskussion) 13:02, 13. Okt. 2016 (CEST)Beantworten
Hallo Christoph Fischer! Die Version davor schien auch schon wieder zu funktionieren, ich nehme aber dann ab jetzt die Version von heute. Viele Grüße und Danke für die Bemühungen, MisterSynergy (Diskussion) 13:05, 13. Okt. 2016 (CEST)Beantworten
@Christoph Fischer: Yep.
Ich hatte dein Experiment mit der Modernizr-Anleihe bereits interessiert verfolgt und hätte dir schon rechtzeitig aufs Brot geschmiert, dass genau diese Methode vor etwas über einem Jahr von MW wegen Inkompatibilität eliminiert worden war.
Ich habe noch einiges allgemein zu dem Thema in der Pipeline; würde aber anempfehlen, @Umherirrender, Raymond: den momentanen Bugfix vorrangig in unser MW zu kopieren.
LG --PerfektesChaos 13:17, 13. Okt. 2016 (CEST)Beantworten
Ich habe es mal übernommen. Es braucht aber wohl noch ein nowiki, da {{errors: Array}} sonst interpretiert wird. Vielleicht reicht auch nur einmal Klammern. Der Umherirrende 20:03, 13. Okt. 2016 (CEST)Beantworten