[Stoq-devel] [Bug 2166] - Implementar sinais para slaves genéricos no lugar de referências de parent

[bugzilla-daemon at async.com.br]

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





------- Comment #4 from jdahlin at async.com.br  2006-01-23 10:22 BRT -------
(From update of attachment 2909)
>Index: stoqlib/gui/lists.py

>+    def setup_slave(self, columns, objects, selection_mode):
>+        self.main.remove(self.main_label)
>+        self.klist = List(columns, objects, selection_mode)

self._klist não?

>+        self.main.add(self.klist)
>+        self.klist.show()

>Index: stoqlib/gui/search.py

>+    gsignal('edit')
>+    gsignal('add')

Não precisa dois signals.

>-    def __init__(self, parent):
>+    def __init__(self):
>         SlaveDelegate.__init__(self, toplevel_name=self.toplevel_name,
>+                               gladefile=self.gladefile, domain='stoqlib')

Acho que não precisa o construtor mais. 
Addiciona domain no classe é vc pode apagar ele

>     def on_edit_button__clicked(self, widget):
>+        self.emit('edit')
> 
>     def on_new_button__clicked(self, *args):
>+        self.emit('add')

>     def setup_slaves(self, **kwargs):
>         self.disable_ok()
>+        self.klist = List(self.get_columns(), mode=self.selection_mode)
>+        # XXX: I think that BasicDialog must redesigned, if so we don't
>+        # need this ".remove" crap
>+        self.main.remove(self.main_label)
>+        self.main.add(self.klist)
>+        self.klist.show()

>     def setup_slaves(self):
>         SearchDialog.setup_slaves(self)
>-        self.toolbar = SearchEditorToolBar(self)
>+        self.toolbar = SearchEditorToolBar()

self._toolbar

>         self.attach_slave('extra_holder', self.toolbar)
>+        self.toolbar.connect("edit", self.edit)
>+        self.toolbar.connect("add", self.new)

self._on_toolbar__edit
self._on_toolbar__new

..

>-    def edit(self, widget, obj=None):
>+    def edit(self, widget=None, obj=None):

>+    def new(self, *args):
>         self.run()

.tão você não precisa essas.