look goods to me

maybe we should have mentioned that this is needed for having the
'autoDestroy' store property working properly. Otherwise it is not clear
why we needed this in the pve context, as all updates store are unqueued
manually with

component.on('destroy', componentstore.stopUpdate);

and the like

On 10/11/2017 11:49 AM, Dominik Csapak wrote:
> the store never fires/gets a destroy event, thus it
> never stops the load_task and never unqueues itself in case
> the store gets destroyed but not explicitely stopped
> 
> with overriding the destroy method, this works now as intended
> 
> Signed-off-by: Dominik Csapak <d.csa...@proxmox.com>
> ---
>  www/manager6/data/UpdateStore.js | 12 ++++++++----
>  1 file changed, 8 insertions(+), 4 deletions(-)
> 
> diff --git a/www/manager6/data/UpdateStore.js 
> b/www/manager6/data/UpdateStore.js
> index 6be36303..a8bc9efe 100644
> --- a/www/manager6/data/UpdateStore.js
> +++ b/www/manager6/data/UpdateStore.js
> @@ -12,6 +12,13 @@ Ext.define('PVE.data.UpdateStore', {
>  
>      isStopped: true,
>  
> +    destroy: function() {
> +     var me = this;
> +     me.load_task.cancel();
> +     PVE.data.UpdateQueue.unqueue(me);
> +     me.callParent();
> +    },
> +
>      constructor: function(config) {
>       var me = this;
>  
> @@ -56,9 +63,6 @@ Ext.define('PVE.data.UpdateStore', {
>  
>       me.callParent([config]);
>  
> -     me.on('destroy', function() {
> -         load_task.cancel();
> -         PVE.data.UpdateQueue.unqueue(me);
> -     });
> +     me.load_task = load_task;
>      }
>  });
> 

_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to