рельсы обновляют атрибуты на основе истинных или ложных утверждений

Question

Я читал и пробовал все, что мало кто из Stackflow предложил здесь без везения. Пока у меня есть;

def save_with_payment 
    if Subscription.where(:subscription_plan => "Yearly") 
    update_attributes(expiry_date: Date.today.next_year) 
    else 
    update_attributes(expiry_date: Date.today.next_month) 
    end 
end 

Каким-то образом, он сохраняет настройки EXPIRY_DATE в следующем году, даже если: subscription_plan => "ежемесячно".

Answer 1

Subscription.where возвращает пустой массив, если нет «ежегодных» подписки.

if [] #returns true 

Итак, ваше первое условие всегда будет правдой. Использовать exists? вместо

def save_with_payment 
    if Subscription.exists?(:subscription_plan => "Yearly") 
    update_attributes(expiry_date: Date.today.next_year) 
    else 
    update_attributes(expiry_date: Date.today.next_month) 
    end 
end 

Answer 2

Это соглашение совершенно неверно.

Subscription.where(:subscription_plan => "Yearly") будет правдивым в большинстве случаев, потому что он предназначен для всего Подписки модели данных. Если одна из записей Yearly, это правда.

Вместо запроса всей модели, сделайте это по собственным данным пользователя.

Пусть пользователь имеет одну подписку, то

def save_with_payment 
    subscription.yearly? ? extend_one_year : extend_one_month 
end 

def extend_one_year 
    update_attributes(expiry_date: Date.today.next_year) 
end 

# def extend_one_year 

Или еще лучше, такой простирающуюся логика должна принадлежит Подписка а не пользователя. Переместите его туда. А также «Расскажите, не спрашивайте»

class Subscription < ActiveRecord::Base 
    belongs_to :user 

    def next_extend_date 
    next_date = subscription_plan.yearly? ? 'next_year' : 'next_month' 
    Date.today.send next_date 
    end 
end 

class User < ActiveRecord::Base 
    has_one :subscription 

    def save_with_payment 
    extend_to subscription.next_extend_date 
    end 

    # But I'm not sure if such logic still need to be in Subscrption. 
    def extend_to(date) 
    update_attribute expiry_date: date 
    end 
end