Merge lp://qastaging/~numerigraphe-team/ocb-addons/7.0-inventory-readonly-lines into lp://qastaging/ocb-addons
| Status: | Rejected |
|---|---|
| Rejected by: | Holger Brunn (Therp) |
| Proposed branch: | lp://qastaging/~numerigraphe-team/ocb-addons/7.0-inventory-readonly-lines |
| Merge into: | lp://qastaging/ocb-addons |
| Diff against target: |
96 lines (+28/-12) 1 file modified
stock/stock_view.xml (+28/-12) |
| To merge this branch: | bzr merge lp://qastaging/~numerigraphe-team/ocb-addons/7.0-inventory-readonly-lines |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Holger Brunn (Therp) | Disapprove | ||
|
Review via email:
|
|||
This proposal supersedes a proposal from 2014-03-10.
Description of the change
As usability improvement in inventories, we propose to make the fields "product", "location" and "serial nb" read-only as soon as an inventory line is recorded.
This keeps users from mistakenly keeping wrong products in stock.
Consider you filled the inventory with the wizard :
Product A: 15
Product B : 20
And physically you really have :
Product A: 15
Product C: 20
It's tempting to change just the product on the 2nd line but it's wrong: that would not remove product B from the stock.
You actually have to set the 2nd line to 0 and add a 3rd line to record the right data.
There is a proposal to merge this into the trunk too, but it's in standby until the new WMS code gets merged.
| Holger Brunn (Therp) (hbrunn) wrote | # |
| Holger Brunn (Therp) (hbrunn) | # |
Unmerged revisions
- 8593. By Numérigraphe
-
[REF] use != instead of XML entities to make the code readble (continued)
- 8592. By Numérigraphe
-
[REF] use != instead of XML entities to make the code readble
- 8591. By Loïc Bellier - Numérigraphe
-
[IMP] make fields readonly on recorded inventory lines, so users are not tempted to change location/
product/ lot instead of creating new lines
Yes, please link a bug report.
For readability, you might want to consider using '!=' instead of '<>'