Frank Kobzik has uploaded a new change for review.

Change subject: frontend: Fix issues with VmRngDevice.Source enum
......................................................................

frontend: Fix issues with VmRngDevice.Source enum

this patch fixes problems with ^ enum parsing in ClusterModel. This enum
is represented as lower-case string in the engine config. When matching
this enum to its string representation, toLowerCase() must be called
firstly.

Change-Id: Ieeb5660fe5da4ea1ba913247150d4c63cecae1bb
Signed-off-by: Frantisek Kobzik <[email protected]>
Bug-Url: https://bugzilla.redhat.com/1122921
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/79/30679/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java
index 4b8a948..33c750e 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java
@@ -1448,10 +1448,10 @@
 
         if (rngSupported) {
             getRngRandomSourceRequired().setEntity(getIsNew()
-                    ? 
defaultRequiredRngSourcesCsv.contains(VmRngDevice.Source.RANDOM.toString())
+                    ? 
defaultRequiredRngSourcesCsv.contains(VmRngDevice.Source.RANDOM.name().toLowerCase())
                     : 
getEntity().getRequiredRngSources().contains(VmRngDevice.Source.RANDOM));
             getRngHwrngSourceRequired().setEntity(getIsNew()
-                    ? 
defaultRequiredRngSourcesCsv.contains(VmRngDevice.Source.HWRNG.toString())
+                    ? 
defaultRequiredRngSourcesCsv.contains(VmRngDevice.Source.HWRNG.name().toLowerCase())
                     : 
getEntity().getRequiredRngSources().contains(VmRngDevice.Source.HWRNG));
         } else { // reset
             getRngRandomSourceRequired().setEntity(false);


-- 
To view, visit http://gerrit.ovirt.org/30679
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ieeb5660fe5da4ea1ba913247150d4c63cecae1bb
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Frank Kobzik <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to