<div class="highlight highlight-source-ts notranslate position-relative overflow-auto" dir="auto" da

Remove misleading warning when using `@wire` on a getter or a setter about lwc HOT 1 OPEN

wjhsf commented on August 9, 2024

Remove misleading warning when using `@wire` on a getter or a setter

from lwc.

Comments (1)

nolanlawson commented on August 9, 2024

This is actually subtle. You can use an @wire with a getter/setter in the following ways:

In all three cases, the wire works as expected but an error is logged:

Invalid @wire value field. Found a duplicate accessor with the same name.

Since this is behavior we support (and I don't see any reason not to support it – a getter/setter is roughly the same as a class property), I would propose we don't remove it and instead get rid of the error-logging.

Based on the tests for the error message BTW, it looks like the intention was to prevent truly egregious mistakes like this one:

lwc/packages/@lwc/integration-karma/test/component/decorators/wire/index.spec.js

Lines 27 to 41 in 9e4ea59

 expect(() => { 

 class DuplicateProperty extends LightningElement { 

 foo = 'observed'; 

 // eslint-disable-next-line no-dupe-class-members 

 @wire(adapter) foo = 'wired'; 

 render() { 

 return duplicatePropertyTemplate; 

 } 

 } 

 Ctor = DuplicateProperty; 

 }).toLogErrorDev( 

 /Invalid observed foo field\. Found a duplicate accessor with the same name\./ 

 );

from lwc.

Remove misleading warning when using `@wire` on a getter or a setter about lwc HOT 1 OPEN

Comments (1)

Related Issues (20)

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	expect(() => {
	class DuplicateProperty extends LightningElement {
	foo = 'observed';
	// eslint-disable-next-line no-dupe-class-members
	@wire(adapter) foo = 'wired';

	render() {
	return duplicatePropertyTemplate;
	}
	}

	Ctor = DuplicateProperty;
	}).toLogErrorDev(
	/Invalid observed foo field\. Found a duplicate accessor with the same name\./
	);