Remove hardcoded shortcut and activate Delete key in one2many

tryton/gui/window/view_form/view/form_gtk/many2many.py

 #This file is part of Tryton.  The COPYRIGHT file at the top level of
 #this repository contains the full copyright notices and license terms.
 import gtk
 from tryton.gui.window.view_form.screen import Screen
 from interface import WidgetInterface
 from one2many import Dialog
 import tryton.rpc as rpc
 from tryton.gui.window.win_search import WinSearch
 from tryton.gui.window.view_form.widget_search.form import _LIMIT
 import tryton.common as common
@@ skipping 39 matching lines @@
         label_remove = gtk.Label(_('Remove'))
         hbox_remove.pack_start(label_remove)
         self.wid_but_remove.add(hbox_remove)
         self.wid_but_remove.set_relief(gtk.RELIEF_HALF)
         self.wid_but_remove.set_focus_on_click(True)
         self.wid_but_remove.connect('clicked', self._sig_remove)
         hbox.pack_start(self.wid_but_remove, expand=False, fill=False)
 
         self.widget.pack_start(hbox, expand=False, fill=False)
 
+        hbox.set_focus_chain([self.wid_text])
+
         self.screen = Screen(attrs['relation'], self._window,
                 view_type=['tree'], views_preload=attrs.get('views', {}),
                 row_activate=self._on_activate)
 
         self.widget.pack_start(self.screen.widget, expand=True, fill=True)
 
+        self.screen.widget.connect('key_press_event', self.on_keypress)
+        self.wid_text.connect('key_press_event', self.on_keypress)
+
         self.old = None
 
     def grab_focus(self):
         return self.wid_text.grab_focus()
 
+    def on_keypress(self, widget, event):
+        if event.keyval == gtk.keysyms.F3:
+            self._sig_add()
+            return False
+        if event.keyval == gtk.keysyms.F2:
+            self._sig_edit()
+        if event.keyval in (gtk.keysyms.Delete, gtk.keysyms.KP_Delete) \
+                and self.screen.widget.is_focus():

[timitos, 2009/11/24 08:00:47]

Deleting is not working with this line on Many2Many. If I comment out this line
everything works fine.

For One2Many this is working.

+            self._sig_remove()
+            return False
+
     def destroy(self):
         self.screen.destroy()
         self.widget.destroy()
         del self.widget
 
     def _sig_add(self, *args):
         domain = self._view.modelfield.domain_get(self._view.model)
         context = self._view.modelfield.context_get(self._view.model)
         value = self.wid_text.get_text()
 
@@ skipping 18 matching lines @@
             res_id = ids[0]
         self.screen.load(ids)
         self.screen.display(res_id=res_id)
         if self.screen.current_view:
             self.screen.current_view.set_cursor()
         self.wid_text.set_text('')
         self.set_value(self._view.model, self._view.modelfield)
 
     def _sig_remove(self, *args):
         self.screen.remove()
-        self.screen.display()
-        self.set_value(self._view.model, self._view.modelfield)
 
     def _sig_activate(self, *args):
         self._sig_add()
         self.wid_text.grab_focus()
 
     def _on_activate(self):
         self._sig_edit()
 
     def _sig_edit(self):
         if self.screen.current_model:
@@ skipping 43 matching lines @@
         return True
 
     def display_value(self):
         return self._view.modelfield.rec_name(self._view.model)
 
     def set_value(self, model, model_field):
         model_field.set_client(model, [x.id for x in self.screen.models.models])
 
     def cancel(self):
         self.old = None