- This topic has 7 replies, 4 voices, and was last updated 8 years, 1 month ago by
support-piotr.
-
AuthorPosts
-
andrei-ivanovParticipantHi,
After upgrading to the pre-release (1.8.1.201705290950) I noticed a new error:
Binding to event attribute ‘onLabel’ is disallowed for security reasons, please use (Label)=…I think it’s a false positive, I don’t think any binding is intented here:
<p-inputSwitch onLabel="I confirm" offLabel="I reject" [(ngModel)]="val"></p-inputSwitch>
May 30, 2017 at 3:26 am #524840
support-swapnaModeratorAndrei,
The error is due to the prefix ‘on’ in the attribute ‘onLabel’, which is a known issue and not specific to the pre-release :
https://github.com/500tech/angular-tree-component/issues/223
https://github.com/valor-software/ngx-bootstrap/issues/1000I see the error in CI 5 as well at my end when I use the prefix ‘on’. Please rename the attribute as a workaround.
Hope this helps. Let us know if you have any questions.
–Swapna
MyEclipse SupportMay 30, 2017 at 4:23 am #524846
andrei-ivanovParticipantHi,
As mentioned in my message above, that is not my component, but a component from PrimeNg.And I don’t think attributes with a ‘on’ prefix are forbidden…
The upgrade to the pre-release fixed the TSLint errors I’ve been having and now the warnings are starting to be shown (so it doesn’t necessarily means it’s a regression), a lot of them wrong, as far as I understand, hence my reports 🙂
May 30, 2017 at 4:25 am #524847
support-piotrParticipantAndrei,
We are considering to fix this issue for the release, as this is a serious issue. It looks like it is happening with the newest versions of angular. We’ll keep you posted. Thanks for letting us know about it!
Best regards,
Piotr TomiakMay 30, 2017 at 4:29 am #524849
andrei-ivanovParticipantThank you too, I’m impressed by the level of support 🙂
June 1, 2017 at 1:57 pm #525278
Brian FernandesModeratorAndrei,
Thank you for the compliment!
We just put out another pre release with a fix to this issue, could you check for updates using help > webclipse > check for updates and let us know if fixed?
We’d like to release this build to everyone in the next 12 – 16 hours, so if you could let us know asap, that would be much appreciated!
June 1, 2017 at 2:25 pm #525284
andrei-ivanovParticipantHi,
I’ve upgraded and that warning has disappeared, great 🙂Next step: https://www.genuitec.com/forums/topic/unknown-attributes-warning/ ?
Thank you
June 2, 2017 at 4:05 am #525357
support-piotrParticipantAndrei,
I’ve just made the fix for “id” and “name”, somehow I misunderstood what you’ve meant in your post. The fix will be available in the release.
-
AuthorPosts