On group creation, if member list is empty report that, not that the group was created.

diff --git a/Changelog b/Changelog
index d7d6840e5fc913e8c4bcd23ff2cfaf3cb3fa207d..b34f6eabbde268ea1843fe83360244ef197e1a7b 100644 (file)
--- a/Changelog
+++ b/Changelog
@@ -32,6 +32,7 @@ Ver 1.75:
 * Add a new directive sql_show_all_groups. If set to true then in user edit page we show all available
   groups with the ones the user is a member of highlighted. The administrator can then directly
   change user group membership by changing membership in this group list.
+* On group creation, if member list is empty report that, not that the group was created.
 Ver 1.72:
 * Move the xlat function to a separate file in lib/xlat.php3
 * Add a lib/sql/nas_list.php3 to also get the nas list from sql (naslist.conf still works)

diff --git a/lib/sql/create_group.php3 b/lib/sql/create_group.php3
index 22ecf69ae9bd21288f7ece9ba99e3a445747b2aa..0539b9902c531fcb93bae15cb59a9bd6e76edd6e 100644 (file)
--- a/lib/sql/create_group.php3
+++ b/lib/sql/create_group.php3
@@ -28,6 +28,8 @@ if ($link){
                        }
                }
        }
+       else
+               echo "<b>Members list is empty!!</b><br>\n";
        if (!$da_abort){
                foreach($show_attrs as $key => $attr){
                        if ($attrmap["$key"] == 'none')
@@ -65,8 +67,8 @@ if ($link){
                        if (!$res || !@da_sql_affected_rows($link,$res,$config))
                                echo "<b>Query failed for attribute $key: " . da_sql_error($link,$config) . "</b><br>\n";
                }
+               echo "<b>Group created successfully</b><br>\n";
        }
-       echo "<b>Group created successfully</b><br>\n";
 }
 else
        echo "<b>Could not connect to SQL database</b><br>\n";