bug: Routing fails when default view is named "module"

[dkent600]

I'm submitting a bug report

• Library Version: see attached gist

Please tell us about your environment:

• Operating System: Windows 10

• Node Version: see attached gist

• NPM Version: see attached gist

• JSPM OR Webpack AND Version see attached gist

• Browser: Chrome 55.0.2883.87 m

• Language: N/

Current behavior: See the attached gist

You will find that the default view will not render. In the console one sees the javascript error message:

aurelia.js:14435 ERROR [app-router] Error: Cannot determine default view strategy for object. at ViewLocator.getViewStrategy (aurelia.js:9143) at RouterView.process (aurelia.js:18215) at _loop (aurelia.js:15809) at NavigationInstruction._commitChanges (aurelia.js:15824) at CommitChangesStep.run (aurelia.js:15676) at next (aurelia.js:15645) at iterate (aurelia.js:16753) at processActivatable (aurelia.js:16756) at ActivateNextStep.run (aurelia.js:16637) at next (aurelia.js:15645)

If you rename the module.html and module.js to any base file name but "module", and modify the route accordingly in app.js, then all works fine.

Expected/desired behavior:

Ability to navigate to a view should not depend on the spelling of the view/module files as long as routing is properly configured.

[EisenbergEffect]

module is a reserved word in JavaScript. I'm suspicious that that is related to this. I'm not sure if we'll be able to fix it. It's not going to be high priority. Feel free to submit a PR with a fix.

[Alexander-Taran]

@bigopon 🤷‍♂️