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

Fixes #10666
One more thing please, do you really think a unit test is neccessary
here?  $.ui.ddmanager.current
is an internal thing, I don't really test the unit's functionality here but
I expect the unit to behave internally better on a certain scenario. What's
your opinion about this?

2014-10-20 12:03 GMT+03:00 חן אשחר <eshcharc@gmail.com>:

> 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>.
>>
>
>