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

diff --git a/dialup_admin/Changelog b/dialup_admin/Changelog
index d7d6840..b34f6ea 100644 (file)
--- a/dialup_admin/Changelog
+++ b/dialup_admin/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/dialup_admin/lib/sql/create_group.php3 b/dialup_admin/lib/sql/create_group.php3
index 22ecf69..0539b99 100644 (file)
--- a/dialup_admin/lib/sql/create_group.php3
+++ b/dialup_admin/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";