Next-gen ruby libs! » github.com/dry-rb » website: https://dry-rb.org » forum: https://discourse.dry-rb.org
Is it possible to do somethinh like this please?
rule(parse_request_id: %i[parse_request_id]) do |parse_request_id|
parse_request_id.filled? > parse_request_repository.find_by(id: value(parse_request_id)).filled?
end
I am trying to avoid this:
validate(exists?: :parse_request_id) do |parse_request_id|
if parse_request.nil?
true
else
parse_request_repository.find_by(id: parse_request_id).present?
end
end
parse_request_id is optional and existence checked against DB if provided
You can’t run arbitrary ruby code inside a
rule block. What I think you want to do is make a cusastom predicate called parse_request_exists? which takes the ID and returns true/false if it exists
so then you have a line like this inside your rule block
I already have that
required(:customer_id).filled(:exists?)
parse_request_id.filled? > parse_request_id.parse_request_exists?
def exists?(value)
record.class.where(id: value).exists?
end
record.class.where(id: value).exists?
end
Ok, so basically I'll create custom methods for every "model" i'd like to check
as I don't like the
record.class.where -> it does not work for multiple checks in one schema
@timriley Thanks :)
alternatively you could provide some way to identify the record as the second arg to the predicate method
so it becomes something like:
request_id.filled? > request_id.record_exists?(:parse_request)TargetRequest::CreateSchema.with(record: customer_repository.new).call(input).to_monad hints: false
Hmm
And use the
:parse_request name inside that predicate method to work out which repo to use or whatever
depends on how flexible you need to make this and how much you care (or not) about making multiple predicate methods
tbh I’d start by duplicating code and seeing how painful that really is in practice
Having unique methods per record does not sound that bad - > it would be more explicit
configure do
def exists?(klass, value)
!!klass.find(value).empty?
end
end
optional(: parse_request_id) {
filled? > exists(ParseRequest)?
}
@igor-alexandrov Nice
Thanks :)
module TargetRequests
CreateSchema = Dry::Validation.Schema(ApplicationSchema) do
required(:customer_id).filled { valid_db_id? & record_exists?(Customer) }
required(:parse_request_id).filled { valid_db_id? & record_exists?(ParseRequest) }
optional(:webhook_id).filled { valid_db_id? & record_exists?(Webhook) }
optional(:target_email_id).filled { valid_db_id? & record_exists?(TargetEmail) }
rule(target: %i[webhook_id target_email_id]) do |webhook_id, target_email_id|
webhook_id.filled? | target_email_id.filled?
end
end
end
What do you think? :)
The only thing that bothers me is the model class "hardcoded"
But that won't change that often, so it's OK I suppose
My initial idea was to somehow incorporate this:
class ApplicationSchema < Dry::Validation::Schema
configure do |config|
option :record
option :user_repository, User
option :customer_repository, Customer
option :document_repository, Document
option :document_content_repository, DocumentContent
option :vendor_repository, Vendor
option :template_repository, Template
option :source_email_repository, SourceEmail
option :parse_request_repository, ParseRequest
option :target_email_repository, TargetEmail
option :webhook_repository, Webhook
config.messages_file = 'config/locales/validations.yml'
end
...
end
Is there any way of Dry::Validation returning only the keys which are part of the schema? I was under impression that this is done automatically, especially with .to_monad extension. But it looks like I was wrong. Or maybe that was some older version as I remember it worked.
I'd like to get rid of strong params
I am looking thru the documentation and source code and can't find any option (or method on Result object) which would help me with that. Or I am blind :))
MySchema.call({i_dont_want_you: 'xxx', customer_id: 1, parse_request_id: "0266fa8b-6878-44aa-ad2e-a1d6e9dffa9d", webhook_id: 1}).to_monad
=> Success({:i_dont_want_you=>"xxx", :customer_id=>1, :parse_request_id=>"0266fa8b-6878-44aa-ad2e-a1d6e9dffa9d", :webhook_id=>1})