Merge pull request Azure#4881 from paulomarquesc/preview

cormacpayne · web-flow · commit 8f1827e8fe12 · 2017-10-30T15:15:36.000-04:00
Fixed a bug related to issue 4828.
diff --git a/src/ResourceManager/Resources/Commands.Resources/Models.Authorization/AuthorizationClient.cs b/src/ResourceManager/Resources/Commands.Resources/Models.Authorization/AuthorizationClient.cs
@@ -219,7 +219,7 @@ public List<PSRoleAssignment> FilterRoleAssignments(FilterRoleAssignmentsOptions
                 }
                 else
                 {
-                    principalId = string.IsNullOrEmpty(options.ADObjectFilter.Id.ToString()) ? adObject.Id.ToString() : options.ADObjectFilter.Id;
+                    principalId = string.IsNullOrEmpty(options.ADObjectFilter.Id) ? adObject.Id.ToString() : options.ADObjectFilter.Id;
                 }
 
                 var tempResult = AuthorizationManagementClient.RoleAssignments.List(

Original file line number	Diff line number	Diff line change
`@@ -219,7 +219,7 @@ public List<PSRoleAssignment> FilterRoleAssignments(FilterRoleAssignmentsOptions`
`219`	`219`	`}`
`220`	`220`	`else`
`221`	`221`	`{`
`222`		`- principalId = string.IsNullOrEmpty(options.ADObjectFilter.Id.ToString()) ? adObject.Id.ToString() : options.ADObjectFilter.Id;`
	`222`	`+ principalId = string.IsNullOrEmpty(options.ADObjectFilter.Id) ? adObject.Id.ToString() : options.ADObjectFilter.Id;`
`223`	`223`	`}`
`224`	`224`
`225`	`225`	`var tempResult = AuthorizationManagementClient.RoleAssignments.List(`