Remove o php-di, valida o método __get e adiciona testes #5
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Olá.
Estou usando o totalvoice em um projeto com slim framework e tive problemas com a dependência do totalvoice com o php-di.
Meu projeto também utiliza o php-di, mas em uma versão mais nova e o composer não permite que existam duas versões diferentes instaladas. Ao analisar o projeto de vocês, vi que o uso do php-di pode ser eliminado sem nenhum prejuízo aos usuários.
Fiz a remoção do php-di e adicionei a restrição do
__get
para um conjunto bem definido de classes da api. O modelo anterior permitia instanciar qualquer classe que estivesse em src/Api, no entanto, como existem classes abstratas nessa pasta, não fica imediatamente claro que classes podem realmente ser instanciadas desse modo.