[POS-commit] r1131 - in stoq/trunk/stoq: domain gui/editors gui/slaves/glade

Christian Robottom Reis kiko at async.com.br
Mon Sep 5 15:17:27 BRT 2005


On Mon, Sep 05, 2005 at 01:46:22PM -0300, Henrique Romano wrote:
> +    def update_comission(self):
> +        category = self.model.category
> +        if self.model.comission or not category:
> +            return
> +        comission = (category.get_comission()
> +                     or category.base_category.get_comission())

Por que o callsite está incluindo esse tipo de algoritmo? Tudo isso
deveria estar em uma classe do domínio --
category.get_default_commision() ou mesmo o category.get_commision()
deveria embutir esse fluxo. Você poderia/deveria até ter um
model.set_commision_from_category(). Qualquer coisa menos colocar isso
em código de interface.

Take care,
--
Christian Robottom Reis | http://async.com.br/~kiko/ | [+55 16] 3376 0125


More information about the POS-commit mailing list