##// END OF EJS Templates
Rails4: replace deprecated find_all_by_* at TimelogController#find_time_entries...
Toshi MARUYAMA -
r12328:b02429e86423
parent child
Show More
@@ -1,297 +1,297
1 1 # Redmine - project management software
2 2 # Copyright (C) 2006-2013 Jean-Philippe Lang
3 3 #
4 4 # This program is free software; you can redistribute it and/or
5 5 # modify it under the terms of the GNU General Public License
6 6 # as published by the Free Software Foundation; either version 2
7 7 # of the License, or (at your option) any later version.
8 8 #
9 9 # This program is distributed in the hope that it will be useful,
10 10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
11 11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 12 # GNU General Public License for more details.
13 13 #
14 14 # You should have received a copy of the GNU General Public License
15 15 # along with this program; if not, write to the Free Software
16 16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
17 17
18 18 class TimelogController < ApplicationController
19 19 menu_item :issues
20 20
21 21 before_filter :find_project_for_new_time_entry, :only => [:create]
22 22 before_filter :find_time_entry, :only => [:show, :edit, :update]
23 23 before_filter :find_time_entries, :only => [:bulk_edit, :bulk_update, :destroy]
24 24 before_filter :authorize, :except => [:new, :index, :report]
25 25
26 26 before_filter :find_optional_project, :only => [:index, :report]
27 27 before_filter :find_optional_project_for_new_time_entry, :only => [:new]
28 28 before_filter :authorize_global, :only => [:new, :index, :report]
29 29
30 30 accept_rss_auth :index
31 31 accept_api_auth :index, :show, :create, :update, :destroy
32 32
33 33 rescue_from Query::StatementInvalid, :with => :query_statement_invalid
34 34
35 35 helper :sort
36 36 include SortHelper
37 37 helper :issues
38 38 include TimelogHelper
39 39 helper :custom_fields
40 40 include CustomFieldsHelper
41 41 helper :queries
42 42 include QueriesHelper
43 43
44 44 def index
45 45 @query = TimeEntryQuery.build_from_params(params, :project => @project, :name => '_')
46 46
47 47 sort_init(@query.sort_criteria.empty? ? [['spent_on', 'desc']] : @query.sort_criteria)
48 48 sort_update(@query.sortable_columns)
49 49 scope = time_entry_scope(:order => sort_clause).
50 50 includes(:project, :user, :issue).
51 51 preload(:issue => [:project, :tracker, :status, :assigned_to, :priority])
52 52
53 53 respond_to do |format|
54 54 format.html {
55 55 @entry_count = scope.count
56 56 @entry_pages = Paginator.new @entry_count, per_page_option, params['page']
57 57 @entries = scope.offset(@entry_pages.offset).limit(@entry_pages.per_page).all
58 58 @total_hours = scope.sum(:hours).to_f
59 59
60 60 render :layout => !request.xhr?
61 61 }
62 62 format.api {
63 63 @entry_count = scope.count
64 64 @offset, @limit = api_offset_and_limit
65 65 @entries = scope.offset(@offset).limit(@limit).preload(:custom_values => :custom_field).all
66 66 }
67 67 format.atom {
68 68 entries = scope.limit(Setting.feeds_limit.to_i).reorder("#{TimeEntry.table_name}.created_on DESC").all
69 69 render_feed(entries, :title => l(:label_spent_time))
70 70 }
71 71 format.csv {
72 72 # Export all entries
73 73 @entries = scope.all
74 74 send_data(query_to_csv(@entries, @query, params), :type => 'text/csv; header=present', :filename => 'timelog.csv')
75 75 }
76 76 end
77 77 end
78 78
79 79 def report
80 80 @query = TimeEntryQuery.build_from_params(params, :project => @project, :name => '_')
81 81 scope = time_entry_scope
82 82
83 83 @report = Redmine::Helpers::TimeReport.new(@project, @issue, params[:criteria], params[:columns], scope)
84 84
85 85 respond_to do |format|
86 86 format.html { render :layout => !request.xhr? }
87 87 format.csv { send_data(report_to_csv(@report), :type => 'text/csv; header=present', :filename => 'timelog.csv') }
88 88 end
89 89 end
90 90
91 91 def show
92 92 respond_to do |format|
93 93 # TODO: Implement html response
94 94 format.html { render :nothing => true, :status => 406 }
95 95 format.api
96 96 end
97 97 end
98 98
99 99 def new
100 100 @time_entry ||= TimeEntry.new(:project => @project, :issue => @issue, :user => User.current, :spent_on => User.current.today)
101 101 @time_entry.safe_attributes = params[:time_entry]
102 102 end
103 103
104 104 def create
105 105 @time_entry ||= TimeEntry.new(:project => @project, :issue => @issue, :user => User.current, :spent_on => User.current.today)
106 106 @time_entry.safe_attributes = params[:time_entry]
107 107
108 108 call_hook(:controller_timelog_edit_before_save, { :params => params, :time_entry => @time_entry })
109 109
110 110 if @time_entry.save
111 111 respond_to do |format|
112 112 format.html {
113 113 flash[:notice] = l(:notice_successful_create)
114 114 if params[:continue]
115 115 if params[:project_id]
116 116 options = {
117 117 :time_entry => {:issue_id => @time_entry.issue_id, :activity_id => @time_entry.activity_id},
118 118 :back_url => params[:back_url]
119 119 }
120 120 if @time_entry.issue
121 121 redirect_to new_project_issue_time_entry_path(@time_entry.project, @time_entry.issue, options)
122 122 else
123 123 redirect_to new_project_time_entry_path(@time_entry.project, options)
124 124 end
125 125 else
126 126 options = {
127 127 :time_entry => {:project_id => @time_entry.project_id, :issue_id => @time_entry.issue_id, :activity_id => @time_entry.activity_id},
128 128 :back_url => params[:back_url]
129 129 }
130 130 redirect_to new_time_entry_path(options)
131 131 end
132 132 else
133 133 redirect_back_or_default project_time_entries_path(@time_entry.project)
134 134 end
135 135 }
136 136 format.api { render :action => 'show', :status => :created, :location => time_entry_url(@time_entry) }
137 137 end
138 138 else
139 139 respond_to do |format|
140 140 format.html { render :action => 'new' }
141 141 format.api { render_validation_errors(@time_entry) }
142 142 end
143 143 end
144 144 end
145 145
146 146 def edit
147 147 @time_entry.safe_attributes = params[:time_entry]
148 148 end
149 149
150 150 def update
151 151 @time_entry.safe_attributes = params[:time_entry]
152 152
153 153 call_hook(:controller_timelog_edit_before_save, { :params => params, :time_entry => @time_entry })
154 154
155 155 if @time_entry.save
156 156 respond_to do |format|
157 157 format.html {
158 158 flash[:notice] = l(:notice_successful_update)
159 159 redirect_back_or_default project_time_entries_path(@time_entry.project)
160 160 }
161 161 format.api { render_api_ok }
162 162 end
163 163 else
164 164 respond_to do |format|
165 165 format.html { render :action => 'edit' }
166 166 format.api { render_validation_errors(@time_entry) }
167 167 end
168 168 end
169 169 end
170 170
171 171 def bulk_edit
172 172 @available_activities = TimeEntryActivity.shared.active
173 173 @custom_fields = TimeEntry.first.available_custom_fields
174 174 end
175 175
176 176 def bulk_update
177 177 attributes = parse_params_for_bulk_time_entry_attributes(params)
178 178
179 179 unsaved_time_entry_ids = []
180 180 @time_entries.each do |time_entry|
181 181 time_entry.reload
182 182 time_entry.safe_attributes = attributes
183 183 call_hook(:controller_time_entries_bulk_edit_before_save, { :params => params, :time_entry => time_entry })
184 184 unless time_entry.save
185 185 logger.info "time entry could not be updated: #{time_entry.errors.full_messages}" if logger && logger.info
186 186 # Keep unsaved time_entry ids to display them in flash error
187 187 unsaved_time_entry_ids << time_entry.id
188 188 end
189 189 end
190 190 set_flash_from_bulk_time_entry_save(@time_entries, unsaved_time_entry_ids)
191 191 redirect_back_or_default project_time_entries_path(@projects.first)
192 192 end
193 193
194 194 def destroy
195 195 destroyed = TimeEntry.transaction do
196 196 @time_entries.each do |t|
197 197 unless t.destroy && t.destroyed?
198 198 raise ActiveRecord::Rollback
199 199 end
200 200 end
201 201 end
202 202
203 203 respond_to do |format|
204 204 format.html {
205 205 if destroyed
206 206 flash[:notice] = l(:notice_successful_delete)
207 207 else
208 208 flash[:error] = l(:notice_unable_delete_time_entry)
209 209 end
210 210 redirect_back_or_default project_time_entries_path(@projects.first)
211 211 }
212 212 format.api {
213 213 if destroyed
214 214 render_api_ok
215 215 else
216 216 render_validation_errors(@time_entries)
217 217 end
218 218 }
219 219 end
220 220 end
221 221
222 222 private
223 223 def find_time_entry
224 224 @time_entry = TimeEntry.find(params[:id])
225 225 unless @time_entry.editable_by?(User.current)
226 226 render_403
227 227 return false
228 228 end
229 229 @project = @time_entry.project
230 230 rescue ActiveRecord::RecordNotFound
231 231 render_404
232 232 end
233 233
234 234 def find_time_entries
235 @time_entries = TimeEntry.find_all_by_id(params[:id] || params[:ids])
235 @time_entries = TimeEntry.where(:id => params[:id] || params[:ids]).all
236 236 raise ActiveRecord::RecordNotFound if @time_entries.empty?
237 237 @projects = @time_entries.collect(&:project).compact.uniq
238 238 @project = @projects.first if @projects.size == 1
239 239 rescue ActiveRecord::RecordNotFound
240 240 render_404
241 241 end
242 242
243 243 def set_flash_from_bulk_time_entry_save(time_entries, unsaved_time_entry_ids)
244 244 if unsaved_time_entry_ids.empty?
245 245 flash[:notice] = l(:notice_successful_update) unless time_entries.empty?
246 246 else
247 247 flash[:error] = l(:notice_failed_to_save_time_entries,
248 248 :count => unsaved_time_entry_ids.size,
249 249 :total => time_entries.size,
250 250 :ids => '#' + unsaved_time_entry_ids.join(', #'))
251 251 end
252 252 end
253 253
254 254 def find_optional_project_for_new_time_entry
255 255 if (project_id = (params[:project_id] || params[:time_entry] && params[:time_entry][:project_id])).present?
256 256 @project = Project.find(project_id)
257 257 end
258 258 if (issue_id = (params[:issue_id] || params[:time_entry] && params[:time_entry][:issue_id])).present?
259 259 @issue = Issue.find(issue_id)
260 260 @project ||= @issue.project
261 261 end
262 262 rescue ActiveRecord::RecordNotFound
263 263 render_404
264 264 end
265 265
266 266 def find_project_for_new_time_entry
267 267 find_optional_project_for_new_time_entry
268 268 if @project.nil?
269 269 render_404
270 270 end
271 271 end
272 272
273 273 def find_optional_project
274 274 if !params[:issue_id].blank?
275 275 @issue = Issue.find(params[:issue_id])
276 276 @project = @issue.project
277 277 elsif !params[:project_id].blank?
278 278 @project = Project.find(params[:project_id])
279 279 end
280 280 end
281 281
282 282 # Returns the TimeEntry scope for index and report actions
283 283 def time_entry_scope(options={})
284 284 scope = @query.results_scope(options)
285 285 if @issue
286 286 scope = scope.on_issue(@issue)
287 287 end
288 288 scope
289 289 end
290 290
291 291 def parse_params_for_bulk_time_entry_attributes(params)
292 292 attributes = (params[:time_entry] || {}).reject {|k,v| v.blank?}
293 293 attributes.keys.each {|k| attributes[k] = '' if attributes[k] == 'none'}
294 294 attributes[:custom_field_values].reject! {|k,v| v.blank?} if attributes[:custom_field_values]
295 295 attributes
296 296 end
297 297 end
General Comments 0
You need to be logged in to leave comments. Login now