Symfony2 form collection

Laat je Action (controller) eens zien?

Dat is inderdaad misschien handig om ook even te laten zien. Dit is mijn controller class:

<?php
namespace AppBundle\Controller;

use AppBundle\Form\CustomerType;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;

class CustomerController extends Controller
{
    /**
     * @Route("/customers", name="customers")
     */
    public function indexAction()
    {
        $em = $this->getDoctrine()->getManager();
        $customers = $em->getRepository('AppBundle:Customer')->findAll();

        return $this->render('customer/index.html.twig', array(
            'customers' => $customers
        ));
    }

    /**
     * @Route("/customer/{id}", name="customer_details")
     */
    public function detailsAction($id, Request $request)
    {
        $result = null;

        $repository = $this->getDoctrine()
            ->getRepository('AppBundle:Customer');
        $customer = $repository
            ->find($id);

        if (!$customer) {
            throw $this->createNotFoundException(
                'Er is geen klant gevonden met ID ' . $id
            );
        }

        $form = $this->createForm(new CustomerType(), $customer);

        $form->handleRequest($request);

        if ($form->isSubmitted()) {
            if ($form->isValid()) {
                $em = $this->getDoctrine()->getManager();

                $em->persist($customer);
                $em->flush();

                $result = new Response(json_encode(array(
                        'url' => $this->generateUrl('customers'))
                ));
                $result->headers->set('Content-Type', 'application/json');
            } else {
                $validator = $this->get('validator');
                $errors = $validator->validate($customer);

                $messages = array();

                foreach ($errors as $error) {
                    $messages['message'] = $error->getMessage();
                    $messages['property'] = $error->getPropertyPath();
                }

                $result = new Response(json_encode(array(
                    'errors' => $messages)
                ));
                $result->headers->set('Content-Type', 'application/json');
            }
        }

        if ($result == null) {
            $result = $this->render('customer/details.html.twig', array(
                'form' => $form->createView()
            ));
        }

        return $result;
    }
}
?>

Het gaat in dit geval om detailsAction(). Offtopic: is de manier met JSON die ik gebruik goed of kan ik hier beter iets anders doen?

Ik heb nog even niet goed naar je probleem gekeken. Maar even wat andere tips:

* Gebruik JsonResponse. Deze geef je een array mee en deze zorgt zelf voor de juiste encoding, headers, etc.
* Ga niet zelf met de validator aan de slag, de Form component doet dit al voor je! Gebruik $form->getErrors().
* Vergeet ook niet je HTTP status codes te veranderen. Je geeft nu altijd 200 terug (Success), terwijl het soms helemaal geen success is (bijv. wanneer het form niet valid is)

Bedankt voor je tips! Scheelt weer wat extra moeite bij validatie. Ik had de validator aangesproken omdat ik dacht dat m'n entities niet gevalideerd werden. Bleek dat ik geen constraints aan m'n entities had gegeven. Daarna ben ik vergeten het weg te halen.

Dan is dit nu m'n nieuwe controller:

<?php
/**
     * @Route("/customer/{id}", name="customer_details")
     */
    public function detailsAction($id, Request $request)
    {
        $result = null;

        $repository = $this->getDoctrine()
            ->getRepository('AppBundle:Customer');
        $customer = $repository
            ->find($id);

        if (!$customer) {
            throw $this->createNotFoundException(
                'Er is geen klant gevonden met ID ' . $id
            );
        }

        $form = $this->createForm(new CustomerType(), $customer);

        $form->handleRequest($request);

        if ($form->isSubmitted()) {
            if ($form->isValid()) {
                $em = $this->getDoctrine()->getManager();

                $em->persist($customer);
                $em->flush();

                $result = new JsonResponse(array(
                    'url' => $this->generateUrl('customers')
                ));
            } else {
                $errors = $form->getErrors();

                $messages = array();

                foreach ($errors as $error) {
                    $messages['message'] = $error->getMessage();
                    $messages['property'] = $error->getPropertyPath();
                }

                $result = new JsonResponse(array(
                    'errors' => $messages
                ), 400);
            }
        }

        if ($result == null) {
            $result = $this->render('customer/details.html.twig', array(
                'form' => $form->createView()
            ));
        }

        return $result;
    }
?>

[size=xsmall]Toevoeging op 03/05/2015 14:36:20:[/size]

Mijn lijst in $form->getErrors() is overigens leeg. Weten jullie hoe dat kan? In m'n validator is hij niet leeg.

<?php

/**
* @ORM\ManyToOne(targetEntity="Value", inversedBy="propertyValues")
* @ORM\JoinColumn(name="value_id", referencedColumnName="id", nullable=true)
*/
private $referencedValue;
?>

Probeerde jouw projectje even na te maken maar nu zie ik het bovenstaande. Heb je dan ook nog een entity Value? en wat is het nut hiervan?

Ik heb inderdaad ook nog een entity Value. Het idee daarachter is dat als er een aangepast veld komt met een dropdownbox, dat Property een collectie heeft van Value. In PropertyValue wordt daarnaar verwezen. Ik vond het nogal een lastig probleem om zowel een vrije input als vooraf gekozen input toe te staan. Nu zijn het allebei nullable velden. Snap je het idee erachter?

Die entity ziet er overigens zo uit:

<?php
namespace AppBundle\Entity;

use Doctrine\ORM\Mapping as ORM;

/**
 * @ORM\Entity
 * @ORM\Table(name="values")
 */
class Value
{
    /**
     * @ORM\Column(type="integer")
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="AUTO")
     */
    private $id;

    /**
     * @ORM\ManyToOne(targetEntity="Property", inversedBy="values")
     * @ORM\JoinColumn(name="property_id", referencedColumnName="id", nullable=false)
     */
    private $property;

    /**
     * @ORM\OneToMany(targetEntity="PropertyValue", mappedBy="referencedValue")
     */
    private $propertyValues;

    /**
     * @ORM\Column(type="string", length=50, nullable=false)
     */
    private $value;
    /**
     * Constructor
     */
    public function __construct()
    {
        $this->propertyValues = new \Doctrine\Common\Collections\ArrayCollection();
    }
    
    // Getters en setters weggelaten
?>

Ben nog op zoek naar de class PropertyValueType en ValueType :p

Wat me nu wel opvalt is dat je een verwijzing hebt van de tabel propertyvalues en van de tabel values naar de tabel property. Lijkt me een beetje dubbel..

Zo werkt Entity Framework ook, twee kanten op. Komt altijd van pas.
PropertyValueType in een form class:

<?php
namespace AppBundle\Form;

use Symfony\Component\Form\AbstractType;
use Symfony\Component\Form\FormBuilderInterface;
use Symfony\Component\OptionsResolver\OptionsResolver;

class PropertyValueType extends AbstractType
{
    public function buildForm(FormBuilderInterface $builder, array $options)
    {
        $builder
            ->add('value', 'text');
    }

    public function configureOptions(OptionsResolver $resolver)
    {
        $resolver->setDefaults(array(
            'data_class' => 'AppBundle\Entity\PropertyValue'
        ));
    }

    public function getName()
    {
        return 'propertyvalue';
    }
}
?>

PropertyType is een geïmproviseerde enum:

<?php
namespace AppBundle\Entity;

abstract class PropertyType
{
    const TextBox = 1;
    const TextArea = 2;
    const CheckBox = 3;
    const CheckList = 4;
    const Selection = 5;
    const Date = 6;
    const DateTime = 7;
}
?>

Hmm verbaasd me niet dat je foutmeldingen krijgt.

a) Je zegt PropertyType. Ik zou verwachten ValueType.
b) de methods buildForm, configureOptions en getName zijn die niet verplicht?
c) namespace AppBundle\Entity in plaats van AppBundle\Form

Ik heb deze dus toegevoegd:

<?php
namespace AppBundle\Form;

use Symfony\Component\Form\AbstractType;
use Symfony\Component\Form\FormBuilderInterface;
use Symfony\Component\OptionsResolver\OptionsResolver;

class ValueType extends AbstractType
{
public function buildForm(FormBuilderInterface $builder, array $options)
{
$builder
->add('value', 'text');
}

public function configureOptions(OptionsResolver $resolver)
{
$resolver->setDefaults(array(
'data_class' => 'AppBundle\Entity\Value'
));
}

public function getName()
{
return 'value';
}
}
?>

en bij mij werkt het zover ik kan beoordelen.

>> b) de methods buildForm, configureOptions en getName zijn die niet verplicht?

Alleen getName(), de andere worden ingevuld als empty methods door AbstractType (als je alleen FormTypeInterface implementeert zijn ze wel verplicht).