@@ -100,7 +100,9 @@ discard block |
||
100 | 100 | * @return Entity |
101 | 101 | */ |
102 | 102 | public function createACLEntry(SharingACL $acl) { |
103 | - if ($acl->getCreated() === null) $acl->setCreated((new \DateTime())->getTimestamp()); |
|
103 | + if ($acl->getCreated() === null) { |
|
104 | + $acl->setCreated((new \DateTime())->getTimestamp()); |
|
105 | + } |
|
104 | 106 | return $this->sharingACL->createACLEntry($acl); |
105 | 107 | } |
106 | 108 | |
@@ -158,13 +160,17 @@ discard block |
||
158 | 160 | $return = []; |
159 | 161 | foreach ($entries as $entry) { |
160 | 162 | // Check if the user can read the credential, probably unnecesary, but just to be sure |
161 | - if (!$entry->hasPermission(SharingACL::READ)) continue; |
|
163 | + if (!$entry->hasPermission(SharingACL::READ)) { |
|
164 | + continue; |
|
165 | + } |
|
162 | 166 | $tmp = $entry->jsonSerialize(); |
163 | 167 | $credential = $this->credential->getCredentialById($entry->getItemId()); |
164 | 168 | $credential = $this->encryptService->decryptCredential($credential); |
165 | 169 | $tmp['credential_data'] = $credential->jsonSerialize(); |
166 | 170 | |
167 | - if (!$entry->hasPermission(SharingACL::FILES)) unset($tmp['credential_data']['files']); |
|
171 | + if (!$entry->hasPermission(SharingACL::FILES)) { |
|
172 | + unset($tmp['credential_data']['files']); |
|
173 | + } |
|
168 | 174 | unset($tmp['credential_data']['shared_key']); |
169 | 175 | $return[] = $tmp; |
170 | 176 | } |
@@ -195,7 +201,9 @@ discard block |
||
195 | 201 | $acl = $this->sharingACL->getItemACL($user_id, $item_guid); |
196 | 202 | |
197 | 203 | // Check if the user can read the credential, probably unnecesary, but just to be sure |
198 | - if (!$acl->hasPermission(SharingACL::READ)) throw new DoesNotExistException("Item not found or wrong access level"); |
|
204 | + if (!$acl->hasPermission(SharingACL::READ)) { |
|
205 | + throw new DoesNotExistException("Item not found or wrong access level"); |
|
206 | + } |
|
199 | 207 | |
200 | 208 | $tmp = $acl->jsonSerialize(); |
201 | 209 | $credential = $this->credential->getCredentialById($acl->getItemId()); |
@@ -203,7 +211,9 @@ discard block |
||
203 | 211 | |
204 | 212 | $tmp['credential_data'] = $credential->jsonSerialize(); |
205 | 213 | |
206 | - if (!$acl->hasPermission(SharingACL::FILES)) unset($tmp['credential_data']['files']); |
|
214 | + if (!$acl->hasPermission(SharingACL::FILES)) { |
|
215 | + unset($tmp['credential_data']['files']); |
|
216 | + } |
|
207 | 217 | unset($tmp['credential_data']['shared_key']); |
208 | 218 | |
209 | 219 | return $tmp; |
@@ -221,7 +231,9 @@ discard block |
||
221 | 231 | */ |
222 | 232 | public function getItemHistory(?string $user_id, string $item_guid) { |
223 | 233 | $acl = $this->sharingACL->getItemACL($user_id, $item_guid); |
224 | - if (!$acl->hasPermission(SharingACL::READ | SharingACL::HISTORY)) return []; |
|
234 | + if (!$acl->hasPermission(SharingACL::READ | SharingACL::HISTORY)) { |
|
235 | + return []; |
|
236 | + } |
|
225 | 237 | |
226 | 238 | return $this->revisions->getRevisions($acl->getItemId()); |
227 | 239 | } |