Deprecate the `:tokenizer` option to `validates_length_of` #19585

Merged
merged 1 commit into from Mar 29, 2015

Conversation

Projects
None yet
2 participants
@sgrif
Member

sgrif commented Mar 29, 2015

As demonstrated by #19570, this option is severely limited, and
satisfies an extremely specific use case. Realistically, there's not
much reason for this option to exist. Its functionality can be trivially
replicated with a normal Ruby method. Let's deprecate this option, in
favor of the simpler solution.

Deprecate the `:tokenizer` option to `validates_length_of`
As demonstrated by #19570, this option is severely limited, and
satisfies an extremely specific use case. Realistically, there's not
much reason for this option to exist. Its functionality can be trivially
replicated with a normal Ruby method. Let's deprecate this option, in
favor of the simpler solution.
@sgrif

This comment has been minimized.

Show comment
Hide comment
Member

sgrif commented Mar 29, 2015

rafaelfranca added a commit that referenced this pull request Mar 29, 2015

Merge pull request #19585 from sgrif/sg-deprecate-tokenizer
Deprecate the `:tokenizer` option to `validates_length_of`

@rafaelfranca rafaelfranca merged commit 77d9a8e into rails:master Mar 29, 2015

@sgrif sgrif deleted the sgrif:sg-deprecate-tokenizer branch Mar 29, 2015

@tagliala tagliala referenced this pull request in DavyJonesLocker/client_side_validations Feb 6, 2018

Open

'unsafe-eval' is not an allowed source of script #733

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment