Project

General

Profile

Actions
Copy link

Feature #5131

closed

Query.has_column() should support has_column(:column_name)

Added by Randy Syring about 14 years ago. Updated over 11 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Code cleanup/refactoring
Target version:
-
Start date:
2010-03-19
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed

Description

(01:36:10 PM) edavis10: rsyring: @query.has_column() expects a QueryColumn object

(01:36:54 PM) edavis10: rsyring: though we should change it to support @query.has_column(:name) and @query.has_column?(QueryColumn....)

Actions
Copy link
 #1

Updated by Randy Syring about 14 years ago

I have a topic branch with this mod and tests completed:

http://github.com/rsyring/redmine/tree/5131_has_column_mod

Actions
Copy link
 #2

Updated by Daniel Felix over 11 years ago

Hi there,

any news on this ticket?

Actions
Copy link
 #3

Updated by Etienne Massip over 11 years ago

  • Status changed from New to Closed
  • Resolution set to Fixed

This has been done some time ago with r8073, see source:/trunk/app/models/query.rb@11174#L390.

It just takes a String instead of a Symbol.

Actions
Copy link

Also available in: Atom PDF