[D-326] Store accessor support for assignable_values

[begerdom]

No description provided.

[FLeinzi]

LGTM!

I don't have any opinion to that suggestion:

As store attribute are its own kind of attribute (like belongs_to relations), they should probably be handled in an own restriction class, like AssignableValues::ActiveRecord::Restriction::StoreAttribute to explicitly handle all peculiarities of store attributes.

For me this change would be enough as store accessors are similar to scalar attributes.

[triskweline]

As store attribute are its own kind of attribute (like belongs_to relations), they should probably be handled in an own restriction class, like AssignableValues::ActiveRecord::Restriction::StoreAttribute to explicitly handle all peculiarities of store attributes.

The original gem design used multiple restriction classes to prevent the if/else clauses seen in this diff. The reader already needs to understand a lot of cases for covering various Rails versions.

If you find the restriction to be similiar to ScalarAttribute, you can inherit or extract a common module.