Erstellt vor 3 Jahren

Geschlossen vor 3 Jahren

#1837 closed Fehler (fixed)

Lieferant auf ungültig setzen, verfälscht Kreditorenbuchungsmaske (mulitbox <-> obsolete?)

Erstellt von: information@… Verantwortlicher: m.bunkus@…
Priorität: normal Meilenstein:
Komponente: kivitendo ERP Version: 2.7.0 unstable
Schweregrad: normal Stichworte: Finanzbuchhaltung
Beobachter: s.schoeling@…

Beschreibung

1.)
Finanzbuchhaltung -> Kreditorenbuchung erfassen auf Lieferant X
2.)
Lieferant X auf ungültig setzen

3.)
Die Kreditorenbuchung aus Schritt 1 aufrufen, die multibox (Drop-Down) springt auf den ersten Eintrag in der Liste

Dasselbe Problem taucht auch bei Verkaufsrechnungen auf und sehr wahrscheinlich auch bei Debitorenbuchungen.

In den Berichtsübersichten als auch im Buchungsjournal werden ungültige Firmen, aber korrekt angezeigt.

Idee A:
Meine optimale Idee wäre, dass es bei Buchungen die einmal geändert werden, gar keine Drop-Down-Liste mehr gibt, sondern nur noch den Namen als Textinfo.

Idee B:
Bei Buchungen die schon eine ID haben, auch entsprechend auf 'ungültige' Werte prüfen, da ja alte Buchungen noch korrekt angezeigt werden müssen.

Soweit meine ersten Ideen ...

Anhänge (1)

hotfix-all_vc.png (36.0 KB) - hinzugefügt von information@… vor 3 Jahren.
erste idee hotfix

Alle Anhänge herunterladen als: .zip

Änderungshistorie (5)

Geändert vor 3 Jahren durch information@…

erste idee hotfix

comment:1 Geändert vor 3 Jahren durch information@…

Und nochmal im Klartext:

Form.pm sub all_vc
+ my $obsolete = "WHERE NOT obsolete" unless $self->{id};

if ($count < $myconfig->{vclimit}) {

$query = qq|SELECT id, name, salesman_id

+ FROM $table $obsolete

comment:2 Geändert vor 3 Jahren durch information@…

  • Lösung auf fixed gesetzt
  • Status von new nach closed geändert

Ich hab nochmal nachgedacht, eigentlich spricht nichts dagegen den Hotfix so einzuspielen, ich mach noch ein Kommentar in den Code dazu.

Commit 325263e0e52dc

comment:3 Geändert vor 3 Jahren durch s.schoeling@…

  • Lösung fixed gelöscht
  • Status von closed nach reopened, s.schoeling@linet-services.de geändert

Hmm, da musst Du nochmal nachbessern Jan. Ein paar Zeilen vorher wird mit einem select count(*) rausgefunden, wieviele Zeilen es gibt. Wenn Deine Änderung da nicht auch greift gibt es häßliche inkonsistente Zustände im Zusammenhang mit vclimit.

comment:4 Geändert vor 3 Jahren durch information@…

  • Lösung auf fixed gesetzt
  • Status von reopened nach closed geändert

Hatte ich auch gesehen, allerdings ignoriert, da ja für alte Buchungen, sowieso keine Änderungen mehr der Auswahl möglich sein sollte ...
Aber für oe und do stimmt das.
Du hast hier Recht ;-).
Ist jetzt auch drin

Hinweis: Hilfe zur Verwendung von Tickets finden Sie in TracTickets.