Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add :select to legacy finder options #5211

Merged

Conversation

carbonin
Copy link
Member

This caused issues when find was called like is is here

VimPerformanceDaily.find(:all, :conditions => cond, :ext_options => options[:ext_options], :select => options[:select])

https://bugzilla.redhat.com/show_bug.cgi?id=1267749

@carbonin carbonin force-pushed the add_select_to_legacy_finder_options branch from 9c8d15b to a418611 Compare October 30, 2015 19:30
@miq-bot
Copy link
Member

miq-bot commented Oct 30, 2015

Checked commit carbonin@a418611 with ruby 1.9.3, rubocop 0.34.2, and haml-lint 0.13.0
1 file checked, 0 offenses detected
Everything looks good. 👍

@gtanzillo
Copy link
Member

👍 Looks good

gtanzillo added a commit that referenced this pull request Oct 30, 2015
…tions

Add :select to legacy finder options
@gtanzillo gtanzillo merged commit 5a719e7 into ManageIQ:master Oct 30, 2015
@chessbyte chessbyte added this to the Sprint 32 Ending Nov 16, 2015 milestone Nov 1, 2015
gtanzillo added a commit that referenced this pull request Nov 10, 2015
…tions

Add :select to legacy finder options
@carbonin carbonin deleted the add_select_to_legacy_finder_options branch November 13, 2015 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants