@@ -89,7 +89,7 @@ |
||
89 | 89 | /** |
90 | 90 | * Get current user permissions for a board |
91 | 91 | * |
92 | - * @param Board|Entity $board |
|
92 | + * @param Board $board |
|
93 | 93 | * @return array|bool |
94 | 94 | * @internal param $boardId |
95 | 95 | */ |
@@ -92,7 +92,7 @@ discard block |
||
92 | 92 | |
93 | 93 | /** |
94 | 94 | * @param $id |
95 | - * @return RelationalEntity if not found |
|
95 | + * @return Entity if not found |
|
96 | 96 | * @throws \OCP\AppFramework\Db\MultipleObjectsReturnedException |
97 | 97 | * @throws \OCP\AppFramework\Db\DoesNotExistException |
98 | 98 | */ |
@@ -136,6 +136,9 @@ discard block |
||
136 | 136 | return parent::delete($entity); |
137 | 137 | } |
138 | 138 | |
139 | + /** |
|
140 | + * @param integer $stackId |
|
141 | + */ |
|
139 | 142 | public function deleteByStack($stackId) { |
140 | 143 | $cards = $this->findAllByStack($stackId); |
141 | 144 | foreach ($cards as $card) { |
@@ -63,7 +63,7 @@ |
||
63 | 63 | * Check if user exists before assigning it to a card |
64 | 64 | * |
65 | 65 | * @param Entity $entity |
66 | - * @return null|Entity |
|
66 | + * @return AssignedUsers|null |
|
67 | 67 | */ |
68 | 68 | public function insert(Entity $entity) { |
69 | 69 | $user = $this->userManager->get($entity->getParticipant()); |
@@ -163,7 +163,7 @@ |
||
163 | 163 | * @param $cardId |
164 | 164 | * @param $attachmentId |
165 | 165 | * @param $request |
166 | - * @return mixed |
|
166 | + * @return \OCP\AppFramework\Db\Entity |
|
167 | 167 | * @throws \OCA\Deck\NoPermissionException |
168 | 168 | * @throws \OCP\AppFramework\Db\DoesNotExistException |
169 | 169 | * @throws \OCP\AppFramework\Db\MultipleObjectsReturnedException |
@@ -30,8 +30,6 @@ |
||
30 | 30 | use OCA\Deck\Db\Stack; |
31 | 31 | use OCA\Deck\Db\StackMapper; |
32 | 32 | use OCA\Deck\StatusException; |
33 | -use OCP\ICache; |
|
34 | -use OCP\ICacheFactory; |
|
35 | 33 | |
36 | 34 | |
37 | 35 | class StackService { |
@@ -80,7 +80,7 @@ discard block |
||
80 | 80 | |
81 | 81 | $entities = []; |
82 | 82 | $cursor = $qb->execute(); |
83 | - while($row = $cursor->fetch()){ |
|
83 | + while ($row = $cursor->fetch()) { |
|
84 | 84 | $entities[] = $this->mapRowToEntity($row); |
85 | 85 | } |
86 | 86 | $cursor->closeCursor(); |
@@ -105,7 +105,7 @@ discard block |
||
105 | 105 | |
106 | 106 | $entities = []; |
107 | 107 | $cursor = $qb->execute(); |
108 | - while($row = $cursor->fetch()){ |
|
108 | + while ($row = $cursor->fetch()) { |
|
109 | 109 | $entities[] = $this->mapRowToEntity($row); |
110 | 110 | } |
111 | 111 | $cursor->closeCursor(); |
@@ -99,16 +99,16 @@ |
||
99 | 99 | $data = []; |
100 | 100 | foreach ($boards as $board) { |
101 | 101 | $fullBoard = $this->boardMapper->find($board->getId(), true, true); |
102 | - $data[$board->getId()] = (array)$fullBoard->jsonSerialize(); |
|
102 | + $data[$board->getId()] = (array) $fullBoard->jsonSerialize(); |
|
103 | 103 | $stacks = $this->stackMapper->findAll($board->getId()); |
104 | 104 | foreach ($stacks as $stack) { |
105 | - $data[$board->getId()]['stacks'][] = (array)$stack->jsonSerialize(); |
|
105 | + $data[$board->getId()]['stacks'][] = (array) $stack->jsonSerialize(); |
|
106 | 106 | $cards = $this->cardMapper->findAllByStack($stack->getId()); |
107 | 107 | foreach ($cards as $card) { |
108 | 108 | $fullCard = $this->cardMapper->find($card->getId()); |
109 | 109 | $assignedUsers = $this->assignedUsersMapper->find($card->getId()); |
110 | 110 | $fullCard->setAssignedUsers($assignedUsers); |
111 | - $data[$board->getId()]['stacks'][$stack->getId()]['cards'][] = (array)$fullCard->jsonSerialize(); |
|
111 | + $data[$board->getId()]['stacks'][$stack->getId()]['cards'][] = (array) $fullCard->jsonSerialize(); |
|
112 | 112 | } |
113 | 113 | } |
114 | 114 | } |
@@ -23,15 +23,11 @@ |
||
23 | 23 | |
24 | 24 | namespace OCA\Deck\Service; |
25 | 25 | |
26 | -use OC\Security\CSP\ContentSecurityPolicyManager; |
|
27 | 26 | use OCA\Deck\Db\Attachment; |
28 | 27 | use OCA\Deck\StatusException; |
29 | 28 | use OCP\AppFramework\Http\ContentSecurityPolicy; |
30 | -use OCP\AppFramework\Http\EmptyContentSecurityPolicy; |
|
31 | 29 | use OCP\AppFramework\Http\FileDisplayResponse; |
32 | 30 | use OCP\AppFramework\Http\StreamResponse; |
33 | -use OCP\Files\Cache\IScanner; |
|
34 | -use OCP\Files\Folder; |
|
35 | 31 | use OCP\Files\IAppData; |
36 | 32 | use OCP\Files\IRootFolder; |
37 | 33 | use OCP\Files\NotFoundException; |
@@ -85,7 +85,7 @@ discard block |
||
85 | 85 | * @throws NotPermittedException |
86 | 86 | */ |
87 | 87 | private function getFolder(Attachment $attachment) { |
88 | - $folderName = 'file-card-' . (int)$attachment->getCardId(); |
|
88 | + $folderName = 'file-card-' . (int) $attachment->getCardId(); |
|
89 | 89 | try { |
90 | 90 | $folder = $this->appData->getFolder($folderName); |
91 | 91 | } catch (NotFoundException $e) { |
@@ -116,7 +116,7 @@ discard block |
||
116 | 116 | * @return array |
117 | 117 | * @throws StatusException |
118 | 118 | */ |
119 | - private function getUploadedFile () { |
|
119 | + private function getUploadedFile() { |
|
120 | 120 | $file = $this->request->getUploadedFile('file'); |
121 | 121 | $error = null; |
122 | 122 | $phpFileUploadErrors = [ |
@@ -190,7 +190,7 @@ discard block |
||
190 | 190 | * @throws \Exception |
191 | 191 | */ |
192 | 192 | private function getFileFromRootFolder(Attachment $attachment) { |
193 | - $folderName = 'file-card-' . (int)$attachment->getCardId(); |
|
193 | + $folderName = 'file-card-' . (int) $attachment->getCardId(); |
|
194 | 194 | $instanceId = $this->config->getSystemValue('instanceid', null); |
195 | 195 | if ($instanceId === null) { |
196 | 196 | throw new \Exception('no instance id!'); |
@@ -59,7 +59,7 @@ |
||
59 | 59 | public function index() { |
60 | 60 | $params = [ |
61 | 61 | 'user' => $this->userId, |
62 | - 'maxUploadSize' => (int)\OCP\Util::uploadLimit(), |
|
62 | + 'maxUploadSize' => (int) \OCP\Util::uploadLimit(), |
|
63 | 63 | ]; |
64 | 64 | |
65 | 65 | if ($this->defaultBoardService->checkFirstRun($this->userId, $this->appName)) { |