diff options
| author | qsm-odoo <qsm@odoo.com> | 2017-03-22 14:55:17 +0100 |
|---|---|---|
| committer | Aaron Bohy <aab@odoo.com> | 2017-04-11 19:44:38 +0200 |
| commit | 3f8569a05702a8d14f18e8303e2f9418ec65a287 (patch) | |
| tree | 0566e821c1b45e3d3c6e253197a8409c7e63fbbf /addons/web/static/src/js/views/basic/basic_controller.js | |
| parent | eaa66d11568b415b8d6b4c7a17a058b5ec1df4a6 (diff) | |
[FIX] web: send array domains to the server in domain field
The `FieldDomain` was sending string domains instead of array domains
to the server (by its associated `BasicModel._fetchSpecialDomain`
function). The server does not do the conversion string -> array for
domains, so the RPC broke everytime.
This commit also modifies the `MockServer` so that tests also break
when sending string domains.
It also improves the way failed rpc are handled. Indeed when an rpc
fails, its deferred is rejected with two arguments: the error object,
and a JQuery event (that will display the error if not
preventDefaulted). The mockRPC didn't pass the second one and thus
there was a crash when reaching code that e.g. calls preventDefault
on that event. Note: this also removes the stringify/parse of the
error object as it is not a correct simulation of a server error
object anyway.
Diffstat (limited to 'addons/web/static/src/js/views/basic/basic_controller.js')
0 files changed, 0 insertions, 0 deletions
