Как правильно обработать форму не связанную с Entity?

Question

[Николай Егоров]

Сделал форму не связанную с сущностью. Фактически это простая форма сортировки записей в админке (в дальнейшем добавлю и фильтрацию по параметрам).

class ImageFilterType extends AbstractType
{
    public function buildForm(FormBuilderInterface $builder, array $options)
    {
        $builder
            ->add('sortBy', ChoiceType::class, [
                'choices' => [
                    'ID' => 'id',
                    'label.createdAt' => 'createdAt',
                    'label.publishedAt' => 'publishedAt',
                    'label.updatedAt' => 'updatedAt',
                ],
                'label' => 'label.sort_by',
                'required' => true,
            ])
            ->add('direction', ChoiceType::class, [
                'choices' => [
                    'label.sort_direction_ASC' => 'ASC',
                    'label.sort_direction_DESC' => 'DESC'
                ],
                'label' => 'label.sort_direction',
                'required' => true,
            ])
            ->add('submit', SubmitType::class, [
                'label' => 'OK'
            ]);
    }
}

В контроллере пытаюсь получить переданные формой параметры:

/**
     * @Route("/images/{page}", defaults={"page": 1}, name="admin_image_list")
     */
    public function index(int $page, Request $request, ImageRepository $imageRepository): Response
    {
        $sortBy = $request->get('sortBy', 'id');
        $direction = $request->get('direction', 'DESC');

        $filterForm = $this->createForm(ImageFilterType::class, [
            'sortBy' => $sortBy,
            'direction' => $direction,
        ])->handleRequest($request);

        $images = $imageRepository->findEntities([], $sortBy, $direction, $page, 10);

        return $this->render('admin/gallery/image_list.html.twig', [
            'images' => $images,
            'filterForm' => $filterForm->createView(),
        ]);
    }

Но в итоге я не получаю в контроллере параметров `$sortBy` и `$direction` - они назначаются по дефолту. В дебаг панели видно, что я получаю не отдельные параметры `sortBy` и `direction`, а массив

image_filter [▼
  "sortBy" => "updatedAt"
  "direction" => "DESC"
  "submit" => ""
  "_token" => "dOketsGIJFAVlVaPvjbBtzx5b3ksQ8nZ6jwY-JT7HDo"
]

Если передавать форму GET методом, то адресная строка такая: `/admin/images?image_filter%5BsortBy%5D=updatedAt&image_filter%5Bdirection%5D=DESC&image_filter%5Bsubmit%5D=&image_filter%5B_token%5D=dOketsGIJFAVlVaPvjbBtzx5b3ksQ8nZ6jwY-JT7HDo`

Как правильно обработать такую форму?

Answer 1

[BoShurik]

$filter = $filterForm->getData();
$images = $imageRepository->findEntities([], $filter['sortBy'], $filter['direction'], $page, 10);

Comments

[Николай Егоров]

Спасибо, да, это работает! Но теперь мне нужно получить эти же параметры в шаблоне, чтобы передать их пагинатору, и это не получается:

{{ pagerfanta(images, 'twitter_bootstrap3_translated', {
                                    routeName: 'admin_image_list',
                                    routeParams: {
                                        'sortBy': app.request.attributes.get('sortBy'),
                                        'direction': app.request.attributes.get('direction')
                                    }
                                }) }}

[Николай Егоров]

Я подозреваю, что и в пагинатор нужно передавать эти параметры с именем формы, да? Иначе форма их не подцепит.

[BoShurik]

Николай Егоров, По идеи pagerfanta автоматически подцепляет все get параметры. По крайней мере knp-paginator делает так.
И вы уж определитесь, вы используется данные формы для получения параметров сортировки, либо отдельные атрибуты. Если первый вариант, то код

$sortBy = $request->get('sortBy', 'id');
$direction = $request->get('direction', 'DESC');

вам не нужен

[BoShurik]

По идеи должно работать так:

{{ pagerfanta(images, 'twitter_bootstrap3_translated') }}

[Николай Егоров]

В общем первый вопрос, вопрос топика - решен, поэтому я его отметил решенным. Спасибо!

Но дальше возникли другие вопросы:
- трудности с доступом к параметрам из TWIG,
- параметры запроса все равно не инициализируются по дефолту (пока не нажал кнопку отправки формы, изза этого форма не рендерится правильно)

Поэтому я у формы убрал имя:

public function getBlockPrefix()
    {
        return '';
    }

И теперь пишу так:

// инициализация параметров запроса и запись их в запрос, если не были инициализированы (форма не отправлялась)
        $request->request->set('direction', $request->get('direction', 'DESC'));
        $request->request->set('sortBy', $request->get('sortBy', 'id'));

        // а дальше в ручную задаю значения в форму, даже если она ранее не отрисовывалась и не отправлялась
        $filterForm = $this->createForm(ImageFilterType::class, [
            'sortBy' => $request->get('sortBy'),
            'direction' => $request->get('direction'),
        ])->handleRequest($request);

        $images = $imageRepository->findEntities(
            [],
            $request->get('sortBy'),
            $request->get('direction'),
            $page, 10);

Сложно описать все, наверное и описал криво... и реализовал не красиво. Но работает как надо.

[BoShurik]

class FilterModel
{
    public $orderBy;
    public $direction;

    public function __construct()
    {
        $this->orderBy = 'id';
        $this->direction = 'DESC';
    }
}

class ImageFilterType extends AbstractType
{
    public function buildForm(FormBuilderInterface $builder, array $options)
    {
        $builder
            ->add('sortBy', ChoiceType::class, [
                'choices' => [
                    'ID' => 'id',
                    'label.createdAt' => 'createdAt',
                    'label.publishedAt' => 'publishedAt',
                    'label.updatedAt' => 'updatedAt',
                ],
                'label' => 'label.sort_by',
                'required' => true,
            ])
            ->add('direction', ChoiceType::class, [
                'choices' => [
                    'label.sort_direction_ASC' => 'ASC',
                    'label.sort_direction_DESC' => 'DESC'
                ],
                'label' => 'label.sort_direction',
                'required' => true,
            ])
            ->add('submit', SubmitType::class, [
                'label' => 'OK'
            ])
        ;
    }

    /**
     * @inheritDoc
     */
    public function configureOptions(OptionsResolver $resolver)
    {
        $resolver->setDefaults([
            'data_class' => FilterModel::class,
        ]);
    }
}

class Controller
{
    public function index(int $page, Request $request, ImageRepository $imageRepository): Response
    {
        $filterModel = new FilterModel();
        $filterForm = $this->createForm(ImageFilterType::class, $filterModel)->handleRequest($request);

        $images = $imageRepository->findEntities([], $filterModel->orderBy, $filterModel->direction, $page, 10);

        return $this->render('admin/gallery/image_list.html.twig', [
            'images' => $images,
            'filterForm' => $filterForm->createView(),
            'filterModel' => $filterModel, // Если нужны параметры, то можно передать в твиг
        ]);
    }
}

[Николай Егоров]

Если не убрать имя формы, то имена параметров ... ну не красивые, длинные, с именем формы. А их нужно:
- инициализировать по дефолту, если форма не отправлялась.
- передать в TWIG пагинатор. У меня он сам не подцепил их автоматом. Не знаю почему.

[BoShurik]

Николай Егоров, я бы сделал так.
PS не рекомендуется рендерить кнопки submit с помощью Symfony Form: https://symfony.com/doc/4.0/best_practices/forms.h...

[BoShurik]

Либо, если не хотите создавать отдельный класс, то

$filterForm = $this->createForm(ImageFilterType::class, [
            'sortBy' => 'id',
            'direction' => 'DESC',
        ])->handleRequest($request);
$filter = $filterForm->getData();
$images = $imageRepository->findEntities([], $filter['sortBy'], $filter['direction'], $page, 10);

[Николай Егоров]

Спасибо, что возитесь со мной)) Нуб я пока)

Так, попробую еще то, что вы предложили, с привязкой к форме класса - я так не делал еще ни разу. Похоже класс задает дефолтные значения параметрам. Тогда остается вопрос с пагинатором - ему тоже ведь нужно передать текущие значения.

[BoShurik]

- инициализировать по дефолту, если форма не отправлялась.

Сейчас вы их странно инициализируете. По сути, вторым аргументом передаются как раз такие дефолтные значения, а вы зачем-то их пытаетесь получить из формы. По сути вам тут эта форма и не нужна: вы всю работу делаете за нее :)

Тогда остается вопрос с пагинатором - ему тоже ведь нужно передать текущие значения.

Если я правильно понял код, то, если не передавать routeName и routeParams, то они автоматически подцепятся. Ну либо просто передайте фильтр, как я предложил выше

[Николай Егоров]

... текущие значения пагинатору я хочу передать, чтобы при переходе на другую страницу у элементов формы изначально устанавливались атрибуты selected на текущих options. Это просто объяснение для чего это мне нужно))

[Николай Егоров]

Я пишу с опозданием, поэтому немного невпопад.

[BoShurik]

Перепроверил на базе symfony/demo,

{{ pagerfanta(images, 'twitter_bootstrap3_translated') }}

Вот так вот все query-параметры сохраняются

[Николай Егоров]

Сейчас вы их странно инициализируете. По сути, вторым аргументом передаются как раз такие дефолтные значения

Если вы про это

$filterForm = $this->createForm(ImageFilterType::class, [
            'sortBy' => $request->get('sortBy'),
            'direction' => $request->get('direction'),
        ])->handleRequest($request);

то да - я форму передаю дефолтные значения, которые инициализирую, если их еще нет, в реквест в начале метода. А в реквест записываю, чтобы обращаться к ним и из TWIG'a. Если не записать в реквест, а просто получить:

$direction = $request->get('direction', 'DESC');
        $sortBy = $request->get('sortBy', 'id');

То в TWIG'e их уже не получишь из реквеста, т.к. изначальный роут '/images/{page}' - их там просто нет. Надеюсь понятно хоть чуть объяснил.

[Николай Егоров]

Я попробую завтра! Спасибо огромное за советы! )

[Николай Егоров]

В общем, блин, вы реально крут! Спасибо огромное за помощь!

Да, если привязать форму к классу, передавать форму методом GET, и в шаблоне писать просто {{ pagerfanta(images, 'twitter_bootstrap3_translated') }} то все работает - форма обрабатывается, пагинатор подтягивает и сохраняет в ссылках все query параметры (поэтому ненужно шаманить в шаблоне пытаясь передать их пагинатору) и при переходе по ссылкам пагинатора форма не теряет выбранные опции!

Я узнал много нового для себя!
Здоровья вам и хорошего настроения!))