SF.net SVN: mailmanager: [2997] MailManager/branches/RELENG_2_0

[<ke...@us...>]

Revision: 2997
Author:   kevca
Date:     2006-05-11 05:33:57 -0700 (Thu, 11 May 2006)
ViewCVS:  http://svn.sourceforge.net/mailmanager/?rev=2997&view=rev

Log Message:
-----------
Fix for
- UI Allows creation of empty groups (#1486433)

Modified Paths:
--------------
    MailManager/branches/RELENG_2_0/CHANGES.txt
    MailManager/branches/RELENG_2_0/MailManager.py
Modified: MailManager/branches/RELENG_2_0/CHANGES.txt
===================================================================
--- MailManager/branches/RELENG_2_0/CHANGES.txt	2006-05-10 09:23:30 UTC (rev 2996)
+++ MailManager/branches/RELENG_2_0/CHANGES.txt	2006-05-11 12:33:57 UTC (rev 2997)
@@ -6,6 +6,7 @@
 - Selecting No change results in a failure (#1467786)
 - Automatic addition of attachments (#1459924)
 - Importing fails with alternative timezone (#1485389)
+- UI Allows creation of empty groups (#1486433)
 
 Version 2.0.8
 * BUG FIXES

Modified: MailManager/branches/RELENG_2_0/MailManager.py
===================================================================
--- MailManager/branches/RELENG_2_0/MailManager.py	2006-05-10 09:23:30 UTC (rev 2996)
+++ MailManager/branches/RELENG_2_0/MailManager.py	2006-05-11 12:33:57 UTC (rev 2997)
@@ -1757,9 +1757,12 @@
     security.declareProtected('MailManager Settings', 'editGroup')
     def editGroup(self, group_name, usernames=[], REQUEST=None):
          """Edits an existing group."""
-         if REQUEST is not None and not usernames:
-             REQUEST.set('error', 'You must select at least one user.')
-             REQUEST.set('flag_users', True)
+         if REQUEST is not None:
+            if not usernames:
+                REQUEST.set('error', 'You must select at least one user.')
+                REQUEST.set('flag_users', True)
+            if REQUEST.get('error'):
+                return self.GroupSettings(self, REQUEST)
          self.sql.deleteGroupMembers(group_name=group_name)
          for username in usernames:
              self.sql.addGroupMember(group_name=group_name, username=username)


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.