Skip to content

Commit 3c4d1cf

Browse files
committed
update code after review
1 parent 30939e5 commit 3c4d1cf

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

src/Resources/Resources/ActiveDirectory/Cmdlets/UpdateAzureADApplicationCommand.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ public override void ExecuteCmdlet()
102102
Homepage = HomePage,
103103
IdentifierUris = IdentifierUri,
104104
ReplyUrls = ReplyUrl,
105-
AvailableToOtherTenants = MyInvocation.BoundParameters.ContainsKey("AvailableToOtherTenants") ? AvailableToOtherTenants : (bool?)null
105+
AvailableToOtherTenants = this.IsParameterBound(c => c.AvailableToOtherTenants) ? AvailableToOtherTenants : (bool?)null
106106
};
107107

108108
if (ShouldProcess(target: ObjectId, action: string.Format("Updating an application with object id '{0}'", ObjectId)))

src/Resources/Resources/ChangeLog.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
- Add new cmdlets for managing deployments at tenant scope: *-AzTenantDeployment
2424
- Refactor *-AzDeployment cmdlets to work specificly at subscription scope
2525
- Created aliases *-AzSubscriptionDeployment for *-AzDeployment cmdlets
26+
* Fix `Update-AzADApplication` when parameter `AvailableToOtherTenants` is not set
2627

2728
## Version 1.10.0
2829
* Make -Scope optional in *-AzPolicyAssignment cmdlets with default to context subscription

0 commit comments

Comments
 (0)