plugin.Base: onOwnerConstructed() => auto-assign the windowId, if possible #6572

Author: tobiu

src/calendar/view/week/Component.mjs

@@ -359,21 +359,19 @@ class Component extends BaseComponent {
                 import('./plugin/DragDrop.mjs'),
                 import('./plugin/EventResizable.mjs')
             ]).then(modules => {
-                let me                  = this,
-                    {appName, windowId} = me,
-                    plugins             = me.plugins || [];
+                let me        = this,
+                    {appName} = me,
+                    plugins   = me.plugins || [];
 
                 plugins.push({
                     module : modules[0].default,
                     appName,
-                    windowId,
                     ...me.pluginDragDropConfig
                 }, {
                     module       : modules[1].default,
                     appName,
                     delegationCls: 'neo-event',
                     directions   : ['b', 't'],
-                    windowId,
                     ...me.pluginEventResizableConfig
                 });
 

src/dialog/Base.mjs

@@ -286,15 +286,14 @@ class Dialog extends Panel {
     afterSetResizable(value, oldValue) {
         if (value && !this.getPlugin('resizable')) {
             import('../plugin/Resizable.mjs').then(module => {
-                let me                  = this,
-                    {appName, windowId} = me,
-                    plugins             = me.plugins || [];
+                let me        = this,
+                    {appName} = me,
+                    plugins   = me.plugins || [];
 
                 plugins.push({
                     module       : module.default,
                     appName,
                     delegationCls: 'neo-dialog',
-                    windowId,
                     ...me.resizablePluginConfig
                 });
 

src/grid/Container.mjs

@@ -232,14 +232,13 @@ class GridContainer extends BaseContainer {
     afterSetCellEditing(value, oldValue) {
         if (value) {
             import('./plugin/CellEditing.mjs').then(module => {
-                let me                  = this,
-                    {appName, windowId} = me,
-                    plugins             = me.plugins || [];
+                let me        = this,
+                    {appName} = me,
+                    plugins   = me.plugins || [];
 
                 plugins.push({
-                    module : module.default,
-                    appName,
-                    windowId
+                    module: module.default,
+                    appName
                 });
 
                 me.plugins = plugins

src/grid/View.mjs

@@ -226,9 +226,7 @@ class GridView extends Component {
                     plugins = me.plugins || [];
 
                 plugins.push({
-                    module  : module.default,
-                    appName : me.appName,
-                    windowId: me.windowId,
+                    module: module.default,
                     ...me.pluginAnimateRowsConfig
                 });
 

src/list/Base.mjs

@@ -206,9 +206,7 @@ class List extends Component {
                     plugins = me.plugins || [];
 
                 plugins.push({
-                    module  : module.default,
-                    appName : me.appName,
-                    windowId: me.windowId,
+                    module: module.default,
                     ...me.pluginAnimateConfig
                 });
 

src/plugin/Base.mjs

@@ -71,7 +71,11 @@ class Plugin extends Base {
      * Override this method to apply changes to the owner Component when it is constructed
      */
     onOwnerConstructed() {
+        let {owner} = this;
 
+        if (owner.windowId) {
+            this.windowId = owner.windowId
+        }
     }
 
     /**

src/table/Container.mjs

@@ -161,14 +161,13 @@ class Container extends BaseContainer {
     afterSetCellEditing(value, oldValue) {
         if (value) {
             import('./plugin/CellEditing.mjs').then(module => {
-                let me                  = this,
-                    {appName, windowId} = me,
-                    plugins             = me.plugins || [];
+                let me        = this,
+                    {appName} = me,
+                    plugins   = me.plugins || [];
 
                 plugins.push({
-                    module : module.default,
-                    appName,
-                    windowId
+                    module: module.default,
+                    appName
                 });
 
                 me.plugins = plugins