Changeset 1437

Timestamp:

03/23/11 17:31:50 (14 months ago)

Author:

zeank

Message:

on successfully registering for a group redisplay list of all
available groups

closes #283

Files:

• HelpIM3/branches/chatgroups/HelpIM/chatgroup/pages/Subscription.py (modified) (3 diffs)

HelpIM3/branches/chatgroups/HelpIM/chatgroup/pages/Subscription.py

@@ -17 +17 @@
     def getPanelDef(self, *args, **kwargs):
         group_id = self.checkNotEmpty('group_id')
+
         if self.site.userIsAuthorized(
             user=self.him_request.session.session_user,
             script=self.editFunction):
-            editLink = {'action': [{'name':self.editFunction, 'text':'Bewerk deze pagina'},
-                                   [{'actionParm': [{'name': 'newsType', 'value':self.newsType }]}]
+            editLink = {'action': [{'name':self.editFunction,
+                                    'text':'Bewerk deze pagina'},
+                                   [{'actionParm': [{'name': 'newsType',
+                                                     'value':self.newsType }]}]
                                    ]}
         else:
@@ -53 +56 @@
                      {'name': self.function,
                       'text': 'aanmelden',
-                      'html:onclick': 'return showChatgroupSubscriptionForm(this);'}
+                      'html:onclick':
+                      'return showChatgroupSubscriptionForm(this);'}
                      ]}]
                  ]}
              ]
             ]}
-        className = ''
-        if not group_id:
-            className = 'hidden'
+        """ we need to redisplay the subscription form only in case of
+        an error when posting to an id"""
+        className = 'hidden'
+        if self.errors and group_id:
+            className = ''
         form = {'edit_panel': [
             {'id': 'subscribe_group_form',
@@ -95 +101 @@
         htmlCode = {  'htmlCode': [ him_text.himTxt_text.replace(' ', ' ') ] }
 
-        if self.checkNotEmpty('group_id'):
+        if self.errors and self.checkNotEmpty('group_id'):
+            """ only display single group if there was an error posting to it """
             groups = [self.site.getChatgroupById(self.site, self.getParam('group_id'))]
         else: