Bug 691544
Summary: | Entering very large numbers in Dynagroups Recalculate Interval field causes stack trace | ||
---|---|---|---|
Product: | [Other] RHQ Project | Reporter: | Corey Welton <cwelton> |
Component: | Core UI | Assignee: | Mike Thompson <mithomps> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Mike Foley <mfoley> |
Severity: | low | Docs Contact: | |
Priority: | low | ||
Version: | 4.0.0.B02 | CC: | hrupp, mazz, mithomps |
Target Milestone: | --- | ||
Target Release: | RHQ 4.3.0 | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | 4.3.0 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2013-09-01 10:18:51 UTC | Type: | --- |
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: | |||
Bug Blocks: | 691508, 735475 |
Description
Corey Welton
2011-03-28 19:38:25 UTC
while we aer in here, lets see if we can fix the units. We have it where you have to enter milliseconds, which is pretty annoying. Perhaps we can allow differrent units or change the default units to something more realistic, like minutes. Under the covers we can convert to milliseconds when we pass the value to the server. I did change the units to minutes (from ms) and kept the representation in the database as milliseconds as I didn't want to break any existing functionality based on those tables. The view shows in minutes and the database is still in millis. I originally started out using various units (via DurationItem), however, things got a little complicated with min/max ranges in the various units so I dropped back to minutes as an acceptable UOM. As indicated, this fulfills the requirement and is simpler/cleaner unless multiple UOM are required. Current maximum set to 1 week (60 * 24 * 7 minutes). I also changed the current resource bundle (Messages.properties) to read (min) instead of (ms) however, i didn't change the other language bundles to reflect the language changes. These changes are in the features/gwt-upgrade branch. These changes are not in master yet (in feature/gwt-upgrade) so I will leave the status as ON_DEV. Git commit id: 5754205 Now in master. Also commented out other foreign resource bundles for re-translation. commit: 223f5fd Bulk closing of items that are on_qa and in old RHQ releases, which are out for a long time and where the issue has not been re-opened since. |