Merge lp:~camptocamp/server-env-tools/7.0-fix-1302500 into lp:~server-env-tools-core-editors/server-env-tools/7.0

[Yannick Vaucher @ Camptocamp]

[Luis Felipe Miléo - http://www.kmee.com.br (mileo)]

LGTM.

Thanks

[Holger Brunn (Therp) (hbrunn)]

Won't you lose the field's context here if there's one set?

review: Needs Information

[lp:~camptocamp/server-env-tools/7.0-fix-1302500]

70. By Yannick Vaucher @ Camptocamp on 2014-04-07

remove set_eval_context as initialized in init

[Yannick Vaucher @ Camptocamp (yvaucher-c2c)]

@Holger Context elements are passed in CompoundContext init with v_context.

From this function, we need to provide a CompundContext or it will raise an error when trying to use add method on the returned value.

I removed the set_eval_context as it isn't needed. `this.__eval_context` is already set to null in init function.

[Holger Brunn (Therp) (hbrunn)]

Thanks Yannick!

review: Approve (code review)

[Raphaël Valyi - http://www.akretion.com (rvalyi)]

Thanks for spotting this Yannick

review: Approve

[Luis Felipe Miléo - http://www.kmee.com.br (mileo)]

@rvalyi

That was the "web" bug that I had spoken last week hangout

[]s

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

Thanks

review: Approve

=== modified file 'web_context_tunnel/static/src/js/context_tunnel.js'
--- web_context_tunnel/static/src/js/context_tunnel.js	2013-12-26 16:23:00 +0000
+++ web_context_tunnel/static/src/js/context_tunnel.js	2014-04-07 13:38:15 +0000
@@ -17,6 +17,7 @@
         }
         if (!v_context) {
             v_context = (this.field || {}).context || {};
+            v_context = new instance.web.CompoundContext(v_context);
         }
         return v_context;
     };