Bug/Lästiges Verhalten bei Buchungsartzuordnung in Liste

[GunterKramp]

Hallo zusammen,

Wenn ich in der Liste der Buchungen in JVerein bin kann ich ja die Buchungsart auf zwei Wegen zuordnen:

Entweder ich klicke die Buchung doppelt an und öffne sie damit. Dann kann ich die Buchungsart zuweisen, speichern und komme dann mit dem Zurück Pfeil in die Buchungsliste zurück. Dabei ist dann auch wieder die Buchung markiert die ich vorher geöffnet habe, ich weiß also wo ich war.

Wenn ich nun aber den schnelleren Weg gehen will und über Rechtsklick auf die Buchung die Buchungsart zuweise wandert nach dem Klick auf Übernehmen und dem Schließen des Auswahlfensters plötzlich der Scrollbar in der Buchungsliste nach Oben und ich bin irgendwo in der Liste.
Das ist sehr nervig, denn in hunderten von Buchungen immer wieder zu suchen wo ich war macht keinen Spaß.
Bei einem Klick auf Abbrechen passiert das übrigens nicht, da bin ich nach wie vor bei der zuvor markierten Buchung.

(Ich habe die Liste nach Datum sortiert und nicht nach Buchungsarten, wo es ja verständlich wäre wenn sich die Liste nach Zuweisung umsortiert. Selbst dann wäre aber sinnvoll weiter auf der zuletzt bearbeiteten Buchung die Markierung zu haben)

Ich verwende JVerein 2.8.10 und Jameica 2.6.1 mit Hibiscus 2.6.12.

Das selbe Problem tritt übrigens auch auf wenn ich einer Buchung ein Mitglied zuordne. Genauso lästig, vermutlich auch die selbe Ursache.

Gunter Kramp

[heiner]

Hallo Gunter,

ich habe ein Ticket aufgemacht. Schaue ich mir demnächst mal an.

Heiner

[GunterKramp]

Danke!

ich hoffe also auf Abhilfe. Hab derweil eh ganz andere Probleme. Davon mehr an passender Stelle.

Herzliche Grüße
Gunter

[wald]

Hallo Heiner,

das oben beschriebene Problem scheint bei allen Zuordnungen über rechts.klick zu passieren, also auch z.B. bei Projekten.
ich würde mich auch über einen Bug-Fix freuen.

Danke

Thomas

[GunterKramp]

Ich habe gerade noch was in dem Zusammenhang bemerkt, was vielleicht damit einen Zusammenhang hat.
Möglicherweise führt nach dem Bearbeiten der Buchung ein Filter-Reset zum kompletten Neuaufbau der Liste so dass die vorherige Position verloren geht?

Denn wenn mensch die Liste der Buchungen filtert und Buchungsart "ohne Buchungsart" auswählt und dann eine Buchung durch doppelt anklicken öffnet, bearbeitet und mit zurück zur Liste zurückkehrt steht der Filter der Liste wieder auf alle Buchungsarten.

Lästig wenn mensch die noch nicht zugeordneten Buchungen zuordnen will.

Bei einer anderen Filterung, also nach einer konkreten Buchungsart konnte ich dieses Verhalten so nicht nachvollziehen.
Habe aber natürlich auch nicht alle durchprobiert.

Herzliche Grüße
Gunter

[wald]

Hallo,

hier noch eine weitere Beobachtung:
Bei Buchungsarten scheint der Filter nur zurück gesetzt zu werden, denn er auf "ohne Buchungsart" steht.
Bei Projekten scheint er immer zurück gesetzt zu werden.

Gruß
Thomas

[wald]

Hallo,

habe den Fehler gefunden, jedenfalls für die Buchungsarten.
Der negative Index (-1) für "ohne Buchung" wird ignoriert.
Hier der Patch:

Code: Alles auswählen

### Eclipse Workspace Patch 1.0
#P jverein
diff --git src/de/jost_net/JVerein/gui/control/BuchungsControl.java src/de/jost_net/JVerein/gui/control/BuchungsControl.java
index f1fbb51..84a8280 100644
--- src/de/jost_net/JVerein/gui/control/BuchungsControl.java
+++ src/de/jost_net/JVerein/gui/control/BuchungsControl.java
@@ -909,7 +909,7 @@
       settings.setAttribute("suchkontoid", "");
     }
     Buchungsart b = (Buchungsart) getSuchBuchungsart().getValue();
-    if (b != null && b.getNummer() >= 0)
+    if (b != null && b.getNummer() >= -1)
     {
       b = (Buchungsart) getSuchBuchungsart().getValue();
       settings.setAttribute(BuchungsControl.BUCHUNGSART, b.getNummer());

Viele Grüße
Thomas

[wald]

Hallo,

hier nun der Patch der den Fehler sowohl für die Buchungsart, als auch für Projekte behebt.

Code: Alles auswählen

### Eclipse Workspace Patch 1.0
#P jverein
diff --git src/de/jost_net/JVerein/gui/control/BuchungsControl.java src/de/jost_net/JVerein/gui/control/BuchungsControl.java
index f1fbb51..0df1a1e 100644
--- src/de/jost_net/JVerein/gui/control/BuchungsControl.java
+++ src/de/jost_net/JVerein/gui/control/BuchungsControl.java
@@ -56,6 +56,7 @@
 import de.jost_net.JVerein.keys.BuchungsartSort;
 import de.jost_net.JVerein.keys.SplitbuchungTyp;
 import de.jost_net.JVerein.keys.Zahlungsweg;
+import de.jost_net.JVerein.rmi.Beitragsgruppe;
 import de.jost_net.JVerein.rmi.Buchung;
 import de.jost_net.JVerein.rmi.Buchungsart;
 import de.jost_net.JVerein.rmi.Jahresabschluss;
@@ -66,6 +67,7 @@
 import de.jost_net.JVerein.util.Dateiname;
 import de.jost_net.JVerein.util.JVDateFormatTTMMJJJJ;
 import de.willuhn.datasource.rmi.DBIterator;
+import de.willuhn.datasource.rmi.ObjectNotFoundException;
 import de.willuhn.jameica.gui.AbstractControl;
 import de.willuhn.jameica.gui.AbstractView;
 import de.willuhn.jameica.gui.Action;
@@ -540,6 +542,22 @@
     {
       return suchprojekt;
     }
+    String proj = settings.getString(PROJEKT, "");
+    Projekt p = null;
+    if (proj.length() > 0)
+    {
+      try
+      {
+        p = (Projekt) Einstellungen.getDBService().createObject(
+            Projekt.class, proj);
+      }
+      catch (ObjectNotFoundException e)
+      {
+        p = (Projekt) Einstellungen.getDBService().createObject(
+            Projekt.class, null);
+      }
+    }
+    
     ArrayList<Projekt> projektliste = new ArrayList<Projekt>();
     Projekt p1 = (Projekt) Einstellungen.getDBService().createObject(
         Projekt.class, null);
@@ -552,8 +570,7 @@
     {
       projektliste.add((Projekt) list.next());
     }
-
-    suchprojekt = new SelectInput(projektliste, null);
+    suchprojekt = new SelectInput(projektliste, p);
     suchprojekt.addListener(new FilterListener());
     suchprojekt.setAttribute("bezeichnung");
     suchprojekt.setPleaseChoose("keine Einschr�nkung");
@@ -909,7 +926,7 @@
       settings.setAttribute("suchkontoid", "");
     }
     Buchungsart b = (Buchungsart) getSuchBuchungsart().getValue();
-    if (b != null && b.getNummer() >= 0)
+    if (b != null && b.getNummer() >= -1)
     {
       b = (Buchungsart) getSuchBuchungsart().getValue();
       settings.setAttribute(BuchungsControl.BUCHUNGSART, b.getNummer());
@@ -921,12 +938,11 @@
     Projekt p = (Projekt) getSuchProjekt().getValue();
     if (p != null)
     {
-      p = (Projekt) getSuchProjekt().getValue();
       settings.setAttribute(BuchungsControl.PROJEKT, p.getID());
     }
     else
     {
-      settings.setAttribute(BuchungsControl.PROJEKT, -2);
+      settings.setAttribute(BuchungsControl.PROJEKT, "");
     }
     settings.setAttribute("suchtext", (String) getSuchtext().getValue());
     settings.setAttribute("suchbetrag", (String) getSuchBetrag().getValue());

Viele Grüße
Thomas

[heiner]

Hallo Thomas,

ich habe deine Patches übernommen.

Heiner

[wald]

Danke,

weisst Du schon, wann Du das nächste Release raus gibst?

Gruß
Thomas