Add scoping by to find_by or create new element

[emaglio]

It might be worth to see if possible or just makes sense to add a scope_by key in the Model arguments.
This comes from experience where people prefer to do this:

step :set_model

def set_model(ctx, params:, current_user:, **)
  ctx[:model] = current_user.broadacsts.find_by(id: params[:id)
end

Over

step Model(Broadcast, :find_by)

Simply for "security" reason.
So this is what we could have:

step Model(Broadcast, :find_by, scope_by: :current_user)

where current_user is passed in ctx -> ctx[:current_user]

We add a step to extract the scope with failure end similar on what is done in Contract::Validate.

Should we think to add this or not really?

[seuros]

How are you thinking to figure out the relation ? You provided a symbol and a model.
i could have

class User < ...
has_many :youtube_broadcasts, class_name: 'Broadcast'
has_many :tags, class_name: 'UserTag'

[yogeshjain999]

Good idea @emaglio

@seuros how about we accept the scope like this ?

step Model(Broadcast, :find_by, scope: { column: :user_id, value: :current_user })