Навести порядок с методом User.active about hackerspace.by HOT 8 CLOSED

@jekhor Усё тут падаецца правільна. скоп маделі(метад класса) і метады інстанса класа active?

User.active != user.active?
розныя аб'екты і розныя метады.
У якім месцы кода блытаніна?

from hackerspace.by.

Мне калі трэба атрымаць спіс дзейсных на дадзены момант юзэраў (не забаненых і не ў саспендзе), то я, убачыўшы метад User.active і ведаючы пра User#active?, проста ні на секунду не буду сумнявацца, што User.active — як раз тое, што мне трэба. А насамрэч не.

from hackerspace.by.

https://github.com/minsk-hackerspace/hackerspace.by/blob/master/app/models/user.rb#L156 напрыклад

from hackerspace.by.

семантыка розная.

from hackerspace.by.

ага, цяпер бачу што не зусім празрыста ўнутры.

трэба рэфакторыць або проста увесці flag які будзе ставіцца актыўнамую юзеру як true і табы будзе менш выбараць і іфчыкаў.

def self.active
(allowed.paid + allowed.signed_in).uniq
end

from hackerspace.by.

Ну мне падаецца, што непатрэбна нам тая сартыроўка па "актыўнасці" юзэраў, якую калісьці праз гэта рабілі ў спісе ўдзельнікаў.

from hackerspace.by.

так, пасля саспенд логікі прыйшоў час пераглядзець гэта.

from hackerspace.by.

Зрабіў змены тут #577

from hackerspace.by.