fix line width and some pylint errors
This commit is contained in:
parent
bb30fb7b65
commit
77e4e6e45f
|
@ -56,8 +56,8 @@ class CommandWindow:
|
||||||
self.account = gajim.connections[account]
|
self.account = gajim.connections[account]
|
||||||
self.jid = jid
|
self.jid = jid
|
||||||
|
|
||||||
self.pulse_id=None # to satisfy self.setup_pulsing()
|
self.pulse_id = None # to satisfy self.setup_pulsing()
|
||||||
self.commandlist=None # a list of (commandname, commanddescription)
|
self.commandlist = None # a list of (commandname, commanddescription)
|
||||||
|
|
||||||
# command's data
|
# command's data
|
||||||
self.commandnode = commandnode
|
self.commandnode = commandnode
|
||||||
|
@ -68,7 +68,8 @@ class CommandWindow:
|
||||||
# retrieving widgets from xml
|
# retrieving widgets from xml
|
||||||
self.xml = gtkgui_helpers.get_gtk_builder('adhoc_commands_window.ui')
|
self.xml = gtkgui_helpers.get_gtk_builder('adhoc_commands_window.ui')
|
||||||
self.window = self.xml.get_object('adhoc_commands_window')
|
self.window = self.xml.get_object('adhoc_commands_window')
|
||||||
self.window.connect('delete-event', self.on_adhoc_commands_window_delete_event)
|
self.window.connect('delete-event',
|
||||||
|
self.on_adhoc_commands_window_delete_event)
|
||||||
for name in ('back_button', 'forward_button',
|
for name in ('back_button', 'forward_button',
|
||||||
'execute_button','close_button','stages_notebook',
|
'execute_button','close_button','stages_notebook',
|
||||||
'retrieving_commands_stage_vbox',
|
'retrieving_commands_stage_vbox',
|
||||||
|
@ -130,7 +131,7 @@ class CommandWindow:
|
||||||
return self.stage_close_button_clicked(*anything)
|
return self.stage_close_button_clicked(*anything)
|
||||||
|
|
||||||
def on_adhoc_commands_window_destroy(self, *anything):
|
def on_adhoc_commands_window_destroy(self, *anything):
|
||||||
# TODO: do all actions that are needed to remove this object from memory...
|
# TODO: do all actions that are needed to remove this object from memory
|
||||||
self.remove_pulsing()
|
self.remove_pulsing()
|
||||||
|
|
||||||
def on_adhoc_commands_window_delete_event(self, *anything):
|
def on_adhoc_commands_window_delete_event(self, *anything):
|
||||||
|
@ -167,7 +168,8 @@ class CommandWindow:
|
||||||
# setup the callbacks
|
# setup the callbacks
|
||||||
self.stage_finish = self.stage1_finish
|
self.stage_finish = self.stage1_finish
|
||||||
self.stage_close_button_clicked = self.stage1_close_button_clicked
|
self.stage_close_button_clicked = self.stage1_close_button_clicked
|
||||||
self.stage_adhoc_commands_window_delete_event = self.stage1_adhoc_commands_window_delete_event
|
self.stage_adhoc_commands_window_delete_event = \
|
||||||
|
self.stage1_adhoc_commands_window_delete_event
|
||||||
|
|
||||||
def stage1_finish(self):
|
def stage1_finish(self):
|
||||||
self.remove_pulsing()
|
self.remove_pulsing()
|
||||||
|
@ -196,8 +198,7 @@ class CommandWindow:
|
||||||
assert len(self.commandlist)>0
|
assert len(self.commandlist)>0
|
||||||
|
|
||||||
self.stages_notebook.set_current_page(
|
self.stages_notebook.set_current_page(
|
||||||
self.stages_notebook.page_num(
|
self.stages_notebook.page_num(self.command_list_stage_vbox))
|
||||||
self.command_list_stage_vbox))
|
|
||||||
|
|
||||||
self.close_button.set_sensitive(True)
|
self.close_button.set_sensitive(True)
|
||||||
self.back_button.set_sensitive(False)
|
self.back_button.set_sensitive(False)
|
||||||
|
@ -208,7 +209,8 @@ class CommandWindow:
|
||||||
first_radio = None
|
first_radio = None
|
||||||
for (commandnode, commandname) in self.commandlist:
|
for (commandnode, commandname) in self.commandlist:
|
||||||
radio = gtk.RadioButton(first_radio, label=commandname)
|
radio = gtk.RadioButton(first_radio, label=commandname)
|
||||||
radio.connect("toggled", self.on_command_radiobutton_toggled, commandnode)
|
radio.connect("toggled", self.on_command_radiobutton_toggled,
|
||||||
|
commandnode)
|
||||||
if not first_radio:
|
if not first_radio:
|
||||||
first_radio = radio
|
first_radio = radio
|
||||||
self.commandnode = commandnode
|
self.commandnode = commandnode
|
||||||
|
@ -266,7 +268,8 @@ class CommandWindow:
|
||||||
self.stage_forward_button_clicked = self.stage3_forward_button_clicked
|
self.stage_forward_button_clicked = self.stage3_forward_button_clicked
|
||||||
self.stage_execute_button_clicked = self.stage3_execute_button_clicked
|
self.stage_execute_button_clicked = self.stage3_execute_button_clicked
|
||||||
self.stage_close_button_clicked = self.stage3_close_button_clicked
|
self.stage_close_button_clicked = self.stage3_close_button_clicked
|
||||||
self.stage_adhoc_commands_window_delete_event = self.stage3_close_button_clicked
|
self.stage_adhoc_commands_window_delete_event = \
|
||||||
|
self.stage3_close_button_clicked
|
||||||
|
|
||||||
def stage3_finish(self):
|
def stage3_finish(self):
|
||||||
pass
|
pass
|
||||||
|
@ -276,10 +279,10 @@ class CommandWindow:
|
||||||
We are in the middle of executing command. Ask user if he really want to
|
We are in the middle of executing command. Ask user if he really want to
|
||||||
cancel the process, then cancel it
|
cancel the process, then cancel it
|
||||||
"""
|
"""
|
||||||
# this works also as a handler for window_delete_event, so we have to return appropriate
|
# this works also as a handler for window_delete_event, so we have to
|
||||||
# values
|
# return appropriate values
|
||||||
if self.form_status == 'completed':
|
if self.form_status == 'completed':
|
||||||
if widget!=self.window:
|
if widget != self.window:
|
||||||
self.window.destroy()
|
self.window.destroy()
|
||||||
return False
|
return False
|
||||||
|
|
||||||
|
@ -291,10 +294,10 @@ class CommandWindow:
|
||||||
self.allow_stage3_close = True
|
self.allow_stage3_close = True
|
||||||
self.window.destroy()
|
self.window.destroy()
|
||||||
|
|
||||||
dialog = dialogs.HigDialog(self.window, gtk.DIALOG_DESTROY_WITH_PARENT | \
|
dialog = dialogs.HigDialog(self.window, gtk.DIALOG_DESTROY_WITH_PARENT \
|
||||||
gtk.DIALOG_MODAL, gtk.BUTTONS_YES_NO, _('Cancel confirmation'),
|
| gtk.DIALOG_MODAL, gtk.BUTTONS_YES_NO, _('Cancel confirmation'),
|
||||||
_('You are in process of executing command. Do you really want to '
|
_('You are in process of executing command. Do you really want to '
|
||||||
'cancel it?'), on_response_yes=on_yes)
|
'cancel it?'), on_response_yes=on_yes)
|
||||||
dialog.popup()
|
dialog.popup()
|
||||||
return True # Block event, don't close window
|
return True # Block event, don't close window
|
||||||
|
|
||||||
|
@ -310,7 +313,7 @@ class CommandWindow:
|
||||||
def stage3_submit_form(self, action='execute'):
|
def stage3_submit_form(self, action='execute'):
|
||||||
self.data_form_widget.set_sensitive(False)
|
self.data_form_widget.set_sensitive(False)
|
||||||
if self.data_form_widget.get_data_form():
|
if self.data_form_widget.get_data_form():
|
||||||
self.data_form_widget.data_form.type='submit'
|
self.data_form_widget.data_form.type = 'submit'
|
||||||
else:
|
else:
|
||||||
self.data_form_widget.hide()
|
self.data_form_widget.hide()
|
||||||
|
|
||||||
|
@ -346,9 +349,10 @@ class CommandWindow:
|
||||||
|
|
||||||
self.data_form_widget.set_sensitive(True)
|
self.data_form_widget.set_sensitive(True)
|
||||||
try:
|
try:
|
||||||
self.data_form_widget.data_form=self.dataform
|
self.data_form_widget.data_form = self.dataform
|
||||||
except dataforms.Error:
|
except dataforms.Error:
|
||||||
self.stage5(error=_('Service sent malformed data'), senderror=True)
|
self.stage5(error=_('Service sent malformed data'),
|
||||||
|
senderror=True)
|
||||||
return
|
return
|
||||||
self.data_form_widget.show()
|
self.data_form_widget.show()
|
||||||
if self.data_form_widget.title:
|
if self.data_form_widget.title:
|
||||||
|
@ -362,7 +366,8 @@ class CommandWindow:
|
||||||
# actions, actions, actions...
|
# actions, actions, actions...
|
||||||
self.close_button.set_sensitive(True)
|
self.close_button.set_sensitive(True)
|
||||||
self.back_button.set_sensitive(actions.getTag('prev') is not None)
|
self.back_button.set_sensitive(actions.getTag('prev') is not None)
|
||||||
self.forward_button.set_sensitive(actions.getTag('next') is not None)
|
self.forward_button.set_sensitive(
|
||||||
|
actions.getTag('next') is not None)
|
||||||
self.execute_button.set_sensitive(True)
|
self.execute_button.set_sensitive(True)
|
||||||
else:
|
else:
|
||||||
self.close_button.set_sensitive(True)
|
self.close_button.set_sensitive(True)
|
||||||
|
@ -376,7 +381,8 @@ class CommandWindow:
|
||||||
self.forward_button.hide()
|
self.forward_button.hide()
|
||||||
self.execute_button.hide()
|
self.execute_button.hide()
|
||||||
self.close_button.show()
|
self.close_button.show()
|
||||||
self.stage_adhoc_commands_window_delete_event = self.stage3_close_button_clicked
|
self.stage_adhoc_commands_window_delete_event = \
|
||||||
|
self.stage3_close_button_clicked
|
||||||
|
|
||||||
note = command.getTag('note')
|
note = command.getTag('note')
|
||||||
if note:
|
if note:
|
||||||
|
@ -396,8 +402,7 @@ class CommandWindow:
|
||||||
self.stage_finish()
|
self.stage_finish()
|
||||||
|
|
||||||
self.stages_notebook.set_current_page(
|
self.stages_notebook.set_current_page(
|
||||||
self.stages_notebook.page_num(
|
self.stages_notebook.page_num(self.no_commands_stage_vbox))
|
||||||
self.no_commands_stage_vbox))
|
|
||||||
|
|
||||||
self.close_button.set_sensitive(True)
|
self.close_button.set_sensitive(True)
|
||||||
self.back_button.set_sensitive(False)
|
self.back_button.set_sensitive(False)
|
||||||
|
@ -442,8 +447,7 @@ class CommandWindow:
|
||||||
assert False
|
assert False
|
||||||
|
|
||||||
self.stages_notebook.set_current_page(
|
self.stages_notebook.set_current_page(
|
||||||
self.stages_notebook.page_num(
|
self.stages_notebook.page_num(self.error_stage_vbox))
|
||||||
self.error_stage_vbox))
|
|
||||||
|
|
||||||
self.close_button.set_sensitive(True)
|
self.close_button.set_sensitive(True)
|
||||||
self.back_button.hide()
|
self.back_button.hide()
|
||||||
|
@ -481,14 +485,15 @@ class CommandWindow:
|
||||||
"""
|
"""
|
||||||
if self.pulse_id:
|
if self.pulse_id:
|
||||||
gobject.source_remove(self.pulse_id)
|
gobject.source_remove(self.pulse_id)
|
||||||
self.pulse_id=None
|
self.pulse_id = None
|
||||||
|
|
||||||
# handling xml stanzas
|
# handling xml stanzas
|
||||||
def request_command_list(self):
|
def request_command_list(self):
|
||||||
"""
|
"""
|
||||||
Request the command list. Change stage on delivery
|
Request the command list. Change stage on delivery
|
||||||
"""
|
"""
|
||||||
query = xmpp.Iq(typ='get', to=xmpp.JID(self.jid), queryNS=xmpp.NS_DISCO_ITEMS)
|
query = xmpp.Iq(typ='get', to=xmpp.JID(self.jid),
|
||||||
|
queryNS=xmpp.NS_DISCO_ITEMS)
|
||||||
query.setQuerynode(xmpp.NS_COMMANDS)
|
query.setQuerynode(xmpp.NS_COMMANDS)
|
||||||
|
|
||||||
def callback(response):
|
def callback(response):
|
||||||
|
@ -512,7 +517,8 @@ class CommandWindow:
|
||||||
self.commandlist = []
|
self.commandlist = []
|
||||||
self.stage4()
|
self.stage4()
|
||||||
else:
|
else:
|
||||||
self.commandlist = [(t.getAttr('node'), t.getAttr('name')) for t in items]
|
self.commandlist = [(t.getAttr('node'), t.getAttr('name')) \
|
||||||
|
for t in items]
|
||||||
self.stage2()
|
self.stage2()
|
||||||
|
|
||||||
self.account.connection.SendAndCallForResponse(query, callback)
|
self.account.connection.SendAndCallForResponse(query, callback)
|
||||||
|
@ -533,9 +539,6 @@ class CommandWindow:
|
||||||
cmdnode.setAttr('sessionid', self.sessionid)
|
cmdnode.setAttr('sessionid', self.sessionid)
|
||||||
|
|
||||||
if self.data_form_widget.data_form:
|
if self.data_form_widget.data_form:
|
||||||
# cmdnode.addChild(node=dataforms.DataForm(tofill=self.data_form_widget.data_form))
|
|
||||||
# FIXME: simplified form to send
|
|
||||||
|
|
||||||
cmdnode.addChild(node=self.data_form_widget.data_form)
|
cmdnode.addChild(node=self.data_form_widget.data_form)
|
||||||
|
|
||||||
def callback(response):
|
def callback(response):
|
||||||
|
@ -564,6 +567,6 @@ class CommandWindow:
|
||||||
|
|
||||||
self.account.connection.send(stanza)
|
self.account.connection.send(stanza)
|
||||||
else:
|
else:
|
||||||
# we did not received any reply from service; FIXME: we should wait and
|
# we did not received any reply from service;
|
||||||
# then send cancel; for now we do nothing
|
# FIXME: we should wait and then send cancel; for now we do nothing
|
||||||
pass
|
pass
|
||||||
|
|
Loading…
Reference in New Issue