8000 SearchExtension - ability to create factory definitions from interfaces by peldax · Pull Request #200 · nette/di · GitHub
[go: up one dir, main page]

Skip to content

SearchExtension - ability to create factory definitions from interfaces#200

Merged
dg merged 2 commits intonette:masterfrom
peldax:patch-2
Jul 24, 2019
Merged

SearchExtension - ability to create factory definitions from interfaces#200
dg merged 2 commits intonette:masterfrom
peldax:patch-2

Conversation

@peldax
Copy link
Contributor
@peldax peldax commented May 10, 2019

This PR extends SearchExtension to be able to search for interfaces and create factory definitions from them.

WIP is added due to some duplicate code that needs to be refactored.

@peldax
Copy link
Contributor Author
peldax commented May 10, 2019

@dg I am making tests for this PR and I am wordering what is desirable outcome when factory interface doesnt implement the create() function

  • Skip the interface silently?
  • Throw exception that interface doesnt have the function?

@peldax peldax changed the title [WIP] SearchExtension - ability to create factory definitions from interfaces SearchExtension - ability to create factory definitions from interfaces Jul 8, 2019
@dg dg force-pushed the master branch 7 times, most recently from fecd1fb to 15978c3 Compare July 19, 2019 11:32
@dg dg merged commit 1237d5d into nette:master Jul 24, 2019
dg pushed a commit that referenced this pull request Jul 24, 2019
dg pushed a commit that referenced this pull request Aug 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0