I observed that issue when implementing IEventdispatcher i.e. when the EventDispatcher constructor has an IEventDispatcher argument - so not for statics. This is not seen when extending EventDispatcher, but in that case the subclass would presumably inherit that method from goog EventTarget. So that setCurrentTarget method in the constructor did not seem to be the complete answer here... that extra hack in the constructor just got things working for now....it works as is but I don't like it.
-Greg [sent from my phone] On 31/08/2016 6:59 PM, "Alex Harui" <aha...@adobe.com> wrote: > > > On 8/30/16, 1:41 PM, "Greg Dove" <greg.d...@gmail.com> wrote: > > >Before I forget, I have 2 extra questions : > > > >1. I wasn't real happy with the change I made for js in > >org.apache.flex.events.EventDispatcher. At the time it was the easiest > >solution. I did not look too deep into the goog base class to see if there > >was a more official way to avoid this. I will take the time to do that in > >the coming days. But if there is not an 'official' way, is the best > >solution to maybe lodge an issue with them (if so, I can, just wanted to > >check first, in case we apply patches or something like that). > > Can you explain when this code is needed? Is this for the static event > dispatcher? Or for a custom IEventDispatcher? > > Thanks, > -Alex > >
