Dolibarr ERP & CRM » Tasks and Feature requests » task #645
Remove field fk_product_fournisseur_price from all "->addline" and "->updateline" method. Remove fields from details tables.Latest modifications
Answer now
Snapshot| Details |
| Submitted on: | 2012-12-19 15:57 | | Submitted by: | Laurent Destailleur (eldy) |
| Last Modified On: | 2012-12-19 17:42 | |
| Summary: | Remove field fk_product_fournisseur_price from all "->addline" and "->updateline" method. Remove fields from details tables. |
| Description: | Remove field fk_product_fournisseur_price from all addline method and details tables.
This field is useless, because the pa_ht is used instead for margin calculation. |
| Priority: | 9 - Highest | | Open to Any Developer: | No |
| Subproject: | None | | Assigned to (multiple): | None |
| Status |
| Start Date: | - | | End Date: | - |
| Completion (0-100%): | 0 | | Status: | Open |
Comments- Laurent Destailleur 2013-06-05 22:34
- Price used when order is defined must be saved into pa_ht.
So price can change after, it is not a problem since value used was saved during order creation. There is no reason to have a link to a table that is a view at t time, so a value useless for calculation since what we need is value when element was created. - address@hidden 2012-12-19 22:34
- This field is used in case supplier price change between order and invoice
- Laurent Destailleur 2012-12-19 17:42
- Remove field fk_product_fournisseur_price from all "->addline" and "->updateline" method.
Remove fields from details tables.
Remove also field label that are duplicate fields and create conflicts.
|
|
So price can change after, it is not a problem since value used was saved during order creation. There is no reason to have a link to a table that is a view at t time, so a value useless for calculation since what we need is value when element was created.