Skip to content

Commit 1314026

Browse files
committed
Minor: Format code
1 parent 1c5d74a commit 1314026

10 files changed

Lines changed: 201 additions & 176 deletions

File tree

src/CoreBundle/Controller/Api/BaseResourceFileAction.php

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@
3232
use Symfony\Component\HttpKernel\Exception\BadRequestHttpException;
3333
use Symfony\Component\HttpKernel\KernelInterface;
3434
use Symfony\Contracts\Translation\TranslatorInterface;
35+
use Throwable;
3536
use ZipArchive;
3637

3738
class BaseResourceFileAction
@@ -272,6 +273,7 @@ public function handleCreateFileRequest(
272273

273274
if (empty($resourceLinkList)) {
274275
$message = 'resourceLinkList is not a valid json. Use for example: [{"cid":1, "visibility":1}]';
276+
275277
throw new InvalidArgumentException($message);
276278
}
277279
}
@@ -288,6 +290,7 @@ public function handleCreateFileRequest(
288290
$resource->setParentResourceNode($parentResourceNodeId);
289291

290292
$uploadedFile = null;
293+
291294
switch ($fileType) {
292295
case 'certificate':
293296
case 'file':
@@ -468,6 +471,7 @@ protected function handleCreateFileRequestUncompress(
468471

469472
if (empty($resourceLinkList) || !\is_array($resourceLinkList)) {
470473
$message = 'resourceLinkList is not a valid json. Use for example: [{"cid":1, "visibility":1}]';
474+
471475
throw new InvalidArgumentException($message);
472476
}
473477
}
@@ -949,11 +953,8 @@ private function assertQuotaForRequest(
949953
foreach ($courses as $course) {
950954
try {
951955
$courseHelper->assertCanStoreDocumentBytes($course, $deltaBytes);
952-
} catch (\Throwable $e) {
953-
throw new BadRequestHttpException(\sprintf(
954-
'Not enough space in course #%d.',
955-
(int) $course->getId()
956-
));
956+
} catch (Throwable $e) {
957+
throw new BadRequestHttpException(\sprintf('Not enough space in course #%d.', (int) $course->getId()));
957958
}
958959
}
959960
}

src/CoreBundle/Controller/Api/ReplaceDocumentFileAction.php

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
use Symfony\Component\HttpFoundation\Response;
2121
use Symfony\Component\HttpKernel\Exception\BadRequestHttpException;
2222
use Symfony\Component\HttpKernel\KernelInterface;
23+
use Throwable;
2324

2425
class ReplaceDocumentFileAction extends BaseResourceFileAction
2526
{
@@ -71,11 +72,8 @@ public function __invoke(
7172
foreach ($courses as $course) {
7273
try {
7374
$courseHelper->assertCanStoreDocumentBytes($course, $deltaBytes);
74-
} catch (\Throwable $e) {
75-
throw new BadRequestHttpException(\sprintf(
76-
'Not enough space in course #%d.',
77-
(int) $course->getId()
78-
));
75+
} catch (Throwable $e) {
76+
throw new BadRequestHttpException(\sprintf('Not enough space in course #%d.', (int) $course->getId()));
7977
}
8078
}
8179
}

src/CoreBundle/Controller/CourseController.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
use Chamilo\CoreBundle\Helpers\CourseStudentInfoHelper;
2323
use Chamilo\CoreBundle\Helpers\UserHelper;
2424
use Chamilo\CoreBundle\Repository\AssetRepository;
25-
use Chamilo\CoreBundle\Repository\CourseCategoryRepository;
2625
use Chamilo\CoreBundle\Repository\ExtraFieldValuesRepository;
2726
use Chamilo\CoreBundle\Repository\LanguageRepository;
2827
use Chamilo\CoreBundle\Repository\LegalRepository;
@@ -73,6 +72,7 @@
7372
use Symfony\Component\Serializer\SerializerInterface;
7473
use Symfony\Component\Validator\Exception\ValidatorException;
7574
use Symfony\Contracts\Translation\TranslatorInterface;
75+
use Throwable;
7676
use UserManager;
7777

7878
/**
@@ -1193,14 +1193,14 @@ public function studentInfoBatchJson(
11931193
$sessionId = (int) ($payload['sid'] ?? 0);
11941194

11951195
$courseIds = $payload['courseIds'] ?? [];
1196-
if (!is_array($courseIds)) {
1196+
if (!\is_array($courseIds)) {
11971197
$courseIds = [];
11981198
}
11991199

12001200
$courseIds = array_values(array_unique(array_map('intval', $courseIds)));
12011201
$courseIds = array_filter($courseIds, static fn (int $id) => $id > 0);
1202-
if (count($courseIds) > 300) {
1203-
$courseIds = array_slice($courseIds, 0, 300);
1202+
if (\count($courseIds) > 300) {
1203+
$courseIds = \array_slice($courseIds, 0, 300);
12041204
}
12051205

12061206
if (empty($courseIds)) {
@@ -1232,7 +1232,7 @@ public function studentInfoBatchJson(
12321232
if (0 === $sessionId) {
12331233
try {
12341234
$this->denyAccessUnlessGranted(CourseVoter::VIEW, $course);
1235-
} catch (\Throwable $e) {
1235+
} catch (Throwable $e) {
12361236
// Skip unauthorized courses without failing the whole batch.
12371237
continue;
12381238
}

src/CoreBundle/Entity/Course.php

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,9 @@
3636
use Symfony\Component\Serializer\Annotation\SerializedName;
3737
use Symfony\Component\Validator\Constraints as Assert;
3838

39+
use const SORT_FLAG_CASE;
40+
use const SORT_NATURAL;
41+
3942
#[ApiResource(
4043
types: ['https://schema.org/Course'],
4144
operations: [
@@ -1266,19 +1269,19 @@ public function getCategoryTitles(): array
12661269
continue;
12671270
}
12681271

1269-
$title = \method_exists($category, 'getTitle')
1272+
$title = method_exists($category, 'getTitle')
12701273
? (string) $category->getTitle()
12711274
: (string) $category;
12721275

1273-
$title = \trim(\strip_tags($title));
1276+
$title = trim(strip_tags($title));
12741277

12751278
if ('' !== $title) {
12761279
$titles[] = $title;
12771280
}
12781281
}
12791282

1280-
$titles = \array_values(\array_unique($titles));
1281-
\sort($titles, \SORT_NATURAL | \SORT_FLAG_CASE);
1283+
$titles = array_values(array_unique($titles));
1284+
sort($titles, SORT_NATURAL | SORT_FLAG_CASE);
12821285

12831286
return $titles;
12841287
}

src/CoreBundle/Helpers/CourseHelper.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1179,7 +1179,7 @@ private function getCourseDocumentUsedBytes(Course $course): int
11791179
* - "500" -> 500 MB
11801180
* - "200M", "200MB" -> 200 MB
11811181
* - "1G", "1GB" -> 1024 MB
1182-
* - large integers that look like bytes -> converted to MB
1182+
* - large integers that look like bytes -> converted to MB.
11831183
*/
11841184
private function parseQuotaRawToMb(string $raw): int
11851185
{

0 commit comments

Comments
 (0)