Roundup Tracker - Issues

Issue 621250

classification
status not set when status=None
Type: Severity: normal
Components: Web interface Versions:
process
Status: closed fixed
:
: richard : ber, richard
Priority: normal :

Created on 2002-10-10 11:15 by ber, last changed 2002-10-10 22:28 by richard.

Files
File name Uploaded Description Edit Remove
better_statusauditor.patch ber, 2002-10-10 11:15 patch
Messages
msg434 Author: [hidden] (ber) Date: 2002-10-10 11:15
statusauditor.py does not set the status to unread when
the issue is created using the standard classic
webform. Because the status key si available, it is
just not set. Thus the code should read like:

 def presetunread(db, cl, nodeid, newvalues):
     ''' Make sure the status is set on new issues
     '''
-    if newvalues.has_key('status'):
+        
+    if newvalues.has_key('status') and
newvalues['status']:
         return

     # ok, do it

Attached a patch that improved behaviour for me,too.
It is not intuitive that last comments on issues reset
the status to "chatting". On the other hand, if they
had no status. We can set it to chatting.
msg435 Author: [hidden] (richard) Date: 2002-10-10 11:18
Logged In: YES 
user_id=6405

The resolved->chatting behaviour is intentional and will remain the 
default - if someone sends a message to a closed issue, it is assumed 
they wish to reopen it. You're free to modify your local status auditor 
to change the behaviour, as you've done :) 
 
Thanks for the other patch. 
msg436 Author: [hidden] (ber) Date: 2002-10-10 13:18
Logged In: YES 
user_id=113859

If it is intentional, why don't you set it to chatting from
"done-cbb" ? :)

My patch also contained that if the status is None it will
be set to chatting if a
msg comes in.
msg437 Author: [hidden] (richard) Date: 2002-10-10 22:28
Logged In: YES 
user_id=6405

Your comment about done-cbb makes sense.  
  
As for your other comment, "Thanks for the other patch." :) 
 
History
Date User Action Args
2002-10-10 11:15:32bercreate