Bug 973140

Summary: Editing dialog for alert conditions does not load condition parameters
Product: [Other] RHQ Project Reporter: Libor Zoubek <lzoubek>
Component: Core UIAssignee: Jirka Kremser <jkremser>
Status: CLOSED CURRENTRELEASE QA Contact: Mike Foley <mfoley>
Severity: high Docs Contact:
Priority: unspecified    
Version: 4.8CC: hrupp, jkremser, theute
Target Milestone: ---   
Target Release: RHQ 4.8   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-08-31 10:09:24 UTC Type: Bug
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: 994904    

Description Libor Zoubek 2013-06-11 11:10:37 UTC 
Version-Release number of selected component (if applicable):
Version: 4.8.0-SNAPSHOT
Build Number: bf44b14

How reproducible:always


Steps to Reproduce:
1. create alert definition and add contition of type 'Measurement Baseline Threshold'
2. select some metric, comparator, put a value, select max (just not default values)
3. Save such condition
4. select condition and pop up edit dialog

Actual results: values are not loaded


Expected results: edit dialog should contain exactly the same values/selections as it was when adding condition.


Additional info: this issue applies to other condition types as well
Comment 1 Jirka Kremser 2013-06-11 13:17:48 UTC 
branch:  master
link:    http://git.fedorahosted.org/cgit/rhq/rhq.git/commit/?id=729e7cd86
time:    2013-06-11 15:05:36 +0200
commit:  729e7cd86fdfc2e1c8f69b980ac58c3784a838ce
author:  Jirka Kremser - jkremser
message: [BZ 973140] - Editing dialog for alert conditions does not load
         condition parameters

    There was a typo. Changing the buildMetricRangeFormItems()
    buildMetricBaselineFormItems() and buildMetricChangeFormItems()
    call order. Also fixing the percentage formatting.
Comment 2 Filip Brychta 2013-08-08 13:20:48 UTC 
Covered by automation - com.redhat.qe.jon.sahi.tests.alerts.AlertDefinitioinTest
Comment 3 Heiko W. Rupp 2013-08-31 10:09:24 UTC 
Bulk close of old bugs in VERIFIED state.