[Question] allow_nil, type, and must supported at the same time? about actor HOT 3 CLOSED

Hey @luizfonseca!

Actually the default is applied only if the argument is not given. When given, the default is never applied, even if it's nil.

The reasoning behind this is to be able to actually pass nil as a value. This is similar to the way Ruby methods assign defaults. For example:

def my_method(weekdays: [0, 1])
  weekdays
end

my_method # => [0, 1]
my_method(weekdays: nil) # => nil

If you want nil to fallback to the default, you could do something like the following:

class MyInteractor < Actor
  input :weekdays,
        type: Array,
        must: {
          be_valid: ->(weekdays) {
            weekdays.nil? || weekdays.all?(SomeValidation)
          }
        },
        allow_nil: true

  def call
    # …
  end

  def weekdays
    super || DEFAULT_WEEKDAYS
  end
end

Does that clarify your issue? I'm open to suggestions as to how to make this simpler. Perhaps we could have another option to allow using the given default when the input is nil? For example, calling it fallback instead of default could be an idea.

from actor.

@sunny It does! Thanks for the clarifications! 🙏

I am going to close the issue, as my original point has been clarified. Thank you for the great work on the library 👍

from actor.

Luiz, I added a spec with your use-case in a0df1f7. Hopefully this clarifies a bit more the current behavior.

from actor.