droptargetContext is undefined when sortable nested

[soonwait]

Like this

<div class="grid"
    data-ng-model="grid.data"
    data-ui-sortable="grid.rowSortableOptions">

    <div class="row"
        data-ng-repeat="row in grid.data track by $index"
        data-ng-model="row.cols"
        data-ui-sortable="grid.colSortableOptions">

        <div class="col"
            data-ng-repeat="col in row.cols track by $index"
            data-ng-model="col.cells"
            data-ui-sortable="grid.cellSortableOptions">

            <div class="cell"
                data-ng-repeat="cell in col.cells track by $index">
            </div>
        </div>
    </div>
</div>

I have fixed myself by writing 'if statement' in line 365, Like

                var droptargetContext = ui.item.sortable._getElementContext(droptarget);
                if(droptargetContext) {
                    ui.item.sortable.droptargetModel = droptargetContext.scope.ngModel;

                    // Cancel the sort (let ng-repeat do the sort for us)
                    // Don't cancel if this is the received list because it has
                    // already been canceled in the other list, and trying to cancel
                    // here will mess up the DOM.
                    element.sortable('cancel');
                }

[thgreasi]

That's quite interesting. Could you fork the provided nested sortable example to reproduce this, so that I can add a test case for that?

[thgreasi]

Something like this perhaps.

[soonwait]

OK, i have forked an example this

you should find out error in console, when drag col or row, which said:

sortable.min.js:8 Uncaught TypeError: Cannot read property 'scope' of undefined
    at HTMLDivElement.C.update (https://cdnjs.cloudflare.com/ajax/libs/angular-ui-sortable/0.17.1/sortable.min.js:8:3604)
    at HTMLDivElement.<anonymous> (https://cdnjs.cloudflare.com/ajax/libs/angular-ui-sortable/0.17.1/sortable.min.js:8:654)
    at t.(anonymous function).(anonymous function)._trigger (https://ajax.googleapis.com/ajax/libs/jqueryui/1.12.1/jquery-ui.min.js:6:11090)
    at t.(anonymous function).(anonymous function)._trigger (https://ajax.googleapis.com/ajax/libs/jqueryui/1.12.1/jquery-ui.min.js:13:2147)
    at t.(anonymous function).(anonymous function)._trigger (https://ajax.googleapis.com/ajax/libs/jqueryui/1.12.1/jquery-ui.min.js:6:4499)
    at t.(anonymous function).(anonymous function).<anonymous> (https://ajax.googleapis.com/ajax/libs/jqueryui/1.12.1/jquery-ui.min.js:13:869)
    at t.(anonymous function).(anonymous function)._clear (https://ajax.googleapis.com/ajax/libs/jqueryui/1.12.1/jquery-ui.min.js:13:1993)
    at t.(anonymous function).(anonymous function)._clear (https://ajax.googleapis.com/ajax/libs/jqueryui/1.12.1/jquery-ui.min.js:6:4499)
    at t.(anonymous function).(anonymous function)._mouseStop (https://ajax.googleapis.com/ajax/libs/jqueryui/1.12.1/jquery-ui.min.js:12:16765)
    at t.(anonymous function).(anonymous function)._mouseStop (https://ajax.googleapis.com/ajax/libs/jqueryui/1.12.1/jquery-ui.min.js:6:4499)

[soonwait]

I also meet another bug(i think), I'll desc that in another issue #534

[thgreasi]

Releasing a fix as v0.17.2. Thank for reporting.