fix(*): remove deprecateDecorators and apply @deprecate tags

Author: mmart1n

.github/CONTRIBUTING.md

@@ -170,17 +170,27 @@ Write migrations.
 
 ## Deprecating methods
 When a method is deprecated a few steps have to be done:
-1. Add deprecation warning message by decorating the method with `@DeprecateMethod` decorator from `deprecateDecorators.ts` file.
+1. Add the `@deprecated` tag at the begging of the method description followed by the version in which the method has been deprecated and what can be used instead. Example:
+```ts
+/**
+ * @deprecated in version 12.1.0. Use 'data' instead
+ *
+ * The data record that populates the row
+ */
+public getRowData(): any {
+    return this.data;
+}
+```
 2. Ensure that the deprecated method is no longer used in IgniteUI for Angular codebase, samples and documentation snippets.
 3. Write migrations.
 
 ## Deprecating class properties
 When a class property is deprecated a few steps have to be done:
-1. Add deprecation warning message by decorating the property with `@DeprecateProperty` decorator from `deprecateDecorators.ts` file.
+1. Add the `@deprecated` tag at the begging of the property description followed by the version in which the property has been deprecated and what can be used instead.
 2. Ensure that the deprecated property is no longer used in IgniteUI for Angular codebase, samples and documentation snippets.
 3. Write migrations.
 
-NOTE: TypeScript disallows decorating both the get and set accessor for a single member. Instead, all decorators for the member must be applied to the first accessor specified in document order. This is because decorators apply to a Property Descriptor, which combines both the get and set accessor, not each declaration separately.
+NOTE: TypeScript disallows adding descriptions to both the get and set accessor for a single member. Instead, the description for the member must be applied to the first accessor specified in document order. Having this in mind the `@deprecated` tag is applied only once.
 
 # Testing a PR
 In order to test a pull request that is awaiting test, perform the following actions.

projects/igniteui-angular/migrations/common/filterSourceDirs.ts

@@ -2,9 +2,9 @@ import { filter, Rule, SchematicContext, Tree } from '@angular-devkit/schematics
 import { getWorkspace, getWorkspacePath, getProjectPaths } from './util';
 
 /**
- * Filter tree to project source dirs
- *
  * @deprecated Temporary
+ *
+ * Filter tree to project source dirs
  */
 export const filterSourceDirs = (host: Tree, context: SchematicContext): Rule => {
     const configPath = getWorkspacePath(host);

projects/igniteui-angular/src/lib/banner/banner.component.ts

@@ -13,8 +13,7 @@ import { ToggleAnimationSettings } from '../expansion-panel/toggle-animation-com
 
 export interface BannerEventArgs extends IBaseEventArgs {
     /**
-     * @deprecated
-     * To get a reference to the banner, use `owner` instead.
+     * @deprecated in 12.1.0. To get a reference to the banner, use `owner` instead
      */
     banner: IgxBannerComponent;
     event?: Event;

projects/igniteui-angular/src/lib/core/deprecateDecorators.ts

@@ -1,108 +1,5 @@
 import { isDevMode } from '@angular/core';
 
-/**
- * @hidden
- */
-export const DeprecateClass = (message: string) => {
-    let isMessageShown = false;
-
-    return <T extends new(...args: any[]) => any>(originalClass: T) => class extends originalClass {
-        constructor(...args) {
-            const target: any = originalClass;
-            const targetName = typeof target === 'function' ? target.name : target.constructor.name;
-            isMessageShown = showMessage(`${targetName}: ${message}`, isMessageShown);
-
-            super(...args);
-        }
-    };
-};
-
-/**
- * @hidden
- */
-export function DeprecateMethod(message: string): MethodDecorator {
-    let isMessageShown = false;
-
-    return function(target: any, key: string, descriptor: PropertyDescriptor) {
-        if (descriptor && descriptor.value) {
-            const originalMethod = descriptor.value;
-
-            descriptor.value = function() {
-                const targetName = typeof target === 'function' ? target.name : target.constructor.name;
-                isMessageShown = showMessage(`${targetName}.${key}: ${message}`, isMessageShown);
-                const args = [];
-                for (const x of arguments) {
-                    args.push(x);
-                }
-                return originalMethod.call(this, ...args);
-            };
-
-            return descriptor;
-        }
-    };
-}
-
-/**
- * @hidden
- */
-export function DeprecateProperty(message: string): PropertyDecorator {
-    return function(target: any, key: string) {
-        let isMessageShown = false;
-        const messageToDisplay = `${target.constructor.name}.${key}: ${message}`;
-
-        // if the target already has the property defined
-        const originalDescriptor = Object.getOwnPropertyDescriptor(target, key);
-        if (originalDescriptor) {
-            const getter = originalDescriptor.get;
-            const setter = originalDescriptor.set;
-
-            if (getter) {
-                originalDescriptor.get = function() {
-                    isMessageShown = showMessage(messageToDisplay, isMessageShown);
-                    return getter.call(this);
-                };
-            }
-
-            if (setter) {
-                originalDescriptor.set = function(value) {
-                    isMessageShown = showMessage(messageToDisplay, isMessageShown);
-                    setter.call(this, value);
-                };
-            }
-
-            return originalDescriptor;
-        }
-
-        // the target doesn't contain a descriptor for that property, so create one
-        // use backing field to set/get the value of the property to ensure there won't be infinite recursive calls
-        const newKey = generateUniqueKey(target, key);
-        Object.defineProperty(target, key, {
-            configurable: true,
-            enumerable: true,
-            set(value) {
-                isMessageShown = showMessage(messageToDisplay, isMessageShown);
-                this[newKey] = value;
-            },
-            get() {
-                isMessageShown = showMessage(messageToDisplay, isMessageShown);
-                return this[newKey];
-            }
-        });
-    };
-}
-
-/**
- * @hidden
- */
-const generateUniqueKey = (target: any, key: string): string => {
-    let newKey = '_' + key;
-    while (target.hasOwnProperty(newKey)) {
-        newKey = '_' + newKey;
-    }
-
-    return newKey;
-};
-
 /**
  * @hidden
  */

projects/igniteui-angular/src/lib/directives/radio/radio-group.directive.ts

@@ -16,7 +16,6 @@ import { IgxRippleModule } from '../ripple/ripple.directive';
 import { takeUntil } from 'rxjs/operators';
 import { noop, Subject } from 'rxjs';
 import { mkenum } from '../../core/utils';
-import { DeprecateProperty } from '../../core/deprecateDecorators';
 
 /**
  * Determines the Radio Group alignment
@@ -129,16 +128,15 @@ export class IgxRadioGroupDirective implements AfterContentInit, ControlValueAcc
     }
 
     /**
-     * An @Input property that allows you to disable the radio group. By default it's false.
-     *
      * @deprecated in version 12.2.0
      *
+     * An input property that allows you to disable the radio group. By default it's false.
+     *
      * @example
      *  ```html
      * <igx-radio-group disabled></igx-radio-group>
      * ```
      */
-    @DeprecateProperty('`disabled` is deprecated.')
     @Input()
     public get disabled(): boolean {
         return this._disabled;
@@ -149,10 +147,10 @@ export class IgxRadioGroupDirective implements AfterContentInit, ControlValueAcc
     }
 
     /**
-     * Sets/gets the position of the `label` in the child radio buttons.
-     *
      * @deprecated in version 12.2.0
      *
+     * Sets/gets the position of the `label` in the child radio buttons.
+     *
      * @remarks
      * If not set, `labelPosition` will have value `"after"`.
      *
@@ -161,7 +159,6 @@ export class IgxRadioGroupDirective implements AfterContentInit, ControlValueAcc
      * <igx-radio-group labelPosition = "before"></igx-radio-group>
      * ```
      */
-    @DeprecateProperty('`labelPosition` is deprecated.')
     @Input()
     public get labelPosition(): RadioLabelPosition | string {
         return this._labelPosition;

projects/igniteui-angular/src/lib/grids/cell.component.ts

@@ -19,7 +19,6 @@ import { GridBaseAPIService } from './api.service';
 import { PlatformUtil } from '../core/utils';
 import { IgxGridBaseDirective } from './grid-base.directive';
 import { IgxGridSelectionService, ISelectionNode } from './selection/selection.service';
-import { DeprecateMethod } from '../core/deprecateDecorators';
 import { HammerGesturesManager } from '../core/touch';
 import { ColumnType } from './common/column.interface';
 import { RowType } from './common/row.interface';
@@ -662,19 +661,6 @@ export class IgxGridCellComponent implements OnInit, OnChanges, OnDestroy {
         private touchManager: HammerGesturesManager,
         protected platformUtil: PlatformUtil) { }
 
-    /**
-     * @deprecated
-     * Gets whether the cell is selected.
-     * ```typescript
-     * let isCellSelected = thid.cell.isCellSelected();
-     * ```
-     * @memberof IgxGridCellComponent
-     */
-    @DeprecateMethod(`'isCellSelected' is deprecated. Use 'selected' property instead.`)
-    public isCellSelected() {
-        return this.selectionService.selected(this.selectionNode);
-    }
-
     /**
      * @hidden
      * @internal
@@ -719,6 +705,20 @@ export class IgxGridCellComponent implements OnInit, OnChanges, OnDestroy {
         });
     }
 
+    /**
+     * @deprecated in version 8.2.0. Use 'selected' property instead
+     *
+     * Gets whether the cell is selected.
+     *
+     * ```typescript
+     * let isCellSelected = thid.cell.isCellSelected();
+     * ```
+     * @memberof IgxGridCellComponent
+     */
+    public isCellSelected() {
+        return this.selectionService.selected(this.selectionNode);
+    }
+
     /**
      * @hidden
      * @internal