Ajustado o problema apontado no phpunit

[GustavoSantarosa]

Ola galera,
Sou Luis Gustavo, programador php a mais de 16 anos, acompanho vocês ja tem uns 7 anos ou mais, e nunca me motivei a contribuir.

Porem hoje voltando a usar o projeto, decidi me juntar a vcs, se me permitirem contribuir e passar um pouco do meu conhecimento, quero ajudar.

Neste PR:

• Rodado phpcs na pasta src conforme as regras do projeto
• Removido uma criação de variavel nao utilizada
• Removido uma criação de variavel nao utilizada
• Configurado um arquivo de helper e junto dois helpers que ajudam bastante a deixar o codigo clean, sem gerar dependencia
• Criado o primeiro config de authorizers para substituir storage\autorizadores.json, pra manter o padrao do projeto, e facilitar a consulta destas informações, sem precisar ficar lendo arquivos.json
• melhorado uma função utilizando os novos helpers, para servir de exemplo
• Faltou renomear o nome do config para a ultima versao em ingles

Meu linkedin: https://www.linkedin.com/in/luis-gustavo-santarosa-pinto

[codecov]

Codecov Report

Attention: Patch coverage is 73.80952% with 33 lines in your changes missing coverage. Please review.

Project coverage is 38.55%. Comparing base (7edd81f) to head (44bb77d).
Report is 9 commits behind head on master.

Files with missing lines	Patch %	Lines
src/Traits/TraitTagAgropecuario.php	0.00%	29 Missing ⚠️
src/Libs/Helpers.php	83.33%	3 Missing ⚠️
src/Complements.php	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #1114      +/-   ##
============================================
+ Coverage     38.26%   38.55%   +0.28%     
+ Complexity     1828     1827       -1     
============================================
  Files            51       53       +2     
  Lines         18104    18191      +87     
============================================
+ Hits           6928     7013      +85     
- Misses        11176    11178       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[gersonfs]

Opa, só acho que não deveríamos ter 2 arquivos com o mesmo propósito: authorizers.php e autorizadores.json senão tem que ficar atualizando 2 arquivos, a chance de esquecer um deles é grande.

[robmachado]

Concordo ! Eu não reparei nisso .. podemos chamar o json dentro do script php ou remover o json

Olhando melhor o código o autorizadores.json foi ignorado !! não é mais usado nos métodos.

[robmachado]

Mas eu não sei até que ponto é útil IMITAR o comportamento do LARAVEL nessa estrutura !
Pode haver algum conflito !?!?

[gersonfs]

Particularmente prefiro trabalhar direto com arquivo PHP, assim já está no formato que o código vai usar, ao contrário do json que precisa ser parseado.
Sobre conflitos, não deve dar nenhum pois toda função adicionada é precedida de um function_exists

[robmachado]

OK ! fica como está então .. vou renomear o json para não conflitar e tirar qq comentário sobre ele

[GustavoSantarosa]

Boa noite, só consegui dar uma atenção agora.

Opa, só acho que não deveríamos ter 2 arquivos com o mesmo propósito: authorizers.php e autorizadores.json senão tem que ficar atualizando 2 arquivos, a chance de esquecer um deles é grande.

• a ideia é depreciar o arquivo json mesmo, pois a forma como ela é importada no código, deixa o codigo um pouco sujo.
• e tambem eu ainda não fiz, mas futuramente, da pra colocar envs no config, e usar os autorizadores como default, e tirar um pouco o engessamento, caso futuramente, os autorizadores venham a ser modificados, os utilizadores, nao precisam ficar aguardando uma atualização do pacote, ou ter a necessidade de ficar fazendo fork ou sobreescrevendo a função, simplesmente, ele cria a env, e ja resolve o problema temporariamente, até que a atualização do projeto seja lançado.

[GustavoSantarosa]

Mas eu não sei até que ponto é útil IMITAR o comportamento do LARAVEL nessa estrutura ! Pode haver algum conflito !?!?

• Não tem chance de dar conflito, pois apesar de imitar o comportamento, unica coisa que foi feita é o helper, que pode ser substituido por outro arquivo helper, no caso do laravel, facilmente.
• Fora isso, ele usa php puro pra executar a função de config
• e a melhor utilidade, é o clean code, e a facil substituição de variavel atraves de env(a ser implementada) por parte de quem usa o pacote