Course: Add custom image support for course links in homepage tools - refs #2863 #6039
christianbeeznest wants to merge
christianbeeznest:GH-2863 into chamilo:masterFailed
36 issues to fix.
Showing 37 of 37 total issues
Missing function doc comment New
New
public function __invoke(CLink $link, CShortcutRepository $shortcutRepository, AssetRepository $assetRepository): Response- Exclude checks
Add a single space around assignment operators New
New
declare(strict_types=1);- Exclude checks
Missing class doc comment New
New
class CLinkImageController- Exclude checks
Missing function doc comment New
New
public function __construct(EntityManagerInterface $entityManager)- Exclude checks
Missing function doc comment New
New
public function __invoke(CLink $link, Request $request): Response- Exclude checks
Missing blank line before return statement New
New
return new Response('Error creating cropped image', Response::HTTP_INTERNAL_SERVER_ERROR);- Exclude checks
Blank line found at end of control structure New
New
- Exclude checks
Only one argument is allowed per line in a multi-line function call New
New
0, 0,- Exclude checks
Only one argument is allowed per line in a multi-line function call New
New
$srcX, $srcY,- Exclude checks
Only one argument is allowed per line in a multi-line function call New
New
$cropWidth, $cropHeight,- Exclude checks
Only one argument is allowed per line in a multi-line function call New
New
120, 120- Exclude checks
Add a single space around assignment operators New
New
declare(strict_types=1);- Exclude checks
Missing class doc comment New
New
final class Version20250118000100 extends AbstractMigrationChamilo- Exclude checks
Missing function doc comment New
New
public function getDescription(): string- Exclude checks
Missing function doc comment New
New
public function up(Schema $schema): void- Exclude checks
Missing function doc comment New
New
public function down(Schema $schema): void- Exclude checks
Missing function doc comment New
New
public function getCustomImageUrl(): ?string- Exclude checks
Missing function doc comment New
New
public function setCustomImageUrl(?string $customImageUrl): self- Exclude checks
Perl-style comments are not allowed. Use "// Comment." or "/* comment */" instead. New
New
#[Groups(['cshortcut:read'])]- Exclude checks
Perl-style comments are not allowed. Use "// Comment." or "/* comment */" instead. New
New
#[ORM\ManyToOne(targetEntity: Asset::class, cascade: ['remove'])]- Exclude checks
Perl-style comments are not allowed. Use "// Comment." or "/* comment */" instead. New
New
#[ORM\JoinColumn(name: 'custom_image_id', referencedColumnName: 'id', onDelete: 'SET NULL')]- Exclude checks
Line indented incorrectly; expected 8 spaces, found 4 New
New
private ?Asset $customImage = null;- Exclude checks
Line indented incorrectly; expected 8 spaces, found 4 New
New
public function getCustomImage(): ?Asset- Exclude checks
Missing function doc comment New
New
public function getCustomImage(): ?Asset- Exclude checks
Line indented incorrectly; expected at least 12 spaces, found 8 New
New
return $this->customImage;- Exclude checks
Line indented incorrectly; expected 8 spaces, found 4 New
New
public function setCustomImage(?Asset $customImage): self- Exclude checks
Missing function doc comment New
New
public function setCustomImage(?Asset $customImage): self- Exclude checks
Line indented incorrectly; expected at least 12 spaces, found 8 New
New
$this->customImage = $customImage;- Exclude checks
Missing blank line before return statement New
New
return $this;- Exclude checks
The method __invoke uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them. New
New
} else { $details['customImageUrl'] = null; }- Read upRead up
- Exclude checks
Remove error control operator '@' on line 61. New
New
public function __invoke(CLink $link, Request $request): Response { $removeImage = $request->request->getBoolean('removeImage', false); $file = $request->files->get('customImage'); - Read upRead up
- Exclude checks
Missing class import via use statement (line '85', column '23'). New
New
throw new \RuntimeException('Invalid image file');- Read upRead up
- Exclude checks
Missing class import via use statement (line '99', column '27'). New
New
throw new \RuntimeException('Unsupported image type');- Read upRead up
- Exclude checks
The method indexJson uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them. New
New
} else { $shortcut->setCustomImageUrl(null); }- Read upRead up
- Exclude checks
Avoid unused parameters such as '$schema'. New
New
public function up(Schema $schema): void- Read upRead up
- Exclude checks
Avoid unused parameters such as '$schema'. New
New
public function down(Schema $schema): void- Read upRead up
- Exclude checks
Missing function doc comment Fixed
- Exclude checks