Dolibarr ERP & CRM » Bugs » bug #1232 Propal/Order/Invoice = >Extrafield on free lines are not saved Latest modifications
Answer now
Snapshot| Details |
| Last Modified On: | 2014-02-03 15:34 | | Submitted by: | HENRY Florian (fhenry) |
| Submitted on: | 2014-01-29 14:12 | |
| Summary: | Propal/Order/Invoice = >Extrafield on free lines are not saved |
| Description: | Define extrafeild on line propal.
Edit a proposal, add a free product line and put a value into extrafeild define. The value of extrafield is not saved.
|
| Step to reproduce bug: | Define extrafeild on line propal.
Edit a proposal, add a free product line and put a value into extrafeild define. The value of extrafield is not saved.
problem come from global from that have similar input name for extrafield into free product line and defined product line. So the value posted are the last (the one from predifined product lines eg nothing because we input value on free product line).
It was working fine when the orm were define into add line template but as from now they are outside, i don't know easy solution. |
| Detected in version: | develop | | Category: | Core Problem |
| Severity: | 5 - Major | | OS Type/Version: | Ubuntu 13.04 |
| PHP version: | PHP 5.4 | | Database type and version: | MysqL 5 |
| Status |
| Status: | Open | | Assigned to: | Laurent Destailleur (eldy) |
| Resolution: | Fixed | |
Comments- Laurent Destailleur 2014-02-17 18:12
- The bug has been corrected inside GIT sources
(http://www.github.com/Dolibarr/dolibarr).
So fix should be available with next stable release. - benke 2014-01-29 14:31
- the problem is caused by the use of only one form/table for all the "addline" screen, i think that the problem is not only on the extrafields fields
the good and proper way seems to create 5 tables and 5 forms for the propal, order and bills main page
1 form/table for the printobjectlines
1 form/table for the formAddObjectLine
1 form/table for the formAddFreeProduct
1 form/table for formAddPredefinedProduct
and finaly 1 form/table for the hook (allready done on the milestone)
for a good structure of code propose that the form/table was written on tpl file
|
|
(http://www.github.com/Dolibarr/dolibarr).
So fix should be available with next stable release.
None→ Laurent Destailleur (eldy)None→ Fixed