class decorator works different from babel5 about babel-plugin-transform-decorators-legacy

[loganfsmyth]

Probably another case of #24 where your plugins are in the wrong order?

[goto100]

Seems not. I'm not using transform-class-properties, my .babelrc file is:

{
  "plugins": [
    "transform-decorators-legacy",
    "add-module-exports",
    "transform-es2015-classes",
    "transform-es2015-destructuring",
    "transform-es2015-function-name",
    "transform-es2015-parameters",
    "transform-es2015-sticky-regex",
    "transform-es2015-unicode-regex",
    "transform-es2015-modules-commonjs"
  ]
}

[goto100]

If I add use strict in the head of the test file, I can even use just one single plugin, the problem still happens.

'use strict';

function classDecorator(Target) {
  return class DecoratedClass extends Target {
    test() {
      return 'decorated';
    }
  };
}

@classDecorator
class Test {
  static test() {
    return 'original';
  }

  static run() {
    // `Test` is not the *decorated* class here, but it is in bebel 5.
    console.log(Test.test());
  }
}

Test.run();

.babelrc

{
  "plugins": [
    "transform-decorators-legacy"
  ]
}

[loganfsmyth]

Ugh, I'm sorry, I misread your issue. This is actually a duplicate of #16, not the one I linked to.

The spec currently is just desugaring examples, which supports the current behavior, rather than the Babel 5 behavior, since it says

@dec
class Foo {
}

should behave like

let Foo = (function () {
  class Foo {
  }

  Foo = dec(Foo) || Foo;
  return Foo;
})();

[goto100]

Sorry for the mistake, the code in this case works fine. So I opened another issue #29, that's the actually problem.