keybinding question

def on_key_event( self, widget, event, signal ):
        # get the keyname
        keyname = gtk.gdk.keyval_name( event.keyval )
        print("pressed key = ",keyname)

        # estop with F1 shold work every time
        # so should also escape aboart actions
        if keyname == "F1":  # will estop the machine, but not reset estop!
        self.command.state( linuxcnc.STATE_ESTOP )

def on_key_event(self,widget, event,state):
        """This catches all key events for Gscreen.
            if will ignore the initial press of shift keys.
            The last key press and its state is help in data.key_event_last.
            This will call a key lookup function that returns a function name.
            then that function is called with the key state info attached.
            If there is a function named in the handler file it will be called
            instead of the default function.
            Setting Gscreen in verbose mode will show key presses / state.
            Any errors in the key look up or function calls are trapped and ignored.
            Setting Gscreen into debug mode will show the errors.
        """
        CNTRL = SHIFT = ALT = 0
        keyname = gtk.gdk.keyval_name(event.keyval)
        self.verbosely_print("Key %s (%d) was pressed state: %d last: %s" % (keyname, event.keyval,state, self.data.key_event_last))
        if event.state & gtk.gdk.CONTROL_MASK:
            CNTRL = 1
            self.verbosely_print("Control was being held down")
        if event.state & gtk.gdk.MOD1_MASK:
            ALT = 1
            self.verbosely_print("Alt was being held down")
        if event.state & gtk.gdk.SHIFT_MASK:
            SHIFT = 1
            self.verbosely_print("Shift was being held down")
        if keyname in( "Shift_L","Shift_R"): return True # ignore shift key press
        if self.data.key_event_last[0] == keyname and self.data.key_event_last[1] == state : return True
        self.data.key_event_last = keyname,state
        try:
            method = self.keylookup.convert(keyname)
            if method:
                try:
                    try:
                        return self.handler_instance[method](state,SHIFT,CNTRL,ALT)
                    except:
                        self.show_try_errors()
                        return self.handler_instance.keybindings[method](state,SHIFT,CNTRL,ALT)
                except:
                    self.show_try_errors()
                    return self[method](state,SHIFT,CNTRL,ALT) 
        except:
            self.show_try_errors()

self.keylookup.add_conversion('Right','JOGX','on_keycall_JOGXPLUS')

def on_keycall_JOGXPLUS(self,state,SHIFT,CNTRL,ALT):
       if state:
            self.on_jog_x_plus_pressed(None)
            return True

def on_keycall_JOGXPLUS(self,state,SHIFT,CNTRL,ALT):
            self.on_jog_x_plus_pressed(None)

def on_keycall_JOGXPLUS(self,state,SHIFT,CNTRL,ALT):
             self.on_jog_x_plus_pressed(None)
             self.on_jog_x_plus_released(None)

F1                      on_keycall_ESTOP
F2                      on_keycall_POWER
escape             on_keycall_ABORT
up                      on_keycall_XPOS
down                 on_keycall_XNEG
right                    on_keycall_YPOS
left                       on_keycall_YNEG
page up             on_keycall_ZPOS
page down             on_keycall_ZNEG
bracket left             on_keycall_APOS
bracket right             on_keycall_ANEG
i                            on_keycall_INCREMENTS

def on_keycall_JOGXPLUS(self,state,SHIFT,CNTRL,ALT):
       if state:
            self.on_jog_x_plus_pressed(None, state) # <---look here
            return True

if keyname == "Right" and state:
            self.command.jog(self.cnc.JOG_CONTINUOUS, 0, self.builder.get_object("jog_speed").get_value()/60)
        if keyname == "Right" and not state:
            self.command.jog(self.cnc.JOG_STOP, 0)