Description of problem: When downloading a saved report, the download file is named after the current date rather than after the report saved date. Example: I download a "VM compliance" report that where scheduled and saved on O2/14/15 01;05:38. The download file is named: 'VM compliance_2015_02_25.pdf", where I would expect it to be named after the saved date, i.e e.g: "VM compliance_2015_02_14.pdf. Consequence: if I download the same day several reports saved at different dates, they all get the same names, that is misleading. Version-Release number of selected component (if applicable): 5.3.1 How reproducible: 100% Steps to Reproduce: 1. Save old report to pdf (notice date) 2. Save new report that was just run 3. Actual results: Expected results: Additional info:
*** Bug 1196681 has been marked as a duplicate of this bug. ***
https://github.com/ManageIQ/manageiq/pull/6087
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/95d3995bb566be138207dd9c97b7d06e992e083d commit 95d3995bb566be138207dd9c97b7d06e992e083d Author: Harpreet Kataria <hkataria> AuthorDate: Thu Jan 7 13:48:51 2016 -0500 Commit: Harpreet Kataria <hkataria> CommitDate: Thu Jan 7 14:57:01 2016 -0500 Appended time to filename for downloadable saved reports. Fixed Date/Time displayed in the footer in text version of downloadable saved reports to show time of when the saved report was last run instead of displaying time when it was downloaded. https://bugzilla.redhat.com/show_bug.cgi?id=1196377 https://bugzilla.redhat.com/show_bug.cgi?id=1292579 https://bugzilla.redhat.com/show_bug.cgi?id=1292574 app/controllers/application_controller.rb | 2 +- app/models/miq_report_result.rb | 4 ++++ lib/report_formatter/text.rb | 4 +++- 3 files changed, 8 insertions(+), 2 deletions(-)
5.5.z MR: http://gitlab.cloudforms.lab.eng.rdu2.redhat.com/cloudforms/cfme/merge_requests/684 5.4.z MR: http://gitlab.cloudforms.lab.eng.rdu2.redhat.com/cloudforms/cfme/merge_requests/685
New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=44d369fb429201d61292b75160aa2652d53d9d85 commit 44d369fb429201d61292b75160aa2652d53d9d85 Author: Harpreet Kataria <hkataria> AuthorDate: Thu Jan 7 17:14:21 2016 -0500 Commit: Harpreet Kataria <hkataria> CommitDate: Thu Jan 7 17:14:21 2016 -0500 Appended time to filename for downloadable saved reports. Fixed Date/Time displayed in the footer in text version of downloadable saved reports to show time of when the saved report was last run instead of displaying time when it was downloaded. https://bugzilla.redhat.com/show_bug.cgi?id=1196377 https://bugzilla.redhat.com/show_bug.cgi?id=1292579 https://bugzilla.redhat.com/show_bug.cgi?id=1292574 (cherry picked from commit 95d3995) vmdb/app/controllers/application_controller.rb | 2 +- vmdb/app/models/miq_report_result.rb | 4 ++++ vmdb/lib/report_formatter/text.rb | 4 +++- 3 files changed, 8 insertions(+), 2 deletions(-)
New commit detected on cfme/5.5.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=4361d61f8154565bf1ea64d76f4331bfec04f58d commit 4361d61f8154565bf1ea64d76f4331bfec04f58d Author: Harpreet Kataria <hkataria> AuthorDate: Thu Jan 7 13:48:51 2016 -0500 Commit: Harpreet Kataria <hkataria> CommitDate: Thu Jan 7 17:11:38 2016 -0500 Appended time to filename for downloadable saved reports. Fixed Date/Time displayed in the footer in text version of downloadable saved reports to show time of when the saved report was last run instead of displaying time when it was downloaded. https://bugzilla.redhat.com/show_bug.cgi?id=1196377 https://bugzilla.redhat.com/show_bug.cgi?id=1292579 https://bugzilla.redhat.com/show_bug.cgi?id=1292574 (cherry picked from commit 95d3995) app/controllers/application_controller.rb | 2 +- app/models/miq_report_result.rb | 4 ++++ lib/report_formatter/text.rb | 4 +++- 3 files changed, 8 insertions(+), 2 deletions(-)
New commit detected on cfme/5.5.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=c160de86eafba1733c5c8235a15a7fc759188003 commit c160de86eafba1733c5c8235a15a7fc759188003 Merge: 3074a1d 4361d61 Author: Dan Clarizio <dclarizi> AuthorDate: Thu Jan 7 18:41:44 2016 -0500 Commit: Dan Clarizio <dclarizi> CommitDate: Thu Jan 7 18:41:44 2016 -0500 Merge branch '55z_pr_6087' into '5.5.z' Appended time to filename for downloadable saved reports. Fixed Date/Time displayed in the footer in text version of downloadable saved reports to show time of when the saved report was last run instead of displaying time when it was downloaded. https://bugzilla.redhat.com/show_bug.cgi?id=1196377 https://bugzilla.redhat.com/show_bug.cgi?id=1292579 https://bugzilla.redhat.com/show_bug.cgi?id=1292574 (cherry picked from commit 95d3995) @dclarizi please review, clean 5.5.z cherry-pick for https://github.com/ManageIQ/manageiq/pull/6087 See merge request !684 app/controllers/application_controller.rb | 2 +- app/models/miq_report_result.rb | 4 ++++ lib/report_formatter/text.rb | 4 +++- 3 files changed, 8 insertions(+), 2 deletions(-)
New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=041e41dffed72123fe94e1906911ba670358fab9 commit 041e41dffed72123fe94e1906911ba670358fab9 Merge: 1972945 44d369f Author: Dan Clarizio <dclarizi> AuthorDate: Thu Jan 7 18:42:16 2016 -0500 Commit: Dan Clarizio <dclarizi> CommitDate: Thu Jan 7 18:42:16 2016 -0500 Merge branch '54z_pr_6087' into '5.4.z' Appended time to filename for downloadable saved reports. Fixed Date/Time displayed in the footer in text version of downloadable saved reports to show time of when the saved report was last run instead of displaying time when it was downloaded. https://bugzilla.redhat.com/show_bug.cgi?id=1196377 https://bugzilla.redhat.com/show_bug.cgi?id=1292579 https://bugzilla.redhat.com/show_bug.cgi?id=1292574 (cherry picked from commit 95d3995) @dclarizi please review, clean 5.4.z cherry-pick for https://github.com/ManageIQ/manageiq/pull/6087 See merge request !685 vmdb/app/controllers/application_controller.rb | 2 +- vmdb/app/models/miq_report_result.rb | 4 ++++ vmdb/lib/report_formatter/text.rb | 4 +++- 3 files changed, 8 insertions(+), 2 deletions(-)
https://github.com/ManageIQ/manageiq/pull/6062
https://github.com/ManageIQ/manageiq/pull/6602
Detected commit referencing this ticket while ticket status is MODIFIED.
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(-)
*** Bug 1306181 has been marked as a duplicate of this bug. ***
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(-)
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(-)
https://github.com/ManageIQ/manageiq/pull/6577
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(-)
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(-)
https://github.com/ManageIQ/manageiq/pull/6666
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/8525c5bfe95e0d879e2e7483fb8b514fee13a12f commit 8525c5bfe95e0d879e2e7483fb8b514fee13a12f Author: Yuri Rudman <yrudman> AuthorDate: Wed Feb 24 15:55:06 2016 -0500 Commit: Yuri Rudman <yrudman> CommitDate: Wed Feb 24 16:26:29 2016 -0500 added test to check if _async_generate_result works when rpt_options is nill https://bugzilla.redhat.com/show_bug.cgi?id=1196377 PR #6602 spec/models/miq_report_result_spec.rb | 23 +++++++++++++++++++++++ 1 file changed, 23 insertions(+)
https://github.com/ManageIQ/manageiq/pull/6946
Spoke to Harpreet.In order to keep the naming consitent with other download files such as list views, export files etc, the downloaded reports are still named after the current date and not after the date when the report was first generated. Marking this as VERIFIED.
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:1348
The needinfo request[s] on this closed bug have been removed as they have been unresolved for 1000 days