[Stoq-devel] [Bug 2604] - Diálogo de pesquisa de produtos deveria oferecer filtragem de estoque por branch company

[bugzilla-daemon at async.com.br]

http://bugs.async.com.br/show_bug.cgi?id=2604  





------- Comment #7 from jdahlin at async.com.br  2006-06-08 17:37 BRT -------
(From update of attachment 3229)
>     def get_branch(self):
>@@ -133,9 +123,27 @@
>                        width=70),
>                 Column('stock', title=_('Stock Total'),
>                        format_func=format_quantity,
>-                       data_type=decimal.Decimal, width=100)]
>+                       data_type=decimal.Decimal, width=100),]

Por que?

Você *precisa* ler os patches antes de você manda eles pra revisão.

É muito importante para fazer as coisas minimas, se você não mesmo precisa uma
mudança, não fez.

>Index: stoqlib/gui/slaves/fiscal.py
>===================================================================
>--- stoqlib/gui/slaves/fiscal.py	(revisão 3670)
>+++ stoqlib/gui/slaves/fiscal.py	(cópia de trabalho)
>@@ -51,6 +51,8 @@
>         SlaveDelegate.__init__(self, gladefile=self.gladefile)
>         self._setup_slaves()
> 
>+    #the code below is duplicated and will be fixed on bug 2651
>+#the duplicated code is in ../stoqlib/gui/slaves/product.py
>     def _setup_slaves(self):
>         items = [(value, key)
>                     for key, value in fiscal_book_entries.items()]

Escreve commentarios tipo isso:

   # Foo Bar Baz Foo Bar Baz Foo Bar Baz Foo Bar Baz Foo Bar Baz 
   # Foo Bar Baz Foo Bar Baz Foo Bar Baz Foo Bar Baz 
   if a == b:
      pass

Não precisa o "path" completeo, slaves/product.py é suficiente.

>Index: stoqlib/gui/slaves/product.py

>+class ProductSearchSlave(SlaveDelegate):

>+    #the code bellow is duplicated and will be fixed on bug 2651
>+    #the duplicated code is in ../stoqlib/gui/slaves/fiscal.py

Commentarios são errados aqui tb.