Skip to content

Commit 097fe08

Browse files
committed
Reformat the method checkCredentials method for a single return
1 parent acc5353 commit 097fe08

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

setup/src/Magento/Setup/Model/PackagesAuth.php

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,7 @@ public function getCredentialBaseUrl()
9999
*/
100100
public function checkCredentials($token, $secretKey)
101101
{
102+
$response = ['success' => true];
102103
$serviceUrl = $this->getPackagesJsonUrl();
103104
$this->curlClient->setCredentials($token, $secretKey);
104105
try {
@@ -107,13 +108,13 @@ public function checkCredentials($token, $secretKey)
107108
$packagesInfo = $this->curlClient->getBody();
108109
$directory = $this->filesystem->getDirectoryWrite(DirectoryList::COMPOSER_HOME);
109110
$directory->writeFile(self::PATH_TO_PACKAGES_FILE, $packagesInfo);
110-
return $this->serializer->serialize(['success' => true]);
111111
} else {
112-
return $this->serializer->serialize(['success' => false, 'message' => 'Bad credentials']);
112+
$response = ['success' => false, 'message' => 'Bad credentials'];
113113
}
114114
} catch (\Exception $e) {
115-
return $this->serializer->serialize(['success' => false, 'message' => $e->getMessage()]);
115+
$response = ['success' => false, 'message' => $e->getMessage()];
116116
}
117+
return $this->serializer->serialize($response);
117118
}
118119

119120
/**

0 commit comments

Comments
 (0)