Ext.mixin.Observable. clearManagedListeners - minor incompatibility with 4.*

1. 
   

   You found a bug! We've classified it as
   
   
   
   EXTJS-13246 .
   
   We encourage you to continue the discussion and to find an acceptable workaround while we work on a permanent fix.

   
   


2. 
   
   
   

   
   

   
   

   Ext.mixin.Observable. clearManagedListeners - minor incompatibility with 4.*

   
   

   
   

   Ext.mixin.Observable. clearManagedListeners - minor incompatibility with 4.*

   
   
   
   

   
   

   There's a minor incompatibility in the clearManagedListeners method. In 4.x, this method has no return value, but in 5.0 it may return 'this'.
   
   

   
   

   Code:

   
   

   
   Ext.define('Ext.mixin.Observable', {
       ...
       clearManagedListeners: function(object) {
           ...
           if (object) {
               ...
               delete managedListeners[id];
               // return this;
               return; // <-- use just return w/o this?
           }
   
           for (id in managedListeners) {
               if (managedListeners.hasOwnProperty(id)) {
                   this.clearManagedListeners(id);
               }
           }
       },
       ...
   

   
   

   
   

   
   

   
   

   
   

   
   
   
   

   ---



3. 
   

   ---