Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename extension_white_list ~> extension_whitelist #1819

Conversation

mehlah
Copy link
Member

@mehlah mehlah commented Jan 2, 2016

Breaking changes:

  • Rename extension_white_list ~> extension_whitelist
  • Rename extension_black_list ~> extension_blacklist.
  • Rename i18n keys extension_black_list_error ~> extension_blacklist_error
    and extension_white_list_error ~> extension_whitelist_error

Rename check_blacklist! internal callback ~> check_extension_blacklist, which
is more precise and less prone to collisions with other callbacks
methods names.

Breaking changes:
* Rename `extension_white_list` ~> `extension_whitelist`
* Rename `extension_black_list` ~> `extension_blacklist`.
* Rename i18n keys `extension_black_list_error` ~> `extension_blacklist_error`
  and `extension_white_list_error` ~> `extension_whitelist_error`

Rename `check_blacklist!` internal callback ~> `check_extension_blacklist`, which
is more precise and less prone to collisions with other callbacks
methods names.
thomasfedb added a commit that referenced this pull request Jan 2, 2016
Rename file extension filtering methods for consistency
@thomasfedb thomasfedb merged commit 8076b86 into carrierwaveuploader:master Jan 2, 2016
@thomasfedb
Copy link
Contributor

@mehlah mehlah deleted the refactor-extensions-white/blacklist branch January 2, 2016 19:34
@ansonhoyt
Copy link

This was mentioned in the 0.11.0 changelog, but my testing still shows the old behavior.

@mshibuya mshibuya mentioned this pull request Aug 11, 2016
weiqingtoh added a commit to Coursemology/coursemology2 that referenced this pull request Jan 4, 2017
weiqingtoh added a commit to Coursemology/coursemology2 that referenced this pull request Jan 14, 2017
weiqingtoh added a commit to Coursemology/coursemology2 that referenced this pull request Jan 15, 2017
weiqingtoh added a commit to Coursemology/coursemology2 that referenced this pull request Jan 23, 2017
weiqingtoh added a commit to Coursemology/coursemology2 that referenced this pull request Jan 24, 2017
allenwq pushed a commit to Coursemology/coursemology2 that referenced this pull request Feb 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants