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 an undefined method `start_with?' for nil:NilClass #7936

Merged
merged 1 commit into from Apr 13, 2016

Conversation

h-kataria
Copy link
Contributor

st_prov_type is nil when adding a Catalog Bundle, st_prov_type is only on the initial screen when user selects type of Catalog Item they want to add.
Issue was introduced in commit 2c95dd5

@bzwei please review.

@bzwei
Copy link
Contributor

bzwei commented Apr 13, 2016

LGTM

@@ -0,0 +1,13 @@
describe "catalog/_form.html.haml" do
before(:each) do
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(:each) is the default and thus you don't need to specify it (minor, so I won't let it hold up the merge)

st_prov_type is nil when adding a Catalog Bundle, st_prov_type is only on the initial screen when user selects type of Catalog Item they want to add.
Issue was introduced in commit 2c95dd5
@miq-bot
Copy link
Member

miq-bot commented Apr 13, 2016

Checked commit h-kataria@1b731e6 with ruby 2.2.3, rubocop 0.37.2, and haml-lint 0.16.1
2 files checked, 0 offenses detected
Everything looks good. ⭐

@h-kataria
Copy link
Contributor Author

@dclarizio @martinpovolny please merge

@Fryguy Fryguy merged commit 725c8a8 into ManageIQ:master Apr 13, 2016
@Fryguy Fryguy added this to the Sprint 39 Ending Apr 18, 2016 milestone Apr 13, 2016
@dclarizio dclarizio deleted the catalog_item_editor_fix branch April 20, 2016 22:04
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.

None yet

4 participants