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

Ensure report columns serialized as hashes have symbolized keys before importing #15273

Merged
merged 1 commit into from
Jun 2, 2017

Conversation

gtanzillo
Copy link
Member

This fixes a bug where a report imported via the Rest API, as json, fails to generate because the keys
in columns serialized as hashes are received as strings.

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

…e importing

This fixes a bug where a report imported via the Rest API, as json, fails to generate because the keys
in columns serialized as hashes are received as strings.

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

miq-bot commented Jun 1, 2017

Checked commit gtanzillo@ef760a7 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 3 offenses detected

app/models/miq_report/import_export.rb

spec/models/miq_report/import_export_spec.rb

@chessbyte chessbyte self-assigned this Jun 2, 2017
@chessbyte chessbyte merged commit 832975b into ManageIQ:master Jun 2, 2017
@chessbyte chessbyte added this to the Sprint 62 Ending Jun 5, 2017 milestone Jun 2, 2017
simaishi pushed a commit that referenced this pull request Jun 5, 2017
Ensure report columns serialized as hashes have symbolized keys before importing
(cherry picked from commit 832975b)

https://bugzilla.redhat.com/show_bug.cgi?id=1458810
@simaishi
Copy link
Contributor

simaishi commented Jun 5, 2017

Euwe backport details:

$ git log -1
commit 409d6797c38e1839baca3b25798aa87377004169
Author: Oleg Barenboim <chessbyte@gmail.com>
Date:   Thu Jun 1 23:00:35 2017 -0400

    Merge pull request #15273 from gtanzillo/report-import-symbolize-keys
    
    Ensure report columns serialized as hashes have symbolized keys before importing
    (cherry picked from commit 832975bd4a6cbb4a041acbaf695a29a0201405ee)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1458810

simaishi pushed a commit that referenced this pull request Jun 9, 2017
Ensure report columns serialized as hashes have symbolized keys before importing
(cherry picked from commit 832975b)

https://bugzilla.redhat.com/show_bug.cgi?id=1460396
@simaishi
Copy link
Contributor

simaishi commented Jun 9, 2017

Fine backport details:

$ git log -1
commit bc3d903ebc4944ce06cc51f44ded190049cb4e59
Author: Oleg Barenboim <chessbyte@gmail.com>
Date:   Thu Jun 1 23:00:35 2017 -0400

    Merge pull request #15273 from gtanzillo/report-import-symbolize-keys
    
    Ensure report columns serialized as hashes have symbolized keys before importing
    (cherry picked from commit 832975bd4a6cbb4a041acbaf695a29a0201405ee)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1460396

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.

4 participants