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

Fixed condition in show_adv_search? method #5440

Merged
merged 1 commit into from
Nov 13, 2015

Conversation

h-kataria
Copy link
Contributor

  • Some explorer screens such as Automate/Simulation do not have tree on the left, there is no need to check whether to display advanced search on those screens.
  • Added spec test to verify fix.

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

@dclarizio please review

- Some explorer screens such as Automate/Simulation do not have tree on the left, there is no need to check whether to display advanced search on those screens.
- Added spec test to verify fix.

https://bugzilla.redhat.com/show_bug.cgi?id=1281860
@miq-bot
Copy link
Member

miq-bot commented Nov 13, 2015

Checked commit h-kataria@fb5f460 with ruby 1.9.3, rubocop 0.34.2, and haml-lint 0.13.0
2 files checked, 2 offenses detected

app/helpers/application_helper.rb

  • 🔴 Error - Line 1288, Col 8 - Syntax - unexpected token tSTRING_CONTENT

spec/helpers/application_helper_spec.rb

dclarizio pushed a commit that referenced this pull request Nov 13, 2015
Fixed condition in show_adv_search? method
@dclarizio dclarizio merged commit 00bc869 into ManageIQ:master Nov 13, 2015
@dclarizio dclarizio deleted the fixed_advanced_search_check branch November 13, 2015 22:01
@dclarizio dclarizio added this to the Sprint 32 Ending Nov 16, 2015 milestone Nov 13, 2015
dclarizio pushed a commit that referenced this pull request Nov 17, 2015
Fixed condition in show_adv_search? method
blomquisg pushed a commit to blomquisg/manageiq that referenced this pull request Nov 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants