[Pocket Query] Prevent map error after PQ deletion

[DieBatzen]

If a pocket query is deleted and parameter settings_pq_previewmap is active, then an error occurs:

Reason is that after pq deletion the page gets reloaded empty and therefore no map container can be added.

[2Abendsegler]

@DieBatzen
Hmm ..., das können wir so nicht lassen.

Du brichst hier das Programm ab, anstatt nur den Block abzubrechen, der nicht fehlerfrei ausgeführt werden kann. Wenn wir so etwas zulassen, müssten wir verifizieren, ob nicht nachgelagert noch irgend etwas laufen muss. Und wenn wir zeitlich später etwas weiter hinten einbauen, was unbedingt noch laufen soll, dann hätten wir hier von diesem Ausstieg sicherlich keine Kenntnis. Damit wir diese Überlegungen nicht anstellen müssen, und schon gar nicht in solche Problematik rein laufen können, machen wir so etwas eigentlich nicht.

Kannst du bitte ein Revert machen und dann deine Prüfung im vorangegangenen if einbauen? Also so etwa

if (settings_pq_previewmap && document.location.href.match(/\.com\/pocket\/gcquery\.aspx/) && !global_isBasic && $('.LatLongTable')[0]) {

---

Noch eine Kleinigkeit.

Ich versuche einen Überblick über alle Änderungen zu behalten und ich versuche auch potentiell für mich problematisches Coding zu verhindern. Dieser PR ist ein Beispiel dafür, warum das Sinn machen kann. Aber es geht auch darum, dass meine Programmierkenntnisse denen von anderen nicht immer gewachsen sind. Gegebenenfalls kann ich ein Coding ablehnen, weil ich mich nicht imstande sehe, es bei Problemen selbst irgendwann zu prüfen und zu korrigieren. Und es geht auch darum meinen Aufwand für eine neue Version möglichst gering zu halten. Ich mache mir deshalb zu den Änderung Notizen, falls es etwas beispielsweise im Changelog zu beachten gilt. Und ich sorge auch beispielsweise dafür, dass alle relevanten Änderungen in den Milestones gelistet sind, weil das die Basis für meine Arbeiten am Changelog bildet.

Ich weiß, dass es manchmal einfach viel schneller und einfacher ist, den PR gleich selbst zu mergen, gerade wenn es sich um einfache Dinge handelt. Und ich mache das ja auch. Meine Motivation dabei ist es aber lediglich, Arbeit bei anderen, momentan also bei dir, so gering wie möglich halten. Schließlich bin ich froh, wenn sich jemand meine aufwendigeren Entwicklungen gewissenhaft ansieht. Bei meinem Kleinkram ist mir das nicht wichtig.

Wir hatten es in der Vergangenheit so gehandhabt, dass alle Änderungen von mir approved werden müssen.
Vielleicht könnten wir das weiterführen.

[DieBatzen]

Du brichst hier das Programm ab, anstatt nur den Block abzubrechen, der nicht fehlerfrei ausgeführt werden kann.

Du hast natürlich vollkommen recht.
Irgendwie war ich geistig im Funktionsmodus und nicht im If-Modus - sorry.

Ich werden den Merge reverten, die Änderung durchführen und zum Review hochladen.

Wir hatten es in der Vergangenheit so gehandhabt, dass alle Änderungen von mir approved werden müssen.
Vielleicht könnten wir das weiterführen.

Gar kein Problem, machen wir so.
In dem Fall dachte ich einfach, die Änderung sei so minimal und sicher, dass kein Review erforderlich ist - so kann man sich täuschen.