Buchungsart Suche statt Combobox

[heiner]

Hallo Thomas,

deinen Patch kann ich nicht direkt in Eclipse verarbeiten.

Heiner

[wald]

Hallo Heiner,

Hast Du einen Tipp, wie ich den erstellen kann?
Den habe ich unter Team > CREATE Patch erstellt.

Ich hätte jetzt erwartet, dass du ihn auch über diesen Weg einspielen kannst.
Rechtsklick auf das JVerein Projekt > Team > Apply Patch.

Danke
Thomas

[heiner]

Hallo Thomas,

schon fast gut. Aber ein Parameter ist noch falsch.

Ich habe 3 Screenshots ins Wiki gepackt. Für Entwickler

Heiner

[wald]

Hallo Heiner,

jetzt hoffentlich im richtigen Format.
Eine kleine Erweiterung habe ich noch gemacht. Auch der Admin > Buchungsart View hat nun eine Suche.

Viele Grüße
Thomas

[heiner]

Hallo Thomas,

ich konnte den Patch jetzt übernehmen. Der Code konnte aber nicht fehlerfrei compiliert werden. Es tauchten noch 7 Fehler auf.

Heiner

[wald]

Hallo Heiner,

ich habe den Patch jetzt bei mir auch getestet.
Aus irgendeinem Grund, werden zwei Stellen nicht gepatcht, obwohl sie im PatchFile aufgeführt sind.

Wenn Du in:

/jverein/src/de/jost_net/JVerein/gui/control/EinstellungControl.java
in Zeile 268 folgende Deklaration einsetzt:

Code: Alles auswählen

 private SelectInput buchungBuchungsartAuswahl;

und in
/jverein/src/de/jost_net/JVerein/gui/control/BuchungsControl.java
Ab Zeile 442 die Methode: de.jost_net.JVerein.gui.control.BuchungsControl.getBuchungsart()
durch folgenden Code ersetzt, sollte der Code keine Fehler mehr haben:

Code: Alles auswählen

	public Input getBuchungsart() throws RemoteException {
		if (buchungsart != null && !buchungsart.getControl().isDisposed()) {
			return buchungsart;
		}
		DBIterator list = Einstellungen.getDBService().createList(
				Buchungsart.class);
		list.setOrder("ORDER BY nummer");

		switch (Einstellungen.getEinstellung().getBuchungBuchungsartAuswahl()) {
		case BuchungBuchungsartAuswahl.ComboBox:
			buchungsart = new SelectInput(list, getBuchung().getBuchungsart());
			((SelectInput) buchungsart).setPleaseChoose("Bitte auswählen");
			break;
		case BuchungBuchungsartAuswahl.SearchInput:
		default: // default soll SearchInput sein. Eigentlich sollten die
					// Settings immer gesetzt sein, aber man weiss ja nie.
			buchungsart = new BuchungsartSearchInput();
			((BuchungsartSearchInput) buchungsart)
					.setAttribute("nrbezeichnung");
			((BuchungsartSearchInput) buchungsart)
					.setSearchString("Zum Suchen tippen ...");
		}

		buchungsart.setValue(getBuchung().getBuchungsart());
		return buchungsart;
	}

Und zum Schluss noch in:
/jverein/src/de/jost_net/JVerein/gui/view/EinstellungenAnzeigeView.java
Einen Einstellungsbreich für die Beitragsart suche
ab Zeile 80 einfügen:

Code: Alles auswählen

    
    cont.addSeparator();
    //Einstellung zum Buchungsdialog 
    cont.addText("Einstellungen Buchungsdialog", false);
    SimpleContainer buchung = new SimpleContainer(cont.getComposite());
    buchung.addLabelPair("Buchungsart Auswahl", control.getBuchungBuchungsartAuswahl());

ich habe jetzt noch mal einen Patch für alle Änderungen erstellt, aber ich glaube nicht, dass der besser funktioniert.
Ich hätte auch die obigen Änderungen als Patch nach dem Patch zur Verfügung gestellt, habe keine Ahnung wie ich das hinkriegen soll, da ich ja nicht commiten kann.

Viele Grüße
Thomas

[heiner]

Hallo Thomas,

ich habe deinen Patch übernommen.

Heiner

[wald]

hallo Heiner,

super Danke!
Du hast nur die Einstellungen etwas anders gemacht. Ich hatte die Umstellung zwischen Liste und Suche extra in einen neuen Block gepackt, weil die anderen Anzeigeeinstellungen sich auf die Mitgliederverwaltung beziehen. Fand das ein Stück übersichtlicher und leichter verständlich, was man damit einstellt.

Gruß
Thomas

[heiner]

Hallo Thomas,

alternativ könnte es auch nach "Einstellungen Buchführung" verschoben werden.

Evtl. mache ich das noch.


Heiner

[wald]

Hallo Heiner,

hier noch eine Erweiterung und ein bisschen Schönheitskorrektur.

Man kann ja auch mehrere Buchungen markieren und dann eine Buchungsart zuordnen.
Auch dort habe ich jetzt den Dialog angepasst, dass es möglich ist zu suchen.

Viele Grüße und schon mal einen guten Rutsch!
Thomas