Witam,

Jak zwalidować w akcji kontrolera dane wyslane z formularza utworzonego manualnie ?

Moj formularz w TWIG:

[HTML] pobierz, plaintext 
<form action="{{ path('typerka_types') }}" method="post">
    <div class="coupon_form">
         {% for key,match in matches %}
            <input type="hidden" name="match_id[]" value="{{ match.match_id }}">
            <div class="coupon_form_host">
                {{ match.host }}
            </div>
            <div class="coupon_form_host_type">
               <input type="text" name="hostType[]">
            </div>
            <div class="coupon_form_guest_type">
                <input type="text" name="guestType[]">
            </div> 
         {% endfor %}
   </div>
    <input type="submit" value="Submit">
</form>
[HTML] pobierz, plaintext 

Kontroler:

[PHP] pobierz, plaintext 
    public function typesAction(Request $request){
 
        $repository = $this->getDoctrine()->getRepository('MyTyperkaBundle:Match');
        $matches = $repository->getMatchesPerMatchday(1);
 
        if ($request->getMethod() == 'POST') { 
            $req = $request->request->all();
                // ... 
                $type = new Type();
                $match = new Match();
                // .... 
                $em->persist($match);
                $em->persist($type);
                $em->flush();
            }
 
            return $this->redirect($this->generateUrl('typerka_types'));
        }
        return array('matches' => $matches);
[PHP] pobierz, plaintext 

Normalnie robi sie to tak :

[PHP] pobierz, plaintext 
        $form->handleRequest($Request);
        if($form->isValid()){
[PHP] pobierz, plaintext 

ale u siebie mnie nie tworze forma w kontrolerze.

jeżeli chcesz zwalidować dane, to uzyj po prostu walidatora na wynikowym (przesłanym) obiekcie. walidator to osobny komponent symfony i może działać bez formularza. musiałbyś jednak dorobić i powielić sporo istniejącej już funkcjonalności m.in. do spopulowania obiektu danymi wejściowymi. Osobiście dla kazdych danych wejściowych (nie ważne, czy pochodzących od uzytkownika, czy automatycznego klienta) staram się tworzyć formularz - czytaj to jako funkcjonalność, a nie rezultat wizualny (formularz można, ale nie trzeba wyrenderować "do widoku"). na Twoim miejscu więc wysiliłbym się i stworzył klase formularza, nawet jeżeli jego pola sa tworzone dynamicznie, to z pewnościa jest jakiś algorytm jego tworzenia następnie skorzystał z $form->isValid() jak biały człowiek

ehh ... no to mnie zmartwiłeś ...

Chciałem to zrobić jak "biały człowiek" jednak miałem problem z uzyskaniem tego co potrzebowałem zrobić, a wiec:
- pobrać dane w RepositoryClass
- wyswietlić je w Twig
- dodać do tych danych pola do wypełnienia
- zapisać dane z wypełnionych pol w bazie

do tego co mi sie wydaje, potrzebujesz połączyć dwie rzeczy: embedded collection oraz dynamical forms. pierwsze pozwoli Ci stworzyć podformularze dla każdej encji z kolekcji (u Ciebie chyba matches), a eventy do dynamicznego tworzenia pól jeżeli masz tam odrobinę więcej logiki potrzebnej

ok chyba nie bede mial wyjscia i powinienem to zrobic jak "bialy czlowiek" ... i rozumiem ze najlepiej zawsze isc ta droga w Symfony ....

Podpowiedz mi tylko czy powinienem tego forma i embeded collection robic dla encji Type czy Match ?
PS. Encja Type to ta ktorej dane bede dodawal do bazy, natomiast encja Match to ta, ktorej dane chce tylko wyswietlic w Twigu i nic wiecej z nimi nie robic.

W takiej relacji jest encja Type z encja Match:

[PHP] pobierz, plaintext 
    /**
     * @ORM\ManyToOne(
     *      targetEntity = "Match",
     * )
     * 
     * @ORM\JoinColumn(
     *      name = "match_id",
     *      referencedColumnName = "id",
     *      onDelete = "SET NULL"
     * )
     */
    private $match;
[PHP] pobierz, plaintext 

Przepraszam że się wpie... między wódkę a zakonskę ale zawsze mnie mnie dziwi dlaczego w Symfony/Zend to musi być takie skomplikowane. Każda pierdoła która powinna być zrobiona na maksa prosto bez udziwnień musi być zrealizowana przy pomocy pierdylardu klas, metod, tricków, hacków

to nie jest kwestia symfony, tylko zasad dobrego programowania. spaghetti możesz tworzyć w kazdym fw, w symfony również

@damianooo:

stwórz formularz osobiście do sprawy podchodze tak, że:

- moje encje nie maja setterów, parametry podaję do konstruktora tylko przy utworzeniu, wymusza to tworzenie metod z logiką, które sprawiają, że encja nie jest "anemiczna"
- tworzę sobie klase formularza opartą o klasę DTO
- tworzę sobie prostą klasę DTO, którą wpinam w formularz; zawiera ona publiczne pola np. kolekcje itd (wiem, ze DTO nie powinno zawierac encji, tylko id's ale w moich przypadkach jest to pomijalne - nie serializuję dto i nie pcham po api)
- tworzę serwis implementujący konkretną ścieżkę działania (przypadek użycia), zapodaje jej wszystkie potrzebne zależności i w jego metodzie handle zawieram logikę - utworzenie, obrobienie i zapis do bazy. oczywiście TDD, więc mam od razu otestowany kod; na koniec działania rzucam event, że działanie wykonane lub exception
- tworzę sobie factory dla encji (jedna dla match i druga dla type), dzięki temu mam w systemie jedno centralne miejsce, gdzie encje są tworzone i nigdzie indziej nie ma czegoś takiego jak new Match itp; robie w niej metodę createEntityFromCommand, która DTO konwertuje do encji
- w kontrolerze: tworzę formularz, wciskam w niego DTO, populuje DTO requestem, robię walidację DTO a nie encji (trzymam jej zasady w validation.yml), jesli valid to na utworzonym serwisie robię po prostu ->handle($command), przekazuje formularz do widoku i np. ustawiam flash dla usera.

voila. wszystko odizolowane i otestowane, a ten sam kod moge wykorzystać na stronie, w api czy commandzie bez różnicy.

[PHP] pobierz, plaintext 
    /**
     * @Route("/rejestracja.html", name="evento_register")
     * @Template()
     */
    public function registerAction(Request $request)
    {
        $response = new Response(null, 200);
        $createCommand = new CreateWorkshop();
        $createCommand->lessons = [new CreateLesson()];
        $form = $this->createForm("register_workshops_form_type", $createCommand, [
            "action" => $this->generateUrl("evento_register"),
            "method" => "POST",
        ]);
        if($request->isMethod("POST"))
        {
            $form->handleRequest($request);
            if($form->isValid())
            {
                $this->get("tactician.commandbus")->handle($form->getData());
                $this->addFlash("success", "Dziękujemy za dodanie nowej informacji, po weryfikacji przez moderatora pojawi się w bazie");
                return $this->redirectToRoute("evento_homepage");
            } else {
                $response->setStatusCode(400);
            }
        }
        return $this->render("AppBundle:Default:register.html.twig", [
            "form" => $form->createView(),
        ], $response);
    }
[PHP] pobierz, plaintext 

Jak chcesz sobie podejrzeć, to mam taką tu małą appkę, nie zastosowałem wydzielenia klas domeny i aplikacji ani TDD ze względu na wielkość, ale to chyba identyczny przypadek jak Twoj - formularz z jedną kolekcją

https://github.com/uirapuru/evento/tree/master/src/AppBundle

to co piszesz to troche wyzsza szkola jazdy wiec pewnie tym razem niestety nie skorzystam bo niestety nie umiem jeszcze tyle ...
bede chcial jednak to zrobic oczywiscie tak jak zasugerowales a wiec z wykorzystniem standardowego formularza ...
i na pewno uproscic jak sie da ... mam nadzieje ze w moim problemie, ktory mam do rozwiazania to mi sie uda ...
podziałam i podziele sie rozwiazaniem tutaj na forum ...

nie mam ambicji bycia nauczycielem, ale myślę, że kazdy moment na naukę dobry, tymbardziej, że jeśli dobrze rozumiem dałem Ci przykład ze swojego kodu 1:1 z Twoim przypadkiem mam nadzieję, że pomogłem.

hmm ... mowisz ze mamy podobienstwo 1:1 ... moze ... nie widze tego na razie

Zrobilem tak w kontrolerze:

[PHP] pobierz, plaintext 
  public function typesAction(Request $request){
 
        $form = $this->createForm(new TypeType(), new Type(), array(
            'action' => $this->generateUrl('typerka_types'),
            'method' => 'POST',
        ));
        $form->add('submit', 'submit');
        $form->handleRequest($request);
        if ($request->getMethod() == 'POST') {
            if ($form->isValid()) {}
        }
        // ...
[PHP] pobierz, plaintext 

Formularz encji Type:

[PHP] pobierz, plaintext 
    public function buildForm(FormBuilderInterface $builder, array $options)
    {
        $builder
            ->add('hostType' )
            ->add('guestType' )
        ;
    }
[PHP] pobierz, plaintext 

niestety potrzebuje jeszcze pobrac dane z encji Match z Repository class:

[PHP] pobierz, plaintext 
    public function getMatchesPerMatchday($matchday){
        $qb = $this->createQueryBuilder('m');
        $qb->select(
                     'm.id AS match_id'
                    ,'tm1.name AS host'
                    ,'tm2.name AS guest'
                    ,'m.description'
                    ,'m.term'
                )
           ->innerJoin('m.hostTeam', 'tm1')
           ->innerJoin('m.guestTeam', 'tm2')
           ->innerJoin('m.matchday', 'md')     
           ->where('md.id = :matchday')
           ->orderBy('m.id')
           ->setParameter('matchday', $matchday)
        ;
        // ...   
    }
[PHP] pobierz, plaintext 

nie wiem jak to wszystko powiazac ...