codebunch / silverstripe-user-management

src/Page/UserRegistrationPage.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
         if ($page = DataObject::get_one(self::class)) {
             return $page;
         }
-        user_error(_t(__CLASS__ . '.NoPage', 'No UserRegistrationPage was found. 
+        user_error(_t(__CLASS__.'.NoPage', 'No UserRegistrationPage was found. 
             Please create one in the CMS!'), E_USER_ERROR);
         return null; // just to keep static analysis happy
     }

src/Reports/UserReport.php

Indentation +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 {
     protected $periodfield = '"Member"."Created"';
 
-     // the name of the report
+        // the name of the report
     public function title()
     {
         return 'Customer List';

Spacing +1 added lines, -1 removed lines

@@ -99,7 +99,7 @@
             $filter = "Member.Created <= '$end'";
         }
         if ($firstName) {
-            $filter = ($filter)? $filter . " AND FirstName Like '%$firstName%'" : "FirstName Like '%$firstName%'";
+            $filter = ($filter) ? $filter." AND FirstName Like '%$firstName%'" : "FirstName Like '%$firstName%'";
         }
 
         return $filter;

src/Forms/SignUpForm.php

Indentation +5 added lines, -5 removed lines

@@ -47,7 +47,7 @@ 
             $signUpPersonal = Member::create();
             $form->saveInto($signUpPersonal);
             if (isset($data['DOB']) && $data['DOB']) {
-                  $signUpPersonal->DOB = date('Y-m-d', strtotime($data['DOB']));
+                    $signUpPersonal->DOB = date('Y-m-d', strtotime($data['DOB']));
             }
             $signUpPersonal->write();
             $Member = Member::get()->byId($signUpPersonal->ID);
@@ -63,10 +63,10 @@ 
     }
 
 
-     /**
-     * @param Controller $controller the controller instance that is being passed to the form
-     * @return FieldList Fields for this form.
-     */
+        /**
+         * @param Controller $controller the controller instance that is being passed to the form
+         * @return FieldList Fields for this form.
+         */
     protected function getFormFields($controller = null)
     {
         $fields = FieldList::create(

src/Forms/ProfileForm.php

Spacing +2 added lines, -2 removed lines

@@ -55,11 +55,11 @@
             try {
                 $form->saveInto($member);
                 $member->write();
-                $msg = $this->getCustomMessage('ProfileUpdateSuccess')!=""
+                $msg = $this->getCustomMessage('ProfileUpdateSuccess') != ""
                     ? $this->getCustomMessage('ProfileUpdateSuccess') : "Profile updated!";
                 $form->sessionMessage($msg, 'good');
             } catch (\Exception $e) {
-                $msg = $this->getCustomMessage('ProfileUpdatError')!=""
+                $msg = $this->getCustomMessage('ProfileUpdatError') != ""
                     ? $this->getCustomMessage('ProfileUpdatError') : "Technical issue, Profile not updated!";
                 $form->sessionMessage($msg, 'bad');
             }

tests/Pages/LostPasswordPageTest.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
     public function testfindlink()
     {
        
-        $page = $this->get("forgotten-password/");  // attempt to access the Lost Forgot Page
+        $page = $this->get("forgotten-password/"); // attempt to access the Lost Forgot Page
         $this->assertEquals(200, $page->getStatusCode(), "a page should load");
     }
 

tests/Extension/UserManagementConfigExtensionTest.php

Indentation +2 added lines, -2 removed lines

@@ -74,8 +74,8 @@
     }
     
     /**
-   * CMS fields test
-   **/
+     * CMS fields test
+     **/
     public function testgetCMSFields()
     {
        

tests/Extension/LoginHandlerExtensionTest.php

Indentation +3 added lines, -3 removed lines

@@ -14,9 +14,9 @@
  */
 class LoginHandlerExtensionTest extends FunctionalTest
 {
-   /**
-   *
-   **/
+    /**
+     *
+     **/
     public function testRedirect()
     {
        

src/Extension/UserManagementConfigExtension.php

Spacing +12 added lines, -12 removed lines

@@ -42,7 +42,7 @@ 
             'Root.UserManagement',
             TreeDropdownField::create(
                 'CustomerGroupID',
-                _t(__CLASS__ . '.CustomerGroup', 'Group to add new customers to'),
+                _t(__CLASS__.'.CustomerGroup', 'Group to add new customers to'),
                 Group::class
             )
         );
@@ -50,7 +50,7 @@ 
             'Root.UserManagement',
             TreeDropdownField::create(
                 'LoginUrlID',
-                _t(__CLASS__ . '.LoginUrl', 'Login Url'),
+                _t(__CLASS__.'.LoginUrl', 'Login Url'),
                 SiteTree::class
             )
         );
@@ -58,7 +58,7 @@ 
             'Root.UserManagement',
             TreeDropdownField::create(
                 'LoginCallBackUrlID',
-                _t(__CLASS__ . '.LoginCallBackUrl', 'Login Call Back Url'),
+                _t(__CLASS__.'.LoginCallBackUrl', 'Login Call Back Url'),
                 SiteTree::class
             )
         );
@@ -66,7 +66,7 @@ 
             'Root.UserManagement',
             TreeDropdownField::create(
                 'LostPasswordUrlID',
-                _t(__CLASS__ . '.LostPasswordUrl', 'Lost Password Url'),
+                _t(__CLASS__.'.LostPasswordUrl', 'Lost Password Url'),
                 SiteTree::class
             )
         );
@@ -74,21 +74,21 @@ 
             'Root.UserManagement',
             TextareaField::create(
                 'ProfileUpdateSuccess',
-                _t(__CLASS__ . '.ProfileUpdateSuccess', 'Profile update Success Message')
+                _t(__CLASS__.'.ProfileUpdateSuccess', 'Profile update Success Message')
             )
         );
         $fields->addFieldToTab(
             'Root.UserManagement',
             TextareaField::create(
                 'ProfileUpdatError',
-                _t(__CLASS__ . '.ProfileUpdatError', 'Profile update Error Message')
+                _t(__CLASS__.'.ProfileUpdatError', 'Profile update Error Message')
             )
         );
         $fields->addFieldToTab(
             'Root.UserManagement',
             CheckboxField::create(
                 'EnableSpamProtection',
-                _t(__CLASS__ . '.EnableSpamProtection', 'Enable Spam Protection')
+                _t(__CLASS__.'.EnableSpamProtection', 'Enable Spam Protection')
             )
         );
         
@@ -113,7 +113,7 @@ 
         if (!$this->owner->LoginUrl()->ID) {
             $LoginUrl = SiteTree::get()
                 ->filter('ClassName', 'UserManagement\Page\UserLoginPage');
-            if($LoginUrl->count() > 0)
+            if ($LoginUrl->count() > 0)
                 return $LoginUrl->first()->ID;
             else
                 return;
@@ -132,7 +132,7 @@ 
         if (!$this->owner->LoginCallBackUrl()->ID) {
             $LoginCallBackUrl = SiteTree::get()
             ->filter('ClassName', 'UserManagement\Page\UserProfilePage');
-            if($LoginCallBackUrl->count() > 0)
+            if ($LoginCallBackUrl->count() > 0)
                 return $LoginCallBackUrl->first()->ID;
             else
                 return;
@@ -150,7 +150,7 @@ 
     {
         if (!$this->owner->LostPasswordUrl()->ID) {
             $LostPasswordUrl = SiteTree::get()->filter('ClassName', 'UserManagement\Page\LostPasswordPage');
-            if($LostPasswordUrl->count()>0)
+            if ($LostPasswordUrl->count() > 0)
                 return $LostPasswordUrl->first()->ID;
             else
                 return;
@@ -169,7 +169,7 @@ 
     {
         if (!$this->owner->CustomerGroup()->ID) {
             $group = Group::get()->filter('Title', 'general');
-            if($group->count() > 0)
+            if ($group->count() > 0)
                 return Group::get()->filter('Title', 'general')->first()->ID;
             else
                 return;
@@ -181,7 +181,7 @@ 
     public function getExportFieldNames()
     {
         $memberFields = Member::create()->getFrontEndFields()->dataFieldNames();
-        $memberFields = array_diff($memberFields, ["FirstName", "Surname", "Email", "TempIDHash", "TempIDExpired", "AutoLoginHash", "AutoLoginExpired","PasswordEncryption","Salt","Locale", "FailedLoginCount", "LockedOutUntil", "Password", "PasswordExpiry"]);
+        $memberFields = array_diff($memberFields, ["FirstName", "Surname", "Email", "TempIDHash", "TempIDExpired", "AutoLoginHash", "AutoLoginExpired", "PasswordEncryption", "Salt", "Locale", "FailedLoginCount", "LockedOutUntil", "Password", "PasswordExpiry"]);
         return array_combine($memberFields, $memberFields);
     }
 }

Braces +20 added lines, -16 removed lines

@@ -113,10 +113,11 @@ 
         if (!$this->owner->LoginUrl()->ID) {
             $LoginUrl = SiteTree::get()
                 ->filter('ClassName', 'UserManagement\Page\UserLoginPage');
-            if($LoginUrl->count() > 0)
-                return $LoginUrl->first()->ID;
-            else
-                return;
+            if($LoginUrl->count() > 0) {
+                            return $LoginUrl->first()->ID;
+            } else {
+                            return;
+            }
         } else {
             return $this->owner->LoginUrl()->ID;
         }
@@ -132,10 +133,11 @@ 
         if (!$this->owner->LoginCallBackUrl()->ID) {
             $LoginCallBackUrl = SiteTree::get()
             ->filter('ClassName', 'UserManagement\Page\UserProfilePage');
-            if($LoginCallBackUrl->count() > 0)
-                return $LoginCallBackUrl->first()->ID;
-            else
-                return;
+            if($LoginCallBackUrl->count() > 0) {
+                            return $LoginCallBackUrl->first()->ID;
+            } else {
+                            return;
+            }
         } else {
             return $this->owner->LoginCallBackUrl()->ID;
         }
@@ -150,10 +152,11 @@ 
     {
         if (!$this->owner->LostPasswordUrl()->ID) {
             $LostPasswordUrl = SiteTree::get()->filter('ClassName', 'UserManagement\Page\LostPasswordPage');
-            if($LostPasswordUrl->count()>0)
-                return $LostPasswordUrl->first()->ID;
-            else
-                return;
+            if($LostPasswordUrl->count()>0) {
+                            return $LostPasswordUrl->first()->ID;
+            } else {
+                            return;
+            }
         } else {
             return $this->owner->LostPasswordUrl()->ID;
         }
@@ -169,10 +172,11 @@ 
     {
         if (!$this->owner->CustomerGroup()->ID) {
             $group = Group::get()->filter('Title', 'general');
-            if($group->count() > 0)
-                return Group::get()->filter('Title', 'general')->first()->ID;
-            else
-                return;
+            if($group->count() > 0) {
+                            return Group::get()->filter('Title', 'general')->first()->ID;
+            } else {
+                            return;
+            }
         } else {
             return $this->owner->CustomerGroup()->ID;
         }