Bug in bulkimport.php ($ad_username)

I think I found a bug in bulkimport.php, which caused logins to fail when there are special character (?) or Umlaute (?, ?, ü) in the username, and automatic disablling of users is activated.

The code causing this is:

// import all relevant users
		$added_users = 0;
		$updated_users = 0;
		foreach ($all_users AS $username) {

/* HERE */		$ad_username = $username;

			// getting user data
			//$user = get_userdatabylogin($username); // deprecated
			$user = get_user_by('login', $username);

for me, changing it to this helped:

// import all relevant users
		$added_users = 0;
		$updated_users = 0;
		foreach ($all_users AS $username) {

			// getting user data
			//$user = get_userdatabylogin($username); // deprecated
			$user = get_user_by('login', $username);
/* HERE */		$ad_username = get_user_meta($user->ID, 'adi_samaccountname', true);

This is because WordPress logon names are created with s instead of ?, a instead of ? and so on.
can anyone confirm this problem / solution?

cheers,
Andreas

https://www.ads-software.com/extend/plugins/active-directory-integration/