silverstripe / silverstripe-ldap

src/Services/LDAPService.php

Spacing +20 added lines, -20 removed lines

@@ -49,7 +49,7 @@ 
      * @var array
      */
     private static $dependencies = [
-        'gateway' => '%$' . LDAPGateway::class
+        'gateway' => '%$'.LDAPGateway::class
     ];
 
     /**
@@ -125,7 +125,7 @@ 
      */
     public static function get_cache()
     {
-        return Injector::inst()->get(CacheInterface::class . '.ldap');
+        return Injector::inst()->get(CacheInterface::class.'.ldap');
     }
 
     /**
@@ -207,14 +207,14 @@ 
         // show better errors than the defaults for various status codes returned by LDAP
         if (!empty($messages[1]) && strpos($messages[1], 'NT_STATUS_ACCOUNT_LOCKED_OUT') !== false) {
             $message = _t(
-                __CLASS__ . '.ACCOUNTLOCKEDOUT',
-                'Your account has been temporarily locked because of too many failed login attempts. ' .
+                __CLASS__.'.ACCOUNTLOCKEDOUT',
+                'Your account has been temporarily locked because of too many failed login attempts. '.
                 'Please try again later.'
             );
         }
         if (!empty($messages[1]) && strpos($messages[1], 'NT_STATUS_LOGON_FAILURE') !== false) {
             $message = _t(
-                __CLASS__ . '.INVALIDCREDENTIALS',
+                __CLASS__.'.INVALIDCREDENTIALS',
                 'The provided details don\'t seem to be correct. Please try again.'
             );
         }
@@ -237,7 +237,7 @@ 
     public function getNodes($cached = true, $attributes = [])
     {
         $cache = self::get_cache();
-        $cacheKey = 'nodes' . md5(implode('', $attributes));
+        $cacheKey = 'nodes'.md5(implode('', $attributes));
         $results = $cache->has($cacheKey);
 
         if (!$results || !$cached) {
@@ -267,7 +267,7 @@ 
     {
         $searchLocations = $this->config()->groups_search_locations ?: [null];
         $cache = self::get_cache();
-        $cacheKey = 'groups' . md5(implode('', array_merge($searchLocations, $attributes)));
+        $cacheKey = 'groups'.md5(implode('', array_merge($searchLocations, $attributes)));
         $results = $cache->has($cacheKey);
 
         if (!$results || !$cached) {
@@ -535,7 +535,7 @@ 
         }
 
         $member->IsExpired = ($data['useraccountcontrol'] & 2) == 2;
-        $member->LastSynced = (string)DBDatetime::now();
+        $member->LastSynced = (string) DBDatetime::now();
 
         foreach ($member->config()->ldap_field_mappings as $attribute => $field) {
             // Special handling required for attributes that don't exist in the response.
@@ -556,7 +556,7 @@ 
                 } else {
                     $this->getLogger()->debug(
                         sprintf(
-                            'Attribute %s configured in Member.ldap_field_mappings, ' .
+                            'Attribute %s configured in Member.ldap_field_mappings, '.
                             'but no available attribute in AD data (GUID: %s, Member ID: %s)',
                             $attribute,
                             $data['objectguid'],
@@ -641,7 +641,7 @@ 
         if ($imageClass !== Image::class && !is_subclass_of($imageClass, Image::class)) {
             $this->getLogger()->warning(
                 sprintf(
-                    'Member field %s configured for ' . $attributeName . ' AD attribute, but it isn\'t a valid relation to an '
+                    'Member field %s configured for '.$attributeName.' AD attribute, but it isn\'t a valid relation to an '
                         . 'Image class',
                     $fieldName
                 )
@@ -667,7 +667,7 @@ 
 
         // Setup variables
         $thumbnailFolder = Folder::find_or_make($member->config()->ldap_thumbnail_path);
-        $filename = sprintf($attributeName . '-%s.jpg', $data['objectguid']);
+        $filename = sprintf($attributeName.'-%s.jpg', $data['objectguid']);
         $filePath = File::join_paths($thumbnailFolder->getFilename(), $filename);
         $fileCfg = [
             // if there's a filename conflict we've got new content so overwrite it.
@@ -760,7 +760,7 @@ 
         }
 
         // Lookup the previous mappings and see if there is any mappings no longer present.
-        $unmappedGroups = $member->Groups()->alterDataQuery(function (DataQuery $query) {
+        $unmappedGroups = $member->Groups()->alterDataQuery(function(DataQuery $query) {
             // join with the Group_Members table because we only want those group members assigned by this module.
             $query->leftJoin("Group_Members", '"Group_Members"."GroupID" = "Group"."ID"');
             $query->where('"Group_Members"."IsImportedFromLDAP" = 1');
@@ -801,7 +801,7 @@ 
             $group->Description = $data['description'];
         }
         $group->DN = $data['dn'];
-        $group->LastSynced = (string)DBDatetime::now();
+        $group->LastSynced = (string) DBDatetime::now();
         $group->write();
 
         // Mappings on this group are automatically maintained to contain just the group's DN.
@@ -864,7 +864,7 @@ 
                 ),
             ]);
         } catch (Exception $e) {
-            throw new ValidationException('LDAP synchronisation failure: ' . $e->getMessage());
+            throw new ValidationException('LDAP synchronisation failure: '.$e->getMessage());
         }
 
         $user = $this->getUserByUsername($member->Username);
@@ -908,7 +908,7 @@ 
                 'distinguishedname' => $dn
             ]);
         } catch (Exception $e) {
-            throw new ValidationException('LDAP group creation failure: ' . $e->getMessage());
+            throw new ValidationException('LDAP group creation failure: '.$e->getMessage());
         }
 
         $data = $this->getGroupByDN($dn);
@@ -964,7 +964,7 @@ 
                 $dn = $newDn;
             }
         } catch (Exception $e) {
-            throw new ValidationException('LDAP move failure: ' . $e->getMessage());
+            throw new ValidationException('LDAP move failure: '.$e->getMessage());
         }
 
         try {
@@ -988,7 +988,7 @@ 
 
             $this->update($dn, $attributes);
         } catch (Exception $e) {
-            throw new ValidationException('LDAP synchronisation failure: ' . $e->getMessage());
+            throw new ValidationException('LDAP synchronisation failure: '.$e->getMessage());
         }
     }
 
@@ -1073,7 +1073,7 @@ 
             try {
                 $this->update($groupDn, ['member' => $members]);
             } catch (Exception $e) {
-                throw new ValidationException('LDAP group membership remove failure: ' . $e->getMessage());
+                throw new ValidationException('LDAP group membership remove failure: '.$e->getMessage());
             }
         }
     }
@@ -1099,7 +1099,7 @@ 
         try {
             $this->update($groupDn, ['member' => $members]);
         } catch (Exception $e) {
-            throw new ValidationException('LDAP group membership add failure: ' . $e->getMessage());
+            throw new ValidationException('LDAP group membership add failure: '.$e->getMessage());
         }
     }
 
@@ -1182,7 +1182,7 @@ 
         try {
             $this->delete($data['distinguishedname']);
         } catch (Exception $e) {
-            throw new ValidationException('LDAP delete user failed: ' . $e->getMessage());
+            throw new ValidationException('LDAP delete user failed: '.$e->getMessage());
         }
     }