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

CCX not properly detected by OU/Org - it's a BUG in the code #53

Closed
KarlHartmann opened this issue Oct 4, 2023 · 1 comment
Closed
Assignees
Labels
investigating we are investigating this report for more information and possible inclusion in a future release

Comments

@KarlHartmann
Copy link

KarlHartmann commented Oct 4, 2023

reporting again, since my first one was just closed
#42

CCX not properly detected by OU/Org
since you check for some string (Orgs only)
in user name - you may hit CommonName parts too

Dear developer, please read and understand your code - and our error about NotesNames

You MUST not search by INSTR for an OU or ORG sting in a Notes Fullname
you might get a result by hitting part of the common name

BAD programming

			If externalNAB Then userNote.ExternalNAB="true"
		If externalORGs(0)<>"" Then 
			For x=0 To UBound(externalORGs)
				If InStr(1,userNote.FullName(0),externalORGs(x),5)<>0 Then userNote.ExternalORG="true":Exit For
			Next
		End If
@notesboy notesboy self-assigned this Sep 16, 2024
@notesboy notesboy added the investigating we are investigating this report for more information and possible inclusion in a future release label Sep 16, 2024
@notesboy
Copy link
Collaborator

Fixed for DLAU V1.2.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
investigating we are investigating this report for more information and possible inclusion in a future release
Projects
None yet
Development

No branches or pull requests

2 participants