| @@ -260,15 +260,6 class ApplicationController < ActionController::Base | |||
|
|
260 | 260 | render_404 |
|
|
261 | 261 | end |
|
|
262 | 262 | |
|
|
263 | # Check if project is unique before bulk operations | |
|
|
264 | def check_project_uniqueness | |
|
|
265 | unless @project | |
|
|
266 | # TODO: let users bulk edit/move/destroy issues from different projects | |
|
|
267 | render_error 'Can not bulk edit/move/destroy issues from different projects' | |
|
|
268 | return false | |
|
|
269 | end | |
|
|
270 | end | |
|
|
271 | ||
|
|
272 | 263 | # make sure that the user is a member of the project (or admin) if project is private |
|
|
273 | 264 | # used as a before_filter for actions that do not require any particular permission on the project |
|
|
274 | 265 | def check_project_privacy |
| @@ -20,8 +20,7 class IssuesController < ApplicationController | |||
|
|
20 | 20 | default_search_scope :issues |
|
|
21 | 21 | |
|
|
22 | 22 | before_filter :find_issue, :only => [:show, :edit, :update] |
|
|
23 |
before_filter :find_issues, :only => [:bulk_edit, :bulk_update, |
|
|
|
24 | before_filter :check_project_uniqueness, :only => [:move, :perform_move] | |
|
|
23 | before_filter :find_issues, :only => [:bulk_edit, :bulk_update, :destroy] | |
|
|
25 | 24 | before_filter :find_project, :only => [:new, :create] |
|
|
26 | 25 | before_filter :authorize, :except => [:index] |
|
|
27 | 26 | before_filter :find_optional_project, :only => [:index] |
General Comments 0
You need to be logged in to leave comments.
Login now