ceef3f4bc9
* Re structure builder logic to allow for step conditionality Concerns - Performance. Look at whether the additional build in the steps controller can be reduced - Does not work if applied to the last step. - Certain conditions will not work with the first step(?) - How should this be scoped to known functionality? * Add indexes and conditions to steps and fields * Complete and add spec * Complete backend * Complete step conditionality and field indexing * Fix failing spec * Update coverage * Apply rubocop * Apply prettier * Apply prettier to wizard js * Fix schema issues created in merge * Remove setting label for force_final * Improve client wizard cache naming * Improve steps controller and spec conditionality * Improve final step attribute naming * Fix failing spec * Linting * Add one more final step test * Linting * Fix eslint issues * Apply prettier * Linting, syntax, merge and copy cleanups * Update wizard-admin.scss * Fix template linting * Rubocop fixes
109 Zeilen
3,3 KiB
Ruby
109 Zeilen
3,3 KiB
Ruby
# frozen_string_literal: true
|
|
class CustomWizard::StepsController < ::ApplicationController
|
|
before_action :ensure_logged_in
|
|
before_action :ensure_can_update
|
|
|
|
def update
|
|
update = update_params.to_h
|
|
|
|
update[:fields] = {}
|
|
if params[:fields]
|
|
field_ids = @step_template['fields'].map { |f| f['id'] }
|
|
params[:fields].each do |k, v|
|
|
update[:fields][k] = v if field_ids.include? k
|
|
end
|
|
end
|
|
|
|
@builder.build
|
|
|
|
updater = @builder.wizard.create_updater(update[:step_id], update[:fields])
|
|
updater.update
|
|
@result = updater.result
|
|
|
|
if updater.success?
|
|
wizard_id = update_params[:wizard_id]
|
|
builder = CustomWizard::Builder.new(wizard_id, current_user)
|
|
@wizard = builder.build
|
|
|
|
current_step = @wizard.find_step(update[:step_id])
|
|
current_submission = @wizard.current_submission
|
|
result = {}
|
|
|
|
if current_step.conditional_final_step && !current_step.last_step
|
|
current_step.force_final = true
|
|
end
|
|
|
|
if current_step.final?
|
|
builder.template.actions.each do |action_template|
|
|
if action_template['run_after'] === 'wizard_completion'
|
|
CustomWizard::Action.new(
|
|
action: action_template,
|
|
wizard: @wizard,
|
|
data: current_submission
|
|
).perform
|
|
end
|
|
end
|
|
|
|
@wizard.save_submission(current_submission)
|
|
|
|
if redirect = get_redirect
|
|
updater.result[:redirect_on_complete] = redirect
|
|
end
|
|
|
|
@wizard.final_cleanup!
|
|
|
|
result[:final] = true
|
|
else
|
|
result[:final] = false
|
|
result[:next_step_id] = current_step.next.id
|
|
end
|
|
|
|
result.merge!(updater.result) if updater.result.present?
|
|
result[:refresh_required] = true if updater.refresh_required?
|
|
result[:wizard] = ::CustomWizard::WizardSerializer.new(
|
|
@wizard,
|
|
scope: Guardian.new(current_user),
|
|
root: false
|
|
).as_json
|
|
|
|
render json: result
|
|
else
|
|
errors = []
|
|
updater.errors.messages.each do |field, msg|
|
|
errors << { field: field, description: msg.join(',') }
|
|
end
|
|
render json: { errors: errors }, status: 422
|
|
end
|
|
end
|
|
|
|
private
|
|
|
|
def ensure_can_update
|
|
@builder = CustomWizard::Builder.new(update_params[:wizard_id], current_user)
|
|
raise Discourse::InvalidParameters.new(:wizard_id) if @builder.template.nil?
|
|
raise Discourse::InvalidAccess.new if !@builder.wizard || !@builder.wizard.can_access?
|
|
|
|
@step_template = @builder.template.steps.select do |s|
|
|
s['id'] == update_params[:step_id]
|
|
end.first
|
|
raise Discourse::InvalidParameters.new(:step_id) if !@step_template
|
|
raise Discourse::InvalidAccess.new if !@builder.check_condition(@step_template)
|
|
end
|
|
|
|
def update_params
|
|
@update_params || begin
|
|
params.require(:step_id)
|
|
params.require(:wizard_id)
|
|
params.permit(:wizard_id, :step_id).transform_values { |v| v.underscore }
|
|
end
|
|
end
|
|
|
|
def get_redirect
|
|
return @result[:redirect_on_next] if @result[:redirect_on_next].present?
|
|
|
|
current_submission = @wizard.current_submission
|
|
return nil unless current_submission.present?
|
|
## route_to set by actions, redirect_on_complete set by actions, redirect_to set at wizard entry
|
|
current_submission[:route_to] || current_submission[:redirect_on_complete] || current_submission[:redirect_to]
|
|
end
|
|
end
|