Decorators and plain object properties

[fatfisz]

I understand that when it comes to property initializers, there might be a need to perform some hocus-pocus with the initializer property, which effectively prevents modifying descriptor's value (it is overwritten by the result of the initializer function in the end).

But for plain objects descriptor's value can be left intact. Accessing it by a decorator is currently unnecessarily prevented.

Could it be not so? If yes, then I volunteer to create a PR with a fix for that :)

[fatfisz]

For now I'm using a decorator that fixes this:

function fixDescriptor(target, key, descriptor) {
  if (descriptor.initializer) {
    descriptor.value = descriptor.initializer.call(target);
    delete descriptor.initializer;
  }
}

... but remembering to put it in front of all properties isn't something I'm looking forward to.

[sebmck]

There's nothing to fix here. This is in the specification/proposal and it wont be changed unless it is there. See the javascript-decorators repo for more information.

[fatfisz]

Could you please point me to the right place, because in the README.md file there is nothing written about plain object properties, and INITIALIZER_INTEROP.md is only concerned with classes.

I've already read that spec carefully before. If you're not convinced, then maybe I'll take it to the spec so that this case will be clarified there.

[sebmck]

wycats/javascript-decorators#36

On Sat, Oct 10, 2015 at 9:22 PM, Rafał Ruciński notifications@github.com
wrote:

Could you please point me to the right place, because in the README.md file there is nothing written about plain object properties, and INITIALIZER_INTEROP.md is only concerned with classes.

I've already read that spec carefully before. If you're not convinced, then maybe I'll take it to the spec so that this case will be clarified there.

Reply to this email directly or view it on GitHub:
#2519 (comment)

[babel-bot]

Comment originally made by Rafał Ruciński (fatfisz) on 2016-03-03T23:42:45.000Z

@sebmck I've recently had a discussion with a peculiar individual in the https://github.com/wycats/javascript-decorators repo (I saw that you too commented on his rude demeanor). That reminded me of this issue, and while looking for it I also found https://phabricator.babeljs.io/T2005, an earlier issue concerning the same problem.
So, for what it's worth, sorry for not searching carefully enough to see if I won't be creating a duplicate. I hope I wasn't rude too much.
And thanks for sticking to the most actual spec, even if some people don't agree with some parts of it.