Unnecessary import of FORM_DIRECTIVES in charts.js causes value accessor error with RC 5 and material 2 Alpha 7

[naveedahmed1]

Hi,

It seems that there is an unnecessary import of FORM_DIRECTIVES in charts.js which causes value accessor error with Angular 2 RC 5 and material 2 Alpha 7.

I removed that from the directives list in charts.js and the error was gone.

Can you please review and check if

common_1.FORM_DIRECTIVES,

is actually necessary to add in directives?

[petewalker]

Same problem here - and from looking at the source, it doesn't appear that FORM_DIRECTIVES are even being used?

Can this be removed?

[naveedahmed1]

@petewalker Yes agree, I removed it and i don't see any side effect.

[raychenfj]

I remove it too. And everything works fine so far.

[jbgarr]

I'm seeing this issue as well and the workaround mentioned by @naveedahmed1 works for now, but a more permanent fix would be really nice. For development it's not that big of a deal, but even for that it means that others working on the same project that happen to run npm install to pull in dependencies have to deal with the workaround as well.

For us the error results in forms being controlled by the FormsModule causing a Template parse error that looks like the following:

zone.js:478 Unhandled Promise rejection: Template parse errors: Reference <#FormNameHere> is defined several times

[cbomkamp]

I see that PR #343 incorporates a fix for this and is in the development branch. Do you have an ETA on when that might make it into the npm package? Thanks!

[paviad]

Closing for inactivity - if issue persists, feel free to comment with a working example which demonstrates the issue.