* non-deterministic execution (depending on having a chart of account
loaded before it), unreliable as demonstration dataset
* hot mess, very debatable purpose
* apparently not used by membership tests (since unreliable execution)
* most expensive single data file when it happens to run in full (12~15s
on runbot)
* heresy
- Move product_product 'Deposit' from demo to data + make it a product_template
- Remove group_no_one on archive button for product_product and template
- Remove a commented import_pdb; pdb.set_trace()
- Add in the sales config 2 fields to edit the account and the taxes for the deposit product
- On the sale orders lines invoicing wizard, display the account and the taxes for the deposit product
instead of the product itself
- Set the product 'Deposit' as inactive to be hidden to the users
If a method used as a default returns an id instead of a RecordSet,
it could fail in 'convert_to_write' (Many2One field) because it will try
to return "value.id". If "value" is an integer, it fails.
Also changed the method to a lambda expression and removed the
'_default_user' method because it's not used anywhere.
Indexes on individual columns (module, name, model, res_id) have been dropped,
and the index on (model, name) was duplicated in a unicity constraint. We have
kept two useful indexes, namely:
- a unique index on (module, name)
- a simple index on (model, res_id)
These templates were deleted with c04065abd8 but there xml_id were still used in the code. Mail template are good for people, so we make them come back from the dead ...
Do not use `openerp.http.addons_manifest` dict to find the absolute
on-disk path of an asset. This dict is only populated when the first
non-xmlrpc http request is made. Avoid crash if this happen by using
`get_resource_path` to find ... the resource path we search.
The button Mark As Todo should be before the button Validate because
most people check things when they transfer stuff in their warehouse.
Moreover, it creates confusion, because people will think Validate means Confirm
(or Mark As Todo).
When validating an empty picking, it will give an error now.
On_change of product on stock moves and pack operations should return
a domain for the UoM.
As every pack operation can have several lots now,
we do an extra check if the qties in these lots details
match the total quantity of the pack operation.
There was an entire_pack key put in the context we want
to avoid by passing it as parameter.
Before this commit, you had to click on "Add a line", then select
a value in the m2o which appeared, then click on "Add". Now, the m2o
is always there in edit mode and you just have to click "Add" once.
When a new line was added, all the others changed their value to 0.
This was because of a property : the sheets propertywas read thanks
to a get('sheets') but as it is an array of object, modifying those
object will modify the sheets (without calling any handler attached
to the 'change:sheets' event). It appeared the form_view set_values
method changed those objects.
A simple solution from a timesheet point of vue was to deep clone the
sheets property before using it.
The test was not stable enough since listview editable and onchange
API changes.
+ Some step were wrong (test of the wrong element)
+ Minimize differences with enterprise version (so that the two tests
are testing the same things)
(backport of enterprise version)
Fix various problems with list view editable :
* Click outside which should discard/save the line in edition
* Warnings which should not appear when line is being discarded/saved
* JS test
+ JS code refactoring
Xavier Morel