Как сделать hasMany с условием в переменной?

Question

[alex4answ]

Добрый день.

Есть комментарии к статьям, мне нужно достать комментарии за последние 7, 28 дней.

сейчас я сделал так:

Article model:
public function getComments7(){
        return $this->hasMany(Comments::className(), ['page_id' => 'id'])->andWhere('date > DATE_SUB(CURRENT_DATE, INTERVAL 7 DAY)');
    }
    
    public function getComments28(){
        return $this->hasMany(Comments::className(), ['page_id' => 'id'])->andWhere('date > DATE_SUB(CURRENT_DATE, INTERVAL 28 DAY)');
    }

и уже в with() при выборке подставляю нужное мне, но проблема вся к том, что обращаться к коментариям нужно по разному
->comments7 и ->comments28, что не очень удобно

Пытался сделать с JoinWith и уже в условии писать за какой период мне нужны комментарии, но результат вообще пуст.

Какие варианты есть кроме LeftJoin, чтобы достать страницы и последние комментарии к ним ?

Не хочу костылить и делать криво косо, потому спрашиваю как лучше поступить)

Comments

[Дмитрий]

Добрый день.
Зачем?
Вы делаете одну и туже связь, только с разными именами.
Это неверно.

[Дмитрий]

Пытался сделать с JoinWith и уже в условии писать за какой период мне нужны комментарии, но результат вообще пуст.

Вот это ближе к истине, а то, что результат пуст, так это, скорей всего, неверный запрос у Вас.

[alex4answ]

slo_nik, я не могу найти как передать условие для hasMany, потому подумал мб так нужно, но видимо нет, спасибо, буду знать)

[Дмитрий]

alex4answ, можно и так, но Вы делаете две одинаковые связи.
Вам надо вывести комментарии за какой-то период. У Вас есть связь между статьями и комментариями.

$model = Articles::find()->where(['id' => $id])->joinWith('comments')->all();

Вы получили статью с каким-то id и всеми её комментариями.
И тут можно извертеться на пупе как хотите.
Возможен такой вариант запроса

// Получить статью с таким-то id, со статусом "активна" и также получить все комментарии к этой статье у которых //тоже статус "активен"
$query = Articles::find()
    ->from(['articles' => Articles::tableName()])
    ->where(['article.status' => Articles::STATUS_ACTIVE, 'article.id' => $id])
    ->with(['commentss'])
    ->innerJoinWith([
        'comments comment',
        'comment.article art' => function(ActiveQuery $query){
            $query->andOnCondition(['art.status' => Comments::STATUS_ACTIVE]);
     }
]);

Вот как-то так. Это пример, работать не будет, но если вдруг, то я не специально)))
Подробней о работе со связями смотрите здесь.

[alex4answ]

slo_nik, спасибо, наколдовал в общем с joinWith и andWhere)

Answer 1

[Максим Тимофеев]

2 связи не нужны, одной достаточно

public function getComments($period=false){
        $query = $this->hasMany(Comments::className(), ['page_id' => 'id']);
        if($period){
             $query->andWhere('date > DATE_SUB(CURRENT_DATE, INTERVAL '.$period.' DAY)');
        }
        return $query;
    }

Comments

[Максим Тимофеев]

alex4answ,
вижу у Вас ->joinWith('comments') Не понятно зачем join, я так понимаю надо -->with('comments'). Ведь скорее всего нужна просто жадная загрузка.
Соответственно создаем методы, кроме описанного выше:

const BIG_COMMENT_SIZE = 28;
const SMALL_COMMENT_SIZE = 7;

public function getBigComments(){
        return $this->getComments(self::BIG_COMMENT_SIZE);
}

public function getSmallComments(){
        return $this->getComments(self::SMALL_COMMENT_SIZE);
}

и вызов

Articles::find()
->with('smallComments')
->all();

[Максим Тимофеев]

alex4answ, https://www.yiiframework.com/doc/api/2.0/yii-db-ac...

[alex4answ]

Максим Тимофеев, та же проблема что и с 2 связями, доступ к данным не ->comments, а smallComments / bigComments, подскажите пожалуйста, как решить это не костыльно?

[Максим Тимофеев]

alex4answ, а что тут костыльного?
Костыльно это 1 супер метод. А много мелких - это правильно. Этим удобно пользоваться и просто поддерживать.