11832cc07d
When customizing CONSUL, one of the most common actions is adding a new
field to a form.
This requires modifying the permitted/allowed parameters. However, in
most cases, the method returning these parameters returned an instance
of `ActionController::Parameters`, so adding more parameters to it
wasn't easy.
So customizing the code required copying the method returning those
parameters and adding the new ones. For example:
```
def something_params
params.require(:something).permit(
:one_consul_attribute,
:another_consul_attribute,
:my_custom_attribute
)
end
```
This meant that, if the `something_params` method changed in CONSUL, the
customization of this method had to be updated as well.
So we're extracting the logic returning the parameters to a method which
returns an array. Now this code can be customized without copying the
original method:
```
alias_method :consul_allowed_params, :allowed_params
def allowed_params
consul_allowed_params + [:my_custom_attribute]
end
```
36 lines
1.1 KiB
Ruby
36 lines
1.1 KiB
Ruby
class RemoteTranslationsController < ApplicationController
|
|
skip_authorization_check
|
|
respond_to :html, :js
|
|
|
|
before_action :set_remote_translations, only: :create
|
|
|
|
def create
|
|
@remote_translations.each do |remote_translation|
|
|
RemoteTranslation.create!(remote_translation) unless translations_enqueued?(remote_translation)
|
|
end
|
|
redirect_to request.referer, notice: t("remote_translations.create.enqueue_remote_translation")
|
|
end
|
|
|
|
private
|
|
|
|
def remote_translations_params
|
|
params.permit(allowed_params)
|
|
end
|
|
|
|
def allowed_params
|
|
[:remote_translations]
|
|
end
|
|
|
|
def set_remote_translations
|
|
remote_translations = remote_translations_params["remote_translations"]
|
|
decoded_remote_translations = ActiveSupport::JSON.decode(remote_translations)
|
|
@remote_translations = decoded_remote_translations.map do |remote_translation|
|
|
remote_translation.slice("remote_translatable_id", "remote_translatable_type", "locale")
|
|
end
|
|
end
|
|
|
|
def translations_enqueued?(remote_translation)
|
|
RemoteTranslation.remote_translation_enqueued?(remote_translation)
|
|
end
|
|
end
|