Merge pull request 'B2026-007 PROMS Security - don't error when add a user to a group but no group selected/C2026-004 – Add a messagebox when create group' (#694) from B2026-007_C2026-004 into Development

good for testing

PROMS/VEPROMS User Interface/dlgManageSecurity.cs

@@ -145,6 +145,9 @@ namespace VEPROMS
             }
 
             LoadRefreshGroupUsers();
+
+			// C2026-004 - messagebox when create group
+			MessageBox.Show($"{txt} group successfully created.", "Create group", MessageBoxButtons.OK, MessageBoxIcon.Information);
 		}
 		private void tvFolders_NodeMouseClick(object sender, TreeNodeMouseClickEventArgs e)
 		{
@@ -187,18 +190,22 @@ namespace VEPROMS
 		//Add a Member to a Group
 		private void addMember_Click(object sender, EventArgs e)
 		{
-			int groupIndex = cbGroupSelection.SelectedIndex;
+			// B2026-007 PROMS Security - don't error when add a user to a group but no group selected
-			string selectedUserID = lstNonMembers.SelectedValue.ToString();
+			if (lstNonMembers.SelectedValue != null)
-			int selectedUID = UserInfo.GetByUserID(selectedUserID).UID;
+			{
-			GroupInfo gi = myGroupInfoList[groupIndex];
+				int groupIndex = cbGroupSelection.SelectedIndex;
-			User selectedUser = User.Get(selectedUID);
+				string selectedUserID = lstNonMembers.SelectedValue.ToString();
+				int selectedUID = UserInfo.GetByUserID(selectedUserID).UID;
+				GroupInfo gi = myGroupInfoList[groupIndex];
+				User selectedUser = User.Get(selectedUID);
 
-			Membership.MakeMembership(selectedUser, Group.Get(gi.GID), null, "");
+				Membership.MakeMembership(selectedUser, Group.Get(gi.GID), null, "");
-			updateMembershipLists();
+				updateMembershipLists();
-			lstNonMembers.SelectedIndex = -1;
+				lstNonMembers.SelectedIndex = -1;
 
-			int index = lstMembers.FindString(selectedUserID);
+				int index = lstMembers.FindString(selectedUserID);
-			lstMembers.SetSelected(index, true);
+				lstMembers.SetSelected(index, true);
+			}
 		}
 
 		//Remove a Member From a Group