Bug 1306699 - UI: Download file from saved report is named after the current date rather than after the report saved date
Summary: UI: Download file from saved report is named after the current date rather th...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.3.0
Hardware: Unspecified
OS: Unspecified
medium
low
Target Milestone: GA
: 5.5.3
Assignee: Harpreet Kataria
QA Contact: Jan Krocil
URL:
Whiteboard:
: 1306809 (view as bug list)
Depends On: 1196377
Blocks: 1292574 1292579
TreeView+ depends on / blocked
 
Reported: 2016-02-11 15:42 UTC by Chris Pelland
Modified: 2019-10-10 11:10 UTC (History)
13 users (show)

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.
Clone Of: 1196377
Environment:
Last Closed: 2016-04-13 18:43:31 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2016:0616 0 normal SHIPPED_LIVE CFME 5.5.3 bug fixes and enhancement update 2016-04-13 22:38:44 UTC

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


Note You need to log in before you can comment on or make changes to this bug.