refactor(cdk/accordion): switch to input transforms

Switches inputs in cdk/accordion to use transforms instead of getters/setters.

Author: crisbeto

src/cdk/accordion/BUILD.bazel

@@ -15,7 +15,6 @@ ng_module(
         exclude = ["**/*.spec.ts"],
     ),
     deps = [
-        "//src/cdk/coercion",
         "//src/cdk/collections",
         "@npm//@angular/core",
         "@npm//rxjs",

src/cdk/accordion/accordion-item.ts

@@ -16,10 +16,10 @@ import {
   ChangeDetectorRef,
   SkipSelf,
   Inject,
+  booleanAttribute,
 } from '@angular/core';
 import {UniqueSelectionDispatcher} from '@angular/cdk/collections';
 import {CDK_ACCORDION, CdkAccordion} from './accordion';
-import {BooleanInput, coerceBooleanProperty} from '@angular/cdk/coercion';
 import {Subscription} from 'rxjs';
 
 /** Used to generate unique ID for each accordion item. */
@@ -59,13 +59,11 @@ export class CdkAccordionItem implements OnDestroy {
   readonly id: string = `cdk-accordion-child-${nextId++}`;
 
   /** Whether the AccordionItem is expanded. */
-  @Input()
+  @Input({transform: booleanAttribute})
   get expanded(): boolean {
     return this._expanded;
   }
-  set expanded(expanded: BooleanInput) {
-    expanded = coerceBooleanProperty(expanded);
-
+  set expanded(expanded: boolean) {
     // Only emit events and update the internal value if the value changes.
     if (this._expanded !== expanded) {
       this._expanded = expanded;
@@ -91,14 +89,7 @@ export class CdkAccordionItem implements OnDestroy {
   private _expanded = false;
 
   /** Whether the AccordionItem is disabled. */
-  @Input()
-  get disabled(): boolean {
-    return this._disabled;
-  }
-  set disabled(disabled: BooleanInput) {
-    this._disabled = coerceBooleanProperty(disabled);
-  }
-  private _disabled = false;
+  @Input({transform: booleanAttribute}) disabled: boolean = false;
 
   /** Unregister function for _expansionDispatcher. */
   private _removeUniqueSelectionListener: () => void = () => {};

src/cdk/accordion/accordion.ts

@@ -6,8 +6,15 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import {BooleanInput, coerceBooleanProperty} from '@angular/cdk/coercion';
-import {Directive, InjectionToken, Input, OnChanges, OnDestroy, SimpleChanges} from '@angular/core';
+import {
+  Directive,
+  InjectionToken,
+  Input,
+  OnChanges,
+  OnDestroy,
+  SimpleChanges,
+  booleanAttribute,
+} from '@angular/core';
 import {Subject} from 'rxjs';
 
 /** Used to generate unique ID for each accordion. */
@@ -39,18 +46,11 @@ export class CdkAccordion implements OnDestroy, OnChanges {
   readonly id: string = `cdk-accordion-${nextId++}`;
 
   /** Whether the accordion should allow multiple expanded accordion items simultaneously. */
-  @Input()
-  get multi(): boolean {
-    return this._multi;
-  }
-  set multi(multi: BooleanInput) {
-    this._multi = coerceBooleanProperty(multi);
-  }
-  private _multi: boolean = false;
+  @Input({transform: booleanAttribute}) multi: boolean = false;
 
   /** Opens all enabled accordion items in an accordion where multi is enabled. */
   openAll(): void {
-    if (this._multi) {
+    if (this.multi) {
       this._openCloseAllActions.next(true);
     }
   }

tools/public_api_guard/cdk/accordion.md

@@ -4,7 +4,6 @@
 
 ```ts
 
-import { BooleanInput } from '@angular/cdk/coercion';
 import { ChangeDetectorRef } from '@angular/core';
 import { EventEmitter } from '@angular/core';
 import * as i0 from '@angular/core';
@@ -19,8 +18,9 @@ import { UniqueSelectionDispatcher } from '@angular/cdk/collections';
 export class CdkAccordion implements OnDestroy, OnChanges {
     closeAll(): void;
     readonly id: string;
-    get multi(): boolean;
-    set multi(multi: BooleanInput);
+    multi: boolean;
+    // (undocumented)
+    static ngAcceptInputType_multi: unknown;
     // (undocumented)
     ngOnChanges(changes: SimpleChanges): void;
     // (undocumented)
@@ -42,14 +42,17 @@ export class CdkAccordionItem implements OnDestroy {
     close(): void;
     readonly closed: EventEmitter<void>;
     readonly destroyed: EventEmitter<void>;
-    get disabled(): boolean;
-    set disabled(disabled: BooleanInput);
+    disabled: boolean;
     get expanded(): boolean;
-    set expanded(expanded: BooleanInput);
+    set expanded(expanded: boolean);
     readonly expandedChange: EventEmitter<boolean>;
     // (undocumented)
     protected _expansionDispatcher: UniqueSelectionDispatcher;
     readonly id: string;
+    // (undocumented)
+    static ngAcceptInputType_disabled: unknown;
+    // (undocumented)
+    static ngAcceptInputType_expanded: unknown;
     ngOnDestroy(): void;
     open(): void;
     readonly opened: EventEmitter<void>;