InversifyJS

InversifyJS copied to clipboard

BindingToSyntax refactor discussion

8

There is common code in each of the "to" methods. For instance ``` public toConstantValue(value: T): interfaces.BindingWhenOnSyntax { this._binding.type = BindingTypeEnum.ConstantValue; this._binding.cache = value; this._binding.dynamicValue = null; this._binding.implementationType = null;...

tonyhallett

Binding typing discussion

7

@notaphplover With respect to https://github.com/inversify/InversifyJS/pull/1132#discussion_r623367620 This is what I think the correct typing should be after #1132. Omitting unnecessary properties. ``` export interface Binding extends Clonable { serviceIdentifier: ServiceIdentifier; dynamicValue:...

tonyhallett

Inversify-express-utils: Inject server level middleware

6

I don't see a clear way to inject server level express middleware, if that middleware has HttpContext as dependency. ## Expected Behavior ``` const expressServer = new InversifyExpressServer(inversifyContainer, null, {...

RPallas92

Sharing a single container among multiple packages

3

I'm looking for best practices or recommendations for my usage of Inversify. I'm defining a few layers of packages that all need to share a container instance. Each layer provides...

wheaney

Looks like incorrect behavior is in _validateActiveBindingCount

2

https://github.com/inversify/InversifyJS/blob/65596eaa2915787d14f852992539a957acc23af0/src/planning/planner.ts#L116 ```typescript switch (bindings.length) { // ... case BindingCount.OnlyOneBindingAvailable: if (!target.isArray()) { return bindings; } case BindingCount.MultipleBindingsAvailable: default: if (!target.isArray()) { // ... } else { return bindings; } }...

tryvols

Is the options inheritance does not work by design?

1

As a user of the library I would expect that the child container will inherit the options of the parent container. Option " skipBaseClassChecks" in particular is not inherited from...

lolik21

Missing required @inject or @multiInject annotation in: argument 0 in class t.

5

I have an issue resolving 3rd party class and the message I get is a bit cryptic. So I decorate PrismaClient with injectable and bind to the container ```typescript decorate(injectable(),...

reflash

Require cycle

2

Using inversify v5.0.5 and metro-bundler (running on react-native with babel) I get this warning. ``` [Fri Mar 19 2021 20:58:19.746] WARN Require cycle: node_modules/inversify/lib/syntax/binding_on_syntax.js -> node_modules/inversify/lib/syntax/binding_when_syntax.js -> node_modules/inversify/lib/syntax/binding_on_syntax.js Require cycles...

t-ricci-enhancers

What should I do if I want to multiInject multiple targetName?

3

I want a Ninja per Weapon in every game board class. ```typescript @injectable() class Katana implements Weapon { public hit() { return "cut!"; } } @injectable() class Shuriken implements Weapon...

ytetsuro

Explore possibility of support for setter injection

2

Suggested by @akosyakov in the chat: > ```ts class Foo { @inject(Bar) set bar(bar: Bar) { this.bar.onChanged(() => this.update()); } } ``` > I understood that i can do it...

remojansen