add question: error messages auf selben seite anzeigen

[astriffe]

vom kunden wurde ja gewünscht, dass wir fehlermeldungen (wie: frage ist leer, frage existiert bereits, ähnliche tags) gleich im frage-formular anzeigen. ähnlich bei addUser, addAnswer,addComment, .... Ich weiss nicht, wie man dies entsprechend umleiten könnte - kann sich das vielleicht einer unserer html-spezialisten kurz anschauen..?=)

[jruef]

Ehm, musst du nicht einfach die Variable die du im HTML Code benutzt, mit den entsprechenden Texten versehen?

[astriffe]

...du meinst die variable message!? so wird es momentan gemacht, dann wird jedoch jeweils eine neue (leere) seite geladen...wenn ich eine route einfüge, die wieder auf die addquestion-seite umleitet, funktionierts aber irgendwie auch nicht..

[jruef]

Ehm nein, dass du es Im Formular feld selbst anzeigts oder ist das keine gute Idee? So wie ich es im edit question gemacht habe.

[astriffe]

... habs gerade ausprobiert - wenn ich aber bei edit question eine leere frage eingebe, wird diese ohne warnung angenommen..!?

[jruef]

dann musst du in der entsprechenden methode im controller eine if schlaufe einbauen, die das ganze abfängt. Und dann deine Variable auf den entsprechenden Wert setzen und übergeben

[astriffe]

ist nun für addQuestion soweit implementiert...werde mir noch editQuestion usw vornehmen.

[astriffe]

erledigt --> close. please review.

[jruef]

Review gemacht folgende Problem:

• Wenn ich eine question mache und dann nur die Tags editiere, kommt die Meldung, dass die Question schon existiert, ich kann also in meiner Question nicht einfach nur die tags editieren.

• Was macht genau, oder was genau sollte das folgende Code Stück machen?

  if (!Question.checkTags(copyTags).isEmpty()) { message = "The following tags already exist: "

  • Question.checkTags(copyTags)
  • ". Please review your tags.\n"; }

[jruef]

Reopened wegen bugs. --> Sry, dass ich nur review machen kann, zeit reicht leider nicht zum selber fixen.

[astriffe]

Kein problem,ich glaube zu wissen,wo der fehler liegt...ich erkläre das codestück,wenn ich am pc bin,ok? (iphone tastatur mit kalten fingern --> des teufels ;-))

[jruef]

Hehe, ja dasch vom tüüfu ;-)

[astriffe]

aalso, zur frage, was das abgebildete Codestück macht: Die methode checkTags überprüft die Distanz der eingebenen Tags zu bereits bestehenden Tags und gibteinen String mit allen Tags zurück, dessen Distanz zu klein ist zu den eingebenen Tags. Ist dieser String also nicht leer, wird die Nachricht ausgegeben, dass die ähnlichen Tags bereits existieren...

Falls noch weiter Fragen existieren -> weiter fragen =)

[astriffe]

der bug ist nun fixed. Ich habe noch ziemlich viel Müll im code, den ich aber noch dort belassen möchte für refactoring (es hat etwas duplicated code...).. ok?

[jruef]

KP

[jruef]

DONE