##// END OF EJS Templates
Updated code comments....
Jean-Philippe Lang -
r13618:8534fc04a0fc
parent child
Show More
@@ -132,8 +132,6 class IssuesController < ApplicationController
132 end
132 end
133 end
133 end
134
134
135 # Add a new issue
136 # The new issue will be created from an existing one if copy_from parameter is given
137 def new
135 def new
138 respond_to do |format|
136 respond_to do |format|
139 format.html { render :action => 'new', :layout => !request.xhr? }
137 format.html { render :action => 'new', :layout => !request.xhr? }
@@ -373,7 +371,6 class IssuesController < ApplicationController
373
371
374 # Used by #edit and #update to set some common instance variables
372 # Used by #edit and #update to set some common instance variables
375 # from the params
373 # from the params
376 # TODO: Refactor, not everything in here is needed by #edit
377 def update_issue_from_params
374 def update_issue_from_params
378 @time_entry = TimeEntry.new(:issue => @issue, :project => @issue.project)
375 @time_entry = TimeEntry.new(:issue => @issue, :project => @issue.project)
379 if params[:time_entry]
376 if params[:time_entry]
@@ -401,7 +398,8 class IssuesController < ApplicationController
401 true
398 true
402 end
399 end
403
400
404 # TODO: Refactor, lots of extra code in here
401 # Used by #new and #create to build a new issue from the params
402 # The new issue will be copied from an existing one if copy_from parameter is given
405 def build_new_issue_from_params
403 def build_new_issue_from_params
406 @issue = Issue.new
404 @issue = Issue.new
407 if params[:copy_from]
405 if params[:copy_from]
@@ -487,6 +485,8 class IssuesController < ApplicationController
487 end
485 end
488 end
486 end
489
487
488 # Returns true if the issue copy should be linked
489 # to the original issue
490 def link_copy?(param)
490 def link_copy?(param)
491 case Setting.link_copied_issue
491 case Setting.link_copied_issue
492 when 'yes'
492 when 'yes'
General Comments 0
You need to be logged in to leave comments. Login now