Skip to content

Commit

Permalink
Revert "feat(forms): formControlName also accepts a number (angular#3…
Browse files Browse the repository at this point in the history
…0606)"

This reverts commit a647298.
  • Loading branch information
kara committed Aug 10, 2019
1 parent 0ddf0c4 commit 0c5077c
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 13 deletions.
2 changes: 1 addition & 1 deletion packages/forms/src/directives/ng_control.ts
Expand Up @@ -36,7 +36,7 @@ export abstract class NgControl extends AbstractControlDirective {
* @description
* The name for the control
*/
name: string|number|null = null;
name: string|null = null;

/**
* @description
Expand Down
Expand Up @@ -145,13 +145,9 @@ export class FormControlName extends NgControl implements OnChanges, OnDestroy {
* @description
* Tracks the name of the `FormControl` bound to the directive. The name corresponds
* to a key in the parent `FormGroup` or `FormArray`.
* Accepts a name as a string or a number.
* The name in the form of a string is useful for individual forms,
* while the numerical form allows for form controls to be bound
* to indices when iterating over controls in a `FormArray`.
*/
// TODO(issue/24571): remove '!'.
@Input('formControlName') name !: string | number | null;
@Input('formControlName') name !: string;

/**
* @description
Expand Down Expand Up @@ -242,9 +238,7 @@ export class FormControlName extends NgControl implements OnChanges, OnDestroy {
* Returns an array that represents the path from the top-level form to this control.
* Each index is the string name of the control on that level.
*/
get path(): string[] {
return controlPath(this.name == null ? this.name : this.name.toString(), this._parent !);
}
get path(): string[] { return controlPath(this.name, this._parent !); }

/**
* @description
Expand Down
4 changes: 2 additions & 2 deletions packages/forms/src/directives/shared.ts
Expand Up @@ -28,8 +28,8 @@ import {SelectMultipleControlValueAccessor} from './select_multiple_control_valu
import {AsyncValidator, AsyncValidatorFn, Validator, ValidatorFn} from './validators';


export function controlPath(name: string | null, parent: ControlContainer): string[] {
return [...parent.path !, name !];
export function controlPath(name: string, parent: ControlContainer): string[] {
return [...parent.path !, name];
}

export function setUpControl(control: FormControl, dir: NgControl): void {
Expand Down
4 changes: 2 additions & 2 deletions tools/public_api_guard/forms/forms.d.ts
Expand Up @@ -255,7 +255,7 @@ export declare class FormControlName extends NgControl implements OnChanges, OnD
readonly formDirective: any;
isDisabled: boolean;
/** @deprecated */ model: any;
name: string | number | null;
name: string;
readonly path: string[];
/** @deprecated */ update: EventEmitter<any>;
readonly validator: ValidatorFn | null;
Expand Down Expand Up @@ -353,7 +353,7 @@ export declare const NG_VALUE_ACCESSOR: InjectionToken<ControlValueAccessor>;

export declare abstract class NgControl extends AbstractControlDirective {
readonly asyncValidator: AsyncValidatorFn | null;
name: string | number | null;
name: string | null;
readonly validator: ValidatorFn | null;
valueAccessor: ControlValueAccessor | null;
abstract viewToModelUpdate(newValue: any): void;
Expand Down

0 comments on commit 0c5077c

Please sign in to comment.