Skip to content

Commit

Permalink
refactor: remove hookDisposablesRunner
Browse files Browse the repository at this point in the history
In v5 this is replaced with `@ember/destriyable`.
  • Loading branch information
buschtoens committed Aug 6, 2020
1 parent 6c4d85e commit 39ad4ee
Show file tree
Hide file tree
Showing 8 changed files with 0 additions and 47 deletions.
3 changes: 0 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -27,9 +27,6 @@ ember install ember-lifeline-decorators ember-lifeline

## Usage

You don't need to register `runDisposables(this)` in the `willDestroy` hook. The
decorators do this for you.

### Available decorators

| Decorator | ember-lifeline | Description |
Expand Down
3 changes: 0 additions & 3 deletions addon/debounce.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ import EmberObject from '@ember/object';

import { debounceTask } from 'ember-lifeline';

import hookDisposablesRunner from './hook-disposables-runner';
import privateAlias from './utils/private-alias';
import { Prototype } from './utils/type-helpers';

Expand All @@ -21,8 +20,6 @@ export default decoratorWithRequiredParams(function debounce<
typeof desc.value === 'function'
);

hookDisposablesRunner(target.constructor);

return privateAlias(
target,
key,
Expand Down
3 changes: 0 additions & 3 deletions addon/disposable.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import { registerDisposable } from 'ember-lifeline';

import { afterMethod } from 'patch-method';

import hookDisposablesRunner from './hook-disposables-runner';
import { Prototype } from './utils/type-helpers';

export default decoratorWithParams(function disposable<
Expand All @@ -21,8 +20,6 @@ export default decoratorWithParams(function disposable<
typeof desc.value === 'function'
);

hookDisposablesRunner(target.constructor);

afterMethod(target.constructor, 'init', function () {
// `.bind` is required because ember-lifeline does not set the correct context
registerDisposable(this, desc.value.bind(this));
Expand Down
2 changes: 0 additions & 2 deletions addon/event-listener.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ import { addEventListener } from 'ember-lifeline';

import { afterMethod } from 'patch-method';

import hookDisposablesRunner from './hook-disposables-runner';
import { Prototype, Constructor } from './utils/type-helpers';

// eslint-disable-next-line @typescript-eslint/naming-convention, @typescript-eslint/no-empty-function
Expand Down Expand Up @@ -45,7 +44,6 @@ export default decoratorWithRequiredParams(function eventListener<
);

collapseProto(target);
hookDisposablesRunner(target.constructor);

afterMethod(
target.constructor as Constructor<Component>,
Expand Down
27 changes: 0 additions & 27 deletions addon/hook-disposables-runner.ts

This file was deleted.

3 changes: 0 additions & 3 deletions addon/later.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ import EmberObject from '@ember/object';

import { runTask } from 'ember-lifeline';

import hookDisposablesRunner from './hook-disposables-runner';
import { Prototype } from './utils/type-helpers';

export default decoratorWithRequiredParams(function later<
Expand All @@ -20,8 +19,6 @@ export default decoratorWithRequiredParams(function later<
typeof desc.value === 'function'
);

hookDisposablesRunner(target.constructor);

return {
...desc,
value(this: InstanceType<typeof target.constructor>, ...args: any[]) {
Expand Down
3 changes: 0 additions & 3 deletions addon/schedule.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ import EmberObject from '@ember/object';

import { scheduleTask } from 'ember-lifeline';

import hookDisposablesRunner from './hook-disposables-runner';
import { Prototype } from './utils/type-helpers';

/**
Expand All @@ -27,8 +26,6 @@ export default decoratorWithRequiredParams(function schedule<
typeof desc.value === 'function'
);

hookDisposablesRunner(target.constructor);

return {
...desc,
value(this: InstanceType<typeof target.constructor>, ...args: any[]) {
Expand Down
3 changes: 0 additions & 3 deletions addon/throttle.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ import EmberObject from '@ember/object';

import { throttleTask } from 'ember-lifeline';

import hookDisposablesRunner from './hook-disposables-runner';
import privateAlias from './utils/private-alias';
import { Prototype } from './utils/type-helpers';

Expand All @@ -21,8 +20,6 @@ export default decoratorWithRequiredParams(function throttle<
typeof desc.value === 'function'
);

hookDisposablesRunner(target.constructor);

return privateAlias(
target,
key,
Expand Down

0 comments on commit 39ad4ee

Please sign in to comment.