@service router is undefined about ember-decorators HOT 8 CLOSED

aha 🦆
I needed to use
this.get('router')
instead of
this.router

from ember-decorators.

That should not be the case, which version of Ember are you using?

from ember-decorators.

@pzuraq I've run into this as well, testing the latest Ember Octane.

"ember-source": "https://s3.amazonaws.com/builds.emberjs.com/canary/shas/707a938e536cb921dbe80ee270c431d1b9717e38.tgz",
"ember-decorators": "5.1.4",

All ES6 classes are affected it looks like; doesn't matter what Ember class it extends from (EmberObject, Route, Controller, Service, and Component are the ones I've run into it with)

from ember-decorators.

@pgrippi can you provide the version of @ember-decorators/babel-transforms you are using? And if it’s not too much trouble, the code of the class itself? That would help a bit in figuring this out

from ember-decorators.

Sure, I'm using (figure it can't hurt to throw the babel version in too):

"@ember-decorators/babel-transforms": "5.1.4"
"ember-cli-babel": "7.6.0"

As for the code, the simplest case is in my application route:

import Route from '@ember/routing/route';
import { inject as service } from '@ember-decorators/service';

export default class ApplicationRoute extends Route {
  @service router;

  init() {
    super.init(...arguments);

    this.router.on('routeDidChange', () => {
      console.debug('route did change');
    });
  }
}

from ember-decorators.

alright, I'll see if I can get a reproduction going. For reference, you should be using the stage 1 versions of the transforms:

"@ember-decorators/babel-transforms": "2.1.2"

We're working on baking these into ember-cli at the moment, I realize this is confusing. If you use the stage 1 transforms, do you have the same problem?

from ember-decorators.

Is there anything else I have to do to use stage 1 other than downgrading @ember-decorators/babel-transforms to 2.1.2? After downgrading to 2.1.2 I still see the issue

from ember-decorators.

I believe this should be fixed on the latest ember-decorators, with the polyfill. Let us know if not!

from ember-decorators.