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

Fix key for regexp in miq_expression.yaml #15452

Merged
merged 1 commit into from
Jun 27, 2017

Conversation

lpichler
Copy link
Contributor

there is a lot of occurrences of :regexp
miq_expresion.rb and also in
app/controllers/application_controller/filter/expression.rb
and it fixes to edit of expression with regexp in UI
(report definition, filter tab)
so there is the typo in yaml.
UI error:
before:
screen shot 2017-06-27 at 09 45 29
after:
screen shot 2017-06-27 at 09 52 28

@miq-bot add_label reporting, bug
@miq-bot assign @gtanzillo

there is lot of occurencies of :regexp
miq_expresion.rb and also in
app/controllers/application_controller/filter/expression.rb
and it fixes editing of expression with regexp in UI
(report definition, filter tab)
so there is typo in yaml.
@miq-bot
Copy link
Member

miq-bot commented Jun 27, 2017

Checked commit lpichler@2d56185 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 🍪

Copy link
Member

@gtanzillo gtanzillo left a comment

Choose a reason for hiding this comment

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

👍

@gtanzillo gtanzillo added this to the Sprint 64 Ending Jul 3, 2017 milestone Jun 27, 2017
@gtanzillo gtanzillo merged commit 1a66602 into ManageIQ:master Jun 27, 2017
@lpichler lpichler deleted the fix_key_for_regexp branch June 27, 2017 14:28
@yrudman
Copy link
Contributor

yrudman commented Jun 28, 2017

This PR should fix BZ https://bugzilla.redhat.com/show_bug.cgi?id=1447374

@simaishi
Copy link
Contributor

simaishi commented Aug 2, 2017

Made the change to lib/miq_expression.rb in Fine branch.

Fine backport details:

$ git log -1
commit de2fbcb8aa7f75efd058ecdcc647c8cb7c7edb5a
Author: Gregg Tanzillo <gtanzill@redhat.com>
Date:   Tue Jun 27 10:15:55 2017 -0400

    Merge pull request #15452 from lpichler/fix_key_for_regexp
    
    Fix key for regexp in miq_expression.yaml
    (cherry picked from commit 1a666022ce92a0691a4193e5e9f8eb7c4d7f5c4e)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1477702
    https://bugzilla.redhat.com/show_bug.cgi?id=1477701

d-m-u pushed a commit to d-m-u/manageiq that referenced this pull request Jun 6, 2018
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.

6 participants