Formatierung EK-Preis bei Einzelteilen in Erzeugnis erstellen falsch

[grichardson@…]

Legt man ein Erzeugnis an und fügt einen Artikel mit EK-Preis als Position unter "Einzelteile" hinzu, wird nach einem erneuten "Erneuern" der EK-Preis falsch formatiert (6,66 -> 666.000,00)

Ein bisect ergibt, dass ich das war (commit 9a7a811d2d254f95f3cf1664566ff4a5335eee31 - Fix für #1647).

Allerdings tritt das Problem durch diesen commit erst zu Tage, liegt aber eigentlich woanders:
Sowohl die Lieferanten-EK-Preise, als auch die EK-Preise der Einzelteile eines Erzeugnisse werden in der Form-Variablen lastcost_$i abgelegt, was natürlich nicht geht.

Wenn man zusätzlich noch einen Lieferanten-Einkaufspreis anlegt, sieht man das Problem. Dieser Preis wird nämlich mit dem des Einzelteils überschrieben.

Hinzu kamen die Lieferanten-EK m.W. nach in commit 85ffde7a577589762b70de99ca08b6f25e9e9d1b

Erzeugnisse sind doch eigentlich immer interne Produkte die aus vorhanden Einzelteilen hergestellt werden, und eben nicht eingekauft werden, daher macht hier auch ein Lieferant keinen Sinn. Nimmt man makemodel aus Erzeugnissen raus ist #1674 zumindest behoben. Patch wird angehängt, vielleicht hat ja jemand Einwände.

Makemodel aus Erzeugnis rausnehmen

Sicher, dass das eine gute Idee ist? Man kann Erzeugnisse ja auch wahlweise entweder im eigenen Hause oder aber mit an den Fertiger gelieferten Einzelteilen (fremd)fertigen lassen.
Wenn das Feld "Lieferant" irgendwann mal im Einkaufsbereich in Verwendung kommt (bisher wird es ja mehr oder weniger der Vollständigkeit halber nur mitgeführt), könnte es durchaus auch bei Erzeugnissen gebraucht werden.

(In reply to comment #5)

Sicher, dass das eine gute Idee ist?

Nö, sonst hätte ich das auch direkt eingecheckt ;-)

Man kann Erzeugnisse ja auch wahlweise
entweder im eigenen Hause oder aber mit an den Fertiger gelieferten
Einzelteilen (fremd)fertigen lassen.

In dem Fall stellt mir der Fertiger aber wahrscheinlich nur eine Rechnung für die Fertigungsdienstleistung, ich kann die Erzeugnisse trotzdem intern über "Erzeugnis fertigen" erstellen. Wenn ich erst die Einzelteile an den Fertiger liefern (und auslagern) würde, und dann die fertige Ware (als Einkauf) zurückbekäme, wäre das nicht nur umständlich, sondern da bräuchte ich das ja auch nicht wie ein Erzeugnis behandeln, sondern könnte das wie eine normale Ware behandeln.
Hier kenne ich aber die typischen Arbeitsabläufe nicht.

Läßt man das makemodel bei Erzeugnissen drin müßte man ansonsten innerhalb der $form die lastcosts besser unterscheiden (nach dem was Bernd geschrieben hat), das habe ich mir gar nicht angeschaut.

(In reply to comment #6)

Hier kenne ich aber die typischen Arbeitsabläufe nicht.

Das war auch nur ein genereller Einwand, und außerdem wird ja bisher nicht verhindert, dass man Erzeugnisse mit einem Lieferschein einlagert. Mein Einwand käme erst dann zum Tragen, wenn die Lieferanten überhaupt mal eine Funktion bekommen.

Ich hatte dieses Ticket komplett vergessen, habe eben in Commit
822b71c22ca6dbe4fe024babaf6a8f0be8d3c4e9 im Prinzip den oben angehängten Patch eingespielt, allerdings an der betreffenden Stelle etwas weniger auskommentiert, wahrscheinlich kann das auch noch raus.

Wenn Erzeugnisse zusammen mit makemodel benutzt werden sollen müssten erst zumindest die lastcosts Unterschieden werden, das bisherige Verhalten war jedenfalls kaputt.