search

socia-platform / htwplus

The social network and new eLearning system for HTW-Berlin.
http://plus.htw-berlin.de
MIT License
14 stars 8 forks source link

Grouptitle only out of whitespace #62

Closed vanbernum closed 9 years ago

vanbernum commented 9 years ago

Aktuell lassen sich Gruppen erstellen, bei denen der Title nur aus White-Space besteht; das sollte so natürlich nicht sein. Vielleicht wäre an der stelle ein Regex sinnvoll bzw. vielleicht stellt JPA bereits Annotationen dafür bereit, um das abzufangen?!

IvenZ commented 9 years ago

Ich gucke es mir mal an. Danke

IvenZ commented 9 years ago

Die Annotationen die ich ausprobiert habe (@NonEmpty, @NotEmpty, @Pattern) waren nicht gleich erfolgreich. Titel wird deshalb einfach getrimmed. Das funktioniert soweit in allen getesteten Fällen. Zukünftig können wir uns trotzdem überlegen, ob wir mit nem Pattern nicht nur bestimmte Buchstaben zulassen.

vanbernum commented 9 years ago

Getrimmt heißt: am Anfang und Ende wird Whitespace entfernt und wenn nichts übrig bleibt, dann Fehler?! Sonderzeichen auszuschließen wäre auch sicherlich sinnvoll...

IvenZ commented 9 years ago

genau und wichtig ist, dass er NACH dem trim den unique check macht. Bei den Annotationen hat er den check entweder davor oder gar nicht gemacht. Sprich, man konnte mit " HTWplus" eine gleiche private HTWplus Gruppe erstellen, die es schon gibt.

vanbernum commented 9 years ago

Top...dann ist so ja jetzt genau richtig.

IvenZ commented 9 years ago

da ziemlich urgent -> direkt auf live