Navigation Menu

Skip to content

Commit

Permalink
Refactor: pull #query_statement_invalid up to ApplicationController.
Browse files Browse the repository at this point in the history
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@3696 e93f8b46-1217-0410-a6f0-8f06a7374b81
  • Loading branch information
edavis10 committed Apr 29, 2010
1 parent e65c3cf commit 488879d
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 17 deletions.
9 changes: 9 additions & 0 deletions app/controllers/application_controller.rb
Expand Up @@ -329,4 +329,13 @@ def api_request?
def render_attachment_warning_if_needed(obj)
flash[:warning] = l(:warning_attachments_not_saved, obj.unsaved_attachments.size) if obj.unsaved_attachments.present?
end

# Rescues an invalid query statement. Just in case...
def query_statement_invalid(exception)
logger.error "Query::StatementInvalid: #{exception.message}" if logger
session.delete(:query)
sort_clear if respond_to?(:sort_clear)
render_error "An error occurred while executing the query and has been logged. Please report this error to your Redmine administrator."
end

end
11 changes: 2 additions & 9 deletions app/controllers/gantts_controller.rb
Expand Up @@ -7,6 +7,8 @@ class GanttsController < ApplicationController
helper :projects
helper :queries
include QueriesHelper
helper :sort
include SortHelper
include Redmine::Export::PDF

def show
Expand Down Expand Up @@ -42,15 +44,6 @@ def show

private

# Rescues an invalid query statement. Just in case...
# TODO: Refactor, move to ApplicationController with IssuesController
def query_statement_invalid(exception)
logger.error "Query::StatementInvalid: #{exception.message}" if logger
session.delete(:query)
sort_clear
render_error "An error occurred while executing the query and has been logged. Please report this error to your Redmine administrator."
end

# TODO: Refactor, duplicates IssuesController
def find_optional_project
@project = Project.find(params[:project_id]) unless params[:project_id].blank?
Expand Down
8 changes: 0 additions & 8 deletions app/controllers/issues_controller.rb
Expand Up @@ -453,14 +453,6 @@ def find_optional_project
render_404
end

# Rescues an invalid query statement. Just in case...
def query_statement_invalid(exception)
logger.error "Query::StatementInvalid: #{exception.message}" if logger
session.delete(:query)
sort_clear
render_error "An error occurred while executing the query and has been logged. Please report this error to your Redmine administrator."
end

# Used by #edit and #update to set some common instance variables
# from the params
# TODO: Refactor, not everything in here is needed by #edit
Expand Down

0 comments on commit 488879d

Please sign in to comment.