how to detach a shortcut command on Qooxdoo? - qooxdoo

Question

I have this code:
qx.Class.define('my.Window', {
extend: qx.ui.window.Window,
construct: function(caption, icon) {
this.base(arguments, caption, icon);
this.setLayout(new qx.ui.layout.Basic());
this.__btn = new qx.ui.form.Button('Shortcut Test');
this.__cmd = new qx.ui.command.Command('Alt+T');
this.__cmd.addListener("execute", function() { alert('FOOBAR'); });
this.__btn.setCommand(this.__cmd);
this.add(this.__btn);
},
members: {
__btn: null,
__cmd: null
}
});
qx.Class.define('my.Compo', {
extend: qx.ui.container.Composite,
construct: function() {
this.base(arguments);
this.setLayout(new qx.ui.layout.HBox());
this.__btnShow = new qx.ui.form.Button("Show Window", "icon/22/apps/internet-web-browser.png");
this.__btnDestroy = new qx.ui.form.Button('Destroy window');
this.__btnNull = new qx.ui.form.Button('Null window');
this.__btnDestroy.addListener('execute', function(){
this.__window.destroy();
}, this);
this.__btnNull.addListener('execute', function(){
this.__window = null;
}, this);
this.__btnShow.addListener("execute", function(e){
if(this.__window) {
console.info('Window exist');
this.__window.open();
this.__window.center();
}
else {
console.info('Window do not exist!');
this.__window = new my.Window("Shortcut test window");
this.__window.setWidth(300);
this.__window.setHeight(200);
this.__window.setShowMinimize(false);
this.__window.open();
this.__window.center();
}
}, this);
this.add(this.__btnShow);
this.add(this.__btnDestroy);
this.add(this.__btnNull);
},
members: {
__btnShow: null,
__btnDestroy: null,
__window: null
}
});
var compo = new my.Compo();
this.getRoot().add(compo);
So, if you try "Alt+T" shortcut before clicking "Show Window" button, nothing happen. After show the Window, the shortcut is available and an alert is showed.
Well, the issue to me is the remaining availability of the shortcut in scenarios where that shortcut must do not exist any more:
When the window is close normally.
When Null window button is execute, and after that if you use "Alt+T" then the alert is showed twice, and so on as many time you switch between the Null and Show buttons.
Same behavior as (2) even if destroy() method of window is called explicitly.
On playground if "Run" button is clicked several times, also equal times is showed alert after use the shortcut.
Thanks no all for your time. :)
On Playground

Answer 1

The qx.ui.command.Command wraps the qx.bom.Shortcut that attaches 2 listeners to document element. When you close the window qx.ui.command.Command instance was not set inactive or destroyed. You'll have to handle the window close event properly.
destruct: function()
{
this.__cmd = null;
}
it does not destroy the command. If you try:
qx.core.ObjectRegistry.fromHashCode("the command object hash code").getActive()
You will find that it exists and is active. You forgot to destroy the command calling
this.__cmd.dispose()
Qooxdoo Playground App object registry is not initialized every time someone pushes "Run". So, qooxdoo object's lifetime is bound to page lifetime or a disposal event.