heintzl / Kernbeisser-Gradle

the Kernbeisser Repository with Gradle Support
5 stars 1 forks source link

Lieferungs-Eingabe überschreibt ungewollt Artikeldaten #624

Closed heintzl closed 2 years ago

heintzl commented 2 years ago

2 schwerwiegende Fehler in der Lieferungseingabe:

  1. Es wird eine Lieferantennummer eingegeben mit einer Stelle zuviel eingegeben, z.B. 512011 statt 51201. Dann erscheint der Artikelname als "Keinen Artikel gefunden", dann wird die Backspacetaste gedrückt um die letzte Stelle zu löschen, und nun kann es passieren, dass der Artikel nicht aktualisiert wird. Wird nun Return gedrückt, wird der Artikel dennoch als Lieferung übernommen, aber die Artikeldaten sind zerstört, der Name ist dann "Keinen Artikel gefunden", Nettopreis, Gebindegröße, Pfandangaben etc. sind mit 0 überschrieben!
  2. Wenn in der Eingabemaske bei einem gefundenen Artikel die Daten (Name, Preis etc.) modifiziert werden, werden diese Änderungen ohne Warnung in die Artikelstammdaten übernommen! Das ist sehr fehleranfällig.
julikiller98 commented 2 years ago

Wäre es ggf. besser wenn wir einen Knopf hätten der den Aktuellen Artikel in der Bearbeitungsmaske öffnet? Oder ist das bearbeiten von Artikeln beim Eingeben praktisch bei jedem eingebenen Artikel nötig? Das meiner Meinung deutlich intuitiver und hätte den Vorteil, das wir das bearbeiten von Daten aus der Maske kommplett abstrahieren könnten.

heintzl commented 2 years ago

Ja, das halte ich für sehr gut! Es sollte darüber hinaus geprüft werden, ob der Artikel schon bei den eingebenen ShoppingItems dabei ist, und ggf. Mit Warnmeldung die vorhandenen Items aktualisiert werden. Denn wenn der Artikel bearbeitet wird, ändert das, wie das Item erstellt wird, z.b. bei Preisänderungen, oder Änderung von weighable. Es könnte sich helfen, wenn der Article von bestehenden items geändert werden kann.