Mitglieder Suche auch bei Vollzahler #409
Conversation
JohannMaierhofer
added
enhancement
|
Wurde in Jameica mit willuhn/jameica@009f987 gefixed. |
Kannst du nochmal kurz Erklären, was genau geändert wurde und welches Problem damit gelöst wird. Ich verstehe gerade nicht genau, was dieser PR erreicht. |
|
Bei Auswahlfeldern für Mitglieder bekommt man eine Dropdown Liste wo man Mitglieder auswählen kann. Bei hunderten Mitgliedern ist diese Liste sehr lang. Da wird es dann schwer ein Mitglied zu finden. Auch wenn die korrigierte Jameica Version freigeben ist stellt sich immer noch die Frage ob wir das dann übernehmen können. Es kann ja sein, dass die User nicht unbedingt auf diese Jameica Version umsteigen. |
Ich weiß nicht ob das unbedingt ein Argument sein sollte. Wen ein Nutzer die neue Jameica Version nicht nutzen will, kann er ja immer noch eine alte JVerein Version nutzen. Ansonsten kann man das ja auch über die Release Notes dokumentieren, dass JVerein die (stand Heute) aktuelleste Jameica Version voraussetzt und evtl. kann man über die Start-Seite sogar eine Warnung einfügen, falls Jameica veraltet ist. |
Du hast Recht, ein Eintrag in den Release Notes sollte reichen. Ein Umstieg auf die neue Version ist ja auch nicht zwingend notwendig. Wenn man das Familien Feature nicht nutzt ist es irrelevant. Nutzt man es kann man ja auch auf Liste umstellen oder man findet eine klick Reihenfolge in der kein disable passiert. |
| @@ -1128,8 +1130,14 @@ else if (bg != null | |||
| getMitglied().setZahlerID(null); | |||
| if (zahler != null) | |||
| { | |||
| zahler.setValue(Einstellungen.getDBService() | |||
| .createObject(Mitglied.class, "")); | |||
| if (zahler instanceof SelectNoScrollInput) | |||
There was a problem hiding this comment.
Puh, dreimal die gleichen Codezeilen in einer Datei. Ich empfehle, Code Duplikationen wann immer zu vermeiden, auch wenn es nur acht bis elf Zeilen sind (je nach dem, ob man die Zeilen davor auch mitzählt.
There was a problem hiding this comment.
Ich habe hier nur zweimal den gleichen Code gesehen. Da aber in den beiden else Zweigen sowieso der gleiche Code war habe ich jetzt einen der beiden gelöscht.
| if (zahler instanceof SelectNoScrollInput) | ||
| { | ||
| ((SelectNoScrollInput) zahler).setPreselected(getMitglied()); | ||
| } | ||
| else if (zahler instanceof VollzahlerSearchInput) | ||
| { | ||
| ((VollzahlerSearchInput) zahler).setValue("Zum Suchen tippen"); | ||
| } | ||
| zahler.setEnabled(false); |
There was a problem hiding this comment.
Hier steht es das dritte Mal bzw. nun das zweite Mal.
|
Bekommen wir diesen PR auch noch vor dem nächsten Release fertig? |
|
Den brauchen wir nicht reinnehmen. Es gibt ja auch noch kein neues Jameica mit behoben Bug. |
Stimmt, hatte ich übersehen, da ich nur den Commit betrachtet habe. @willuhn könnte ja nach nunmehr einem Dreivierteljahr ein Bugfix release 2.10.5 herausgeben. 😏 |
|
Ja, das stimmt. Ich kümmere mich in den nächsten Tagen mal um ein neues Release. |
Ich habe auch die Mitglied Eingabe bei Vollmitglied um eine Suchmöglichkeit erweitert.
Der PR kann aber erst bei einem Update von Jameica übernommen werden.
Es ist ein Fehler in der Klasse SearchInput. Nach disable und wieder enable kann man im Feld keinen Text mehr eingeben. Ich habe Olaf bereits informiert.
Zum Testen kann man in der Klasse SearchInput in der Methode setEnabled die folgende Zeile einfügen:
text.setEditable(enabled);
Oder main/latest bzw. nightly von Jameica verwenden.