Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

event.source.itemScope is undefined #347

Closed
noullet opened this issue Oct 20, 2016 · 12 comments · May be fixed by #359
Closed

event.source.itemScope is undefined #347

noullet opened this issue Oct 20, 2016 · 12 comments · May be fixed by #359

Comments

@noullet
Copy link

noullet commented Oct 20, 2016

Hi,

Since 1.3.7, event.source.itemScope is undefined when accessed in orderChanged and itemMoved.

The change 29b56a1 motivated by #93 is the cause of this issue.

In my case, I want to get the model value:

orderChanged: function(event) {
    var model = event.source.itemScope.modelValue;
    // ...
},

And here is what I get:

angular.js:13920 TypeError: Cannot read property 'modelValue' of undefined

I don't think I'm doing anything special but I am surprised that this issue hasn't be raised yet. Please tell me if you can't reproduce it.

@pwojee
Copy link

pwojee commented Oct 20, 2016

This is also happening to me.

@mnberti
Copy link

mnberti commented Oct 23, 2016

bug exists for me as well.

@mhmaguire
Copy link

Happening here also

@pedrosanta
Copy link

+1, Also having this issue. 😕

@nonplus
Copy link

nonplus commented Oct 26, 2016

+1

3 similar comments
@barkerjeff
Copy link

barkerjeff commented Oct 31, 2016

+1

@dllabs
Copy link

dllabs commented Nov 3, 2016

+1

@skython
Copy link

skython commented Nov 8, 2016

+1

@juanto
Copy link

juanto commented Nov 9, 2016

Happening here too

@natan88
Copy link

natan88 commented Nov 21, 2016

In version 1.3.6 has this error.
In version 1.3.4 does not have this error.

@jedatu jedatu mentioned this issue Dec 19, 2016
@flpwgr
Copy link

flpwgr commented Dec 21, 2016

👍

@a5hik
Copy link
Owner

a5hik commented Apr 3, 2017

fixed in 1.3.8

@a5hik a5hik closed this as completed Apr 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.