From 0824c6ccc8640546db05d8f1ee7c5e1c1e521723 Mon Sep 17 00:00:00 2001 From: richard Date: Wed, 16 Jan 2002 03:02:42 +0000 Subject: [PATCH] #503793 ] changing assignedto resets nosy list git-svn-id: http://svn.roundup-tracker.org/svnroot/roundup/trunk@551 57a73879-2fb5-44c3-a270-3262357dd7e2 --- CHANGES.txt | 1 + roundup/cgi_client.py | 23 ++++++++++++++++++++--- 2 files changed, 21 insertions(+), 3 deletions(-) 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 -- 2.30.2