laravel-pagseguro icon indicating copy to clipboard operation
laravel-pagseguro copied to clipboard

[IMPROVEMENT] - mudança do arquivo de configuração

Open marabesi opened this issue 6 years ago • 0 comments

Motivação

Hoje o laravel-pagseguro usa o padrão PSR0 para carregar as classes e possui um estilo de código razoável.

Porém analisando o código fonte existe um arquivo chamado application-config.php dentro do diretório src/laravel/pagseguro/Config. Porém esse arquivo não é uma classe é apenas um esqueleto das configurações disponíveis para definir URL's e outras coisas necessárias ao projeto.

Acompanhando a evolução do Laravel, é possível perceber que os arquivos que não possuem o padrão do autoload foram removidos de dentro do "coração" das classes, como por exemplo os arquivos de rotas. Em versÕes antigas são armazenados no diretório app/Http/routes.php e nas mais recentes em routes/web.php.

Sugestão

Estou abrindo essa issue para ver o que acham da seguinte sugestão:

Mover o arquivo de configurações do diretório laravel-pagseguro/src/laravel/pagseguro/Config para a pastaconfig.

Para facilitar o entendimento, veja a estrutura agora:

|__ src
      |___ Config 
              |___ application-config.php

Como ficará

|__ src
|___ config
       |___ application-config.php

Ou seja, a pasta config ficará no mesmo nível do src, assim poderemos considerar que tudo que existe em src condiz com a PSR0.

marabesi avatar Sep 04 '17 00:09 marabesi