From 9381d8cc3a167dbac73241324f13fb6cb40f228d Mon Sep 17 00:00:00 2001 From: Karel Picman Date: Wed, 25 Sep 2013 15:50:34 +0200 Subject: [PATCH] Do not assign an empty workflow --- app/controllers/dmsf_workflows_controller.rb | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/app/controllers/dmsf_workflows_controller.rb b/app/controllers/dmsf_workflows_controller.rb index f19319ae..429e2667 100644 --- a/app/controllers/dmsf_workflows_controller.rb +++ b/app/controllers/dmsf_workflows_controller.rb @@ -123,14 +123,16 @@ class DmsfWorkflowsController < ApplicationController revision = DmsfFileRevision.find_by_id params[:dmsf_file_revision_id] if revision revision.set_workflow(params[:dmsf_workflow_id], params[:action]) - revision.assign_workflow(params[:dmsf_workflow_id]) - if request.post? - if revision.save - file = DmsfFile.find_by_id revision.dmsf_file_id - file.lock! if file - flash[:notice] = l(:notice_successful_update) - else - flash[:error] = l(:error_workflow_assign) + if params[:dmsf_workflow_id].present? + revision.assign_workflow(params[:dmsf_workflow_id]) + if request.post? + if revision.save + file = DmsfFile.find_by_id revision.dmsf_file_id + file.lock! if file + flash[:notice] = l(:notice_successful_update) + else + flash[:error] = l(:error_workflow_assign) + end end end end