Merge lp:~openerp-dev/openobject-addons/6.1-opw-573631-msh into lp:openobject-addons/6.1

Proposed by Mohammed Shekha(Open ERP)
Status: Approved
Approved by: Naresh(OpenERP)
Approved revision: 6742
Proposed branch: lp:~openerp-dev/openobject-addons/6.1-opw-573631-msh
Merge into: lp:openobject-addons/6.1
Diff against target: 23 lines (+3/-3)
1 file modified
share/static/src/js/share.js (+3/-3)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/6.1-opw-573631-msh
Reviewer Review Type Date Requested Status
Naresh(OpenERP) (community) Approve
Review via email: mp+102099@code.launchpad.net

Description of the change

Hello,

Fixed the issue of context not passed in share from sidebar.

Here we have web-addons share in which we have added a share link in sidebar and launch wizard by calling the "go_step_1" method but here we have forgot to pass context, due to this the string of wizard title never comes language specific.

Hence changed the code.

Thanks.

To post a comment you must log in.
Revision history for this message
Naresh(OpenERP) (nch-openerp) :
review: Approve
Revision history for this message
Naresh(OpenERP) (nch-openerp) wrote :

Hello,

This bug was qualified as Confirmed on Trunk (means still existing and reproducible). A Merge Proposal for trunk was created to fix it. Here is the link to follow the MP on Launchpad https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-opw-573631-port-mma/+merge/132239 and be informed once it's been merged in trunk: ... If this Merge Proposal could not be merged in v6.1 at the release of v7.0, it will be closed.

Unmerged revisions

6742. By Mohammed Shekha(Open ERP)

[FIX]Fixed the issue of context not passed in share.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'share/static/src/js/share.js'
--- share/static/src/js/share.js 2011-12-27 20:36:18 +0000
+++ share/static/src/js/share.js 2012-04-16 12:48:19 +0000
@@ -11,16 +11,16 @@
11 self.rpc('/web/session/eval_domain_and_context', {11 self.rpc('/web/session/eval_domain_and_context', {
12 domains: [domain],12 domains: [domain],
13 contexts: [view.dataset.context]13 contexts: [view.dataset.context]
14 }, function (result) {14 }, function (results) {
15 Share.create({15 Share.create({
16 name: action.name,16 name: action.name,
17 domain: result.domain,17 domain: results.domain,
18 action_id: action.id,18 action_id: action.id,
19 user_type: user_type || 'embedded',19 user_type: user_type || 'embedded',
20 view_type: view.fields_view.type,20 view_type: view.fields_view.type,
21 }, function(result) {21 }, function(result) {
22 var share_id = result.result;22 var share_id = result.result;
23 var step1 = Share.call('go_step_1', [[share_id],], function(result) {23 var step1 = Share.call('go_step_1', [[share_id], results.context], function(result) {
24 var action = result;24 var action = result;
25 self.do_action(action);25 self.do_action(action);
26 });26 });