Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

External users by organization name is case sensitive #78

Open
perlausten opened this issue Oct 18, 2024 · 0 comments
Open

External users by organization name is case sensitive #78

perlausten opened this issue Oct 18, 2024 · 0 comments

Comments

@perlausten
Copy link

The check for organization names for external users is case sensitive. So if you mark "SomeCompany" as an external organization name and you have users from that external organization where the organization unit is called "Somecompany" than those users are marked as CCB users and not CCX users.

Suggestion: use LCase in the check.

Original code:

If userName.Admd=externalORGs(x) Or userName.Country=externalORGs(x) Or userName.Organization=externalORGs(x)_
Or userName.Orgunit1=externalORGs(x) Or userName.Orgunit2=externalORGs(x) Or userName.Orgunit3=externalORGs(x)_
Or userName.Orgunit4=externalORGs(x) Or userName.Prmd=externalORGs(x) Then
	userNote.ExternalORG = "true"
	Exit For
End If

Suggested new code:

If LCase(userName.Admd)=LCase(externalORGs(x)) Or LCase(userName.Country)=LCase(externalORGs(x)) Or LCase(userName.Organization)=LCase(externalORGs(x)) _
Or LCase(userName.Orgunit1)=LCase(externalORGs(x)) Or LCase(userName.Orgunit2)=LCase(externalORGs(x)) Or LCase(userName.Orgunit3)=LCase(externalORGs(x)) _
Or LCase(userName.Orgunit4)=LCase(externalORGs(x)) Or LCase(userName.Prmd)=LCase(externalORGs(x)) Then
	userNote.ExternalORG = "true"
	Exit For
End If
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant