From afd098697f15ba6fd26e8586f0d27de7ce30ffd0 Mon Sep 17 00:00:00 2001
From: richard <richard@57a73879-2fb5-44c3-a270-3262357dd7e2>
Date: Wed, 12 Dec 2001 23:55:00 +0000
Subject: [PATCH] Fixed some problems with user editing

git-svn-id: http://svn.roundup-tracker.org/svnroot/roundup/trunk@460 57a73879-2fb5-44c3-a270-3262357dd7e2
---
 frontends/ZRoundup/ZRoundup.py |  7 +++++--
 roundup/cgi_client.py          | 11 +++++++----
 2 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/frontends/ZRoundup/ZRoundup.py b/frontends/ZRoundup/ZRoundup.py
index 2b57957..d73d7d8 100644
--- a/frontends/ZRoundup/ZRoundup.py
+++ b/frontends/ZRoundup/ZRoundup.py
@@ -14,7 +14,7 @@
 # BASIS, AND THERE IS NO OBLIGATION WHATSOEVER TO PROVIDE MAINTENANCE,
 # SUPPORT, UPDATES, ENHANCEMENTS, OR MODIFICATIONS.
 # 
-# $Id: ZRoundup.py,v 1.2 2001-12-12 23:33:58 richard Exp $
+# $Id: ZRoundup.py,v 1.3 2001-12-12 23:55:00 richard Exp $
 #
 ''' ZRoundup module - exposes the roundup web interface to Zope
 
@@ -113,7 +113,7 @@ class ZRoundup(Item, PropertyManager, Implicit, Persistent):
 
     security.declarePrivate('_opendb')
     def _opendb(self):
-        '''Open the roundup instnace database for a transaction
+        '''Open the roundup instance database for a transaction.
         '''
         instance = roundup.instance.open(self.instance_home)
         request = RequestWrapper(self.REQUEST['RESPONSE'])
@@ -163,6 +163,9 @@ modulesecurity.apply(globals())
 
 #
 # $Log: not supported by cvs2svn $
+# Revision 1.2  2001/12/12 23:33:58  richard
+# added some implementation notes
+#
 # Revision 1.1  2001/12/12 23:27:13  richard
 # Added a Zope frontend for roundup.
 #
diff --git a/roundup/cgi_client.py b/roundup/cgi_client.py
index 59b587b..f4fa8d1 100644
--- a/roundup/cgi_client.py
+++ b/roundup/cgi_client.py
@@ -15,7 +15,7 @@
 # BASIS, AND THERE IS NO OBLIGATION WHATSOEVER TO PROVIDE MAINTENANCE,
 # SUPPORT, UPDATES, ENHANCEMENTS, OR MODIFICATIONS.
 # 
-# $Id: cgi_client.py,v 1.80 2001-12-12 23:27:14 richard Exp $
+# $Id: cgi_client.py,v 1.81 2001-12-12 23:55:00 richard Exp $
 
 __doc__ = """
 WWW request handler (also used in the stand-alone server).
@@ -395,15 +395,15 @@ class Client:
                 props, changed = parsePropsFromForm(self.db, user, self.form,
                     self.nodeid)
                 set_cookie = 0
-                if self.nodeid == self.getuid() and 'password' in changed:
+                if self.nodeid == self.getuid() and changed.has_key('password'):
                     password = self.form['password'].value.strip()
                     if password:
                         set_cookie = password
                     else:
+                        # no password was supplied - don't change it
                         del props['password']
-                        del changed[changed.index('password')]
+                        del changed['password']
                 user.set(self.nodeid, **props)
-                self._post_editnode(self.nodeid)
                 # and some feedback for the user
                 message = _('%(changes)s edited ok')%{'changes':
                     ', '.join(changed.keys())}
@@ -1090,6 +1090,9 @@ def parsePropsFromForm(db, cl, form, nodeid=0):
 
 #
 # $Log: not supported by cvs2svn $
+# Revision 1.80  2001/12/12 23:27:14  richard
+# Added a Zope frontend for roundup.
+#
 # Revision 1.79  2001/12/10 22:20:01  richard
 # Enabled transaction support in the bsddb backend. It uses the anydbm code
 # where possible, only replacing methods where the db is opened (it uses the
-- 
2.30.2