Валидация в Rails, как сделать запрос в соседнее поле?

Question

[Кирилл Казаков]

Провожу валидацию в модели введенных данных:

#app/models/validate.rb
validates :email, :presence=> true,
:format=> VALID_EMAIL_REGEX,
:uniqueness=> true

Не могу понять, как сделать доп. запрос в БД в ту же таблицу в соседнее поле, но которое не участвует при наборе формы (тут в модель передается результаты ввода формы params[:subscribtion]) ) ?

Задача : проверить email на уникальность ( что сделано выше ), а так же проверить что в соседнем поле subscribe - значение true .
subscribe не участвует при вводе.

Все кручусь вокруг has_one, но не могу понять как правильно заюзать .
Так же мне нужно видимо юзать объединение условий валидации, но с этим позже, пока не могу понять как сделать запрос в соседнее поле.

UPD:
Сейчас запрос такой

SELECT 1 FROM `subscriptions` WHERE (`subscriptions`.`email` = BINARY '[e-mail]') LIMIT 1

а надо

SELECT 1 FROM `subscriptions` WHERE (`subscriptions`.`email` = BINARY '[e-mail]') AND (`subscriptions`.'subscribe' = '1') LIMIT 1

Comments

[e̳͇̳̰̯̠̗̤͖̔̑̓̀̈́̇̈̍͞b̊̒̑͗̄́͊͢͡͠]

если поле принадлежит этой же модели, то просто self.subscribe

[Кирилл Казаков]

Dmitry Sharikov:

class Subscription < ActiveRecord::Base
before_save { self.email = email.downcase }
VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i
us_s = self.subscribe
validates :email, :presence=> true,
:format=> VALID_EMAIL_REGEX,
:uniqueness=> true
end

В БД

Табл. - subscriptions
поля
id
email
name
subscribe

В такой записи в моделе я вижу ошибку
NoMethodError (undefined method `subscribe' ).

[e̳͇̳̰̯̠̗̤͖̔̑̓̀̈́̇̈̍͞b̊̒̑͗̄́͊͢͡͠]

Кирилл Казаков: а что ты пытаешься сделать? я вижу как ты пытаешь присвоить что-то константе

[e̳͇̳̰̯̠̗̤͖̔̑̓̀̈́̇̈̍͞b̊̒̑͗̄́͊͢͡͠]

а не, у тебя переменная us_s и ты с ней что-то хочешь сделать. что именно?

[Кирилл Казаков]

Dmitry Sharikov: Хотел присвоить результат запроса (чтобы убедиться, что запрос выполняется, а далее уже добавить валидацию вместе с этим значением).
Вопрос обновил, написал какой сейчас запрос, написал, какой хочу запрос.

[e̳͇̳̰̯̠̗̤͖̔̑̓̀̈́̇̈̍͞b̊̒̑͗̄́͊͢͡͠]

Кирилл Казаков: зайди в консоль рельсовую, сделай Subscription.first.subscribe? выхлоп сюда

[e̳͇̳̰̯̠̗̤͖̔̑̓̀̈́̇̈̍͞b̊̒̑͗̄́͊͢͡͠]

Кирилл Казаков: удаляй свою переменную отсюда from /srv/www/voinaimir.tv/htdocs/app/models/subscription.rb:12

[Кирилл Казаков]

Dmitry Sharikov: Сорри, там эксперементировал , не удалил кусок

Subscription.first.subscribe?
=> false

[Кирилл Казаков]

Dmitry Sharikov: Да, уже, извини.

[e̳͇̳̰̯̠̗̤͖̔̑̓̀̈́̇̈̍͞b̊̒̑͗̄́͊͢͡͠]

и еще раз покажи выхлоп консоли

[Кирилл Казаков]

Dmitry Sharikov:

1.8.7-p374 :006 > Subscription.first.subscribe?
=> false

[e̳͇̳̰̯̠̗̤͖̔̑̓̀̈́̇̈̍͞b̊̒̑͗̄́͊͢͡͠]

ну собственно, все работает. что ты хочешь проверить через это поле? поставить условие валидации?

[Кирилл Казаков]

Dmitry Sharikov: Да, именно так.
Уникальность email - сделано AND "subscribe" = "1"

[e̳͇̳̰̯̠̗̤͖̔̑̓̀̈́̇̈̍͞b̊̒̑͗̄́͊͢͡͠]

Кирилл Казаков: тогда вот, rusrails.ru/active-record-validations#ispolzovanie...

[Кирилл Казаков]

Dmitry Sharikov: Спасибо !
Прямо как в примере сделал, а rails перестал запрашивать БД вообще

class Subscription < ActiveRecord::Base

before_save { self.email = email.downcase }
VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i
validates :email, :presence=> true,
:format=> VALID_EMAIL_REGEX,
:if=> :subscribe_on?,
:uniqueness=> true

def subscribe_on?
self.subscribe
end

end

[e̳͇̳̰̯̠̗̤͖̔̑̓̀̈́̇̈̍͞b̊̒̑͗̄́͊͢͡͠]

Кирилл Казаков: def subscribe_on?
self.subscribe?
end

[e̳͇̳̰̯̠̗̤͖̔̑̓̀̈́̇̈̍͞b̊̒̑͗̄́͊͢͡͠]

Кирилл Казаков: :if=> :subscribe_on? должно быть в конце

[Кирилл Казаков]

Dmitry Sharikov:
Не запрашивает БД
Так тоже не работает

def subscribe_on?
subscribe == "1"
end

Не понимаю , убираю из валидации :if=> :subscribe_on? , начинаются запросы в БД на уникальность , вместе с условием ни одного запроса в БД .

[Кирилл Казаков]

Dmitry Sharikov: И так тоже ничего

class Subscription < ActiveRecord::Base

before_save { self.email = email.downcase }
VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i
validates :email, :presence=> true,
:format=> VALID_EMAIL_REGEX,
:uniqueness=> true,
:if=> :subscribe_on?

def subscribe_on?
self.subscribe?
end

end

[e̳͇̳̰̯̠̗̤͖̔̑̓̀̈́̇̈̍͞b̊̒̑͗̄́͊͢͡͠]

Кирилл Казаков: давай лог

[Кирилл Казаков]

Dmitry Sharikov:

Started POST "/subscribtions/validate" for my_ip at Tue Oct 27 18:13:49 +0400 2015
DEPRECATION WARNING: Setting filter_parameter_logging in ActionController is deprecated and has no longer effect, please set 'config.filter_parameters' in config/application.rb instead. (called from /app/controllers/application_controller.rb:5)
Processing by SubscribtionsController#validate as JSON
Parameters: {"subscribtion"=>{"email"=>"[e-mail]"}}
Completed 200 OK in 5ms (Views: 0.1ms | ActiveRecord: 0.0ms)

[e̳͇̳̰̯̠̗̤͖̔̑̓̀̈́̇̈̍͞b̊̒̑͗̄́͊͢͡͠]

Кирилл Казаков: возьми два объекта Subscription с subscribe = true and subscribe = false, и оба лога дай

[Кирилл Казаков]

Dmitry Sharikov: Боюсь я не понял, что надо сделать.
Сделать запрос из модели, где subscribe = true , subscribe = false ?

[e̳͇̳̰̯̠̗̤͖̔̑̓̀̈́̇̈̍͞b̊̒̑͗̄́͊͢͡͠]

Кирилл Казаков: короч, модель у тебя норм, тебе нужно тестить валидацию с разными параметрами и только так найдешь проблему

[Кирилл Казаков]

Dmitry Sharikov: Ok , спасибо за уделенное время !

Answer 1

[Артем Пьянков]

Не совсем понял что ты хочешь сделать. Но чтоб получить требуемый запрос, сделай так:

validates :email, uniqueness: { scope: :subscribe }, if: :subscribe?

Так валидация будет запускаться только если в поле subscribe значение true. Если нужно так же проверять, что subscribe всегда true, то есть такой вариант:
validates :subscribe, inclusion: { in: [true] }

Мне он не очень нравится. Лучше задуматься над кастомным валидатором или методом валидации.

Comments

[Кирилл Казаков]

Не совсем понял что ты хочешь сделать. Но чтоб получить требуемый запрос, сделай так:

Хочу :
Чтобы
1)шла проверка на уникальность email
2)Если email не уникален, то проверить , подписан ли этот email на рассылку,
а)В случае если не подписан (subscribe= false), то отправить ему письмо с предложением подписаться еще раз.
б) Если не уникален и подписан, то выдать ошибку

[Артем Пьянков]

Кирилл Казаков: валидации созданы для простой проверки пользовательского ввода. То, что описываешь ты, лучше сделать с помощью отдельного класса или, например, интерактора (https://github.com/collectiveidea/interactor).

[Кирилл Казаков]

ibub1ik: Благодарю , завтра буду разбираться.
Отлично, что наставили на правильный путь !

Answer 2

[Orky]

"а надо
SELECT 1 FROM `subscriptions` WHERE (`subscriptions`.`email` = BINARY '[e-mail]') AND (`subscriptions`.'subscribe' = '1') LIMIT 1" - запрос неверный, потому что может быть два мыла в базе, одно со включенной подпиской, одно с выключенной.

А насчет валидации согласен с ibub1ik - лучше сделать отдельным классом.