'max tags per topic' failing silently

Since this week. we no longer see an error message when you more than max tags per topic on a topic. Instead, the topic just saves, but the additional tags are discarded silently.

2 „Gefällt mir“

Hi,

it should be fixed by

https://github.com/discourse/discourse/pull/9018

I added few tests, hopefully we don’t regress on this anymore.

Thanks.

8 „Gefällt mir“

merged

Ich habe dieses Thema gefunden, als ich melden wollte, dass max_tags_per_topic bei aktuellen Tests, die bestanden wurden, stillschweigend fehlschlägt, wenn es auf 0 gesetzt ist.

Reproduktion:

  • Setzen Sie max_tags_per_topic auf 0
  • Fügen Sie einem Thema ein Tag hinzu
  • Sehen Sie, dass es nicht hinzugefügt wird

Wenn max_tags_per_topic auf 1 gesetzt ist und Sie versuchen, ein zweites Tag hinzuzufügen, erhalten Sie die Fehlermeldung „Sie können nur 1 Element auswählen“.

Ich vermute, der Fehler liegt hier, wenn maximum 0 ist, dann ist der if-Ausdruck falsch.

3 „Gefällt mir“

Danke! Wir haben es hier behoben:

1 „Gefällt mir“

Dieses Thema wurde nach 19 Stunden automatisch geschlossen. Neue Antworten sind nicht mehr möglich.