Usando o Respect (RFC) #57
Conversation
|
Me dêem o feedback nessa bomba que eu já ou removo ou faço o merge dessa birosca aqui 🎉 /cc @rogeriopradoj @diegoholiveira |
|
+1 pro Respect Por mim pode dar merge |
|
+1 pro Respect |
|
😎 @hussani e @MichelAraujo Hoho, Valeu pelo feedback :D |
|
Já falei que sou a favor do uso do Respect em outra issue mas vai aqui o meu 👍 |
|
@augustohp tá ótimo. só faço uma sugestão: o alganet postou no repositório de samples do respect um exemplo de como ele usa o Respect Config para fazer o setup das rotas em um arquivo ini, eu gostei demais desse exemplo dele porque deixou o código muito limpo, bem direto ao ponto. Talvez isso possa ser implementado aqui. O que acha? |
|
@diegoholiveira Dê uma olhada na branch do Respect que o @augustohp criou. Lá está sendo usado o Rest, o Config e o Validation. |
|
@diegoholiveira Eu to ligado, tb curti mas esse branch eu tinha feito há muito tempo... To fazendo revisão e melhorando ele ainda. Mas essa semana já rol o merge. On Sat, Jul 13, 2013 at 12:15 PM, Victor Ferreira
|
|
Acho melhor fazer o merge da issue #14 antes de mergear esse cara aqui. Óbvio que o Travis me fodeu, mas na minha máquina local ta rolando... Vou mexer (provavelmente) em algunas coisas ainda, mas um code-review até aqui seria muito bem vindo. :P |
|
Cara, por favor não fica bravo, mas ficou engraçada a frase: [...] mas na minha máquina local ta rolando :-) |
Melhora a cobertura por testes unitários e corrige um bug encontrado em `PDODataAccess::getAll()` que utilizava uma classe inexsitente para o método `PdoStatement::setFetchMode`. PS: Pagando o preço do TDD =P
As rotas sao implementadas por duas classes:
- Ophportunidades/Route/AllPositions.php
- Ophportunidades/Route/OnePosition.php
São elas (as rotas):
- [GET] /position Retorna uma posição
- [GET] /positions Retorna todas as posições cadastradas
- [POST] /position Cria uma noca posição
Usei o Respect/Config para configurar as dependências das classes
já existentes no `conf/app.ini` e fiz um teste simples te *integration*
só pra garantir que as coisas funcionam como deveriam.
O README tava um pouco esquecido. Dei uma atualizada bem bunda nele mas que pelo menos cobre um **workflow de GIT** bem básico pra gente se organizar e como o resto do pessoal pode fazer pra contribuir com o desenvolvimento da aplicação. PS: Eu sei que apesar de lista o Vagrant como requisito do README ele não está disponível ainda, mas pelo menos isso torna desnecessária a edição do README quando ele for implementado (espero) num futuro breve.
- Criado FACADE para permitir regras dentro da nossa namespace. - Criada regra para validação de IDs. - Implementada regra para validação de IDs.
- Geração de coverage dentro do diretório "reports" - Diretório "reports" acrescentado ao .gitignore
- Remove bootstrap alternativo sem autoload do Composer. - Cria configurações da aplicação baseada em ambientes, tornando o antigo conf/app.ini obsoleto. - Move configurações de rotas do index.php para conf/httpRoutes.ini usando o Respect/Config. - Move configuração de acesso a dados do obsoleto conf/app.ini para conf/dataAccess.ini. - Define configuração de ambiente no Boxfile para o Pagodabox. O processo de bootstrap agora se divide em 3 etapas: - Carregamento do composer e declaração de constantes de diretório. - Declaração de constantes de ambiente e construção do container de configuração. - Configuração da aplicação.
|
@rogeriopradoj Nem me fala! AHUuHAhuAhuahua |
| @@ -5,7 +5,11 @@ | |||
| }, | |||
| "require-dev": { | |||
| "phpunit/phpunit": "3.7@stable", | |||
| "squizlabs/php_codesniffer": "dev-master" | |||
| "squizlabs/php_codesniffer": "dev-master", | |||
| "phpunit/phpunit": "3.7@stable", | |||
There was a problem hiding this comment.
Pra ter certeza que ele instala né! #not
PS: Fiz o merge desse conflito igual minha bunda. Valeu pelo review!
|
@augustohp tá escavando ae? 😂 |
HAHAHAHA. Descobri o link "Pull Requests" no cabeçalho do GitHub agora. Tem uma penca de coisa velha aberta que eu nem lembrava mais que existia. 😜 |
Criei este branch apenas pra um estudo e ensaio de como utilizar o Respect mantendo o máximo do código que temos hoje. Gostaria de opiniões para um merge ao develop.
Fora os testes eu não fiz mas nenhuma verificação, isto é: as rotas deveriam funcionar com o Respect\Rest.
Existe um refactor que pode definitivamente ser aproveitado (
git cherry-pick) no commit 21bd822.Eu preciso ainda criar uma forma melhor de utilizar regras de validação da nossa namespace com o
Respect\Validation, pretendo fazer isso nos próximos dias.Já aproveitei e fiz o merge do branch do travis com um MakeFile meio crú mas que está funcionando.
Suite rodando no PHP 5.4 do Travis