Code review comment for lp:~fnegrini/openerp.pt-br-localiz/openerp.pt-br-localiz

Olá Fabio,

Mais uma vez obrigado pela contribuição, eu fiz alguns testes e existe um problema, eu tinha colocado no wizard de busca duas tela, uma com os campos do endereço principais obrigatórios para as pessoas preencherem esses campos antes da busca do ceps para reduzir o resultado da busca, porque os campos no endereço do parceiro são obrigatórios e como você eliminou no wizard a tela com os campos obrigatórios de endereço e esta pegando diretamente os campos do parceiros para aparecer o resultado diretamente (o que achei interessante), existe um problema que se você não preencher os campos do endereço como pais, estado, municio, bairro e endereço, o wizard tenta trazer todos os ceps e como a base de cep tem alguns mil registro, acaba trabando o OpenERP.

Neste casso eu sugiro no método de busca você conferir se ao pressionar o botão de busca pelo menos esses campos pais, estado, municio, bairro ou endereço ou cep estão preenchidos, caso não estejam preenchidos você pode fazer o raise falando que para busca pelo menos esses campos devem ser preenchido, uma outra solução e também no método search do cep verificar se há mais de 80 ou 100 elementos na lista com os IDs caso tenha mais você pode cortar a lista e trazer somente os 80 ou 100 primeiros