diff options
| author | Paramjit Singh Sahota <psa@tinyerp.com> | 2013-04-04 12:37:23 +0530 |
|---|---|---|
| committer | Paramjit Singh Sahota <psa@tinyerp.com> | 2013-04-04 12:37:23 +0530 |
| commit | b32cc3b4427c6d59581da13d039bde931ce82a5c (patch) | |
| tree | ea7a54e00b863c7ebc8bbc5ac82a3da99d957801 /addons/document/document_view.xml | |
| parent | 06986e47375175b8228e13b498d3503ec1615373 (diff) | |
[IMP] Improved code
1) Edit Subscriptions of -> Edit Subscription (without 's') of
2) Instead of defining a new 'apply_edited_subtype', you can use 'message_subscribe' that has an optional 'subtype_ids' parameter that already allows to manage subtypes of a subscription.
3) In _get_subscription_data: do not use context when it is not necessary. Please add a parameter user_id=None that will be used instead of uid to find user_pid.
4) Instead of defining a new 'on_apply_subtype', can't you reuse 'do_update_subscription' with maybe some update to handle the classic use / edit widget ?
The above changes has been done.
bzr revid: psa@tinyerp.com-20130404070723-5v1v35fte6vl2b2j
Diffstat (limited to 'addons/document/document_view.xml')
0 files changed, 0 insertions, 0 deletions
