emberjs/ember.js

@tracked: lazy evaluation

Open

#18.118 aberto em 19 de jun. de 2019

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)JavaScript (4.216 forks)batch import
DocumentationHelp Wanted

Métricas do repositório

Stars
 (22.437 stars)
Métricas de merge de PR
 (Mesclagem média 11d 14h) (20 fundiu PRs em 30d)

Description

While investigating #18075, I noticed a minor issue, that persists even after the change to WeakMap in #18091, because of the lazy evaluation

class Foo {
  @service someService;

  @tracked someProp = this.someService.serviceProp;
}

Assume Foo gets created by a container lookup or get manually called setOwner on.

Intuitively I would expect the above code to work. And it does. If this.someProp is only accessed later in the template or in init, everything works fine.

However, if you already access this.someProp in the constructor, you get a TypeError, because this.someService is undefined.

While I understand why, I still think this is inconsistent. If we decide to keep lazy evaluation of the initializers, we need to document it well. Would the new static decorators proposal support lazy evaluation in the first place?

Guia do colaborador