diff options
author | Morris Jobke <hey@morrisjobke.de> | 2018-02-13 21:55:24 +0100 |
---|---|---|
committer | Morris Jobke <hey@morrisjobke.de> | 2018-02-13 21:55:24 +0100 |
commit | e2974f1133693c022585d3758df142e0f385cb3e (patch) | |
tree | f8d1110d68580b8307e78e1844964b2df486d071 | |
parent | 3fc6d6234e39914d8e6e7d4b9ffe0b5420d3aa2d (diff) | |
download | nextcloud-server-e2974f1133693c022585d3758df142e0f385cb3e.tar.gz nextcloud-server-e2974f1133693c022585d3758df142e0f385cb3e.zip |
Simplify return statement
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
-rw-r--r-- | apps/dav/lib/CardDAV/SyncService.php | 4 | ||||
-rw-r--r-- | apps/dav/lib/Connector/Sabre/Auth.php | 3 | ||||
-rw-r--r-- | apps/dav/lib/Upload/UploadHome.php | 3 | ||||
-rw-r--r-- | apps/encryption/lib/Crypto/Crypt.php | 7 | ||||
-rw-r--r-- | apps/files_sharing/lib/Helper.php | 4 | ||||
-rw-r--r-- | apps/user_ldap/lib/Access.php | 9 | ||||
-rw-r--r-- | apps/user_ldap/lib/Group_LDAP.php | 8 | ||||
-rw-r--r-- | apps/user_ldap/lib/Jobs/CleanUp.php | 4 | ||||
-rw-r--r-- | lib/private/Files/Storage/Wrapper/Encryption.php | 4 |
9 files changed, 13 insertions, 33 deletions
diff --git a/apps/dav/lib/CardDAV/SyncService.php b/apps/dav/lib/CardDAV/SyncService.php index 4a5e45ecb80..ee68a4a26ec 100644 --- a/apps/dav/lib/CardDAV/SyncService.php +++ b/apps/dav/lib/CardDAV/SyncService.php @@ -236,9 +236,7 @@ class SyncService { $root->appendChild($sync); $root->appendChild($prop); $dom->appendChild($root); - $body = $dom->saveXML(); - - return $body; + return $dom->saveXML(); } /** diff --git a/apps/dav/lib/Connector/Sabre/Auth.php b/apps/dav/lib/Connector/Sabre/Auth.php index 456bb17138a..fcd1b34edbc 100644 --- a/apps/dav/lib/Connector/Sabre/Auth.php +++ b/apps/dav/lib/Connector/Sabre/Auth.php @@ -152,8 +152,7 @@ class Auth extends AbstractBasic { */ function check(RequestInterface $request, ResponseInterface $response) { try { - $result = $this->auth($request, $response); - return $result; + return $this->auth($request, $response); } catch (NotAuthenticated $e) { throw $e; } catch (Exception $e) { diff --git a/apps/dav/lib/Upload/UploadHome.php b/apps/dav/lib/Upload/UploadHome.php index ec31e93f4f9..6976ece9943 100644 --- a/apps/dav/lib/Upload/UploadHome.php +++ b/apps/dav/lib/Upload/UploadHome.php @@ -90,7 +90,6 @@ class UploadHome implements ICollection { } $view = new View('/' . $user->getUID() . '/uploads'); $rootInfo = $view->getFileInfo(''); - $impl = new Directory($view, $rootInfo); - return $impl; + return new Directory($view, $rootInfo); } } diff --git a/apps/encryption/lib/Crypto/Crypt.php b/apps/encryption/lib/Crypto/Crypt.php index 4d5c20ea75f..49b1bcb41aa 100644 --- a/apps/encryption/lib/Crypto/Crypt.php +++ b/apps/encryption/lib/Crypto/Crypt.php @@ -196,9 +196,7 @@ class Crypt { // combine content to encrypt the IV identifier and actual IV $catFile = $this->concatIV($encryptedContent, $iv); $catFile = $this->concatSig($catFile, $sig); - $padded = $this->addPadding($catFile); - - return $padded; + return $this->addPadding($catFile); } /** @@ -495,8 +493,7 @@ class Crypt { */ private function createSignature($data, $passPhrase) { $passPhrase = hash('sha512', $passPhrase . 'a', true); - $signature = hash_hmac('sha256', $data, $passPhrase); - return $signature; + return hash_hmac('sha256', $data, $passPhrase); } diff --git a/apps/files_sharing/lib/Helper.php b/apps/files_sharing/lib/Helper.php index 684c20c4544..559386149ed 100644 --- a/apps/files_sharing/lib/Helper.php +++ b/apps/files_sharing/lib/Helper.php @@ -210,9 +210,7 @@ class Helper { } $sliced = array_slice($split, 2); - $relPath = implode('/', $sliced); - - return $relPath; + return implode('/', $sliced); } /** diff --git a/apps/user_ldap/lib/Access.php b/apps/user_ldap/lib/Access.php index 3d182077764..c02cc968637 100644 --- a/apps/user_ldap/lib/Access.php +++ b/apps/user_ldap/lib/Access.php @@ -419,8 +419,7 @@ class Access extends LDAPUtility implements IUserTools { $domainParts[] = $part; } } - $domainDN = implode(',', $domainParts); - return $domainDN; + return implode(',', $domainParts); } /** @@ -800,8 +799,7 @@ class Access extends LDAPUtility implements IUserTools { public function fetchUsersByLoginName($loginName, $attributes = array('dn')) { $loginName = $this->escapeFilterPart($loginName); $filter = str_replace('%uid', $loginName, $this->connection->ldapLoginFilter); - $users = $this->fetchListOfUsers($filter, $attributes); - return $users; + return $this->fetchListOfUsers($filter, $attributes); } /** @@ -814,8 +812,7 @@ class Access extends LDAPUtility implements IUserTools { public function countUsersByLoginName($loginName) { $loginName = $this->escapeFilterPart($loginName); $filter = str_replace('%uid', $loginName, $this->connection->ldapLoginFilter); - $users = $this->countUsers($filter); - return $users; + return $this->countUsers($filter); } /** diff --git a/apps/user_ldap/lib/Group_LDAP.php b/apps/user_ldap/lib/Group_LDAP.php index e1e6a78c44b..6cb56941463 100644 --- a/apps/user_ldap/lib/Group_LDAP.php +++ b/apps/user_ldap/lib/Group_LDAP.php @@ -370,9 +370,7 @@ class Group_LDAP extends BackendUtility implements \OCP\GroupInterface, IGroupLD } $filterParts[] = $this->access->connection->ldapGidNumber .'=' . $groupID; - $filter = $this->access->combineFilterWithAnd($filterParts); - - return $filter; + return $this->access->combineFilterWithAnd($filterParts); } /** @@ -534,9 +532,7 @@ class Group_LDAP extends BackendUtility implements \OCP\GroupInterface, IGroupLD } $filterParts[] = 'primaryGroupID=' . $groupID; - $filter = $this->access->combineFilterWithAnd($filterParts); - - return $filter; + return $this->access->combineFilterWithAnd($filterParts); } /** diff --git a/apps/user_ldap/lib/Jobs/CleanUp.php b/apps/user_ldap/lib/Jobs/CleanUp.php index df7d888a902..f905256c696 100644 --- a/apps/user_ldap/lib/Jobs/CleanUp.php +++ b/apps/user_ldap/lib/Jobs/CleanUp.php @@ -172,9 +172,7 @@ class CleanUp extends TimedJob { return false; } - $enabled = $this->isCleanUpEnabled(); - - return $enabled; + return $this->isCleanUpEnabled(); } /** diff --git a/lib/private/Files/Storage/Wrapper/Encryption.php b/lib/private/Files/Storage/Wrapper/Encryption.php index e40a4830953..b65db054055 100644 --- a/lib/private/Files/Storage/Wrapper/Encryption.php +++ b/lib/private/Files/Storage/Wrapper/Encryption.php @@ -351,9 +351,7 @@ class Encryption extends Wrapper { // need to stream copy file by file in case we copy between a encrypted // and a unencrypted storage $this->unlink($path2); - $result = $this->copyFromStorage($this, $path1, $path2); - - return $result; + return $this->copyFromStorage($this, $path1, $path2); } /** |