Droppable: $.ui.ddmanager.current released when droppable destroys

Fixes #10666
if ($.ui.ddmanager.current && $.ui.ddmanager.current.element &&
this.element &&
           ($.ui.ddmanager.current.element.get(0) === this.element.get(0)))
{
            $.ui.ddmanager.current = null;
        }

will that do?

Regarding the style issues, I think I did follow the instruction. Isn't
passing the linter and jsHint enough?

Working on a unit test.

2014-10-19 21:40 GMT+03:00 Scott González <notifications@github.com>:

> In ui/droppable.js:
>
> > @@ -100,6 +100,10 @@ $.widget( "ui.droppable", {
> >  		this._splice( drop );
> >
> >  		this.element.removeClass( "ui-droppable ui-droppable-disabled" );
> > +
> > +        if($.ui.ddmanager.current && ($.ui.ddmanager.current.element === this.element)) {
>
> Comparing jQuery objects for equality is never a good idea. Check the
> underlying element.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jquery/jquery-ui/pull/1360/files#r19062350>.
>