Por que o estado do Pedido é definido com um atributo integer?

okfn-brasil / cuidando2

Cuidando do Meu Bairro 2.0 é uma reescrita do projeto Cuidando do Meu Bairro ( cuidando.org.br ) que busca mapear a execução do orçamento municipal de São Paulo.

GNU Affero General Public License v3.0

6 stars 2 forks source link

Por que o estado do Pedido é definido com um atributo integer? #103

Open alexandre opened 8 years ago

alexandre commented 8 years ago

Definir o estado do pedido utiizando 0 e 1 é um pouco vago, IMO. Além do comentário do models.py,é necessário adicionar essa explicação em outros lugares.

alexandre commented 8 years ago

Outro detalhe: o método get_state verifica o estado e retorna um objeto string. Por que não salvar o estado com uma string direto?

andresmrm commented 8 years ago

Então, eu montei isso em um primeiro momento, quando não tinha bem claro como funcionaria o sistema. Acho que podemos mudar sim. Você acha melhor deixar uma string livre ou um enumerate?

alexandre commented 8 years ago

@andresmrm , IMO: string + uppercase (e.g. RECEBIDO/ATENDIDO).