Проект

Общее

Профиль

Feature #413

закрыто

Feature #410: Создать фид для сайта

Переработать прежний фид (добавить категорию Фонари.), протестировать.

Добавил(а) Василий Малинин около 2 лет назад. Обновлено около 2 лет назад.

Статус:
Closed
Приоритет:
Normal
Назначена:
Василий Малинин
Дата начала:
23.02.2023
Срок завершения:
Готовность:

0%

Оценка временных затрат:
Трудозатраты:
Оплачена:
Нет
В спринте:
Да
Тестирование:
Сделана во время:

Описание

Прежний фид (http://192.168.20.100:3000/issues/190)
Контроллер /www/feed/controllers/ExportController.php

Обновлено Василий Малинин около 2 лет назад

  • Параметр Тема изменился с Переработать прежний фид. на Переработать прежний фид (добавить категорию Фонари.), протестировать.
  • Описание обновлено (Разница(diff))

Обновлено Василий Малинин около 2 лет назад

Обновлено Вячеслав Науменко около 2 лет назад

  • Параметр В спринте изменился на Да

Обновлено Василий Малинин около 2 лет назад

  • Параметр Статус изменился с New на Resolved

Обновлено Вячеслав Науменко около 2 лет назад

  • Параметр Статус изменился с Resolved на Feedback

Для этого кода сделать улучшения:

   public function actionAuroragrow()
     {
-        $category = ProductCategory::findOne(['name' => 'Автопрожекторы.']);
-        $searchModel = new ProductSearch(['product_category_id' => $category->id]);
+        $categories = ProductCategory::findAll(['name' => [
+            'Автопрожекторы.',
+            'Фонари.',
+            ]
+        ]);
+        $categoryIds = [];
+        foreach ($categories as $category) {
+            $categoryIds[] = $category->id;
+        }
+        $searchModel = new ProductSearch(['product_category_id' => $categoryIds]);
         $query = $searchModel->query();
  1. Форматирование лучше сделать так
        $categories = ProductCategory::findAll([
            'name' => [
                'Автопрожекторы.',
                'Фонари.',
            ]
        ]);

  1. Получение идентификаторов сделать через функцию, а не через foreach
$categoryIds = ArrayHelper::getColumn($categories, 'id');

также стоит учесть, что массив $categoryIds может быть пустым и запрос ниже поломает, поэтому после получения выполняем код:

$categoryIds[] = 0;

Обновлено Василий Малинин около 2 лет назад

  • Параметр Статус изменился с Feedback на Resolved

Обновлено Вячеслав Науменко около 2 лет назад

  • Параметр Статус изменился с Resolved на Closed

Экспортировать в Atom PDF