Bug 1306699

Summary: UI: Download file from saved report is named after the current date rather than after the report saved date
Product: Red Hat CloudForms Management Engine Reporter: Chris Pelland <cpelland>
Component: UI - OPSAssignee: Harpreet Kataria <hkataria>
Status: CLOSED ERRATA QA Contact: Jan Krocil <jkrocil>
Severity: low Docs Contact:
Priority: medium    
Version: 5.3.0CC: cpelland, dajohnso, dclarizi, fdewaley, hkataria, jdeubel, jhardy, jprause, mfeifer, mpovolny, obarenbo, snansi, xlecauch
Target Milestone: GAKeywords: ZStream
Target Release: 5.5.3   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.5.3.2 Doc Type: Bug Fix
Doc Text:
In the previous version of CloudForms Management Engine, the file name of a saved report would be set to the download date, instead of the report creation date. There was no way to tell when the report was saved from the downloaded report. This bug was fixed by saving the report creation date in the report footer, and using the saved date in the downloaded report file's name. Saved reports have the expected file name in the new version of CloudForms Management Engine.
Story Points: ---
Clone Of: 1196377 Environment:
Last Closed: 2016-04-13 18:43:31 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1196377    
Bug Blocks: 1292574, 1292579    

Comment 1 CFME Bot 2016-02-11 23:55:39 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/70e112d0edaaada97caadfcecf7bfaa3ca2f80c4

commit 70e112d0edaaada97caadfcecf7bfaa3ca2f80c4
Author:     Harpreet Kataria <hkataria>
AuthorDate: Wed Feb 10 15:01:45 2016 -0500
Commit:     Harpreet Kataria <hkataria>
CommitDate: Thu Feb 11 10:53:32 2016 -0500

    Fixed an issue where rpt_options could be nil for some OOTB reports.
    
    Downloading txt reports can fail on merge! if rpt_options is nil
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1196377
    https://bugzilla.redhat.com/show_bug.cgi?id=1306699
    https://bugzilla.redhat.com/show_bug.cgi?id=1292579

 app/models/miq_report_result.rb | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Comment 3 CFME Bot 2016-02-12 14:58:45 UTC
New commit detected on cfme/5.5.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=cbb2d5fa0ffe21d7cf120ab5fad53ec2de6adb48

commit cbb2d5fa0ffe21d7cf120ab5fad53ec2de6adb48
Author:     Harpreet Kataria <hkataria>
AuthorDate: Wed Feb 10 15:01:45 2016 -0500
Commit:     Harpreet Kataria <hkataria>
CommitDate: Fri Feb 12 09:11:09 2016 -0500

    Fixed an issue where rpt_options could be nil for some OOTB reports.
    
    Downloading txt reports can fail on merge! if rpt_options is nil
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1196377
    https://bugzilla.redhat.com/show_bug.cgi?id=1306699
    https://bugzilla.redhat.com/show_bug.cgi?id=1292579
    (cherry picked from commit 70e112d)

 app/models/miq_report_result.rb | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Comment 4 CFME Bot 2016-02-12 14:59:02 UTC
New commit detected on cfme/5.5.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=8355600203c0e5863b1b655457597e11a9c38a09

commit 8355600203c0e5863b1b655457597e11a9c38a09
Merge: c2694eb cbb2d5f
Author:     Dan Clarizio <dclarizi>
AuthorDate: Fri Feb 12 09:54:24 2016 -0500
Commit:     Dan Clarizio <dclarizi>
CommitDate: Fri Feb 12 09:54:24 2016 -0500

    Merge branch '55z_pr_6602' into '5.5.z'
    
    Fixed an issue where rpt_options could be nil for some OOTB reports.
    
    Downloading txt reports can fail on merge! if rpt_options is nil
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1196377
    https://bugzilla.redhat.com/show_bug.cgi?id=1306699
    https://bugzilla.redhat.com/show_bug.cgi?id=1292579
    (cherry picked from commit 70e112d)
    
    @dclarizi please review, clean cherry-pick for https://github.com/ManageIQ/manageiq/pull/6602
    
    See merge request !786

 app/models/miq_report_result.rb | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Comment 5 Yuri Rudman 2016-02-12 15:55:55 UTC
*** Bug 1306809 has been marked as a duplicate of this bug. ***

Comment 7 CFME Bot 2016-02-19 15:24:51 UTC
New commit detected on cfme/5.4.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=a1f3a85ef5a5905fd0b5a799c8b513fe262c7950

commit a1f3a85ef5a5905fd0b5a799c8b513fe262c7950
Author:     Harpreet Kataria <hkataria>
AuthorDate: Fri Feb 12 09:14:21 2016 -0500
Commit:     Harpreet Kataria <hkataria>
CommitDate: Fri Feb 12 09:14:21 2016 -0500

    Fixed an issue where rpt_options could be nil for some OOTB reports.
    
    Downloading txt reports can fail on merge! if rpt_options is nil
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1196377
    https://bugzilla.redhat.com/show_bug.cgi?id=1306699
    https://bugzilla.redhat.com/show_bug.cgi?id=1292579
    (cherry picked from commit 70e112d)

 vmdb/app/models/miq_report_result.rb | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Comment 8 CFME Bot 2016-02-19 15:25:01 UTC
New commit detected on cfme/5.4.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=7665a7c4e5f2cb465516da3aeb60f6117ffc36d9

commit 7665a7c4e5f2cb465516da3aeb60f6117ffc36d9
Merge: 26e38bc a1f3a85
Author:     Dan Clarizio <dclarizi>
AuthorDate: Fri Feb 19 10:06:43 2016 -0500
Commit:     Dan Clarizio <dclarizi>
CommitDate: Fri Feb 19 10:06:43 2016 -0500

    Merge branch '54z_pr_6602' into '5.4.z'
    
    Fixed an issue where rpt_options could be nil for some OOTB reports.
    
    Downloading txt reports can fail on merge! if rpt_options is nil
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1196377
    https://bugzilla.redhat.com/show_bug.cgi?id=1306699
    https://bugzilla.redhat.com/show_bug.cgi?id=1292579
    (cherry picked from commit 70e112d)
    
    @dclarizi please review, clean 5.4.z cherry-pick for https://github.com/ManageIQ/manageiq/pull/6602
    
    See merge request !787

 vmdb/app/models/miq_report_result.rb | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Comment 10 Jan Krocil 2016-03-29 08:22:59 UTC
Verified fixed in 5.5.3.2 - 5.5.3.2.20160318152106_92d2d67.

Files contain a footer with date when report was originally created and filenames contain date of actual file creation.

Comment 12 errata-xmlrpc 2016-04-13 18:43:31 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2016:0616