Как поменять поле, которое проверяется в условии ларавель?

Question

[Иван]

Добрый день. Такая проблема. Проверяю активных юзеров, и если он не обновлялся больше трех месяцев, хочу сделать его архивным, меняя active на 0. Сделал такое условие с методом. Проверяю запуском сидера.

$users = User::where('active', 1)->whereDate('updated_at', '<=', date('Y-m-d H:i:s', strtotime('-90 days')))->get();
        if (!empty($users)) {
            foreach ($users as $user) {
                if (!empty($user)) {
                    if ($user->active == 1) {
                        $user->active = 0;
                        $user->save();
                    }
                }
            }
        }

При запуске этого сидера, если выполняется условие, его зацикливает, или еще чего. Висит он короче, помогает Ctrl+C. При этом первую подходящую запись в таблице он все таки обновляет. Если пробовать изменить другое поле, не active, все работает как задумано. Пробовал без всех этих ифов, только форич, тоже самое. В чем прикол и как исправить?

Comments

[NubasLol]

if (!empty($user))

Что делает этот код?

[Иван]

NubasLol,Проверяет не пусто ли там. лучше перебдеть, чем недобдеть. Но я также пробовал без всех этих ифов, только форич, тоже самое.

[NubasLol]

Иван, И каким образом переменные которые ты проверяешь, могут быть пустыми?

[Alex Wells]

Иван,

Проверяет не пусто ли там. лучше перебдеть, чем недобдеть

Лучше всего не говнокодить.

Answer 1

[jazzus]

\DB::table('users')
   ->where('active', true)
   ->whereDate('updated_at', '<=', now()->subMonths(3))
   ->update(['active' => false]);

Comments

[Иван]

Спасибо, работает как надо. Но надо через модель, и чтоб при этом updated_at не обновлялся. Может подскажите, как сделать так, чтоб это поле updated_at не обновлялось?

[jazzus]

Иван, зачем через модель? Это конечный запрос к БД на обновление.

Answer 2

[Alex Wells]

Зачем тебе active в принципе? Почему ты в выборках не можешь просто юзать updated_at где тебе нужно? Судя по тому, что ты задаешь этот вопрос, никаких причин нет.

Comments

[Иван]

Если active и так 0, зачем мне что то менять. А если active =1 и updated_at >3 месяца, тогда active надо сделать 0.

[Alex Wells]

Иван, еще раз: зачем тебе active?

По этому условию проверяй: updated_at >3 месяца

[jazzus]

Иван, active это обычно модераторское поле, чтобы например полностью заблокировать сущность, но не удалять ее. А для фильтрации по last updated в Ларавел уже создано поле updated_at и завязывать ее на active смысла нет никакого. Тебе достаточно сделать скоп в модели

public function scopeRecentlyUpdated($query)
{
   return $query->whereDate('updated_at', '>', now()->subMonths(3));
}

и запрашивать где нужно

User::recentlyUpdated()
    ->get();

Answer 3

[Daria Motorina]

До дебага сначала стоит отрефакторить код) чтобы уменьшить цикломатическую сложность и увеличить читаемость:

$lastActiveDate = date('Y-m-d H:i:s', strtotime('-90 days'));

$users = User::where('active', 1)
    ->whereDate('updated_at', '<=', $lastActiveDate)
    ->get();

if ($users->isEmpty()) {
    return;
}

foreach ($users as $user) {
    if ($user->isEmpty()) {
        continue;
    }
    
    if ($user->active == 1) {
        $user->active = 0;
        $user->save();
    }  
}

Я не помню точно, но обычная функция empty не будет работать на коллекцию $users, на $user в форыче возможно
UPD. Вообще-то юзеры c is_active=1 уже отобраны в запросе, поэтому форыч можно свести к:

foreach ($users as $user) {
    $user->active = 0;
    $user->save();
}

Comments

[NubasLol]

Метод get() всегда вернет коллекцию, которая будет итерируема. Нет смысла в

if ($users->isEmpty()) {
    return;
}

форич и так не будет выполнятся. Так же и $user не может быть ничем кроме объекта модельки

[Daria Motorina]

NubasLol, уже заметила, добавила в UPD)

[Иван]

Упростил для теста до не могу, и все равно виснет,именно если менять active. В чем дело без понятия...

public function run()
    {
        $users = Users::whereDate('updated_at', '<=', date('Y-m-d H:i:s', strtotime('-60 days')))->get();
        foreach ($users as $user) {
            $user->active = 0;
            $user->save();
        }
    }

[NubasLol]

Ага) А еще лучше

$users = User::where('active', 1)
    ->whereDate('updated_at', '<=', $lastActiveDate)
    ->update(['active' => 0]);

[Daria Motorina]

NubasLol, кот_с_белиссимо.jpg )))

[Иван]

NubasLol, Да, спасибо. Рабочий вариант и компактней. Остановился на таком

User::where('active', 1)->whereDate('updated_at', '<=', date('Y-m-d H:i:s', strtotime('-90 days'))->update(['active' => false']);

А вот еще возник вопрос. Как указать там не обновлять updated_at ?