Nova implementação para BancoDoBrasil

[gilbertoca]

Nova implementação para BancoDoBrasil contemplando as novas regras para geração de Boleto de Combrança - #158 . Referência: www.bb.com.br/docs/pub/emp/mpe/espeboletobb.pdf Útimo acesso: 06/01/2017

Conserta #204 Conserta #161

[angeliski]

Salve @gilbertoca Você chegou a validar essa implementação direto com o banco do brasil?

[gilbertoca]

Sim, @angeliski. Comentei aqui https://github.com/caelum/caelum-stella/issues/158 quando solicitei a revisão do código.

[angeliski]

Show de bola @gilbertoca ! Vou revisar e pedir pro @Turini ou o @mariofts Acompanhar com a gente. :)

[gilbertoca]

@angeliski @Turini @mariofts Travis engasgou!

[angeliski]

Opa @gilbertoca Aqui pra mim ele parece que quebrou só um teste

[gilbertoca]

@angeliski @Turini @mariofts Fork atualizado:

[INFO] --- maven-compiler-plugin:2.3.2:compile (default-compile) @ vraptor-validator-example ---
[INFO] Compiling 2 source files to /home/gilberto.andrade/tmp/caelum-stella/examples/vraptor-validator-example/src/main/webapp/WEB-INF/classes
[INFO] 
[INFO] --- maven-resources-plugin:2.6:testResources (default-testResources) @ vraptor-validator-example ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory /home/gilberto.andrade/tmp/caelum-stella/examples/vraptor-validator-example/src/test/resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.3.2:testCompile (default-testCompile) @ vraptor-validator-example ---
[INFO] No sources to compile
[INFO] 
[INFO] --- maven-surefire-plugin:2.4.2:test (default-test) @ vraptor-validator-example ---
[INFO] Tests are skipped.
[INFO] 
[INFO] --- maven-surefire-plugin:2.4.2:test (run unit tests) @ vraptor-validator-example ---
[INFO] No tests to run.
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Caelum Stella ...................................... SUCCESS [  3.222 s]
[INFO] Stella Core ........................................ SUCCESS [ 10.183 s]
[INFO] Stella Bean Validation ............................. SUCCESS [  4.636 s]
[INFO] Stella Hibernate User-Types ........................ SUCCESS [ 10.526 s]
[INFO] Stella Faces ....................................... SUCCESS [  4.727 s]
[INFO] Stella Boleto ...................................... SUCCESS [ 27.493 s]
[INFO] Stella Frete ....................................... SUCCESS [  1.586 s]
[INFO] Example: Stella Faces 1x ........................... SUCCESS [  0.790 s]
[INFO] Example: Stella Faces 2x ........................... SUCCESS [  1.551 s]
[INFO] Example: Validator and Persistence ................. SUCCESS [  1.972 s]
[INFO] Example: Validator and VRaptor ..................... SUCCESS [  1.830 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 01:09 min
[INFO] Finished at: 2017-11-08T13:03:31-03:00
[INFO] Final Memory: 102M/445M
[INFO] ------------------------------------------------------------------------
[13:03] gilberto.andrade@BEM-INF007:caelum-stella (master u=) $ 

O legal seria ver todos os testes executarem - assim teríamos uma noção do todo.

[Turini]

@gilbertoca executei no travis novamente e parece estar tudo ok agora!

[gilbertoca]

Pessoal @angeliski @Turini @mariofts , o que vcs acham? Dá pra incorporar esse PR?

[angeliski]

Eu acredito que esteja tudo certo. Podemos fazer o merge @Turini ?

[Turini]

oi, pessoal

ainda nao consegui parar pra olhar o código com cuidado e testar. vou fazer isso até o final da semana, ta bem?

@angeliski você consegue dar pull e testar gerar um boleto desse tipo, pra garantir que está funcionando bem em outros environments? ajudaria bastante

[angeliski]

Opa @Turini , Acho que quarta a noite eu consigo fazer os testes e coloco o feedback aqui :+1:

[angeliski]

@Turini Eu fiz um teste e me parece estar tudo certo. Vou anexar os boletos do Santander e do Banco do Brasil que foram gerados. @gilbertoca Eu só fiquei com uma dúvida. Tem uma regra na classo do BancoDoBrasil (Linha 84), onde o campoLivre tem que ser exatamente 25 digitos. Isso confere? Não pode ser mais nem menos? Porque como esse campo é composto de vários outros, eu achei meio estranho.

Santander.pdf Banco do Brasil.pdf

[gilbertoca]

Confere sim @angeliski ! Veja na documentação que linquei no inicio do post.

2.3 Leiaute do Código de Barras 2.3.2 Conteúdo - veja o quadro com a dimensão/composição do código de barras(acredito que vale para todos os bancos)

[gilbertoca]

@angeliski @Turini @mariofts , o travis tá limpo - e agora?

[angeliski]

Falta algo @Turini ?

[georgelima1]

Ainda falta alguma alteracao para esse PR ser aprovado? Essas alteracoes estao sendo muito esperadas pelos devs, inclusive eu.

[angeliski]

Hey @Turini precisa alguma outra alteração nesse PR?

[disparter]

Hey @Turini precisa alguma outra alteração nesse PR?

[Turini]

oi pessoal

eu não consigo testar por agora, mas como parece impactar o trabalho de mais de uma pessoa vou mergear e tentar fazer um release -beta com essa versão. Depois de vocês homologarem bem e testarem em um projeto em produção, lançamos uma versão final com essa implementação, tá bem?

e aproveitando; o projeto precisa de manteiners, como vocês viram. Se alguém se interessar em contribuir oficialmente fique a vontade de me mandar um email, que coloco em contato com o pessoal da caelum.

Seria uma ajuda muito bem-vinda.

[Turini]

release feito. Deve ficar disponível em algumas horas no maven.