Bug 1370570
| Summary: | C&U - WEB UI crashes when moving from calendar to daily/hourly selection | ||
|---|---|---|---|
| Product: | Red Hat CloudForms Management Engine | Reporter: | Satoe Imaishi <simaishi> |
| Component: | UI - OPS | Assignee: | Yaacov Zamir <yzamir> |
| Status: | CLOSED ERRATA | QA Contact: | Einat Pacifici <epacific> |
| Severity: | high | Docs Contact: | |
| Priority: | high | ||
| Version: | 5.6.0 | CC: | cpelland, fsimonce, hkataria, jhardy, mpovolny, nachandr, obarenbo, simaishi, yzamir |
| Target Milestone: | GA | Keywords: | ZStream |
| Target Release: | 5.6.3 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | container:c&u | ||
| Fixed In Version: | 5.6.3.0 | Doc Type: | If docs needed, set a value |
| Doc Text: | Story Points: | --- | |
| Clone Of: | 1366233 | Environment: | |
| Last Closed: | 2016-11-30 19:53:09 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | Container Management | Target Upstream Version: | |
| Embargoed: | |||
| Bug Depends On: | 1366233 | ||
| Bug Blocks: | |||
|
Comment 2
Federico Simoncelli
2016-08-29 13:54:22 UTC
Backported: https://github.com/ManageIQ/manageiq/pull/10458 Assigning the BZ to Einat for verification. Assigned. This still occurs in 5.6.2. Same steps to reproduce. New commit detected on ManageIQ/manageiq/darga: https://github.com/ManageIQ/manageiq/commit/3959bbc655f180d29e89954a490cbe63309ffbd1 commit 3959bbc655f180d29e89954a490cbe63309ffbd1 Author: yaacov <kobi.zamir> AuthorDate: Sun Sep 25 18:29:58 2016 +0300 Commit: yaacov <kobi.zamir> CommitDate: Tue Sep 27 16:23:16 2016 +0300 C&U - WEB UI crashes when moving from calendar to daily/hourly selection It is posible to send an empty string as a date from the UI, we currently only protect against nil date but not against an empty string. In this PR we add a check for empty string date. PR #10458 did not reset the date in cases where the date was empty, it only protected the sanity checks. This PR also reset the date in cases where the date string is mepty. Bugzilla https://bugzilla.redhat.com/show_bug.cgi?id=1370570 .../application_controller/performance.rb | 4 +- .../application_controller/performance_spec.rb | 50 ++++++++++++++++++++++ 2 files changed, 52 insertions(+), 2 deletions(-) Merged upstream: https://github.com/ManageIQ/manageiq/pull/11506 Verified. UI no longer crashes. 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://rhn.redhat.com/errata/RHSA-2016-2839.html |