search

hschottm / tags

Contao tags extension
11 stars 16 forks source link

Tagcloud fehlt, wenn Tags mit % vorhanden sind #11

Closed seaneble closed 10 years ago

seaneble commented 10 years ago

Ein Redakteur hat es bei mir geschafft, ein %-Zeichen in einen Tag einzugeben. Ab dem Zeitpunkt war es dann nicht mehr möglich, das Tagcloud-Feld im Backend zu sehen, es waren nur der Hilfetext und der Feldname zu sehen.

Bis ich jedoch darauf kam, dass es an dem Prozent-Zeichen lag, verging einige Zeit. Letztendlich ist das Problem, dass die Methode TagField->generate() ein sprintf verwendet. In meinem Fall enthielt $list eine unglückliche Kombination aus % und weiteren Zeichen, sodass sprintf gerne ein Argument mehr gesehen hätte und daher einfach false zurückgab.

Es wäre gut, wenn entweder Redakteure kein %-Zeichen in Tags eingeben dürften oder dieses Zeichen vor der Übergabe an sprintf maskiert würde.

hschottm commented 10 years ago

Fixed. Der String hat in der sprintf-Anweisung eh nichts verloren. Danke für den Hinweis!

seaneble commented 10 years ago

Super, danke!

seaneble commented 10 years ago

Weißt du schon, wann eine neue Version rauskommen wird?

hschottm commented 10 years ago

Ich hab das gleich vor zwei Wochen überall in die tags-Module eingebaut. Ich habe nur die Versionsnummer nicht erhöht. Wenn du aus dem ER aktualisierst, sollte der Fix eigentlich drin sein.

seaneble commented 10 years ago

Ja, die Version habe ich schon. Das macht man aber eigentlich nicht, gell ;-)