Project

General

Profile

Fehler #3

2478 Preis bei Erzeugnissen

Added by Jan Büren about 5 years ago. Updated over 4 years ago.

Status:
Erledigt
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
08/28/2014
Due date:
% Done:

0%

Estimated time:

Description

Wird ein Preis in den Erzeugnissen mit Komma gemacht, so wird er beim Erneuern verändert.
Aus 15,55 wird 1555,00

Abhilfe zZ. Ohne erneuern speichern.

https://trac.kivitendo.de/ticket/2478

History

#1 Updated by G. Richardson over 4 years ago

In Trac war das Ticket https://trac.kivitendo.de/ticket/2478

Das Problem ist, daß bei Erzeugnissen sellprice (und listprice) 2x geparsed wird:
https://github.com/kivitendo/kivitendo-erp/blob/78912e55533a48109adc11e798101b29e174728d/bin/mozilla/ic.pl#L1792
https://github.com/kivitendo/kivitendo-erp/blob/78912e55533a48109adc11e798101b29e174728d/bin/mozilla/io.pl#L681

Das ist früher nicht aufgefallen, weil der sellprice in check_form früher immer auf 0 gesetzt wurde und dann aus den Verkaufspreisen der Einzelteile berechnet wurde.

In Commit ca48cabb wurde das automatische Berechnen allerdings zu Gunsten einer freien Eingabe durch den Benutzer abgelöst: https://github.com/kivitendo/kivitendo-erp/commit/ca48cabb1cda087e3a27fa04dc64e4352e150446

Das Gleiche Verhalten gab es auch bei weight, wo dies aber durchaus noch sinnvoll ist, im Gegensatz zum Preis würde man beim Gewicht eher keinen Aufschlag oder Rabatt haben wollen.

Eventuell wäre es nützlich die Neuberechnung beizubehalten, aber nur auszuführen, wenn der Verkaufspreis 0 ist. Sobald ein Preis vorgegeben oder geändert wird würde nichts neu berechnet werden.

Zur Lösung des Problems: Bei normalen Artikeln funktioniert es, auch mit check_form, am Besten behandelt man das gleich. Das Gleiche für den Listenpreis.

#2 Updated by Bernd Bleßmann over 4 years ago

  • Subject changed from 2487 Preis bei Erzeugnissen to 2478 Preis bei Erzeugnissen
  • Description updated (diff)

#3 Updated by G. Richardson over 4 years ago

  • Description updated (diff)

#4 Updated by G. Richardson over 4 years ago

  • Status changed from Neu to Erledigt

Behoben in Commit 69c0d29be2f

Also available in: Atom PDF