Merge pull request #2720 from gautamdsheth/feature/removeObsoleteUnif…

…iedGroupMethod Fix: removed usage of obsolete ListUnifiedGroups method
pnp · Jun 11, 2020 · d8bb581 · d8bb581
2 parents c297004 + 3136ad4
commit d8bb581
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -12,6 +12,7 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/).
 ### Changed
 
 ### Contributors
+- Gautam Sheth [gautamdsheth]
 
 ## [3.22.2006.2]
 

diff --git a/Commands/Base/PipeBinds/UnifiedGroupPipeBind.cs b/Commands/Base/PipeBinds/UnifiedGroupPipeBind.cs
@@ -52,10 +52,10 @@ public UnifiedGroupEntity GetGroup(string accessToken, bool includeClassificatio
             }
             else if (!string.IsNullOrEmpty(DisplayName))
             {
-                var groups = UnifiedGroupsUtility.ListUnifiedGroups(accessToken, DisplayName, includeSite: true, includeClassification:includeClassification);
+                var groups = UnifiedGroupsUtility.GetUnifiedGroups(accessToken, DisplayName, includeSite: true, includeClassification:includeClassification);
                 if (groups == null || groups.Count == 0)
                 {
-                    groups = UnifiedGroupsUtility.ListUnifiedGroups(accessToken, mailNickname: DisplayName, includeSite: true, includeClassification:includeClassification);
+                    groups = UnifiedGroupsUtility.GetUnifiedGroups(accessToken, mailNickname: DisplayName, includeSite: true, includeClassification:includeClassification);
                 }
                 if (groups != null && groups.Any())
                 {

diff --git a/Commands/Graph/GetUnifiedGroup.cs b/Commands/Graph/GetUnifiedGroup.cs
@@ -65,7 +65,7 @@ protected override void ExecuteCmdlet()
             else
             {
                 // Retrieve all the groups
-                groups = UnifiedGroupsUtility.ListUnifiedGroups(AccessToken, includeSite: !ExcludeSiteUrl.IsPresent, includeClassification:IncludeClassification.IsPresent, includeHasTeam: IncludeHasTeam.IsPresent);
+                groups = UnifiedGroupsUtility.GetUnifiedGroups(AccessToken, includeSite: !ExcludeSiteUrl.IsPresent, includeClassification:IncludeClassification.IsPresent, includeHasTeam: IncludeHasTeam.IsPresent);
             }
 
             if (group != null)

diff --git a/Commands/Graph/NewUnifiedGroup.cs b/Commands/Graph/NewUnifiedGroup.cs
@@ -70,7 +70,7 @@ protected override void ExecuteCmdlet()
 
             if (!Force)
             {
-                var candidates = UnifiedGroupsUtility.ListUnifiedGroups(AccessToken,
+                var candidates = UnifiedGroupsUtility.GetUnifiedGroups(AccessToken,
                     mailNickname: MailNickname,
                     endIndex: 1);
                 // ListUnifiedGroups retrieves groups with starts-with, so need another check