Merge branch 'xorg/maint'
authorNicolas Iooss <nicolas.iooss_git@polytechnique.org>
Sun, 24 Nov 2013 15:48:40 +0000 (16:48 +0100)
committerNicolas Iooss <nicolas.iooss_git@polytechnique.org>
Sun, 24 Nov 2013 15:48:40 +0000 (16:48 +0100)
1  2 
modules/xnetgrp.php

diff --combined modules/xnetgrp.php
@@@ -831,9 -831,13 +831,13 @@@ class XnetGrpModule extends PLModul
                           S::i('uid'), $globals->asso('id'));
              $admins = $globals->asso()->iterToNotify();
              $admin = $admins->next();
-             $to = $admin->bestEmail();
-             while ($admin = $admins->next()) {
-                 $to .= ', ' . $admin->bestEmail();
+             if (!is_null($admin)) {
+                 $to = $admin->bestEmail();
+                 while ($admin = $admins->next()) {
+                     $to .= ', ' . $admin->bestEmail();
+                 }
+             } else {
+                 $to = '';
              }
  
              $append = "\n"
  
              Group::subscribe($globals->asso('id'), $user->id());
              $this->removeSubscriptionRequest($user->id());
 +            
 +            // Check if the group has more than 1000 members, if so, disable the "send mail" function.
 +            $full_count = XDB::fetchOneCell('SELECT COUNT(*)
 +                                               FROM group_members
 +                                              WHERE asso_id = {?}',
 +                                $globals->asso('id'));
 +            if ($full_count > 999) {
 +                XDB::execute(
 +                    "UPDATE groups
 +                    SET disable_mails = 1
 +                    WHERE id = {?}",
 +                    $globals->asso('id')); 
 +            }
 +            
              if ($suggest_account_activation) {
                  pl_redirect('member/suggest/' . $user->login() . '/' . $email . '/' . $globals->asso('nom'));
              } else {