UF inválido

[r-martins]

Precisa fazer algo para converter os diferentes tipo de estado (uf) digitados ou selecionados via dropdown, em siglas (SP,AC,BA,MG, etc).

[deniscsz]

Ricardo, me corrija se entendi o código errado.
A coleta dos endereços está sendo feita na função getCreditCardApiCallParams, e depois essas informações são enviadas para o pagseguro pela _callApi sem tratamento.

Não seria melhor criar uma função para coletar as informações do pedido (já validando UF e outras informações) e depois utilizá-la na getCreditCardApiCallParams? Vou alterar aqui e envio o pull para você analisar.

[r-martins]

Oi Denis,
Obrigado pelo primeiro comentário! kk
Eu procurei juntar tudo que diz respeito ao tratamento dos parametros
enviados dentro do helper/Params.php. Como em geral são coisas de string e
afins, não vi pq deixar muito ligado ao Model. Tipo tratamento de espaços
em telefones, e coisas do tipo. Acho que seria melhor fazer la no
Params.php.
O que acha?

2014-05-02 15:06 GMT-03:00 Denis Colli Spalenza notifications@github.com:

Ricardo, me corrija se entendi o código errado.
A coleta dos endereços está sendo feita na função
getCreditCardApiCallParams, e depois essas informações são enviadas para o
pagseguro pela _callApi sem tratamento.

Não seria melhor criar uma função para coletar as informações do pedido
(já validando UF e outras informações) e depois utilizá-la na
getCreditCardApiCallParams? Vou alterar aqui e envio o pull para você
analisar.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/1#issuecomment-42060464
.

[deniscsz]

Não tinha olhado o Params.php com atenção. Acho melhor fazer como você sugeriu mesmo. Assim que eu tirar um tempinho aqui contribuo e envio o pull para você analisar. Valeu!

[r-martins]

Valeu Denis.
Esse do UF deve ser o proximo passo que eu vou pegar, pois nao to
conseguindo usar na minha loja por conta disso rs.
Antes de fazer da uma olhada pra ver se já nao comitei algo, rs.
Mas valeu de qquer forma.

Vou adicionar mais coisas aqui nessa lista....

Abraços!

2014-05-02 15:42 GMT-03:00 Denis Colli Spalenza notifications@github.com:

Não tinha olhado o Params.php com atenção. Acho melhor fazer como você
sugeriu mesmo. Assim que eu tirar um tempinho aqui contribuo e envio o pull
para você analisar. Valeu!

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/1#issuecomment-42064228
.