]> source.dussan.org Git - redmine.git/commitdiff
code layout clean up app/controllers/issues_controller.rb
authorToshi MARUYAMA <marutosijp2@yahoo.co.jp>
Fri, 22 Nov 2019 16:49:43 +0000 (16:49 +0000)
committerToshi MARUYAMA <marutosijp2@yahoo.co.jp>
Fri, 22 Nov 2019 16:49:43 +0000 (16:49 +0000)
git-svn-id: http://svn.redmine.org/redmine/trunk@19176 e93f8b46-1217-0410-a6f0-8f06a7374b81

app/controllers/issues_controller.rb

index 82f165025723d959954ec2bddc44df1e3458209e..a3ca3bc6ab9d7be674d15eb2df93c1f1ed3656eb 100644 (file)
@@ -599,7 +599,9 @@ class IssuesController < ApplicationController
   # Saves @issue and a time_entry from the parameters
   def save_issue_with_child_records
     Issue.transaction do
-      if params[:time_entry] && (params[:time_entry][:hours].present? || params[:time_entry][:comments].present?) && User.current.allowed_to?(:log_time, @issue.project)
+      if params[:time_entry] &&
+           (params[:time_entry][:hours].present? || params[:time_entry][:comments].present?) &&
+           User.current.allowed_to?(:log_time, @issue.project)
         time_entry = @time_entry || TimeEntry.new
         time_entry.project = @issue.project
         time_entry.issue = @issue
@@ -608,10 +610,19 @@ class IssuesController < ApplicationController
         time_entry.safe_attributes = params[:time_entry]
         @issue.time_entries << time_entry
       end
-
-      call_hook(:controller_issues_edit_before_save, { :params => params, :issue => @issue, :time_entry => time_entry, :journal => @issue.current_journal})
+      call_hook(
+        :controller_issues_edit_before_save,
+        {:params => params, :issue => @issue,
+         :time_entry => time_entry,
+         :journal => @issue.current_journal}
+      )
       if @issue.save
-        call_hook(:controller_issues_edit_after_save, { :params => params, :issue => @issue, :time_entry => time_entry, :journal => @issue.current_journal})
+        call_hook(
+          :controller_issues_edit_after_save,
+          {:params => params, :issue => @issue,
+           :time_entry => time_entry,
+           :journal => @issue.current_journal}
+        )
       else
         raise ActiveRecord::Rollback
       end