Skip to content

Commit

Permalink
Kendo Form Array template bug fixed (closes #437)
Browse files Browse the repository at this point in the history
  • Loading branch information
@udos86
udos86 committed Jul 10, 2017
1 parent 7baf7b9 commit de6bc38
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion modules/ui-kendo/src/dynamic-form-kendo.component.html
View file
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
<div *ngFor="let groupModel of model.groups; let idx = index" role="group"
[formGroupName]="idx" [ngClass]="[model.cls.element.group, model.cls.grid.group]">

<dynamic-form-kendo-control *ngFor="let controlModel of groupModel.group"
<dynamic-form-kendo-control *ngFor="let item of groupModel.group"
[bindId]="false"
[context]="groupModel"
[group]="control.at(idx)"
Expand Down

2 comments on commit de6bc38

@stekontar
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@udos86
please also change [model] attribute form "controlModel" to "item"
Regards,
S.

<dynamic-form-kendo-control *ngFor="let item of groupModel.group"
                                            [bindId]="false"
                                            [context]="groupModel"
                                            [group]="control.at(idx)"
                                            [hasErrorMessaging]="item.hasErrorMessages"
                                            **[model]="controlModel"**
                                            [nestedTemplates]="templates"
                                            [ngClass]="[item.cls.element.host, item.cls.grid.host]"
                                            (blur)="onFocusChange($event)"
                                            (change)="onValueChange($event)"
                                            (focus)="onFocusChange($event)"></dynamic-form-kendo-control>

@udos86
Copy link
Owner Author

@udos86 udos86 commented on de6bc38 Jul 18, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stekontar Thank for checking. Guess, I was in a hurry. Sorry for that. I'll release a new version asap.

Please sign in to comment.