@odoo/owl 1.4.5 → 1.4.9

package/README.md

@@ -124,7 +124,7 @@ npm install @odoo/owl
 
 If you want to use a simple `<script>` tag, the last release can be downloaded here:
 
-- [owl-1.4.5](https://github.com/odoo/owl/releases/tag/v1.4.5)
+- [owl-1.4.9](https://github.com/odoo/owl/releases/tag/v1.4.9)
 
 ## License
 

package/dist/owl.cjs.js

@@ -360,6 +360,7 @@ const isRightSeparator = (token) => token && (token.type === "RIGHT_BRACE" || to
  * the list of variables so it does not get replaced by a lookup in the context
  */
 function compileExprToArray(expr, scope) {
+    const localVars = new Set();
     scope = Object.create(scope);
     const tokens = tokenize(expr);
     let i = 0;
@@ -385,7 +386,7 @@ function compileExprToArray(expr, scope) {
                 if (groupType === "LEFT_BRACE" &&
                     isLeftSeparator(prevToken) &&
                     isRightSeparator(nextToken)) {
-                    tokens.splice(i + 1, 0, { type: "COLON", value: ":" }, { ...token });
+                    tokens.splice(i + 1, 0, { type: "COLON", value: ":" }, Object.assign({}, token));
                     nextToken = tokens[i + 1];
                 }
                 if (prevToken.type === "OPERATOR" && prevToken.value === ".") {
@@ -408,12 +409,14 @@ function compileExprToArray(expr, scope) {
                     if (tokens[j].type === "SYMBOL" && tokens[j].originalValue) {
                         tokens[j].value = tokens[j].originalValue;
                         scope[tokens[j].value] = { id: tokens[j].value, expr: tokens[j].value };
+                        localVars.add(tokens[j].value);
                     }
                     j--;
                 }
             }
             else {
                 scope[token.value] = { id: token.value, expr: token.value };
+                localVars.add(token.value);
             }
         }
         if (isVar) {
@@ -428,6 +431,13 @@ function compileExprToArray(expr, scope) {
         }
         i++;
     }
+    // Mark all variables that have been used locally.
+    // This assumes the expression has only one scope (incorrect but "good enough for now")
+    for (const token of tokens) {
+        if (token.type === "SYMBOL" && localVars.has(token.value)) {
+            token.isLocal = true;
+        }
+    }
     return tokens;
 }
 function compileExpr(expr, scope) {
@@ -579,7 +589,18 @@ class CompilationContext {
         const done = new Set();
         return tokens
             .map((tok) => {
-            if (tok.varName) {
+            // "this" in captured expressions should be the current component
+            if (tok.value === "this") {
+                if (!done.has("this")) {
+                    done.add("this");
+                    this.addLine(`const this_${argId} = utils.getComponent(context);`);
+                }
+                tok.value = `this_${argId}`;
+            }
+            // Variables that should be looked up in the scope. isLocal is for arrow
+            // function arguments that should stay untouched (eg "ev => ev" should
+            // not become "const ev_1 = scope['ev']; ev_1 => ev_1")
+            if (tok.varName && !tok.isLocal) {
                 if (!done.has(tok.varName)) {
                     done.add(tok.varName);
                     this.addLine(`const ${tok.varName}_${argId} = ${tok.value};`);
@@ -892,16 +913,16 @@ function createKeyToOldIdx(children, beginIdx, endIdx) {
     }
     return map;
 }
-const hooks = ["create", "update", "remove", "destroy", "pre", "post"];
+const hooks$1 = ["create", "update", "remove", "destroy", "pre", "post"];
 function init(modules, domApi) {
     let i, j, cbs = {};
     const api = domApi !== undefined ? domApi : htmlDomApi;
-    for (i = 0; i < hooks.length; ++i) {
-        cbs[hooks[i]] = [];
+    for (i = 0; i < hooks$1.length; ++i) {
+        cbs[hooks$1[i]] = [];
         for (j = 0; j < modules.length; ++j) {
-            const hook = modules[j][hooks[i]];
+            const hook = modules[j][hooks$1[i]];
             if (hook !== undefined) {
-                cbs[hooks[i]].push(hook);
+                cbs[hooks$1[i]].push(hook);
             }
         }
     }
@@ -2510,7 +2531,8 @@ QWeb.addDirective({
         ctx.addLine(`if (!_${arrayID}) { throw new Error('QWeb error: Invalid loop expression')}`);
         let keysID = ctx.generateID();
         let valuesID = ctx.generateID();
-        ctx.addLine(`let _${keysID} = _${valuesID} = _${arrayID};`);
+        ctx.addLine(`let _${keysID} = _${arrayID};`);
+        ctx.addLine(`let _${valuesID} = _${arrayID};`);
         ctx.addIf(`!(_${arrayID} instanceof Array)`);
         ctx.addLine(`_${keysID} = Object.keys(_${arrayID});`);
         ctx.addLine(`_${valuesID} = Object.values(_${arrayID});`);
@@ -2924,9 +2946,6 @@ Object.defineProperty(config, "mode", {
 This is not suitable for production use.
 See https://github.com/odoo/owl/blob/master/doc/reference/config.md#mode for more information.`);
         }
-        else {
-            console.log(`Owl is now running in 'prod' mode.`);
-        }
     },
 });
 Object.defineProperty(config, "enableTransitions", {
@@ -3179,7 +3198,12 @@ QWeb.addDirective({
             else if (!name.startsWith("t-")) {
                 if (name !== "class" && name !== "style") {
                     // this is a prop!
-                    props[name] = ctx.formatExpression(value) || "undefined";
+                    if (value.includes("=>")) {
+                        props[name] = ctx.captureExpression(value);
+                    }
+                    else {
+                        props[name] = ctx.formatExpression(value) || "undefined";
+                    }
                 }
             }
         }
@@ -3662,6 +3686,7 @@ class Fiber {
         // build patchQueue
         const patchQueue = [];
         const doWork = function (f) {
+            f.component.__owl__.currentFiber = null;
             patchQueue.push(f);
             return f.child;
         };
@@ -3704,8 +3729,9 @@ class Fiber {
                 component.__patch(target, fiber.vnode);
             }
             else {
-                if (fiber.shouldPatch) {
-                    component.__patch(component.__owl__.vnode, fiber.vnode);
+                const vnode = component.__owl__.vnode;
+                if (fiber.shouldPatch && vnode) {
+                    component.__patch(vnode, fiber.vnode);
                     // When updating a Component's props (in directive),
                     // the component has a pvnode AND should be patched.
                     // However, its pvnode.elm may have changed if it is a High Order Component
@@ -3718,10 +3744,6 @@ class Fiber {
                     component.__owl__.pvnode.elm = component.__owl__.vnode.elm;
                 }
             }
-            const compOwl = component.__owl__;
-            if (fiber === compOwl.currentFiber) {
-                compOwl.currentFiber = null;
-            }
         }
         // insert into the DOM (mount case)
         let inDOM = false;
@@ -4375,7 +4397,6 @@ class Component {
     __callMounted() {
         const __owl__ = this.__owl__;
         __owl__.status = 3 /* MOUNTED */;
-        __owl__.currentFiber = null;
         this.mounted();
         if (__owl__.mountedCB) {
             __owl__.mountedCB();
@@ -4668,7 +4689,7 @@ function partitionBy(arr, fn) {
         return acc;
     }, []);
 }
-class Context extends EventBus {
+class Context$1 extends EventBus {
     constructor(state = {}) {
         super();
         this.rev = 1;
@@ -4781,7 +4802,7 @@ function useContextWithCB(ctx, component, method) {
  * will return an observed object (or array).  Changes to that value will then
  * trigger a rerendering of the current component.
  */
-function useState(state) {
+function useState$1(state) {
     const component = Component.current;
     const __owl__ = component.__owl__;
     if (!__owl__.observer) {
@@ -4917,7 +4938,7 @@ function useExternalListener(target, eventName, handler, eventParams) {
 
 var _hooks = /*#__PURE__*/Object.freeze({
     __proto__: null,
-    useState: useState,
+    useState: useState$1,
     onMounted: onMounted,
     onWillUnmount: onWillUnmount,
     onWillPatch: onWillPatch,
@@ -4931,7 +4952,7 @@ var _hooks = /*#__PURE__*/Object.freeze({
     useExternalListener: useExternalListener
 });
 
-class Store extends Context {
+class Store$1 extends Context$1 {
     constructor(config) {
         super(config.state);
         this.actions = config.actions;
@@ -4970,7 +4991,7 @@ function useStore(selector, options = {}) {
     const component = Component.current;
     const componentId = component.__owl__.id;
     const store = options.store || component.env.store;
-    if (!(store instanceof Store)) {
+    if (!(store instanceof Store$1)) {
         throw new Error(`No store found when connecting '${component.constructor.name}'`);
     }
     let result = selector(store.state, component.props);
@@ -5517,15 +5538,15 @@ function makeExtractionRegExp(path) {
  *
  * Note that dynamic values, such as a date or a commit hash are added by rollup
  */
-const Context$1 = Context;
-const useState$1 = useState;
+const Context = Context$1;
+const useState = useState$1;
 const core = { EventBus, Observer };
 const router = { Router, RouteComponent, Link };
-const Store$1 = Store;
+const Store = Store$1;
 const utils = _utils;
 const tags = _tags;
 const misc = { AsyncRoot, Portal };
-const hooks$1 = Object.assign({}, _hooks, {
+const hooks = Object.assign({}, _hooks, {
     useContext: useContext,
     useDispatch: useDispatch,
     useGetters: useGetters,
@@ -5534,23 +5555,23 @@ const hooks$1 = Object.assign({}, _hooks, {
 const __info__ = {};
 
 exports.Component = Component;
-exports.Context = Context$1;
+exports.Context = Context;
 exports.QWeb = QWeb;
-exports.Store = Store$1;
+exports.Store = Store;
 exports.__info__ = __info__;
 exports.browser = browser;
 exports.config = config;
 exports.core = core;
-exports.hooks = hooks$1;
+exports.hooks = hooks;
 exports.misc = misc;
 exports.mount = mount;
 exports.router = router;
 exports.tags = tags;
-exports.useState = useState$1;
+exports.useState = useState;
 exports.utils = utils;
 
 
-__info__.version = '1.4.5';
-__info__.date = '2021-09-22T14:34:13.251Z';
-__info__.hash = '37313c4';
+__info__.version = '1.4.9';
+__info__.date = '2021-12-07T09:21:41.690Z';
+__info__.hash = '73f94fb';
 __info__.url = 'https://github.com/odoo/owl';