Skip to content

Commit

Permalink
Fix plan filter in AzureADPlanImpl (#580)
Browse files Browse the repository at this point in the history
<!-- Thank you for submitting a Pull Request. If you're new to
contributing to BCApps please read our pull request guideline below
* https://github.com/microsoft/BCApps/Contributing.md
-->
#### Summary <!-- Provide a general summary of your changes -->
Add missing `&` to the filter on plan in `AzureAdPlanImpl`

#### Work Item(s) <!-- Add the issue number here after the #. The issue
needs to be open and approved. Submitting PRs with no linked issues or
unapproved issues is highly discouraged. -->
Fixes
[AB#498639](https://dynamicssmb2.visualstudio.com/Dynamics%20SMB/_workitems/edit/498639/)
  • Loading branch information
stkillen authored Feb 14, 2024
1 parent 99ee993 commit c21ffd8
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -510,7 +510,7 @@ codeunit 9018 "Azure AD Plan Impl."
// Check if the user is a member of the Device group
if IsDeviceRole(GraphUserInfo) then begin
// Only assign the device plan if the user doesn't have any other plans (except possibly Internal Admin or M365 Collaboration)
TempPlan.SetFilter("Plan ID", '<>%1&<>%2&<>%3<>%4', PlanIds.GetGlobalAdminPlanId(), PlanIds.GetD365AdminPlanId(), PlanIds.GetBCAdminPlanId(), PlanIds.GetMicrosoft365PlanId());
TempPlan.SetFilter("Plan ID", '<>%1&<>%2&<>%3&<>%4', PlanIds.GetGlobalAdminPlanId(), PlanIds.GetD365AdminPlanId(), PlanIds.GetBCAdminPlanId(), PlanIds.GetMicrosoft365PlanId());

if TempPlan.IsEmpty() then begin
// Remove the Internal Admin and M365 Collaboration plans, if assigned
Expand Down

0 comments on commit c21ffd8

Please sign in to comment.