Fix #3098 - issue with Get-PnPTenantSyncClientRestriction not showing… (#3099)

* Fix #3098 - issue with Get-PnPTenantSyncClientRestriction not showing values

* Update CHANGELOG.md

---------

Co-authored-by: Gautam Sheth <gautam.sheth@staffbase.com>

Author: gautamdsheth

CHANGELOG.md

@@ -30,6 +30,7 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/).
 - Fixed issue where `Add-PnPSiteCollectionAdmin -PrimarySiteCollectionAdmin <user>` would require owners to be specified as well. [#3035](https://github.com/pnp/powershell/pull/3035)
 - Fixed `Get-PnPAzureADGroup` returns only 100 results even if there are more groups present in Azure AD. [#3085](https://github.com/pnp/powershell/pull/3085)
 - Fixed `Get-PnPAccessToken` cmdlet to correctly retrieve the access token for the specified resource URL. [#3091](https://github.com/pnp/powershell/pull/3091)
+- Fixed issue with `Get-PnPTenantSyncClientRestriction` cmdlet not populating the necessary properties. [#3099](https://github.com/pnp/powershell/pull/3099)
 
 ### Changed
 

src/Commands/Admin/GetTenantSyncClientRestriction.cs

@@ -10,8 +10,7 @@ public class GetPnPTenantSyncClientRestriction : PnPAdminCmdlet
     {
         protected override void ExecuteCmdlet()
         {
-            AdminContext.Load(Tenant);
-            AdminContext.Load(Tenant, t => t.HideDefaultThemes);
+            AdminContext.Load(Tenant);            
             AdminContext.ExecuteQueryRetry();
             WriteObject(new SPOTenantSyncClientRestriction(Tenant));
         }

src/Commands/Admin/RemoveTenantSyncClientRestriction.cs

@@ -11,12 +11,12 @@ public class RemovePnPTenantSyncClientRestriction : PnPAdminCmdlet
     {
         protected override void ExecuteCmdlet()
         {
-            this.Tenant.IsUnmanagedSyncClientForTenantRestricted = false;
-            this.Tenant.BlockMacSync = false;
-            this.Tenant.ExcludedFileExtensionsForSyncClient = new List<string>();
-            this.Tenant.OptOutOfGrooveBlock = false;
-            this.Tenant.OptOutOfGrooveSoftBlock = false;
-            this.Tenant.DisableReportProblemDialog = false;
+            Tenant.IsUnmanagedSyncClientForTenantRestricted = false;
+            Tenant.BlockMacSync = false;
+            Tenant.ExcludedFileExtensionsForSyncClient = new List<string>();
+            Tenant.OptOutOfGrooveBlock = false;
+            Tenant.OptOutOfGrooveSoftBlock = false;
+            Tenant.DisableReportProblemDialog = false;
             AdminContext.ExecuteQueryRetry();
             WriteObject(new SPOTenantSyncClientRestriction(Tenant));
         }

src/Commands/Model/SPOTenantSyncClientRestriction.cs

@@ -1,4 +1,5 @@
 using Microsoft.Online.SharePoint.TenantAdministration;
+using Microsoft.SharePoint.Client;
 using System;
 using System.Collections.Generic;
 
@@ -50,6 +51,7 @@ public class SPOTenantSyncClientRestriction
         /// <param name="tenant">Tenant instance to get the properties from to fill this instance</param>
         public SPOTenantSyncClientRestriction(Tenant tenant)
         {
+            tenant.EnsureProperties(t => t.IsUnmanagedSyncClientForTenantRestricted, t => t.AllowedDomainListForSyncClient, t => t.BlockMacSync, t => t.ExcludedFileExtensionsForSyncClient, t => t.OptOutOfGrooveBlock, t => t.OptOutOfGrooveSoftBlock, t => t.DisableReportProblemDialog);
             AllowedDomainList = new List<Guid>(tenant.AllowedDomainListForSyncClient);
             BlockMacSync = tenant.BlockMacSync;
             ExcludedFileExtensions = new List<string>(tenant.ExcludedFileExtensionsForSyncClient);