From 5a257d02a6de8e66c2529552321f129dd3dc8d4d Mon Sep 17 00:00:00 2001 From: Paul Gschwendtner Date: Fri, 1 Feb 2019 14:39:17 +0100 Subject: [PATCH] build: fix "person_management" playground example (#28490) Currently when someone serves the "person_management" playground example, there will be runtime exceptions by `@angular/forms` if someone clicks on one of the two buttons rendered in the example. This happens because the example is outdated and the input elements using `ngModel` do not specify a proper "name" while being inside of a `
` element. A name is required inside of a form. The failure is not specific to Ivy and is not covered by any test because the e2e tests for this example are just asserting that the page properly loads (the error only shows up one of the buttons has been clicked) This is the reason why these errors were never visibile to the e2e tests. Though in order to make this example work, we should this fix these failures so that the example can work as expected. ``` FullNameComponent.html:7 ERROR Error: If ngModel is used within a form tag, either the name attribute must be set or the form control must be defined as 'standalone' in ngModelOptions. Example 1: Example 2: at Function.TemplateDrivenErrors.missingNameException (template_driven_errors.ts:40) at NgModel._checkName (ng_model.ts:319) at NgModel._checkForErrors (ng_model.ts:302) at NgModel.ngOnChanges (ng_model.ts:215) at Object.checkAndUpdateDirectiveInline (provider.ts:208) at checkAndUpdateNodeInline (view.ts:429) at checkAndUpdateNode (view.ts:389) at debugCheckAndUpdateNode (services.ts:431) at debugCheckDirectivesFn (services.ts:392) at Object.eval [as updateDirectives] (FullNameComponent.html:7) ``` PR Close #28490 --- .../playground/src/person_management/index.ts | 25 ++++++++++++------- 1 file changed, 16 insertions(+), 9 deletions(-) diff --git a/modules/playground/src/person_management/index.ts b/modules/playground/src/person_management/index.ts index 3638ccecff..d981acad14 100644 --- a/modules/playground/src/person_management/index.ts +++ b/modules/playground/src/person_management/index.ts @@ -85,13 +85,15 @@ export class DataService {
@@ -115,29 +117,34 @@ export class FullNameComponent {
- +
- +
- + Age: {{person.age}}
\
- - + + {{person.mom.fullName}}
- - + + {{person.dad.fullName}}