Changeset 177

Show
Ignore:
Timestamp:
01/14/09 13:54:24 (10 months ago)
Author:
bettse
Message:

re #2889
use a temp variable instead of req.args so we don't overwrite variables used for posting.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trac-plugins/accountmanagerplugin/trunk/acct_mgr/web_ui.py

    r169 r177  
    394394            #This code is based on the commented out code in _create_user. 
    395395            #Creating a Session object and doing a save() on it would have been more elegant but didn't seem to work 
    396             req.args['name'] = user['userinfo']['firstname'] + ' ' + user['userinfo']['lastname'] 
    397             req.args['email'] = user['userinfo']['email'] 
     396            temp = {} 
     397            temp['name'] = user['userinfo']['firstname'] + ' ' + user['userinfo']['lastname'] 
     398            temp['email'] = user['userinfo']['email'] 
    398399 
    399400            db = self.env.get_db_cnx() 
    …… 
    401402 
    402403            for key in ('name', 'email'): 
    403                 value = req.args.get(key) 
     404                value = temp.get(key) 
    404405                if not value: 
    405406                    continue  
    …… 
    413414                                   (user['userinfo']['username'] , key, value)) 
    414415            db.commit() 
    415             self.log.info("New session saved for user %s" % req.args['name'] ) 
     416            self.log.info("New session saved for user %s" % temp['name'] ) 
    416417         
    417418