Decorator bug with target parameter about babel-plugin-transform-decorators-legacy

[loganfsmyth]

The target of a method decorator is the class prototype, not the constructor function.

[rsumilang]

Thanks for the quick reply! What about static methods? It seems that they receive a constructor like a class instead of a prototype:

function dec(debug) {
  return (target, property, descriptor) => {
    console.log(debug);
    console.log(target);
  }
}

class A {

  @dec('from static method:')
  static doSomething(){

  }
}

Console:

from static method:
> function A () {
   _classCallback(this, A);
}

Is this intended functionality?

[loganfsmyth]

Yes. A method decorator's target, whether static or non-static, is the object that the property is getting added to. When you specify a non-static method, even without the decorator the method is available at Constructor.prototype.foo, with a static method it's at Constructor.foo. The decorator just gets whatever object the descriptor will be defined on.

[rsumilang]

I see (said the blind man). ;) Thanks for the explanation!

[loganfsmyth]

No problem, always feel free to drop by #javascript on Freenode or Babel's Slack if you have more ES6 questions.