silverstripe / silverstripe-ldap

src/Forms/LDAPChangePasswordForm.php

Spacing +2 added lines, -2 removed lines

@@ -59,7 +59,7 @@ 
         ) {
             $emailField = TextField::create(
                 'Email',
-                _t(__CLASS__ . '.USERNAMEOREMAIL', 'Email'),
+                _t(__CLASS__.'.USERNAMEOREMAIL', 'Email'),
                 $member->Email,
                 null,
                 $this
@@ -68,7 +68,7 @@ 
         if (!empty($data['samaccountname'])) {
             $usernameField = TextField::create(
                 'Username',
-                _t(__CLASS__ . '.USERNAME', 'Username'),
+                _t(__CLASS__.'.USERNAME', 'Username'),
                 $data['samaccountname'],
                 null,
                 $this

src/Services/LDAPService.php

Spacing +21 added lines, -21 removed lines

@@ -47,7 +47,7 @@ 
      * @var array
      */
     private static $dependencies = [
-        'gateway' => '%$' . LDAPGateway::class
+        'gateway' => '%$'.LDAPGateway::class
     ];
 
     /**
@@ -123,7 +123,7 @@ 
      */
     public static function get_cache()
     {
-        return Injector::inst()->get(CacheInterface::class . '.ldap');
+        return Injector::inst()->get(CacheInterface::class.'.ldap');
     }
 
     /**
@@ -192,14 +192,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.'
             );
         }
@@ -222,7 +222,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) {
@@ -252,7 +252,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) {
@@ -520,13 +520,13 @@ 
         }
 
         $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) {
             if (!isset($data[$attribute])) {
                 $this->getLogger()->notice(
                     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'],
@@ -544,7 +544,7 @@ 
                 ) {
                     $this->getLogger()->warning(
                         sprintf(
-                            'Member field %s configured for thumbnailphoto AD attribute, but it isn\'t a ' .
+                            'Member field %s configured for thumbnailphoto AD attribute, but it isn\'t a '.
                             'valid relation to an Image class',
                             $field
                         )
@@ -554,8 +554,8 @@ 
                 }
 
                 $filename = sprintf('thumbnailphoto-%s.jpg', $data['samaccountname']);
-                $path = ASSETS_DIR . '/' . $member->config()->ldap_thumbnail_path;
-                $absPath = BASE_PATH . '/' . $path;
+                $path = ASSETS_DIR.'/'.$member->config()->ldap_thumbnail_path;
+                $absPath = BASE_PATH.'/'.$path;
                 if (!file_exists($absPath)) {
                     Filesystem::makeFolder($absPath);
                 }
@@ -567,13 +567,13 @@ 
                 }
 
                 // The image data is provided in raw binary.
-                file_put_contents($absPath . '/' . $filename, $data[$attribute]);
+                file_put_contents($absPath.'/'.$filename, $data[$attribute]);
                 $record = new $imageClass();
                 $record->Name = $filename;
-                $record->Filename = $path . '/' . $filename;
+                $record->Filename = $path.'/'.$filename;
                 $record->write();
 
-                $relationField = $field . 'ID';
+                $relationField = $field.'ID';
                 $member->{$relationField} = $record->ID;
             } else {
                 $member->$field = $data[$attribute];
@@ -714,7 +714,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.
@@ -777,7 +777,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);
@@ -821,7 +821,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);
@@ -986,7 +986,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());
             }
         }
     }
@@ -1012,7 +1012,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());
         }
     }
 
@@ -1095,7 +1095,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());
         }
     }
 

src/Model/LDAPGroupMapping.php

Spacing +6 added lines, -6 removed lines

@@ -47,7 +47,7 @@ 
      * @var array
      */
     private static $dependencies = [
-        'ldapService' => '%$' . LDAPService::class,
+        'ldapService' => '%$'.LDAPService::class,
     ];
 
     /**
@@ -59,8 +59,8 @@ 
         $fields = parent::getCMSFields();
         $fields->removeByName('DN');
 
-        $field = DropdownField::create('DN', _t(__CLASS__ . '.LDAPGROUP', 'LDAP Group'));
-        $field->setEmptyString(_t(__CLASS__ . '.SELECTONE', 'Select one'));
+        $field = DropdownField::create('DN', _t(__CLASS__.'.LDAPGROUP', 'LDAP Group'));
+        $field->setEmptyString(_t(__CLASS__.'.SELECTONE', 'Select one'));
         $groups = $this->ldapService->getGroups(true, ['dn', 'name']);
         if ($groups) {
             foreach ($groups as $dn => $record) {
@@ -74,12 +74,12 @@ 
         $fields->removeByName('Scope');
         $fields->addFieldToTab(
             'Root.Main',
-            DropdownField::create('Scope', _t(__CLASS__ . '.SCOPE', 'Scope'), [
+            DropdownField::create('Scope', _t(__CLASS__.'.SCOPE', 'Scope'), [
                 'Subtree' => _t(
-                    __CLASS__ . '.SUBTREE_DESCRIPTION',
+                    __CLASS__.'.SUBTREE_DESCRIPTION',
                     'Users within this group and all nested groups within'
                 ),
-                'OneLevel' => _t(__CLASS__ . '.ONELEVEL_DESCRIPTION', 'Only users within this group'),
+                'OneLevel' => _t(__CLASS__.'.ONELEVEL_DESCRIPTION', 'Only users within this group'),
             ])
         );
 

src/Model/LDAPUtil.php

Spacing +5 added lines, -5 removed lines

@@ -42,8 +42,8 @@ 
         for ($k = 1; $k <= 2; ++$k) {
             $hex_guid_to_guid_str .= substr($hex_guid, 16 - 2 * $k, 2);
         }
-        $hex_guid_to_guid_str .= '-' . substr($hex_guid, 16, 4);
-        $hex_guid_to_guid_str .= '-' . substr($hex_guid, 20);
+        $hex_guid_to_guid_str .= '-'.substr($hex_guid, 16, 4);
+        $hex_guid_to_guid_str .= '-'.substr($hex_guid, 20);
 
         return strtoupper($hex_guid_to_guid_str);
     }
@@ -69,7 +69,7 @@ 
 
         if ($escape) {
             $escaped = '\\';
-            for ($i = 0; $i < strlen($octet_str); $i+=2) {
+            for ($i = 0; $i < strlen($octet_str); $i += 2) {
                 $escaped .= substr($octet_str, $i, 2);
                 if ($i != strlen($octet_str) - 2) {
                     $escaped .= '\\';
@@ -95,11 +95,11 @@ 
 
         for ($x = 0; $x < $subcount; $x++) {
             $subauth[$x] = hexdec(self::little_endian(substr($hex_sid, 16 + ($x * 8), 8)));
-            $result .= '-' . $subauth[$x];
+            $result .= '-'.$subauth[$x];
         }
 
         // Cheat by tacking on the S-
-        return 'S-' . $result;
+        return 'S-'.$result;
     }
 
     /**

src/Jobs/LDAPAllSyncJob.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
      */
     public function getTitle()
     {
-        return _t(__CLASS__ . '.SYNCTITLE', 'Sync all groups and users from Active Directory, and set mappings up.');
+        return _t(__CLASS__.'.SYNCTITLE', 'Sync all groups and users from Active Directory, and set mappings up.');
     }
 
     /**

src/Jobs/LDAPMemberSyncJob.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
      */
     public function getTitle()
     {
-        return _t(__CLASS__ . '.SYNCTITLE', 'Sync all users from Active Directory');
+        return _t(__CLASS__.'.SYNCTITLE', 'Sync all users from Active Directory');
     }
 
     /**