]> wagnertech.de Git - kivitendo-erp.git/blobdiff - SL/PriceSource.pm
DATEV: Warnungen: Uninitialisierte Variablen
[kivitendo-erp.git] / SL / PriceSource.pm
index b34bf41638343bce8d91d10b68d5bd190ed9787d..831f68ebc99a0d035ac07d8053b16ca556b08d2a 100644 (file)
@@ -149,8 +149,12 @@ and it is up to the user to change a price.
 
 =item 2.
 
-If a price is set from a source, it is read only. A price edited manually is by
-definition not a sourced price.
+If a price is set from a source then the system will try to prevent the user
+from messing it up. By default this means the price will be read-only.
+Implementations can choose to make prices editable, but even then deviations
+from the calculatied price will be marked.
+
+A that is not set from a source will not have any of this.
 
 =item 3.
 
@@ -285,18 +289,27 @@ to be aware of units and price_factors. This is madness.
 
 =item *
 
-A common complaint is that prices from certain vendors are always negotiated
-and should use a default value but must be editable (like free prices) by
-default. This should be orthogonal for all prices.
-
-=item *
-
 The current implementation of lastcost is useless. Since it's one of the
 master_data prices it will always compete with listprice. But in real scenarios
 the listprice tends to go up, while lastcost stays the same, so lastcost
 usually wins. Lastcost could be lower priority, but a better design would be
 nice.
 
+=item *
+
+Guarantee 1 states that price sources will never change prices on their own.
+Further testing in the wild has shown that this is desirable within a record,
+but not when copying items from one record to another within a workflow.
+
+Specifically when changing from sales to purchase records prices don't make
+sense anymore. The guarantees should be updated to reflect this and
+transposition guidelines should be documented.
+
+=item *
+
+Prices were originally planned as a context element rather than a modal popup.
+It would be great to have this now with better framework.
+
 =back
 
 =head1 AUTHOR