Merge pull request #1610 from RotherOSS/issue-#1608-empty_RolePermissionsFromLDAP

bschmalhofer · web-flow · commit a74cb98b6721 · 2022-02-11T18:58:46.000+01:00
Issue #1608: handling of %RolePermissionsFromLDAP
diff --git a/Kernel/System/Auth/Sync/LDAP.pm b/Kernel/System/Auth/Sync/LDAP.pm
@@ -776,10 +776,8 @@ sub Sync {
                 }
             }
         }
-    }
 
-    # compare role permissions from ldap with current user role permissions and update if necessary
-    if (%RolePermissionsFromLDAP) {
+        # compare role permissions from ldap with current user role permissions and update if necessary
 
         # get current user roles
         my %UserRoles = $GroupObject->PermissionUserRoleGet(

Original file line number	Diff line number	Diff line change
`@@ -776,10 +776,8 @@ sub Sync {`
`776`	`776`	`}`
`777`	`777`	`}`
`778`	`778`	`}`
`779`		`- }`
`780`	`779`
`781`		`- # compare role permissions from ldap with current user role permissions and update if necessary`
`782`		`- if (%RolePermissionsFromLDAP) {`
	`780`	`+ # compare role permissions from ldap with current user role permissions and update if necessary`
`783`	`781`
`784`	`782`	`# get current user roles`
`785`	`783`	`my %UserRoles = $GroupObject->PermissionUserRoleGet(`