Passed
Push — master ( 877e1b...2b6564 )
by Sathish
12:37
created
src/Extension/UserManagementConfigExtension.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
             'Root.UserManagement',
43 43
             TreeDropdownField::create(
44 44
                 'CustomerGroupID',
45
-                _t(__CLASS__ . '.CustomerGroup', 'Group to add new customers to'),
45
+                _t(__CLASS__.'.CustomerGroup', 'Group to add new customers to'),
46 46
                 Group::class
47 47
             )
48 48
         );
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
             'Root.UserManagement',
51 51
             TreeDropdownField::create(
52 52
                 'LoginUrlID',
53
-                _t(__CLASS__ . '.LoginUrl', 'Login Url'),
53
+                _t(__CLASS__.'.LoginUrl', 'Login Url'),
54 54
                 SiteTree::class
55 55
             )
56 56
         );
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
             'Root.UserManagement',
59 59
             TreeDropdownField::create(
60 60
                 'LoginCallBackUrlID',
61
-                _t(__CLASS__ . '.LoginCallBackUrl', 'Login Call Back Url'),
61
+                _t(__CLASS__.'.LoginCallBackUrl', 'Login Call Back Url'),
62 62
                 SiteTree::class
63 63
             )
64 64
         );
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
             'Root.UserManagement',
67 67
             TreeDropdownField::create(
68 68
                 'LostPasswordUrlID',
69
-                _t(__CLASS__ . '.LostPasswordUrl', 'Lost Password Url'),
69
+                _t(__CLASS__.'.LostPasswordUrl', 'Lost Password Url'),
70 70
                 SiteTree::class
71 71
             )
72 72
         );
@@ -74,21 +74,21 @@  discard block
 block discarded – undo
74 74
             'Root.UserManagement',
75 75
             TextareaField::create(
76 76
                 'ProfileUpdateSuccess',
77
-                _t(__CLASS__ . '.ProfileUpdateSuccess', 'Profile update Success Message')
77
+                _t(__CLASS__.'.ProfileUpdateSuccess', 'Profile update Success Message')
78 78
             )
79 79
         );
80 80
         $fields->addFieldToTab(
81 81
             'Root.UserManagement',
82 82
             TextareaField::create(
83 83
                 'ProfileUpdatError',
84
-                _t(__CLASS__ . '.ProfileUpdatError', 'Profile update Error Message')
84
+                _t(__CLASS__.'.ProfileUpdatError', 'Profile update Error Message')
85 85
             )
86 86
         );
87 87
         $fields->addFieldToTab(
88 88
             'Root.UserManagement',
89 89
             CheckboxField::create(
90 90
                 'EnableSpamProtection',
91
-                _t(__CLASS__ . '.EnableSpamProtection', 'Enable Spam Protection')
91
+                _t(__CLASS__.'.EnableSpamProtection', 'Enable Spam Protection')
92 92
             )
93 93
         );
94 94
         
@@ -185,7 +185,7 @@  discard block
 block discarded – undo
185 185
     public function getExportFieldNames()
186 186
     {
187 187
         $memberFields = Member::create()->getFrontEndFields()->dataFieldNames();
188
-        $memberFields = array_diff($memberFields, ["FirstName", "Surname", "Email", "TempIDHash", "TempIDExpired", "AutoLoginHash", "AutoLoginExpired","PasswordEncryption","Salt","Locale", "FailedLoginCount", "LockedOutUntil", "Password", "PasswordExpiry"]);
188
+        $memberFields = array_diff($memberFields, ["FirstName", "Surname", "Email", "TempIDHash", "TempIDExpired", "AutoLoginHash", "AutoLoginExpired", "PasswordEncryption", "Salt", "Locale", "FailedLoginCount", "LockedOutUntil", "Password", "PasswordExpiry"]);
189 189
         return array_combine($memberFields, $memberFields);
190 190
     }
191 191
 
Please login to merge, or discard this patch.
tests/Pages/UserRegistrationPageTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@
 block discarded – undo
18 18
      */
19 19
     public function testfindlink()
20 20
     {
21
-        $page = $this->get("user-registration/");  // attempt to access the signup Page
21
+        $page = $this->get("user-registration/"); // attempt to access the signup Page
22 22
         $this->assertEquals(200, $page->getStatusCode(), "a page should load");
23 23
         $this->assertEquals(UserRegistrationPage::find_link(false), "/user-registration/", "user-registration page exists");
24 24
     }
Please login to merge, or discard this patch.
tests/Pages/LostPasswordPageTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
     public function testfindlink()
20 20
     {
21 21
        
22
-        $page = $this->get("forgotten-password/");  // attempt to access the Lost Forgot Page
22
+        $page = $this->get("forgotten-password/"); // attempt to access the Lost Forgot Page
23 23
         $this->assertEquals(200, $page->getStatusCode(), "a page should load");
24 24
         $this->assertEquals(LostPasswordPage::find_link(false), "/forgotten-password/", "LostPasswordPage exists");
25 25
     }
Please login to merge, or discard this patch.
tests/Pages/UserProfilePageTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
     {
24 24
         $member = Member::get()->filter("Email", "[email protected]")->first();
25 25
         Injector::inst()->get(IdentityStore::class)->logIn($member);
26
-        $page = $this->get("my-profile/");  // attempt to access the profile Page
26
+        $page = $this->get("my-profile/"); // attempt to access the profile Page
27 27
         $this->assertEquals(200, $page->getStatusCode(), "a page should load");
28 28
         $this->assertEquals(UserProfilePage::find_link(false), "/my-profile/", "My profile page exists");
29 29
     }
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
 
43 43
         $this->assertEquals(
44 44
             1,
45
-            Member::get()->filter("Email", "[email protected]")->count()>0 ? 1 : 0,
45
+            Member::get()->filter("Email", "[email protected]")->count() > 0 ? 1 : 0,
46 46
             'testMyForm() returns the email from profile page.'
47 47
         );
48 48
     }
Please login to merge, or discard this patch.
tests/Pages/UserLoginPageTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@
 block discarded – undo
18 18
      */
19 19
     public function testfindlink()
20 20
     {
21
-        $page = $this->get("user-login/");  // attempt to access the user login Page
21
+        $page = $this->get("user-login/"); // attempt to access the user login Page
22 22
         $this->assertEquals(200, $page->getStatusCode(), "a page should load");
23 23
         $this->assertEquals(UserLoginPage::find_link(false), "/user-login/", "User Login page exists");
24 24
     }
Please login to merge, or discard this patch.
src/Reports/UserReport.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -120,7 +120,7 @@
 block discarded – undo
120 120
     public function FilterByName($filter, $firstName)
121 121
     {
122 122
         if ($firstName) {
123
-            $filter = ($filter)? $filter . " AND FirstName Like '%$firstName%'" : "FirstName Like '%$firstName%'";
123
+            $filter = ($filter) ? $filter." AND FirstName Like '%$firstName%'" : "FirstName Like '%$firstName%'";
124 124
         }
125 125
         return $filter;
126 126
     }
Please login to merge, or discard this patch.
Indentation   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -55,9 +55,9 @@  discard block
 block discarded – undo
55 55
     }
56 56
 
57 57
     /**
58
-    * Returns columns names of the reports
59
-    * @return array
60
-    */
58
+     * Returns columns names of the reports
59
+     * @return array
60
+     */
61 61
     public function columns()
62 62
     {
63 63
         $fields = [
@@ -77,9 +77,9 @@  discard block
 block discarded – undo
77 77
     }
78 78
     
79 79
     /**
80
-    * Return a FieldList of the fields that can be used to filter
81
-    * @return array
82
-    */
80
+     * Return a FieldList of the fields that can be used to filter
81
+     * @return array
82
+     */
83 83
     public function parameterFields()
84 84
     {
85 85
         $member = Security::getCurrentUser() ? Security::getCurrentUser() : Member::create();
@@ -94,11 +94,11 @@  discard block
 block discarded – undo
94 94
     }
95 95
     
96 96
     /**
97
-    * @param string $start
98
-    * @param string $end
99
-    * @param string $firstName
100
-    * @return string | false
101
-    */
97
+     * @param string $start
98
+     * @param string $end
99
+     * @param string $firstName
100
+     * @return string | false
101
+     */
102 102
     public function FilterByDate($start, $end)
103 103
     {
104 104
         $filter = false;
@@ -113,10 +113,10 @@  discard block
 block discarded – undo
113 113
     }
114 114
 
115 115
     /**
116
-    * @param string $firstName
117
-    * @param string $filter
118
-    * @return string | false
119
-    */
116
+     * @param string $firstName
117
+     * @param string $filter
118
+     * @return string | false
119
+     */
120 120
     public function FilterByName($filter, $firstName)
121 121
     {
122 122
         if ($firstName) {
Please login to merge, or discard this patch.
src/Page/UserProfilePageController.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -41,7 +41,7 @@
 block discarded – undo
41 41
     */
42 42
     public function ProfileForm()
43 43
     {
44
-        $form =  new ProfileForm($this, 'ProfileForm');
44
+        $form = new ProfileForm($this, 'ProfileForm');
45 45
         if ($this->member) {
46 46
             $form->loadDataFrom($this->member);
47 47
         }
Please login to merge, or discard this patch.
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -36,9 +36,9 @@
 block discarded – undo
36 36
     }
37 37
     
38 38
     /**
39
-    * Returns the profile form
40
-    * @return \UserManagement\Forms\ProfileForm
41
-    */
39
+     * Returns the profile form
40
+     * @return \UserManagement\Forms\ProfileForm
41
+     */
42 42
     public function ProfileForm()
43 43
     {
44 44
         $form =  new ProfileForm($this, 'ProfileForm');
Please login to merge, or discard this patch.
tests/Extension/LoginHandlerExtensionTest.php 1 patch
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -15,8 +15,8 @@
 block discarded – undo
15 15
 class LoginHandlerExtensionTest extends FunctionalTest
16 16
 {
17 17
     /**
18
-    * Login redirect
19
-    */
18
+     * Login redirect
19
+     */
20 20
     public function testRedirect()
21 21
     {
22 22
         $this->markTestIncomplete('Test user config CMS fields');
Please login to merge, or discard this patch.