[SF][SF2][Symfony2] Walidacja danych wysłanych z niestandardowego forma., Manualne tworzenie formularza i jego obsługa Opcje

[damianooo]

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.

[uirapuru]

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

Ten post edytował uirapuru 25.03.2016, 09:16:19

[damianooo]

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

Ten post edytował damianooo 25.03.2016, 09:36:26

[uirapuru]

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

[damianooo]

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 

[Pyton_000]

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

[uirapuru]

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

Ten post edytował uirapuru 25.03.2016, 10:32:30

[damianooo]

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 ...

[uirapuru]

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.

[damianooo]

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 ...