Inkonsistente Benennungskonventionen für PNG- und PDF-Uploads

[robbie.morrison]

Hallo Devs,

ich habe kürzlich (mit Discourse 2.4.0.beta7 und der aktuellen Version) zwei inhaltlich identische Dateien im PNG- und PDF-Format hochgeladen (im openmod-Forum). Discourse analysiert und protokolliert automatisch den ursprünglichen Dateinamen im Markdown wie folgt:

Hochgeladener Dateiname	Angezeigte Datei	Kommentar
bridge-2-a5-flier.04.png	04	annotiert 1
bridge-2-a5-flier.04.pdf	bridge-2-a5-flier.04.pdf	annotiert 2

Nachfolgend ein Screenshot, mit den Anmerkungen „1“ und „2“, um zu zeigen, wie dies funktioniert:

Meiner Ansicht nach sollte im PNG-Fall der ursprüngliche Dateiname protokolliert werden oder, falls das nicht möglich ist, zumindest der gesamte Basisname gemäß: basename "bridge-2-a5-flier.04.png" ".png".

Ich habe nach ähnlichen Fehlerberichten gesucht, konnte aber nur Diskussionen zur Generierung der anonymisierten Namenszeichenkette finden.

Vielen Dank wie immer für diese wunderbare Software. R

[codinghorror (Jeff Atwood)]

Stimme zu, sieht nach einem Fehler aus, der auf den Punkt im Dateinamen zurückzuführen ist.

[martin (Martin Brennan)]

Ich habe es mir angesehen, und es handelt sich definitiv um einen Fehler, der auf das . im Dateinamen zurückzuführen ist. Composer versucht lediglich, die Dateiendung zu entfernen, sodass wir im Markdown bridge-2-a5-flier.04|WIDTHxHEIGHT erhalten. Es wird jedoch einfach am . getrennt und der erste Teil der Datei vor der Erweiterung genommen, was offensichtlich nicht korrekt ist.

Soweit ich sehen kann, ist dieser Fehler seit zwei Jahren vorhanden. Vielen Dank, dass du so -scharfäugig warst und ihn entdeckt hast! Ich werde heute eine Korrektur einspielen.

[martin (Martin Brennan)]

Behoben mit FIX: Image file names with dots were showing incorrectly in composer markdown by martin-brennan · Pull Request #8465 · discourse/discourse · GitHub