Looks like a typo/carryover.


On Sat, May 17, 2014 at 10:10 AM, Jasper Tey <notifications@github.com>wrote:

> In src/knockout-sortable.js:
>
> > +
> > +    		dropActual = droppable.options.drop;
> > +
> > +    		$element.droppable(ko.utils.extend(droppable.options, {
> > +    			drop: function (event, ui) {
> > +    				var sourceParent, targetParent, targetIndex, i, targetUnwrapped, arg,
> > +						   el = ui.draggable[0],
> > +						   item = ko.utils.domData.get(el, ITEMKEY) || ko.utils.domData.get(el, DRAGKEY);
> > +
> > +    				if (item && item.clone)
> > +    					item = item.clone();
> > +
> > +    				if (item) {
> > +    					sourceParent = ko.utils.domData.get(el, PARENTKEY);
> > +    					targetParent = droppable.data
> > +    					targetIndex;
>
> What's the purpose of targetIndex sitting here on its own line soing
> nothing? Am I missing something or is it just a typo?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/rniemeyer/knockout-sortable/pull/36/files#r12768955>
> .
>