Emoji-Auswahl stürzt den Nachrichten-Editor in Android / Chrome ab

Beitragen Fehler
3

Ich kann den Fehler reproduzieren. Hier ist der Stack Trace:

_application-bfbda341c2eb6dd7d61c681e17bdccec057c30e045ddc332927a7363150e9b1b.js:16386 Uncaught TypeError: Cannot read property '0' of null
    at HTMLLIElement.<anonymous> (application-bfbda341c2eb6dd7d61c681e17bdccec057c30e045ddc332927a7363150e9b1b.br.js:1)
    at HTMLLIElement.dispatch (ember_jquery-36a23101c869ab0dc53fc908de69adb785731593573d32bdeef416acc1076ef4.br.js:1)
    at HTMLLIElement.d.handle (ember_jquery-36a23101c869ab0dc53fc908de69adb785731593573d32bdeef416acc1076ef4.br.js:1)
(anonymous) @ application-bfbda341c2eb6dd7d61c681e17bdccec057c30e045ddc332927a7363150e9b1b.br.js:1
dispatch @ ember_jquery-36a23101c869ab0dc53fc908de69adb785731593573d32bdeef416acc1076ef4.br.js:1
d.handle @ ember_jquery-36a23101c869ab0dc53fc908de69adb785731593573d32bdeef416acc1076ef4.br.js:1

Das ist diese Zeile:

https://github.com/discourse/discourse/blob/master/app/assets/javascripts/discourse/app/lib/autocomplete.js#L308

Der Fehler tritt auf, weil selectedOption 0 ist (ein einziger Vorschlag, also der erste), während autocompleteOptions irgendwie null ist.

Ich untersuche gerade, warum das so ist…

Bislang bin ich mir nicht sicher, warum das passiert. Anfangs habe ich diesen PR von @Osama verdächtigt:

Aber ich habe einige Breakpoints hinzugefügt und kann nicht wirklich herausfinden, “wer” autocompleteOptions mutiert und auf null setzt.

Dass autocompleteOptions aus dem Bereich des übergeordneten Closures zwei Ebenen höher stammt, ist ebenfalls ziemlich seltsam und macht den Code etwas schwerer nachvollziehbar und zu debuggen.

8 „Gefällt mir“