Author: asterix
Date: 2007-12-02 23:37:08 +0100 (Sun, 02 Dec 2007)
New Revision: 9087
Modified:
trunk/src/common/connection.py
trunk/src/config.py
Log:
create new account in 2 steps:
1/ connect, get form and disconnect
2/ re-connect, get form, check it and send filled form
Modified: trunk/src/common/connection.py
===================================================================
--- trunk/src/common/connection.py 2007-12-02 19:29:22 UTC (rev 9086)
+++ trunk/src/common/connection.py 2007-12-02 22:37:08 UTC (rev 9087)
@@ -119,6 +119,7 @@
self.time_to_reconnect = None
self.last_time_to_reconnect = None
self.new_account_info = None
+ self.new_account_form = None
self.bookmarks = []
self.annotations = {}
self.on_purpose = False
@@ -184,7 +185,7 @@
self.retrycount = 0
# We are doing disconnect at so many places, better use one function in all
- def disconnect(self, on_purpose = False):
+ def disconnect(self, on_purpose=False):
self.on_purpose = on_purpose
self.connected = 0
self.time_to_reconnect = None
@@ -259,8 +260,47 @@
return
is_form = data[2]
conf = data[1]
- self.dispatch('NEW_ACC_CONNECTED', (conf, is_form))
- return
+ if self.new_account_form:
+ def _on_register_result(result):
+ if not common.xmpp.isResultNode(result):
+ self.dispatch('ACC_NOT_OK', (result.getError()))
+ return
+ if USE_GPG:
+ self.gpg = GnuPG.GnuPG(gajim.config.get(
+ 'use_gpg_agent'))
+ self.dispatch('ACC_OK', (self.new_account_info))
+ self.new_account_info = None
+ self.new_account_form = None
+ if self.connection:
+ self.connection.UnregisterDisconnectHandler(
+ self._on_new_account)
+ self.disconnect(on_purpose=True)
+ # it's the second time we get the form, we have info user
+ # typed, so send them
+ if is_form:
+ #TODO: Check if form has changed
+ iq = Iq('set', NS_REGISTER, to=self._hostname)
+ iq.setTag('query').addChild(node=self.new_account_form)
+ self.connection.SendAndCallForResponse(iq,
+ _on_register_result)
+ else:
+ if self.new_account_form.keys().sort() != \
+ conf.keys().sort():
+ # requested config has changed since first connection
+ self.dispatch('ACC_NOT_OK', (_(
+ 'Server %s provided a different registration form')\
+ % data[0]))
+ return
+ common.xmpp.features_nb.register(self.connection,
+ self._hostname, self.new_account_form,
+ _on_register_result)
+ return
+ else:
+ self.dispatch('NEW_ACC_CONNECTED', (conf, is_form))
+ self.connection.UnregisterDisconnectHandler(
+ self._on_new_account)
+ self.disconnect(on_purpose=True)
+ return
if not data[1]: # wrong answer
self.dispatch('ERROR', (_('Invalid answer'),
_('Transport %s answered wrongly to register request: %s') % \
@@ -1029,18 +1069,6 @@
groups = groups)
def send_new_account_infos(self, form, is_form):
- def _on_register_result(result):
- if not common.xmpp.isResultNode(result):
- self.dispatch('ACC_NOT_OK', (result.getError()))
- return
- if USE_GPG:
- self.gpg = GnuPG.GnuPG(gajim.config.get('use_gpg_agent'))
- gajim.connections[self.name] = self
- self.dispatch('ACC_OK', (self.new_account_info))
- self.new_account_info = None
- if self.connection:
- self.connection.UnregisterDisconnectHandler(self._on_new_account)
- self.disconnect(on_purpose=True)
if is_form:
# Get username and password and put them in new_account_info
for field in self._data_form.iter_fields():
@@ -1048,17 +1076,14 @@
self.new_account_info['name'] = field.value
if field.var == 'password':
self.new_account_info['password'] = field.value
- iq=Iq('set', NS_REGISTER, to = self._hostname)
- iq.setTag('query').addChild(node = form)
- self.connection.SendAndCallForResponse(iq, _on_register_result)
else:
# Get username and password and put them in new_account_info
if form.has_key('username'):
self.new_account_info['name'] = form['username']
if form.has_key('password'):
self.new_account_info['password'] = form['password']
- common.xmpp.features_nb.register(self.connection, self._hostname,
- form, _on_register_result)
+ self.new_account_form = form
+ self.new_account(self.name, self.new_account_info)
def new_account(self, name, config, sync = False):
# If a connection already exist we cannot create a new account
Modified: trunk/src/config.py
===================================================================
--- trunk/src/config.py 2007-12-02 19:29:22 UTC (rev 9086)
+++ trunk/src/config.py 2007-12-02 22:37:08 UTC (rev 9087)
@@ -3088,6 +3088,7 @@
self.notebook.set_current_page(5) # show finish page
self.show_vcard_checkbutton.set_active(False)
elif cur_page == 2:
+ # We are creating a new account
server = self.xml.get_widget('server_comboboxentry1').child.get_text()\
.decode('utf-8')
@@ -3172,7 +3173,7 @@
return True # loop forever
def new_acc_connected(self, form, is_form):
- '''connection to server succeded, present the form to the user'''
+ '''connection to server succeded, present the form to the user.'''
if self.update_progressbar_timeout_id is not None:
gobject.source_remove(self.update_progressbar_timeout_id)
self.back_button.show()