Colocando as variaveis em camel case

nfephp-org / sped-da

Classes para geração dos documentos auxiliares usados pelos padrões Sped

Other

139 stars 238 forks source link

Colocando as variaveis em camel case #336

Closed p4bl04p closed 4 years ago

p4bl04p commented 4 years ago

Bom dia.

Alteração das variaveis para camel case, pois algumas estavam com este padrão outras não, desconfigurando a impressão do danfe

Variaveis alteradas:

$margsup -.> margSup $margesq -.> margEsq

robmachado commented 4 years ago

Pablo na ultima semana eu estava fazendo uma refatoração inicial nos blocos. Antes de subir ajustes por favor veja como está agora. Por exemplo, você fez um PR com referencia ao numero do DPEC mas isso tem um método próprio agora no DaCommon::class. Eu estou tentando fazer uma limpeza em alguns códigos pois ao longo dos anos um "monte de gambiarras" foram sendo colocadas nessas classes. E eu estou tentando melhorar isso, mas é muito trabalho para mim ono tempo disponivel.

p4bl04p commented 4 years ago

Boa noite Roberto. Olhei a classe que informou e fiz algumas alterações, quando tiver um tempo da uma olhada se ficou tudo OK. Nos teste que fiz aqui esta tudo funcionando.

robmachado commented 4 years ago

Já foram ajustadas as variáveis