modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/inventory/resource/discovery/ResourceAutodiscoveryView.java | 13 ++++------ 1 file changed, 6 insertions(+), 7 deletions(-)
New commits: commit 1163194d1955b334dfe783876b7f052b306bc737 Author: John Sanda jsanda@redhat.com Date: Tue Jan 10 14:25:39 2012 -0500
[BZ 772681] Avoid ClassCastException
There were a couple places previously casting the ListNode to a TreeNode, and this was causing a ClassCastException. This might be due to a SmartGWT defect introduced back in version 2.5. See these forum posts for related info,
http://forums.smartclient.com/showthread.php?t=18457&highlight=TreeNode+... http://forums.smartclient.com/showthread.php?t=19912
diff --git a/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/inventory/resource/discovery/ResourceAutodiscoveryView.java b/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/inventory/resource/discovery/ResourceAutodiscoveryView.java index 690ea75..afbaeaf 100644 --- a/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/inventory/resource/discovery/ResourceAutodiscoveryView.java +++ b/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/inventory/resource/discovery/ResourceAutodiscoveryView.java @@ -18,10 +18,6 @@ */ package org.rhq.enterprise.gui.coregui.client.inventory.resource.discovery;
-import java.util.ArrayList; -import java.util.EnumSet; -import java.util.List; - import com.google.gwt.user.client.rpc.AsyncCallback; import com.smartgwt.client.data.Criteria; import com.smartgwt.client.data.DataSource; @@ -50,7 +46,6 @@ import com.smartgwt.client.widgets.toolbar.ToolStrip; import com.smartgwt.client.widgets.tree.TreeGrid; import com.smartgwt.client.widgets.tree.TreeGridField; import com.smartgwt.client.widgets.tree.TreeNode; - import org.rhq.core.domain.resource.InventoryStatus; import org.rhq.enterprise.gui.coregui.client.CoreGUI; import org.rhq.enterprise.gui.coregui.client.RefreshableView; @@ -64,6 +59,10 @@ import org.rhq.enterprise.gui.coregui.client.util.selenium.LocatableIButton; import org.rhq.enterprise.gui.coregui.client.util.selenium.LocatableTreeGrid; import org.rhq.enterprise.gui.coregui.client.util.selenium.LocatableVLayout;
+import java.util.ArrayList; +import java.util.EnumSet; +import java.util.List; + /** * @author Greg Hinkle */ @@ -220,7 +219,7 @@ public class ResourceAutodiscoveryView extends LocatableVLayout implements Refre } selectionChangedHandlerDisabled = true;
- final TreeNode selectedNode = (TreeNode) selectionEvent.getRecord(); + final TreeNode selectedNode = treeGrid.getTree().findById(selectionEvent.getRecord().getAttribute("id")); TreeNode parentNode = treeGrid.getTree().getParent(selectedNode); boolean isPlatform = treeGrid.getTree().isRoot(parentNode); boolean isCheckboxMarked = treeGrid.isSelected(selectedNode); @@ -409,7 +408,7 @@ public class ResourceAutodiscoveryView extends LocatableVLayout implements Refre boolean unignoreOk = false;
for (ListGridRecord listGridRecord : treeGrid.getSelection()) { - TreeNode node = (TreeNode) listGridRecord; + TreeNode node = treeGrid.getTree().findById(listGridRecord.getAttribute("id")); String status = node.getAttributeAsString("status"); TreeNode parentNode = treeGrid.getTree().getParent(node); boolean isPlatform = treeGrid.getTree().isRoot(parentNode);
rhq-commits@lists.fedorahosted.org