chore: output() function migration

Author: hakimio

libs/cdk/virtual-table/abstract-table-builder-api.directive.ts

@@ -11,14 +11,13 @@ import {
     Directive,
     effect,
     ElementRef,
-    EventEmitter,
     inject,
     input,
     NgZone,
     OnChanges,
     OnDestroy,
     OnInit,
-    Output,
+    output,
     QueryList,
     SimpleChanges,
     ViewChild,
@@ -167,19 +166,13 @@ export abstract class AbstractTableBuilderApi<T>
 
     public readonly isVirtualTable = input(true, {alias: 'is-virtual-table'});
 
-    @Output()
-    public readonly afterRendered = new EventEmitter<boolean>();
-
-    @Output()
-    public readonly schemaChanges = new EventEmitter<TableUpdateSchema>();
+    public readonly afterRendered = output<boolean>();
+    public readonly schemaChanges = output<TableUpdateSchema>();
 
     // TODO: should be rename (breaking changes)
     // eslint-disable-next-line @angular-eslint/no-output-on-prefix
-    @Output()
-    public readonly onChanges = new EventEmitter<Nullable<T[]>>();
-
-    @Output()
-    public readonly sortChanges = new EventEmitter<OrderedField[]>();
+    public readonly onChanges = output<Nullable<T[]>>();
+    public readonly sortChanges = output<OrderedField[]>();
 
     @ContentChild(NgxOptions, {static: false})
     public columnOptions: Nullable<NgxOptions> = null;

libs/cdk/virtual-table/components/ngx-context-menu/ngx-context-menu-item/ngx-context-menu-item.component.ts

@@ -4,13 +4,12 @@ import {
     ChangeDetectorRef,
     Component,
     ElementRef,
-    EventEmitter,
     inject,
     input,
     NgZone,
     OnDestroy,
     OnInit,
-    Output,
+    output,
     ViewChild,
     ViewEncapsulation,
 } from '@angular/core';
@@ -59,8 +58,7 @@ export class NgxContextMenuItem<T = any> implements OnInit, OnDestroy {
 
     // TODO: should be rename (breaking changes)
     // eslint-disable-next-line @angular-eslint/no-output-on-prefix
-    @Output()
-    public readonly onClick = new EventEmitter<ContextItemEvent>();
+    public readonly onClick = output<ContextItemEvent>();
 
     @ViewChild('item', {static: false})
     public itemRef: Nullable<ElementRef<HTMLDivElement>> = null;

libs/cdk/virtual-table/components/ngx-header/ngx-header.component.ts

@@ -2,9 +2,8 @@ import {NgClass} from '@angular/common';
 import {
     ChangeDetectionStrategy,
     Component,
-    EventEmitter,
     input,
-    Output,
+    output,
     ViewEncapsulation,
 } from '@angular/core';
 import {SIGNAL} from '@angular/core/primitives/signals';
@@ -21,8 +20,7 @@ import {TableContentDirective} from '../../directives/table-content.directive';
     changeDetection: ChangeDetectionStrategy.OnPush,
 })
 export class NgxHeader extends TableContentDirective {
-    @Output()
-    public readonly expandedChange = new EventEmitter<InputBoolean>();
+    public readonly expandedChange = output<InputBoolean>();
 
     public readonly expanded = input<boolean, InputBoolean>(true, {
         transform: coerceBoolean,

libs/cdk/virtual-table/components/table-tbody/table-tbody.component.ts

@@ -4,11 +4,10 @@ import {
     ChangeDetectionStrategy,
     ChangeDetectorRef,
     Component,
-    EventEmitter,
     inject,
     input,
     NgZone,
-    Output,
+    output,
     ViewEncapsulation,
 } from '@angular/core';
 import {getValueByPath} from '@angular-ru/cdk/object';
@@ -120,8 +119,7 @@ export class TableTbody<T> {
         alias: 'column-schema',
     });
 
-    @Output()
-    public readonly changed = new EventEmitter<void>(true);
+    public readonly changed = output();
 
     public selection = inject(SelectionService<T>);
     public contextMenu = inject(ContextMenuService<T>);

libs/cdk/virtual-table/components/table-thead/table-thead.component.ts

@@ -3,11 +3,10 @@ import {NgClass, NgStyle, NgTemplateOutlet, TitleCasePipe} from '@angular/common
 import {
     ChangeDetectionStrategy,
     Component,
-    EventEmitter,
     HostListener,
     inject,
     input,
-    Output,
+    output,
     ViewEncapsulation,
 } from '@angular/core';
 import {fadeInLinearAnimation} from '@angular-ru/cdk/animations';
@@ -61,14 +60,9 @@ export class TableThead<T> {
         alias: 'column-schema',
     });
 
-    @Output()
-    public readonly resizing = new EventEmitter<ResizeEvent>();
-
-    @Output()
-    public readonly sortByKey = new EventEmitter<string>();
-
-    @Output()
-    public readonly openContextMenu = new EventEmitter<MouseEvent>();
+    public readonly resizing = output<ResizeEvent>();
+    public readonly sortByKey = output<string>();
+    public readonly openContextMenu = output<MouseEvent>();
 
     public orderType: typeof SortOrderType = SortOrderType;
     public limit: number = OVERLOAD_WIDTH_TABLE_HEAD_CELL;

libs/cdk/virtual-table/directives/auto-height.directive.ts

@@ -1,14 +1,13 @@
 import {
     Directive,
     ElementRef,
-    EventEmitter,
     inject,
     input,
     NgZone,
     OnChanges,
     OnDestroy,
     OnInit,
-    Output,
+    output,
     SimpleChanges,
 } from '@angular/core';
 import {Nullable} from '@angular-ru/cdk/typings';
@@ -43,8 +42,7 @@ export class AutoHeight<T> implements OnInit, OnChanges, OnDestroy {
 
     public readonly sourceRef = input<T[]>([]);
 
-    @Output()
-    public readonly recalculatedHeight = new EventEmitter<void>(true);
+    public readonly recalculatedHeight = output();
 
     public get destroy$(): Subject<boolean> {
         return this._destroy$;

libs/cdk/virtual-table/directives/observer-view.directive.ts

@@ -2,12 +2,11 @@ import {
     AfterViewInit,
     Directive,
     ElementRef,
-    EventEmitter,
     inject,
     input,
     NgZone,
     OnDestroy,
-    Output,
+    output,
 } from '@angular/core';
 import {Nullable} from '@angular-ru/cdk/typings';
 import {isNotNil} from '@angular-ru/cdk/utils';
@@ -25,8 +24,7 @@ export class ObserverView implements AfterViewInit, OnDestroy {
 
     public readonly observerRootMargin = input<string>();
 
-    @Output()
-    public readonly observeVisible = new EventEmitter<boolean>(true);
+    public readonly observeVisible = output<boolean>();
 
     public ngAfterViewInit(): void {
         this.ngZone.runOutsideAngular((): void => {

libs/cdk/virtual-table/directives/rows/abstract-template-cell-common.directive.ts

@@ -1,14 +1,6 @@
 /* eslint-disable @angular-eslint/no-input-rename */
 import type {Signal} from '@angular/core';
-import {
-    Directive,
-    EventEmitter,
-    inject,
-    input,
-    Output,
-    signal,
-    TemplateRef,
-} from '@angular/core';
+import {Directive, inject, input, output, signal, TemplateRef} from '@angular/core';
 import {Nullable, PlainObject} from '@angular-ru/cdk/typings';
 
 import {TableEvent} from '../../interfaces/table-builder.external';
@@ -29,11 +21,9 @@ export abstract class AbstractTemplateCellCommonDirective<T> {
 
     // TODO: should be rename (breaking changes)
     // eslint-disable-next-line @angular-eslint/no-output-on-prefix
-    @Output()
-    public readonly onClick = new EventEmitter<TableEvent<T | any, any>>();
+    public readonly onClick = output<TableEvent<T | any, any>>();
 
-    @Output()
-    public readonly dblClick = new EventEmitter<TableEvent<T | any, any>>();
+    public readonly dblClick = output<TableEvent<T | any, any>>();
 
     public type: Signal<Nullable<string>> = signal(null);
 }

libs/cdk/virtual-table/interfaces/table-builder.external.ts

@@ -1,4 +1,5 @@
-import {EmbeddedViewRef, EventEmitter, TemplateRef} from '@angular/core';
+import type {OutputEmitterRef} from '@angular/core';
+import {EmbeddedViewRef, TemplateRef} from '@angular/core';
 import {ExcelType} from '@angular-ru/cdk/excel';
 import {DeepPartial, Nullable, PlainObject} from '@angular-ru/cdk/typings';
 
@@ -10,7 +11,7 @@ export enum ImplicitContext {
     CELL = 'CELL',
 }
 
-export type TableClickEventEmitter<T, K> = Nullable<EventEmitter<TableEvent<T, K>>>;
+export type TableClickEventEmitter<T, K> = Nullable<OutputEmitterRef<TableEvent<T, K>>>;
 
 export interface TableCellOptions<T = any> {
     class: Nullable<PlainObject | string[] | string>;
@@ -22,8 +23,8 @@ export interface TableCellOptions<T = any> {
     height: Nullable<number>;
     template?: Nullable<TemplateRef<T>>;
     context: ImplicitContext;
-    onClick: EventEmitter<any>;
-    dblClick: EventEmitter<any>;
+    onClick: OutputEmitterRef<any>;
+    dblClick: OutputEmitterRef<any>;
 }
 
 export interface TableHeadCellOptions {

libs/cdk/virtual-table/services/sortable/sortable.service.ts

@@ -1,4 +1,4 @@
-import {EventEmitter, inject, Injectable, NgZone} from '@angular/core';
+import {inject, Injectable, NgZone, OutputEmitterRef} from '@angular/core';
 import {Nullable, PlainObjectOf, SortOrderType} from '@angular-ru/cdk/typings';
 import {isNotNil, isTruthy} from '@angular-ru/cdk/utils';
 import {WebWorkerThreadService} from '@angular-ru/cdk/webworker';
@@ -14,7 +14,7 @@ export class SortableService<T> {
     private readonly zone = inject(NgZone);
 
     private skipInternalSort = false;
-    private sortChanges: Nullable<EventEmitter<OrderedField[]>> = null;
+    private sortChanges: Nullable<OutputEmitterRef<OrderedField[]>> = null;
     public definition: PlainObjectOf<SortOrderType> = {};
     public positionMap: PlainObjectOf<number> = {};
     public sortableCount = 0;
@@ -52,7 +52,7 @@ export class SortableService<T> {
         this.skipInternalSort = skipInternalSort;
     }
 
-    public setSortChanges(emitter: EventEmitter<OrderedField[]>): void {
+    public setSortChanges(emitter: OutputEmitterRef<OrderedField[]>): void {
         this.sortChanges = emitter;
     }