View GitHub Repository waca

Schedule Inspection
Subscribe
Completed
Push — newinternal-releasecandidate ( 410e59...fe35c3 )
by Simon
17s queued 14s
created
Status
Category
Toggle Patches
includes/Security/CredentialProviders/PasswordCredentialProvider.php 1 patch
Braces   +3 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -26,8 +26,7 @@  discard block
 
 block discarded – undo
26 26 
     public function authenticate(User $user, $data)
27 27 
     {
28 28 
         $storedData = $this->getCredentialData($user->getId());
29 
-        if($storedData === null)
30 
-        {
29 
+        if($storedData === null) {
31 30 
             // No available credential matching these parameters
32 31 
             return false;
33 32 
         }
 
@@ -38,7 +37,7 @@  discard block
 
 block discarded – undo
38 37 
         }
39 38
40 39 
         if(password_verify($data, $storedData->getData())) {
41 
-            if(password_needs_rehash($storedData->getData(), self::PASSWORD_ALGO, array('cost' => self::PASSWORD_COST))){
40 
+            if(password_needs_rehash($storedData->getData(), self::PASSWORD_ALGO, array('cost' => self::PASSWORD_COST))) {
42 41 
                 $this->setCredential($user, $storedData->getFactor(), $data);
43 42 
             }
44 43 
 
@@ -52,7 +51,7 @@  discard block
 
 block discarded – undo
52 51 
     {
53 52 
         $storedData = $this->getCredentialData($user->getId());
54 53
55 
-        if($storedData === null){
54 
+        if($storedData === null) {
56 55 
             $storedData = $this->createNewCredential($user);
57 56 
         }
58 57
Please login to merge, or discard this patch.