Size: a a a

2021 September 24

МФ

Максим Федоров... in symfony
Только dql? Без логики?
источник

A

Anthony in symfony
а чего, репо отменили чтоли? Зачем оно в контроллере?
Просто идеологически, это не работа контроллера: знать как конкретно добывать данные.
Контроллер знает только какие сервисы дернуть и куда вернуть ответ
источник

МФ

Максим Федоров... in symfony
Что за сервис? И зачем?
источник

R

Roman in symfony
public function view(int $proof_id, EntityManagerInterface $em): Response
   {
       $dql = "SELECT p as proof, SUM(s.vote) AS stat
                   FROM App:Proof p                                          
                   LEFT JOIN App:Statistic s WITH p.id = s.proof
                   WHERE p.id = $proof_id";

       $result = $em->createQuery($dql)->getOneOrNullResult();

       $proof = $result['proof'];

       if (!$proof) {
           throw new NotFoundHttpException(self::PROOF_NO_FOUND);
       }

       if (!$proof->getIsPublic() && $proof->getUser() != $this->getUser()) {
           throw new AccessDeniedException("Error access to proof $proof_id");
       }

       return $this->render('Logic/proof_view.html.twig', [
           'proof' => $proof,
           'vote_value' => $result['stat'] ?: 0,
       ]);
   }
источник

A

Anthony in symfony
источник

КГ

Константин Грачев... in symfony
источник

✨Basic_Instinct✨ in symfony
источник

A

Anthony in symfony
AccessDeniedException
это вот вообще до контроллера дойти не должно было
источник

КГ

Константин Грачев... in symfony
Внимание говнокод, уберите детей от экранов
источник

A

Anthony in symfony
за такое надо уголовно карать
источник

КГ

Константин Грачев... in symfony
Кстати это исключение разве не 500 должно вернуть при выбрасывании из контроллера. Чтобы 403 был там должен быть *HttpException
источник

A

Anthony in symfony
ну зависит от того, что это вообще за эксепшн
источник

СВ

Сергей Вершинин... in symfony
👍
источник

SF

Sanan Fatullazade in symfony
вот это я конечно идиот
источник

КГ

Константин Грачев... in symfony
кстати именно из-за sql оно сейчас поломано.. 😢
источник

SF

Sanan Fatullazade in symfony
у меня просто были две папки с одинаковым названием в разных местах и я просто не туда смотрел
источник

SF

Sanan Fatullazade in symfony
аахахахахах
источник

SF

Sanan Fatullazade in symfony
фейспалм
источник

МФ

Максим Федоров... in symfony
ну да, я увидел метку у папок и потому сказал
там шторм роутит раздельно
источник

КГ

Константин Грачев... in symfony
О важности нейминга..
источник