[Symfony][SF2][Symfony2]Dodanie walidacji do formularza nie używającego encji Opcje

[Crash89]

Chcę tutaj porównać jedną daną pobraną w przesyłanym formularzu z drugą.
Wyczytałem gdzieś że do rozwiązania tego problemu, przy tworzeniu formularza FormType należy wykorzystać obiekt Callback, konstrukcja wygląda mniej więcej tak:

[PHP] pobierz, plaintext 
use Symfony\Component\Validator\ExecutionContextInterface;
use Symfony\Component\Validator\Constraints\Callback;
 
 
    public function buildForm(FormBuilderInterface $builder, array $options)
{
$builder->add('pole','type',array(
'mapped' => false,
'constraints' => array(
new Callback(array($this,'validate')
))));
// i treść funkcji validate
 
public function validate(ExecutionContextInterface $context){
       if($data['pole1'] >= $data->['pole2']){
           $context->addViolation("Tekst błędu");
       }
}
 
[PHP] pobierz, plaintext 

Problem w tym że nie wiem gdzie mam umieścić funckję validate.
Umieszczałem ją w FormType i nie działa, także próbowałem w kontrolerze, ale tam też jakoś na to nie reaguje.
Przy formularzu wykorzystującym entity, umieszczam ją w entity i wszystko działa, ale tutaj nie mam takiej klasy.



Więc, gdzie umieścić tą funkcję?

[Crozin]

A skąd wziąłeś sobie niby zmienną $data w metodzie validate()? Masz tam dostęp do obiektu ExecutionContextInterface i przez niego powinieneś jak już dobrać się do właściwości poszczególnych pól.

[Crash89]

$data powinienem brać z formularza:

[PHP] pobierz, plaintext 
$data = $form->getData();
[PHP] pobierz, plaintext 

Wygląda na to że miałbym umieścić tą funkcję w kontrolerze, ale wydaje mi sie że to nie jest dobra praktyka.
Mam stworzyć oddzielną klase walidacji dla tego przypadku?
np. class validatePole?

Ten post edytował Crash89 16.07.2015, 13:48:52

[Crozin]

Komponent od walidacji danych nie ma na dobrą sprawę zbyt dużego pojęcia na temat czegoś takiego jak "formularz". Pod $context->getValue() zostanie zwrócona Ci wartość, która jest akutualnie poddawana sprawdzaniu. Możesz w tym miejscu dokonać porównania dwóch właściwości i w przypadku błędu za pomocą $context->buildViolation() dodać błąd.

[blahy]

Co to znaczy nie dziala?

Nie dziala bo zgodnie z dokumentacja: http://symfony.com/doc/current/reference/c...callback-option
callback dostaje:

Concrete callbacks receive an ExecutionContextInterface instance as only argument.
Static or closure callbacks receive the validated object as the first argument and the ExecutionContextInterface instance as the second argument.

Wiec Twoja metoda validate jest w dobrym miejscu (tzn moglaby byc gdzie indziej niz w form type, ale tutaj tez jest ok), ale powinna miec taka sygnature:
public function validate($value, ExecutionContextInterface $ec).

Tak jak ty masz powinienes dostac:
Catchable Fatal Error: Argument 1 passed to AppBundle\Form\TestType::validate() must implement interface Symfony\Component\Validator\Context\ExecutionContextInterface, string given
albo cos podobnego - bylby to lepszy komunikat niz "nie dziala". Jesli nie widzisz tych bledow to polecam developowac przez kontroler developerski

Dla pewnosci wysylam przykladowy form type, ktory dziala. Jesli w pole "Pole" wpiszemy "a" to formularz jest prawidlowy, a jesli b to dostaniemy blad podpiety pod pole: Pole musi byc a

[PHP] pobierz, plaintext 
<?php
 
namespace AppBundle\Form;
 
use Symfony\Component\Form\AbstractType;
use Symfony\Component\Form\FormBuilderInterface;
use Symfony\Component\Validator\Constraints\Callback;
use Symfony\Component\Validator\Context\ExecutionContextInterface;
 
class TestType extends AbstractType
{
    public function buildForm(FormBuilderInterface $builder, array $options)
    {
        $builder->add('pole','text',array(
            'mapped' => false,
            'constraints' => array(
                new Callback(array($this,'validate')
        ))));
    }
 
    public function validate($value, ExecutionContextInterface $ec)
    {
        if ($value !== 'a') {
            $ec->buildViolation('Pole musi byc a')
            ->atPath('pole')
            ->addViolation();
        }
    }
 
    public function getName()
    {
        return 'test';
    }
}
[PHP] pobierz, plaintext 

no i kontroler:

[PHP] pobierz, plaintext 
<?php
 
namespace AppBundle\Controller;
 
use AppBundle\Form\TestType;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
 
class DefaultController extends Controller
{
    /**
     * @Route("/app/example", name="homepage")
     */
    public function indexAction(\Symfony\Component\HttpFoundation\Request $request)
    {
        $form = $this->createForm(new TestType());
 
        $form->handleRequest($request);
 
        if ($form->isValid()) {
            die('valid');
        }
 
        return $this->render('default/index.html.twig', [
            'form' => $form->createView(),
        ]);
    }
}
[PHP] pobierz, plaintext 

[Crash89]

Dzięki waszym postom udało mi sie rozwiązać problem

Ostatecznie rozwiązałem to tak:

W klasie FormType umieściłem funkcję validate.

[PHP] pobierz, plaintext 
 
public function validate($value,ExecutionContextInterface $context){
            $form = $context->getRoot();
 
         if($form->get('pole1')->getData() <= $form->get('pole2')->getData();
             $context->buildViolation('Text błędu')
                          ->atPath('pole2')
                          ->addViolation();
}
}
[PHP] pobierz, plaintext 

[blahy]

jest ok, ale jesli ta metoda walidacji ma dotyczyc calego formularza (bo uzywa wiekszej ilosci pol) to moze byc lepszym pomyslem dodanie walidatora do calego formularza, a nie do pojedynczego pola.

Do klasy FormType mozna dodac metode:

[PHP] pobierz, plaintext 
public function setDefaultOptions(OptionsResolverInterface $resolver) {
        $resolver->setDefaults(array(
            'constraints' => array(
                new Callback(array($this,'validate'))
            ),
        ));
    }
[PHP] pobierz, plaintext 

Wtedy w metodzie validate w 1 argumencie dostaniemy array z wszystkimi polami z formularza (pola nie powinny miec mapped => false, i tak nie przekazujemy obiektu):

[PHP] pobierz, plaintext 
public function validate($value, ExecutionContextInterface $ec)
    {
        var_dump($value); // array(1) { ["pole"]=> string(3) "fff" } 
[PHP] pobierz, plaintext 

[Crash89]

jest ok, ale jesli ta metoda walidacji ma dotyczyc calego formularza (bo uzywa wiekszej ilosci pol) to moze byc lepszym pomyslem dodanie walidatora do calego formularza, a nie do pojedynczego pola.

Do klasy FormType mozna dodac metode:

[PHP] pobierz, plaintext 
public function setDefaultOptions(OptionsResolverInterface $resolver) {
        $resolver->setDefaults(array(
            'constraints' => array(
                new Callback(array($this,'validate'))
            ),
        ));
    }
[PHP] pobierz, plaintext 

Wtedy w metodzie validate w 1 argumencie dostaniemy array z wszystkimi polami z formularza (pola nie powinny miec mapped => false, i tak nie przekazujemy obiektu):

[PHP] pobierz, plaintext 
public function validate($value, ExecutionContextInterface $ec)
    {
        var_dump($value); // array(1) { ["pole"]=> string(3) "fff" } 
[PHP] pobierz, plaintext 

Używając powyższej metody, zwraca mi komunikat podobny do: "cannot use DateTime as Array".
Co jest dziwne, bo gdy porównuje te wartości z $value['pole1'] z $form->get('pole1')->getData() przez var_dump(), są takie same.

[blahy]

Moze zmieniles funkcje validate a zostawiles constraint tez dla pola, tak jak miales wczesniej. Dlatego dalej wartosc datetime przekazywana jest do funkcji validate.
Jesli nie, to znowu przydalby sie kod