[pve-devel] [PATCH v2 manager 18/20] allow child classes of ContentView to specify their own listeners

Fabian Ebner f.ebner at proxmox.com
Wed Sep 16 14:50:39 CEST 2020


Ext.apply would simply overwrite possibly existing listeners.

Signed-off-by: Fabian Ebner <f.ebner at proxmox.com>
---

new in v2

 www/manager6/storage/ContentView.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/www/manager6/storage/ContentView.js b/www/manager6/storage/ContentView.js
index e563d3a3..c5dd5206 100644
--- a/www/manager6/storage/ContentView.js
+++ b/www/manager6/storage/ContentView.js
@@ -351,11 +351,11 @@ Ext.define('PVE.storage.ContentView', {
 	    selModel: sm,
 	    tbar: me.tbar,
 	    columns: columns,
-	    listeners: {
-		activate: reload
-	    }
 	});
 
+	// don't overwrite existing listeners
+	me.listeners = Object.assign({}, { activate: reload }, me.listeners);
+
 	me.callParent();
     }
 }, function() {
-- 
2.20.1





More information about the pve-devel mailing list