From 52362421181b71797137a58970efc60f8b49ce62 Mon Sep 17 00:00:00 2001 From: Nathaniel Furniss Date: Tue, 13 Jul 2021 15:06:01 -0700 Subject: [PATCH] Remove deprecated disabledWhen --- .../glimmer/lib/components/-link-to.ts | 6 -- .../glimmer/lib/components/link-to.ts | 35 -------- .../components/link-to/routing-angle-test.js | 81 +----------------- .../components/link-to/routing-curly-test.js | 83 +------------------ 4 files changed, 3 insertions(+), 202 deletions(-) diff --git a/packages/@ember/-internals/glimmer/lib/components/-link-to.ts b/packages/@ember/-internals/glimmer/lib/components/-link-to.ts index 642eeb64605..f0a8e651e13 100644 --- a/packages/@ember/-internals/glimmer/lib/components/-link-to.ts +++ b/packages/@ember/-internals/glimmer/lib/components/-link-to.ts @@ -879,12 +879,6 @@ const LinkComponent = EmberComponent.extend({ loadingHref: '#', didReceiveAttrs() { - let { disabledWhen } = this; - - if (disabledWhen !== undefined) { - this.set('disabled', disabledWhen); - } - let { params } = this; if (!params || params.length === 0) { diff --git a/packages/@ember/-internals/glimmer/lib/components/link-to.ts b/packages/@ember/-internals/glimmer/lib/components/link-to.ts index 40ee661d959..a4100a09665 100644 --- a/packages/@ember/-internals/glimmer/lib/components/link-to.ts +++ b/packages/@ember/-internals/glimmer/lib/components/link-to.ts @@ -614,7 +614,6 @@ if (EMBER_MODERNIZED_BUILT_IN_COMPONENTS) { }); } - // @disabledWhen { let superIsSupportedArgument = prototype['isSupportedArgument']; @@ -622,24 +621,6 @@ if (EMBER_MODERNIZED_BUILT_IN_COMPONENTS) { configurable: true, enumerable: false, value: function isSupportedArgument(this: LinkTo, name: string): boolean { - if (this.modernized) { - if (name === 'disabledWhen') { - deprecate( - 'Passing the `@disabledWhen` argument to is deprecated. ' + - 'Use the `@disabled` argument instead.', - false, - { - id: 'ember.link-to.disabled-when', - for: 'ember-source', - since: {}, - until: '4.0.0', - } - ); - - return true; - } - } - return superIsSupportedArgument.call(this, name); }, }); @@ -657,22 +638,6 @@ if (EMBER_MODERNIZED_BUILT_IN_COMPONENTS) { configurable: true, enumerable: false, get: function isDisabled(this: LinkTo): boolean { - if ('disabledWhen' in this.args.named) { - deprecate( - 'Passing the `@disabledWhen` argument to is deprecated. ' + - 'Use the `@disabled` argument instead.', - false, - { - id: 'ember.link-to.disabled-when', - for: 'ember-source', - since: {}, - until: '4.0.0', - } - ); - - return Boolean(this.named('disabledWhen')); - } - return superGetter.call(this); }, }); diff --git a/packages/@ember/-internals/glimmer/tests/integration/components/link-to/routing-angle-test.js b/packages/@ember/-internals/glimmer/tests/integration/components/link-to/routing-angle-test.js index 60ac6577dac..b44e1c4bd10 100644 --- a/packages/@ember/-internals/glimmer/tests/integration/components/link-to/routing-angle-test.js +++ b/packages/@ember/-internals/glimmer/tests/integration/components/link-to/routing-angle-test.js @@ -106,64 +106,6 @@ moduleFor( assert.strictEqual(this.$('#about-link').attr('href'), null, 'there is no href attribute'); } - async [`@test [DEPRECATED] it applies a 'disabled' class when disabledWhen`](assert) { - this.addTemplate( - 'index', - ` - About - About - ` - ); - - let controller; - - this.add( - 'controller:index', - class extends Controller { - constructor(...args) { - super(...args); - controller = this; - } - - dynamicDisabledWhen = true; - } - ); - - await expectDeprecationAsync( - () => this.visit('/'), - 'Passing the `@disabledWhen` argument to is deprecated. Use the `@disabled` argument instead.', - EMBER_MODERNIZED_BUILT_IN_COMPONENTS - ); - - assert.equal( - this.$('#about-link-static.disabled').length, - 1, - 'The static link is disabled when its disabledWhen is true' - ); - assert.equal( - this.$('#about-link-dynamic.disabled').length, - 1, - 'The dynamic link is disabled when its disabledWhen is true' - ); - - expectDeprecation( - () => runTask(() => controller.set('dynamicDisabledWhen', false)), - 'Passing the `@disabledWhen` argument to is deprecated. Use the `@disabled` argument instead.', - EMBER_MODERNIZED_BUILT_IN_COMPONENTS - ); - - assert.equal( - this.$('#about-link-static.disabled').length, - 1, - 'The static link is disabled when its disabledWhen is true' - ); - assert.strictEqual( - this.$('#about-link-dynamic.disabled').length, - 0, - 'The dynamic link is re-enabled when its disabledWhen becomes false' - ); - } - async [`@test it applies a 'disabled' class when disabled`](assert) { this.addTemplate( 'index', @@ -343,27 +285,6 @@ moduleFor( ); } - async [`@test [DEPRECATED] it does not respond to clicks when disabledWhen`](assert) { - this.addTemplate( - 'index', - `About` - ); - - await expectDeprecationAsync( - () => this.visit('/'), - 'Passing the `@disabledWhen` argument to is deprecated. Use the `@disabled` argument instead.', - EMBER_MODERNIZED_BUILT_IN_COMPONENTS - ); - - await expectDeprecationAsync( - () => this.click('#about-link'), - 'Passing the `@disabledWhen` argument to is deprecated. Use the `@disabled` argument instead.', - EMBER_MODERNIZED_BUILT_IN_COMPONENTS - ); - - assert.strictEqual(this.$('h3.about').length, 0, 'Transitioning did not occur'); - } - async [`@test it does not respond to clicks when disabled`](assert) { this.addTemplate( 'index', @@ -410,7 +331,7 @@ moduleFor( assert.equal( this.$('h3.about').length, 1, - 'Transitioning did occur when disabledWhen became false' + 'Transitioning did occur when disabled became false' ); } diff --git a/packages/@ember/-internals/glimmer/tests/integration/components/link-to/routing-curly-test.js b/packages/@ember/-internals/glimmer/tests/integration/components/link-to/routing-curly-test.js index 138a66f7068..5e8c4df70b0 100644 --- a/packages/@ember/-internals/glimmer/tests/integration/components/link-to/routing-curly-test.js +++ b/packages/@ember/-internals/glimmer/tests/integration/components/link-to/routing-curly-test.js @@ -111,64 +111,6 @@ moduleFor( ); } - async [`@test [DEPRECATED] it applies a 'disabled' class when disabledWhen`](assert) { - this.addTemplate( - 'index', - ` - - - ` - ); - - let controller; - - this.add( - 'controller:index', - class extends Controller { - constructor(...args) { - super(...args); - controller = this; - } - - dynamicDisabledWhen = true; - } - ); - - await expectDeprecationAsync( - () => this.visit('/'), - 'Passing the `@disabledWhen` argument to is deprecated. Use the `@disabled` argument instead.', - EMBER_MODERNIZED_BUILT_IN_COMPONENTS - ); - - assert.equal( - this.$('#about-link-static > a.disabled').length, - 1, - 'The static link is disabled when its disabledWhen is true' - ); - assert.equal( - this.$('#about-link-dynamic > a.disabled').length, - 1, - 'The dynamic link is disabled when its disabledWhen is true' - ); - - expectDeprecation( - () => runTask(() => controller.set('dynamicDisabledWhen', false)), - 'Passing the `@disabledWhen` argument to is deprecated. Use the `@disabled` argument instead.', - EMBER_MODERNIZED_BUILT_IN_COMPONENTS - ); - - assert.equal( - this.$('#about-link-static > a.disabled').length, - 1, - 'The static link is disabled when its disabledWhen is true' - ); - assert.strictEqual( - this.$('#about-link-dynamic > a.disabled').length, - 0, - 'The dynamic link is re-enabled when its disabledWhen becomes false' - ); - } - async [`@test it applies a 'disabled' class when disabled`](assert) { this.addTemplate( 'index', @@ -210,7 +152,7 @@ moduleFor( assert.equal( this.$('#about-link-static > a.disabled').length, 1, - 'The static link is disabled when its disabledWhen is true' + 'The static link is disabled when its disabled is true' ); assert.strictEqual( this.$('#about-link-dynamic > a.disabled').length, @@ -355,27 +297,6 @@ moduleFor( ); } - async [`@test [DEPRECATED] it does not respond to clicks when disabledWhen`](assert) { - this.addTemplate( - 'index', - `` - ); - - await expectDeprecationAsync( - () => this.visit('/'), - 'Passing the `@disabledWhen` argument to is deprecated. Use the `@disabled` argument instead.', - EMBER_MODERNIZED_BUILT_IN_COMPONENTS - ); - - await expectDeprecationAsync( - () => this.click('#about-link > a'), - 'Passing the `@disabledWhen` argument to is deprecated. Use the `@disabled` argument instead.', - EMBER_MODERNIZED_BUILT_IN_COMPONENTS - ); - - assert.strictEqual(this.$('h3.about').length, 0, 'Transitioning did not occur'); - } - async [`@test it does not respond to clicks when disabled`](assert) { this.addTemplate( 'index', @@ -422,7 +343,7 @@ moduleFor( assert.equal( this.$('h3.about').length, 1, - 'Transitioning did occur when disabledWhen became false' + 'Transitioning did occur when disabled became false' ); }