Your project must not use PHP super globals
- Read doc
- Productivity
- Critical
More information: https://insight.symfony.com/what-we-analyse/symfony.use_super_globals
- public function __invoke(SendContactEmail $message): void
- {
- $email = (new Email())
- ->from($message->getEmail())
- ->to($_ENV['EMAIL_TO'])
Last edited by Luigi Gandemer- ->subject("Portfolio contact from {$message->getName()}")
- ->html("
- <h2>New contact from your portfolio</h2>
- <p><strong>Name:</strong> {$message->getName()}</p>
- <p><strong>Email:</strong> {$message->getEmail()}</p>
Your project files must be encoded in UTF-8
- Read doc
- Reputation
- Critical
More information: https://insight.symfony.com/what-we-analyse/web.non_utf8_encoding
Last edited by Luigi Gandemer
Your project must provide a favicon in its public directory
- Read doc
- Reputation
- Critical
More information: https://insight.symfony.com/what-we-analyse/web.missing_favicon
Last edited by Luigi Gandemer
-
public
- index.php
Your project error pages must be customised 2
- Read doc
- Reputation
- Critical
More information: https://insight.symfony.com/what-we-analyse/symfony.configuration.error_pages_should_be_customised
Your project must not contain invalid function or method calls 10
- Read doc
- Reliability
- Major
More information: https://insight.symfony.com/what-we-analyse/php.invalid_call
- public function __invoke(SendContactEmail $message): void
- {
- $email = (new Email())
- ->from($message->getEmail())
- ->to($_ENV['EMAIL_TO'])
Last edited by Luigi Gandemer- ->subject("Portfolio contact from {$message->getName()}")
- ->html("
- <h2>New contact from your portfolio</h2>
- <p><strong>Name:</strong> {$message->getName()}</p>
- <p><strong>Email:</strong> {$message->getEmail()}</p>
- $this->mailer->send($email);
- $contact = $this->contactRepository->find($message->getContactId());
- if ($contact) {
- $contact->setStatus('sent');
Last edited by Luigi Gandemer- $this->entityManager->flush();
- }
- }
- }
- $username = $input->getArgument('username');
- $password = $input->getArgument('password');
- $user = new User();
- $user->setUsername($username);
Last edited by Luigi Gandemer- $user->setRoles(['ROLE_ADMIN']);
- $user->setPassword(
- $this->passwordHasher->hashPassword($user, $password)
- );
- $user = new User();
- $user->setUsername($username);
- $user->setRoles(['ROLE_ADMIN']);
- $user->setPassword(
- $this->passwordHasher->hashPassword($user, $password)
Last edited by Luigi Gandemer- );
- $this->entityManager->persist($user);
- $this->entityManager->flush();
- return;
- }
- $contact = $this->contactRepository->find($message->getContactId());
- if ($contact) {
- $contact->setStatus('failed');
Last edited by Luigi Gandemer- $this->entityManager->flush();
- }
- }
- }
- private readonly MessageBusInterface $bus,
- ) {}
- public function process(mixed $data, Operation $operation, array $uriVariables = [], array $context = []): Contact
- {
- $this->entityManager->persist($data);
Last edited by Luigi Gandemer- $this->entityManager->flush();
- $this->bus->dispatch(new SendContactEmail(
- $data->getName(),
- $data->getEmail(),
- {
- $this->entityManager->persist($data);
- $this->entityManager->flush();
- $this->bus->dispatch(new SendContactEmail(
- $data->getName(),
Last edited by Luigi Gandemer- $data->getEmail(),
- $data->getMessage(),
- $data->getId(),
- ));
- $this->entityManager->persist($data);
- $this->entityManager->flush();
- $this->bus->dispatch(new SendContactEmail(
- $data->getName(),
- $data->getEmail(),
Last edited by Luigi Gandemer- $data->getMessage(),
- $data->getId(),
- ));
- return $data;
- $this->entityManager->flush();
- $this->bus->dispatch(new SendContactEmail(
- $data->getName(),
- $data->getEmail(),
- $data->getMessage(),
Last edited by Luigi Gandemer- $data->getId(),
- ));
- return $data;
- }
- $this->bus->dispatch(new SendContactEmail(
- $data->getName(),
- $data->getEmail(),
- $data->getMessage(),
- $data->getId(),
Last edited by Luigi Gandemer- ));
- return $data;
- }
- }
Your project must not contain invalid instantiations 5
- Read doc
- Reliability
- Major
More information: https://insight.symfony.com/what-we-analyse/php.invalid_instantiation
- {
- $this->entityManager->persist($data);
- $this->entityManager->flush();
- $this->bus->dispatch(new SendContactEmail(
- $data->getName(),
Last edited by Luigi Gandemer- $data->getEmail(),
- $data->getMessage(),
- $data->getId(),
- ));
- $this->entityManager->persist($data);
- $this->entityManager->flush();
- $this->bus->dispatch(new SendContactEmail(
- $data->getName(),
- $data->getEmail(),
Last edited by Luigi Gandemer- $data->getMessage(),
- $data->getId(),
- ));
- return $data;
- $this->entityManager->flush();
- $this->bus->dispatch(new SendContactEmail(
- $data->getName(),
- $data->getEmail(),
- $data->getMessage(),
Last edited by Luigi Gandemer- $data->getId(),
- ));
- return $data;
- }
- $this->bus->dispatch(new SendContactEmail(
- $data->getName(),
- $data->getEmail(),
- $data->getMessage(),
- $data->getId(),
Last edited by Luigi Gandemer- ));
- return $data;
- }
- }
- use App\Kernel;
- require_once dirname(__DIR__).'/vendor/autoload_runtime.php';
- return function (array $context) {
- return new Kernel($context['APP_ENV'], (bool) $context['APP_DEBUG']);
Last edited by Luigi Gandemer- };
Your project should not use invalid return types 5
- Read doc
- Reliability
- Major
More information: https://insight.symfony.com/what-we-analyse/php.invalid_return_typehint
- $data->getEmail(),
- $data->getMessage(),
- $data->getId(),
- ));
- return $data;
Last edited by Luigi Gandemer- }
- }
- public function getRoles(): array
- {
- $roles = $this->roles;
- $roles[] = 'ROLE_USER';
- return array_unique($roles);
Last edited by Luigi Gandemer- }
- public function setRoles(array $roles): static
- {
- $this->roles = $roles;
- return $this;
- }
- public function getUserIdentifier(): string
- {
- return (string) $this->username;
Last edited by Luigi Gandemer- }
- public function eraseCredentials(): void {}
- }
- parent::__construct($registry, Experience::class);
- }
- public function findAllWithTasks(): array
- {
- return $this->createQueryBuilder('e')
Last edited by Luigi Gandemer- ->addSelect('t')
- ->leftJoin('e.tasks', 't')
- ->orderBy('e.startedAt', 'DESC')
- ->getQuery()
- ->getResult();
- parent::__construct($registry, Project::class);
- }
- public function findAllWithTechnologies(): array
- {
- return $this->createQueryBuilder('p')
Last edited by Luigi Gandemer- ->addSelect('t', 'i')
- ->leftJoin('p.technologies', 't')
- ->leftJoin('p.image', 'i')
- ->orderBy('p.id', 'DESC')
- ->getQuery()
Your project should not use invalid parameter and return typehints
- Read doc
- Reliability
- Major
More information: https://insight.symfony.com/what-we-analyse/php.invalid_typehint
- $roles = $this->roles;
- $roles[] = 'ROLE_USER';
- return array_unique($roles);
- }
- public function setRoles(array $roles): static
Last edited by Luigi Gandemer- {
- $this->roles = $roles;
- return $this;
- }
Your project should use return types 4
- Read doc
- Reliability
- Major
More information: https://insight.symfony.com/what-we-analyse/php.missing_return_typehint
- public function __construct(ManagerRegistry $registry)
- {
- parent::__construct($registry, Experience::class);
- }
- public function findAllWithTasks(): array
Last edited by Luigi Gandemer- {
- return $this->createQueryBuilder('e')
- ->addSelect('t')
- ->leftJoin('e.tasks', 't')
- ->orderBy('e.startedAt', 'DESC')
- public function __construct(ManagerRegistry $registry)
- {
- parent::__construct($registry, Project::class);
- }
- public function findAllWithTechnologies(): array
Last edited by Luigi Gandemer- {
- return $this->createQueryBuilder('p')
- ->addSelect('t', 'i')
- ->leftJoin('p.technologies', 't')
- ->leftJoin('p.image', 'i')
- {
- $this->companyName = $companyName;
- return $this;
- }
- public function getTasks(): Collection { return $this->tasks; }
Last edited by Luigi Gandemer- public function addTask(Task $task): static
- {
- if (!$this->tasks->contains($task)) {
- $this->tasks->add($task);
- {
- $this->image = $image;
- return $this;
- }
- public function getTechnologies(): Collection { return $this->technologies; }
Last edited by Luigi Gandemer- public function addTechnology(Technology $technology): static
- {
- if (!$this->technologies->contains($technology)) {
- $this->technologies->add($technology);
Your project should contain a robots.txt file
- Read doc
- Reputation
- Major
More information: https://insight.symfony.com/what-we-analyse/web.missing_robots_txt
Last edited by Luigi Gandemer
-
public
- index.php
Your project session configuration could expose your users data
- Read doc
- Legal
- Data leak
- Major
More information: https://insight.symfony.com/what-we-analyse/symfony.request.session_cookie_default_name
Web applications should contain a site.webmanifest file
- Read doc
- Reputation
- Minor
More information: https://insight.symfony.com/what-we-analyse/web.missing_web_manifest
Last edited by Luigi Gandemer
-
public
- index.php
Text files should end with a valid new line character. 34
- Read doc
- Productivity
- Info
More information: https://insight.symfony.com/what-we-analyse/missing_e_o_l
- if ($contact) {
- $contact->setStatus('sent');
- $this->entityManager->flush();
- }
- }
- }
Last edited by Luigi Gandemer
- | - - - - - - - - - - - - - - - - - - - - - | : - - - - - - - - - - - - - - - - - - - - - - - - - : |
- | A P I | h t t p : / / l o c a l h o s t : 8 0 8 0 |
- | L i t e S p e e d A d m i n | h t t p : / / l o c a l h o s t : 7 0 8 0 |
- | R a b b i t M Q U I | h t t p : / / l o c a l h o s t : 1 5 6 7 2 |
- | M a i l h o g U I | h t t p : / / l o c a l h o s t : 8 0 2 5 |
- | p h p M y A d m i n | h t t p : / / l o c a l h o s t : 8 0 8 1 |
Last edited by Luigi Gandemer
- $io->success("Admin user '{$username}' created successfully.");
- return Command::SUCCESS;
- }
- }
Last edited by Luigi Gandemer
- if ($contact) {
- $contact->setStatus('failed');
- $this->entityManager->flush();
- }
- }
- }
Last edited by Luigi Gandemer
- $data->getId(),
- ));
- return $data;
- }
- }
Last edited by Luigi Gandemer
- {
- return (string) $this->username;
- }
- public function eraseCredentials(): void {}
- }
Last edited by Luigi Gandemer
- ->leftJoin('e.tasks', 't')
- ->orderBy('e.startedAt', 'DESC')
- ->getQuery()
- ->getResult();
- }
- }
Last edited by Luigi Gandemer
- ->leftJoin('p.image', 'i')
- ->orderBy('p.id', 'DESC')
- ->getQuery()
- ->getResult();
- }
- }
Last edited by Luigi Gandemer
- $task->setExperience(null);
- }
- }
- return $this;
- }
- }
Last edited by Luigi Gandemer
- public function removeTechnology(Technology $technology): static
- {
- $this->technologies->removeElement($technology);
- return $this;
- }
- }
Last edited by Luigi Gandemer
- enable_re_doc: false
- enable_scalar: true
- mapping:
- paths:
- - '%kernel.project_dir%/src/Entity'
- - '%kernel.project_dir%/src/Controller'
Last edited by Luigi Gandemer
- failed:
- dsn: 'doctrine://default?queue_name=failed'
- routing:
- App\Message\SendContactEmail: async
Last edited by Luigi Gandemer
- paths:
- '^/api/':
- allow_origin: ['%env(CORS_ALLOW_ORIGIN)%']
- allow_headers: ['Content-Type', 'Authorization']
- allow_methods: ['GET', 'POST', 'PUT', 'PATCH', 'DELETE', 'OPTIONS']
- max_age: 3600
Last edited by Luigi Gandemer
- access_control:
- - { path: ^/api/login, roles: PUBLIC_ACCESS }
- - { path: ^/api/portfolio, roles: PUBLIC_ACCESS }
- - { path: ^/api/contact, roles: PUBLIC_ACCESS }
- - { path: ^/api/docs, roles: PUBLIC_ACCESS }
- - { path: ^/api, roles: ROLE_ADMIN }
Last edited by Luigi Gandemer
- controllers:
- resource: routing.controllers
- api_login:
- path: /api/login
- methods: [POST]
Last edited by Luigi Gandemer
- resource: '../src/'
- App\OpenApi\PortfolioOpenApiDecorator:
- decorates: 'api_platform.openapi.factory'
- arguments:
- $decorated: '@.inner'
Last edited by Luigi Gandemer
- PMA_PASSWORD: ${MYSQL_PASSWORD}
- depends_on:
- - db
- volumes:
- db_data:
Last edited by Luigi Gandemer
- null,
- ['groups' => ['technology:read']]
- ),
- ]);
- }
- }
Last edited by Luigi Gandemer
- public function setStatus(string $status): static
- {
- $this->status = $status;
- return $this;
- }
- }
Last edited by Luigi Gandemer
- public function setImage(Image $image): static
- {
- $this->image = $image;
- return $this;
- }
- }
Last edited by Luigi Gandemer
- $this->filename = $filename;
- return $this;
- }
- public function getUploadedAt(): DateTimeImmutable { return $this->uploadedAt; }
- }
Last edited by Luigi Gandemer
- public function setDescription(string $description): static
- {
- $this->description = $description;
- return $this;
- }
- }
Last edited by Luigi Gandemer
- public function setExperience(?Experience $experience): static
- {
- $this->experience = $experience;
- return $this;
- }
- }
Last edited by Luigi Gandemer
- public function setImage(Image $image): static
- {
- $this->image = $image;
- return $this;
- }
- }
Last edited by Luigi Gandemer
- public function getName(): string { return $this->name; }
- public function getEmail(): string { return $this->email; }
- public function getMessage(): string { return $this->message; }
- public function getContactId(): int { return $this->contactId; }
- }
Last edited by Luigi Gandemer
- )
- ));
- return $openApi;
- }
- }
Last edited by Luigi Gandemer
- {
- public function __construct(ManagerRegistry $registry)
- {
- parent::__construct($registry, Contact::class);
- }
- }
Last edited by Luigi Gandemer
- {
- public function __construct(ManagerRegistry $registry)
- {
- parent::__construct($registry, Degree::class);
- }
- }
Last edited by Luigi Gandemer
- {
- public function __construct(ManagerRegistry $registry)
- {
- parent::__construct($registry, Image::class);
- }
- }
Last edited by Luigi Gandemer
- {
- public function __construct(ManagerRegistry $registry)
- {
- parent::__construct($registry, Skill::class);
- }
- }
Last edited by Luigi Gandemer
- {
- public function __construct(ManagerRegistry $registry)
- {
- parent::__construct($registry, Task::class);
- }
- }
Last edited by Luigi Gandemer
- {
- public function __construct(ManagerRegistry $registry)
- {
- parent::__construct($registry, Technology::class);
- }
- }
Last edited by Luigi Gandemer
- }
- $user->setPassword($newHashedPassword);
- $this->getEntityManager()->persist($user);
- $this->getEntityManager()->flush();
- }
- }
Last edited by Luigi Gandemer
- self::EDIT => in_array('ROLE_ADMIN', $user->getRoles()),
- self::DELETE => in_array('ROLE_ADMIN', $user->getRoles()),
- default => false,
- };
- }
- }
Last edited by Luigi Gandemer
Your project should not contain PHP files defining multiple classes
- Read doc
- Productivity
- Info
More information: https://insight.symfony.com/what-we-analyse/php.file_contains_more_than_one_class
Last edited by Luigi Gandemer