From: richard <richard@57a73879-2fb5-44c3-a270-3262357dd7e2>
Date: Wed, 16 Jan 2002 03:02:42 +0000 (+0000)
Subject: #503793 ] changing assignedto resets nosy list
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=0824c6ccc8640546db05d8f1ee7c5e1c1e521723;p=roundup.git

#503793 ] changing assignedto resets nosy list


git-svn-id: http://svn.roundup-tracker.org/svnroot/roundup/trunk@551 57a73879-2fb5-44c3-a270-3262357dd7e2
---

diff --git a/CHANGES.txt b/CHANGES.txt
index 673d445..3276532 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -27,6 +27,7 @@ Fixed:
  . #502956 ] filtering by multilink not supported
  . #503340 ] creating issue with [asignedto=p.ohly]
  . #502949 ] index view for non-issues and redisplay
+ . #503793 ] changing assignedto resets nosy list
 
 
 2002-01-08 - 0.4.0b1
diff --git a/roundup/cgi_client.py b/roundup/cgi_client.py
index 5d84479..5489426 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.98 2002-01-14 02:20:14 richard Exp $
+# $Id: cgi_client.py,v 1.99 2002-01-16 03:02:42 richard Exp $
 
 __doc__ = """
 WWW request handler (also used in the stand-alone server).
@@ -343,8 +343,16 @@ class Client:
             return
         assignedto_id = props['assignedto']
         if not props.has_key('nosy'):
-            props['nosy'] = [assignedto_id]
-        elif assignedto_id not in props['nosy']:
+            # load current nosy
+            if self.nodeid:
+                cl = self.db.classes[self.classname]
+                l = cl.get(self.nodeid, 'nosy')
+		if assignedto_id in l:
+		    return
+		props['nosy'] = l
+            else:
+                props['nosy'] = []
+        if assignedto_id not in props['nosy']:
             props['nosy'].append(assignedto_id)
 
     def _changenode(self, props):
@@ -1165,6 +1173,15 @@ def parsePropsFromForm(db, cl, form, nodeid=0):
 
 #
 # $Log: not supported by cvs2svn $
+# Revision 1.98  2002/01/14 02:20:14  richard
+#  . changed all config accesses so they access either the instance or the
+#    config attriubute on the db. This means that all config is obtained from
+#    instance_config instead of the mish-mash of classes. This will make
+#    switching to a ConfigParser setup easier too, I hope.
+#
+# At a minimum, this makes migration a _little_ easier (a lot easier in the
+# 0.5.0 switch, I hope!)
+#
 # Revision 1.97  2002/01/11 23:22:29  richard
 #  . #502437 ] rogue reactor and unittest
 #    in short, the nosy reactor was modifying the nosy list. That code had