Bug 888369 - Time range setting on monitoring table should be more robust
Summary: Time range setting on monitoring table should be more robust
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: Core UI
Version: 4.5
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ---
: RHQ 4.6
Assignee: Jirka Kremser
QA Contact: Mike Foley
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-12-18 15:05 UTC by Jirka Kremser
Modified: 2013-09-03 14:41 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-09-03 14:41:53 UTC
Embargoed:


Attachments (Terms of Use)

Description Jirka Kremser 2012-12-18 15:05:48 UTC
Description of problem:
If the start time later than the end time, exception is thrown with reasonable message (for instance: "Start date 1418876100000 is not before 1355832903395")

If the format of the time is not valid in those Time Range - Start/Stop inputs, another exception is thrown (Exception caught: (TypeError)). 


Version-Release number of selected component (if applicable):
4.6.0-SNAPSHOT

How reproducible:
always

Steps to Reproduce:
1. navigate to Inventory and select some platform
2. Under the Monitoring tab type following into the "Time Range - Start" and "Time Range - Stop" fields, lets call them x and y
3. a) x > y (for any times, i.e. start is after end)
   b) x = "foo"
  
Actual results:
a) javax.ejb.EJBTransactionRolledbackException:Start date 1387318500000 is not before 1355832903395

b) (TypeError) stack: com_smartgwt_client_widgets_form_fields_DateItem_$getValueAsDate

Expected results:
It should be handled on the client side without bothering the server with nonsense inputs.

Additional info:

Comment 1 Jirka Kremser 2012-12-21 15:58:33 UTC
http://git.fedorahosted.org/git/?p=rhq/rhq.git;a=commitdiff;h=ea3d203dc

time:    Fri Dec 21 16:54:02 2012 +0100
commit:  ea3d203dcd9fd9bb110bb0f6dce6f1379e88bacb
author:  Jirka Kremser - jkremser
message: [BZ 888369 - Time range setting on monitoring table should be more robust] Adding form validators for the format of the times as well as for the start time < end time; if the end time is after the current server time, the end time is set to the current server time. This caused the NaNs in the table before. However, It is still not perfect, because the user with bad time settings will have prefilled values in the start/end time fields. The simple form ("last n hours") on the monitoring tab is working as it should.

Comment 2 Heiko W. Rupp 2013-09-03 14:41:53 UTC
Bulk closing of issues in old RHQ releases that are in production for a while now.

Please open a new issue when running into an issue.


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