Symfony 2, как можно использовать проверку внутри QueryBuilder-а?

Question

[Антон Натаров]

В репозиторий могут приходить две переменных, искомый текст ($search) и дополнительное условие ($country).

Нужно: Если переменная есть, добавить дополнительное условие, если нету, QueryBuilder использует только стандартный запрос.

public function findByPersonAction($search, $country)
    {       
        $query = $this->getEntityManager()
            ->createQueryBuilder('p')
            ->select('p')
            ->from('DroidUserBundle:Person', 'pi')
            ->leftJoin('DroidUserBundle:PersonaAddress', 'c', 'WITH', 'p.person_id = c.person_id')
            ->where('lower(p.firstname) LIKE lower(:search)')

            if($country != NULL)
             { $query->andWhere('c.country_id = :country')
                          ->setParameter('country', $country)
             }

            ->setParameter('search', '%'.$search.'%')
            ->getQuery();
  
        $result = $query->getResult();
        
        return $result;
    }

Я реализовал это таким подходом. Выдает Fatal error синтаксическую ошибку if либо ругается на { }
Что делаю не так?

Answer 1

[Алексей Романенко]

у вас синтаксическая ошибка.
нет ; после ->where('lower(p.firstname) LIKE lower(:search)')
и не понятно к чему эта строка ->setParameter('search', '%'.$search.'%')

Comments

[Антон Натаров]

По поводу строки. Это связано с тем, что использую PostgreSQL и его LIKE Не такой же как у MySQL, но есть аналог ILIKE, которого в Doctrine нету.

Не смотря, на точку с запятой, он все равно ругается на скобки цикла.

[BoShurik]

$query->setParameter('search', '%'.$search.'%')

[Алексей Романенко]

HanDroid: читайте что вам пишет пхп когда вы пытаетесь выполнить свой код. там все написано.

[Антон Натаров]

BoShurik: и что вас смущает, наличие того, что могут быть двойные имена?
Анна-Мария например. Введя Мария или просто Мар он найдет и этого пользователя.

[Антон Натаров]

Алексей Романенко: Parse Error: syntax error, unexpected '}' in
Сразу стало "все понятно" =)

[Алексей Романенко]

HanDroid: вы исправили 2 ошибки которые я вам написал?

[Антон Натаров]

Алексей Романенко: Да, поправил.

[Алексей Романенко]

HanDroid: покажите что получилось и какую ошибку пишет с указанием строки.

[Антон Натаров]

Алексей Романенко: Благодарю за помощь. Я уже разобрался в чем была ошибка.

[Алексей Романенко]

HanDroid: велкам

Answer 2

[Антон Натаров]

Решение ситуации таково:
Точку с запятой до цикла, внутри цикла после запроса и продолжение запроса начинать с $query->

->where('lower(p.firstname) LIKE lower(:search)');
        if($country != NULL){
            $query->andWhere('c.country_id = :country')->setParameter('country', $country);
           }
            $query->setParameter('search', '%'.$search.'%')

Answer 3

[Ivanq]

У вас до скобки не стоит ; и перевод строки перед -> лишний.
И все равно - будет Fatal Error. Нужно сверху вместо

public function findByPersonAction($search, $country)

подставить

public function findByPersonAction($search, $country=null)

. Этим мы ставим значение по умолчанию. Без этого будет ошибка неверное количество параметров функции

Comments

[BoShurik]

Может быть там findByPersonAction($search, null);

[Алексей Романенко]

совсем не обязательно делать =null

[Антон Натаров]

Все проверки есть в контроллере, нужно было именно что бы QB имел такую возможность "достраивать запрос", а не в качестве безопасности. И как советует BoShurik делать не стоит. Во первых не ясно что именно Null, а во вторых это ломает всю функцию и приводит к ошибке синтаксиса.

[BoShurik]

Антон Натаров: Не то чтобы я за вызов функций таким образом. Приведенный пример показывает, что не обязательно делать параметра необязательным. В любом случае вполне себе валидное использование. Ясно, что вторым параметром передаем null и никакой ошибки синтаксиса тут нет.