Symfony Form Error: "Expected argument of type "Doctrine\ORM\QueryBuilder", "Doctrine\ORM\Query" given"

Question

I have two form inside the Controller the first form works fine however my second Form does not work as expected.

MyController:

// Second Form
$formTwo = $this->get('form.factory')->createNamedBuilder('form2name', new CarType(), null, array())
        ->getForm();

if('POST' === $request->getMethod()) {

if ($request->request->has('form1name')) {
    // handle the first form  
}

if ($request->request->has('form2name')) {
    // handle the second form 
    // get the id value of the selected value. 
  }
}

My CarType:

public function buildForm(FormBuilderInterface $builder, array $options){

   $builder->add('makename','entity',array(
                    'class'=> 'MyTestBundle:Car\CarModel',
                    'query_builder'=>function(EntityRepository $er){
                        return $query = $er->createQueryBuilder('s')
                                    ->select('s.makename')
                                    ->distinct()
                                    ->orderBy('s.makename','ASC');
                }

   ));
  $builder->add('search','submit',array());
}

My Car Entity

Full Stack Trace

Error : "Expected argument of type "Doctrine\ORM\QueryBuilder", "Doctrine\ORM\Query" given"

Symfony Version : 2.7

Remove the getQuery() and maybe change $query to $qb just to make it clear that you are dealing with a query builder object and not a query object. Two very different things. symfony.com/doc/current/reference/forms/types/…. You might get a different error. Not sure what distinct will do to your query. Have to try it and see. — Cerad
– Cerad, Commented Feb 22, 2016 at 15:12
Remove the ->getQuery(). Without that you return the query builder, with it you return a query object. Also you can remove the $query = as it's never used so redundant. — qooplmao
– qooplmao, Commented Feb 22, 2016 at 15:12

Delphine · Accepted Answer · 2021-06-16 09:18:28Z

5

In query_builder (for building your form) you have to return a QueryBuilder object. Currently, you return a Query object.

Just remove getQuery()

 $builder->add('makename','entity',array(
                    'class'=> 'MyTestBundle:Car\CarModel',
                    'query_builder'=>function(EntityRepository $er){
                        return $er->createQueryBuilder('s')
                                    ->select('s.makename')
                                    ->distinct()
                                    //->getQuery(); remove this line
                }

edited Jun 16, 2021 at 9:18

answered Feb 22, 2016 at 15:14

Delphine

8989 silver badges24 bronze badges

Sign up to request clarification or add additional context in comments.

6 Comments

hasumedic Over a year ago

Also the assignation to the $query variable

gabf Hahn Over a year ago

I get a different Error now : Warning: spl_object_hash() expects parameter 1 to be object, string given

Delphine Over a year ago

Ok, could you update your post and copy the full trace in order to know exactly when this error is thrown and what is this string, please ?

gabf Hahn Over a year ago

yes because i want to create dynamic dropdown and "makename" is the first dropdown option and I think now it works... I removed "select" and added "choice_label"

$builder->add('makename', 'entity', array(             'class' => 'MyTestBundle:Car\CarModel',             'choice_label'=>'makename',             'query_builder' => function (EntityRepository $er) {                 return $qb = $er->createQueryBuilder('s')                     ->GroupBy('s.makename')                     ->orderBy('s.makename','ASC');             }         ));

gabf Hahn Over a year ago

And I also had to change from distinct() to GroupBy() and got it working... Thanks :)

|

Collectives™ on Stack Overflow

Symfony Form Error: "Expected argument of type "Doctrine\ORM\QueryBuilder", "Doctrine\ORM\Query" given"

1 Answer 1

6 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

6 Comments

Your Answer

Sign up or log in

Post as a guest

Related