lp:~fabien-morin/unifield-server/fm-us-2888

Branch merges

Propose for merging

No branches dependent on this one.

Rejected for merging into lp:unifield-server

UniField Reviewer Team: Pending requested 2017-05-17

Diff: 365 lines (+127/-60)

10 files modified

bin/addons/base/ir/ir_translation.py (+1/-1)
bin/addons/msf_doc_import/initial_stock_inventory.py (+3/-1)
bin/addons/msf_tools/msf_tools.py (+20/-20)
bin/addons/product/product.py (+2/-2)
bin/addons/reason_types_moves/reason_type.py (+6/-6)
bin/addons/specific_rules/specific_rules.py (+2/-0)
bin/addons/specific_rules/stock.py (+2/-1)
bin/addons/stock/stock.py (+73/-21)
bin/osv/expression.py (+10/-5)
bin/osv/fields.py (+8/-3)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

4357. By Fabien MORIN on 2017-05-22

US-2888 [IMP] if offset is used, then the NO_ORDER is dangerous and should not
be present.

4356. By Fabien MORIN on 2017-05-22

US-2888 [FIX] as initial.stock.inventory share code with stock.inventory,
handle the case of initial inventory call of action_confirm.

4355. By Fabien MORIN on 2017-05-22

US-2888 [IMP] Do not order on place there is no need to do it:
order a search to give the result to a read/read_flat is useless as it will be
reordered in this methods.
order to build a dict is useless as a dict is not ordered.

4354. By Fabien MORIN on 2017-05-22

US-2888 [FIX] not_chained=True must be set during the _inventory_line_hook. It
has been removed by mistake
[IMP] cosmetic changes

4353. By Fabien MORIN on 2017-05-22

US-2888 [MERGE] with latest trunk

4352. By Fabien MORIN on 2017-05-18

US-2888 [IMP] revert to the original revision

4351. By Fabien MORIN on 2017-05-18

US-2888 [FIX] do not escape the '' as there is no more need to do it.
[FIX] update move_vals in all cases as the value getted on inventory_line can
be different than the one in move_vals. This doesn't change the small perf
improvments on the Physical Inventory validation as in this case inventory_line
is None (because the values are always set into move_vals).
[FIX] revert the order='NO_ORDER' has it have some side effect: in some cases
the name_get method don't use a read but a browse (which don't re-order the
result but uses the same order than ids), in this case the order would be
different from the previous behaviour.

4350. By Fabien MORIN on 2017-05-12

US-2888 [ADD] explanations

4349. By Fabien MORIN on 2017-05-12

US-2888 [IMP] make inventory line faster by doing less request:
before starting to iterate on all lines, gather all informations needed by
doing one request per information for all lines (instead of one request per
information per line).

Do not use browse which is horrible in term of performance when iterate on a
lot of object and access a lot of sub objects of them, for example:

location_id = line.product_id.product_tmpl_id.property_stock_inventory.id

=> this line is horribly slow.

[IMP] pass some informations to the methods when we have it to avoid the method
to do requests to find it (comment, reason_type_id)

4348. By Fabien MORIN on 2017-05-12

US-2888 [IMP] add index on columns that are slow and frequently used