Fixed that Query#has_column? returns false with default columns.

git-svn-id: http://svn.redmine.org/redmine/trunk@16217 e93f8b46-1217-0410-a6f0-8f06a7374b81

app/models/query.rb

@@ -679,7 +679,8 @@ class Query < ActiveRecord::Base
   end
 
   def has_column?(column)
-    column_names && column_names.include?(column.is_a?(QueryColumn) ? column.name : column)
+    name = column.is_a?(QueryColumn) ? column.name : column
+    columns.detect {|c| c.name == name}
   end
 
   def has_custom_field_column?

test/unit/query_test.rb

@@ -1170,6 +1170,14 @@ class QueryTest < ActiveSupport::TestCase
     assert !q.has_column?(category_column)
   end
 
+  def test_has_column_should_return_true_for_default_column
+    with_settings :issue_list_default_columns => %w(tracker subject) do
+      q = IssueQuery.new
+      assert q.has_column?(:tracker)
+      assert !q.has_column?(:category)
+    end
+  end
+
   def test_inline_and_block_columns
     q = IssueQuery.new
     q.column_names = ['subject', 'description', 'tracker']