getCtor returns undefined when class property refers to another class that is defined after it

[hookyns]

getCtor returns undefined when class property refers to another class that is defined after it

[avin-kavish]

What I did is more like this. The link is not running but I think you can check the generated code.

[Hookyns]

@avin-kavish

It works for me.

import { getType } from "tst-reflect";

function entity(_: any)
{
}

@entity
export class Cat
{
	id = 0;
	toys: Toy[] = [];
}

@entity
export class Toy
{
	id = 0;
}

const type = getType<Toy>();
type.getCtor().then(ctor => console.log(ctor)); // It prints [class Toy]; working with Cat class too.

Can you send repro, eg. on StackBlitz?

[Hookyns]

Okay, there are three issues.

• First issue is not generated "ctor" property.

• Second issue is how decorators work.
  There is __decorate call inside generated TypeScript which is called before type is exported from the module so you cannot import it from the decorator. This is a "no go".
  You will receive type, there is no problem, but import is gonna return undefined, always.
  Solution is simple. Use decorator's parameter. First parameter is the decorated class.

/**
 * @reflect
 */
export function entity<TType extends Function>(Class: TType) {
	const typeT = getType<TType>();
	console.log(typeT.name, Class);
}

• And there is the third problem. Simple decorators are broken, cuz of how I handle them so you will never receive the decorated target.

I've already fixed the issue with decorators (in [email protected]) so you can use the solution from the second point.
Meanwhile, I will fix the issue from the first point, but you cannot use it in decorator anyway.

[avin-kavish]

Alright. Actually, 2nd point is not a problem. I did it that way just for the stackblitz example. In my actual implementation I push it to a container and later once all the modules are loaded, I check the types.

/**
 * @reflect
 */
export function entity<TType extends Constructor>(_: any) {
  const typeInfo = getType<TType>()
  EntityTypeStore.push(typeInfo)
}

[avin-kavish]

It's happening here also. https://stackblitz.com/edit/tst-reflect-issue-49-htnd32?file=package.json,index.ts

[avin-kavish]

Happened again. This time seemingly random, class next to it gets imported just fine.

[Hookyns]

Fixed in v1. Not in the current version.