Как исправить ошибку Call to a member function fetchAll() on array в php?

Question

[ProggRamm]

Есть функция фильтра

public function filter($showActiveOnly, $showNotAdded, $showHalfAdded, $showDeclined)
    {
        $newRow = [];

        $statuses = [
            'dodano' => $showActiveOnly,
            'nedodano' => $showNotAdded,
            'casdodano' => $showHalfAdded,
            'storno' => $showDeclined,
        ];

        foreach ($statuses as $status => $addFilter)
        {
            if ($addFilter)
            {
                $newRow[] = $this->db->table(self::TABLE_NAME)->where(array('status' => $status));
                // or :
                // array_push($newRow, $this->db->table(self::TABLE_NAME)->where(array('status' => $status)));
            }
        }
        return $newRow ? : $this->db->table(self::TABLE_NAME);
    }

и фунция в которой fetchAll и она вызывает функцию filter

public function getFiltered($showActiveOnly, $showNotAdded, $showHalfAdded, $showDeclined)
    {
        return $this->filter($showActiveOnly, $showNotAdded, $showHalfAdded, $showDeclined)->fetchAll();

        //$rows = $this->filter($showActiveOnly, $showNotAdded, $showHalfAdded, $showDeclined);
        //$rows->execute();
        //$results = $rows->fetchAll();
        //return  $results;
    }

Вылазит ошибка Call to a member function FetchAll() on array
При чём если из функции filter убрать квадратные скобки возле newRow который в цикле то ошибка пропадаер и функция работает но не так как нужно

Comments

[Siverius]

это дубликат вопроса. https://qna.habr.com/q/1176500?e=12859832

$newRow = $this->db->table(self::TABLE_NAME)->where(array('status' => $status));

[Олег]

return $newRow ? : $this->db->table(self::TABLE_NAME);
кто родил этот перл ?

[Siverius]

Олег, я

[Siverius]

Хотя здесь я бы написал ещё перлее

public function filter($showActiveOnly, $showNotAdded, $showHalfAdded, $showDeclined)
    {
        $statuses = [
            'dodano' => $showActiveOnly,
            'nedodano' => $showNotAdded,
            'casdodano' => $showHalfAdded,
            'storno' => $showDeclined,
        ];

        foreach ($statuses as $status => $addFilter)
        {
            if ($addFilter)
            {
                return $this->db->table(self::TABLE_NAME)->where(array('status' => $status));
            }
        }
        return $this->db->table(self::TABLE_NAME);
    }

[Олег]

Ну так думай, как убрать грабли, которые сам себе положил.
У Вас функция возврашает массив найденных или квери билдер.
к массиву лепить запрос дальше не получиться

[Siverius]

Олег, грабли-шмабли.
Я бы, зная кодовую базу и что вообще человеку надо, вообще возвращал бы один лишь массив, но увы, товарищ вопрошающий дал слишком мало инфы.

[Олег]

Кодеигнитор ?
https://www.codeigniter.com/user_guide/database/qu...
прочитали

А по смыслу нужен наверное where_in
поле имеет значение входящее в массив нужных
добавляете в зависимости от аргументов в массив значения. Если массив не пустой
return $this->db->table(self::TABLE_NAME)->where_in('status' => $statuses);
иначе
return $this->db->table(self::TABLE_NAME);

Answer 1

[Дмитрий]

1. В случае фильтрации вы возвращаете массив - и у массива нет метода fetchAll. Как собственно и других методов.
2. В случае если вы убираете квадратные скобки то тогда у вас возвращается только данные с первым статусом.

Если я правильно понял то вам надо посмотреть есть ли метод whereIn у вашего $this->db->table и использовать его.

Comments

[ProggRamm]

А как узнать это?

[Дмитрий]

ProggRamm, посмотреть код этого класса или документацию. Я то точно не знаю что это такое.