Cannot read property 'name' of undefined when using avoid-scope-typos

[davidmcclelland]

Happy to provide any additional info I can to help reproduce, just let me know what you need.

eslint [email protected]
[email protected]

Stacktrace:
Cannot read property 'name' of undefined
TypeError: Cannot read property 'name' of undefined
at check ([snip]\node_modules\eslint-plugin-angular\rules\avoid-scope-typos.js:24:75)
at Linter.Identifier ([snip]\node_modules\eslint-plugin-angular\rules\avoid-scope-typos.js:31:17)
at emitOne (events.js:101:20)
at Linter.emit (events.js:188:7)
at NodeEventGenerator.applySelector ([snip]\node_modules\eslint\lib\util\node-event-generator.js:265:26)
at NodeEventGenerator.applySelectors ([snip]\node_modules\eslint\lib\util\node-event-generator.js:294:22)
at NodeEventGenerator.enterNode ([snip]\node_modules\eslint\lib\util\node-event-generator.js:308:14)
at CodePathAnalyzer.enterNode ([snip]\node_modules\eslint\lib\code-path-analysis\code-path-analyzer.js:602:23)
at Traverser.enter ([snip]\node_modules\eslint\lib\linter.js:926:36)
at Traverser.__execute ([snip]\node_modules\estraverse\estraverse.js:397:31)
at Traverser.traverse ([snip]\node_modules\estraverse\estraverse.js:501:28)

[EmmanuelDemey]

Thanks for the issue. Can you also share the code causing this error ? I will add it to our unit tests.

[davidmcclelland]

I tried to log the node being entered in linter.js's traverser.traverse, but the output was rather verbose. I can't even figure out which file of mine is causing the exception. According to #508 it happens with the source for jquery 3.1.1 with attachments for it. You may be better off using that to test.

[fungiboletus]

Got the error with a function broadcast() {} in a factory. Renaming it to broadcast2 or something else fixed the error.