Passed
Push — master ( 3c5ecd...a595a7 )
by Andre
04:04
created
src/services/Providers.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
     public function loginUrl($handle, $options = [])
73 73
     {
74 74
         $provider = $this->getProviderByHandle($handle);
75
-        if (is_null($provider)){
75
+        if (is_null($provider)) {
76 76
             throw new NotSupportedException('Provider not found or disabled: '.$handle);
77 77
         }
78 78
 
@@ -139,12 +139,12 @@  discard block
 block discarded – undo
139 139
         ]];
140 140
 
141 141
         foreach ($fields as $field) {
142
-            if (!$this->validateFieldClass($field)){
142
+            if (!$this->validateFieldClass($field)) {
143 143
                 continue;
144 144
             }
145 145
 
146 146
             $option = [
147
-                'label' => $field->name. ' ('.$field->handle.')',
147
+                'label' => $field->name.' ('.$field->handle.')',
148 148
                 'value' => $field->handle
149 149
             ];
150 150
 
@@ -167,7 +167,7 @@  discard block
 block discarded – undo
167 167
           Dropdown::class => 1
168 168
         ];
169 169
 
170
-        if (isset($supportedClasses[$fieldClass])){
170
+        if (isset($supportedClasses[$fieldClass])) {
171 171
             return true;
172 172
         }
173 173
 
@@ -381,7 +381,7 @@  discard block
 block discarded – undo
381 381
     {
382 382
         $providers = $this->getAllProviderTypes();
383 383
 
384
-        if ($excludeCreated){
384
+        if ($excludeCreated) {
385 385
             $providers = $this->getExcludeCreatedProviders();
386 386
         }
387 387
 
@@ -576,11 +576,11 @@  discard block
 block discarded – undo
576 576
         $userProfile = $adapter->getUserProfile();
577 577
         $user = Craft::$app->getUser()->getIdentity();
578 578
 
579
-        if (!$user){
579
+        if (!$user) {
580 580
             $user = $this->retrieveUser($userProfile, $provider);
581 581
         }
582 582
 
583
-        if (!$user){
583
+        if (!$user) {
584 584
             Craft::error("Not user to login", __METHOD__);
585 585
             return false;
586 586
         }
@@ -607,7 +607,7 @@  discard block
 block discarded – undo
607 607
      */
608 608
     private function retrieveUser(Profile $userProfile, Provider $provider): User
609 609
     {
610
-        if (is_null($userProfile->email)){
610
+        if (is_null($userProfile->email)) {
611 611
             throw new \Exception("Email address is not provided, please check the settings of your application");
612 612
         }
613 613
 
@@ -618,7 +618,7 @@  discard block
 block discarded – undo
618 618
         }
619 619
         $settings = Socializer::$app->settings->getSettings();
620 620
 
621
-        if (!$settings->enableUserSignUp){
621
+        if (!$settings->enableUserSignUp) {
622 622
             return null;
623 623
         }
624 624
 
@@ -632,14 +632,14 @@  discard block
 block discarded – undo
632 632
         // validate populate
633 633
         $user = $this->populateUserModel($user, $provider, $userProfile);
634 634
 
635
-        if (!Craft::$app->elements->saveElement($user)){
635
+        if (!Craft::$app->elements->saveElement($user)) {
636 636
             Craft::error("Unable to create user: ".json_encode($user->getErrors()));
637 637
             throw new \Exception("Something went wrong while creating the user");
638 638
         }
639 639
 
640
-        if ($settings->userGroupId){
640
+        if ($settings->userGroupId) {
641 641
             $userGroup = Craft::$app->getUserGroups()->getGroupById($settings->userGroupId);
642
-            if ($userGroup){
642
+            if ($userGroup) {
643 643
                 Craft::$app->getUsers()->assignUserToGroups($user->id, [$userGroup->id]);
644 644
             }
645 645
         }
@@ -668,10 +668,10 @@  discard block
 block discarded – undo
668 668
         }
669 669
 
670 670
         foreach ($fieldMapping as $item) {
671
-            if(isset($item['targetUserField']) && $item['targetUserField']){
671
+            if (isset($item['targetUserField']) && $item['targetUserField']) {
672 672
                 $profileValue = $profile->{$item['sourceFormField']};
673 673
                 $field = $user->getFieldLayout()->getFieldByHandle($item['targetUserField']);
674
-                if ($field){
674
+                if ($field) {
675 675
                     $user->setFieldValue($item['targetUserField'], $profileValue);
676 676
                 }
677 677
             }
@@ -694,7 +694,7 @@  discard block
 block discarded – undo
694 694
             $providerRecord = ProviderRecord::findOne($provider->id);
695 695
 
696 696
             if (!$providerRecord) {
697
-                throw new Exception(Craft::t("enupal-socializer",'No Provider exists with the ID “{id}”', ['id' => $provider->id]));
697
+                throw new Exception(Craft::t("enupal-socializer", 'No Provider exists with the ID “{id}”', ['id' => $provider->id]));
698 698
             }
699 699
         }
700 700
 
Please login to merge, or discard this patch.