我的模型有两个自定义的ActiveRecord验证方法,它们应该同时触发。但是,无论我做什么,都只执行方法 A (setup_ssh_user
),而不是执行 B (save_ssh_on_server
)。
我已经尝试过:
- 使用
byebug
和Rails.logger.info('funciton B is called')
检查方法 B 是否确实从未被调用 - 注释掉方法 A
- 重命名方法 B
- 使用
validate
而不是before_validation
- 交换方法 A 和 B 的代码
class Postgresql < ApplicationRecord
# Validations
validates :host, presence: true
validates :port, presence: true
validates :user, presence: true
validates :dbname, presence: true
before_validation :setup_ssh_user, :save_ssh_on_server, on: :create, if: :ssh_tunnel?
before_validation :save_ssh_on_server, on: :update, if: :ssh_public_key_changed?
validate :rollback_ssh_setup, if: :rollback_needed
# Associations
has_one :target, as: :resource, inverse_of: :resource, autosave: true
accepts_nested_attributes_for :target, reject_if: :all_blank, allow_destroy: true
def rollback_needed
return self.target.deleted || errors[:ssh_public_key].any? || errors[:ssh_user].any? ? true : false
end
def rollback_ssh_setup
status1, status2, status3 = SshTunnel::Setup.new().rollback_ssh_setup(self.ssh_user, self.ssh_public_key)
if status1.success? && status2.success? && status3.success?
return true
else
errors.add(:ssh_tunnel, "Rollback of ssh setup failed.")
return false
end
end
def setup_ssh_user
status = SshTunnel::Setup.new().create_user(self.ssh_user)
if status.success?
return true
else
errors.add(:ssh_user, "User Creation failed, please try again.")
return false
end
end
def save_ssh_on_server
Rails.logger.info("ssh saving function called")
status = SshTunnel::Setup.new().save_public_key(self.ssh_public_key, self.ssh_user)
if status.success?
return true
else
errors.add(:ssh_public_key, "Saving SSH Key failed, please try again.")
return false
end
end
end
由于这两种方法都非常相似,我不明白为什么一种永远不会被执行,而另一种是。也许我误解了活动记录验证的工作原理?非常感谢任何改进或调试建议。
不幸的是,对于为什么上面的代码不起作用,这并不是一个真正的答案,但我找到了解决方法 - 我希望这可以帮助其他可能遇到类似情况的人。
我替换了:
before_validation :setup_ssh_user, :save_ssh_on_server, on: :create, if: :ssh_tunnel?
跟:
before_validation on: :create do
if self.ssh_tunnel
self.setup_ssh_user
self.save_ssh_on_server
end
end