You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@labs.apache.org by si...@apache.org on 2008/10/28 14:05:22 UTC
svn commit: r708562 - in /labs/magma/trunk/foundation-beans-view/src:
main/java/org/apache/magma/view/tree/Zone.java
test/java/org/apache/magma/view/BeanViewTreeTest.java
Author: simoneg
Date: Tue Oct 28 06:05:22 2008
New Revision: 708562
URL: http://svn.apache.org/viewvc?rev=708562&view=rev
Log:
LABS-219 : fix
Modified:
labs/magma/trunk/foundation-beans-view/src/main/java/org/apache/magma/view/tree/Zone.java
labs/magma/trunk/foundation-beans-view/src/test/java/org/apache/magma/view/BeanViewTreeTest.java
Modified: labs/magma/trunk/foundation-beans-view/src/main/java/org/apache/magma/view/tree/Zone.java
URL: http://svn.apache.org/viewvc/labs/magma/trunk/foundation-beans-view/src/main/java/org/apache/magma/view/tree/Zone.java?rev=708562&r1=708561&r2=708562&view=diff
==============================================================================
--- labs/magma/trunk/foundation-beans-view/src/main/java/org/apache/magma/view/tree/Zone.java (original)
+++ labs/magma/trunk/foundation-beans-view/src/main/java/org/apache/magma/view/tree/Zone.java Tue Oct 28 06:05:22 2008
@@ -75,6 +75,7 @@
Node node = this.nodes.get(i);
if (node instanceof HiddenNode) {
Set<Node> allNodes = node.getAllNodes();
+ allNodes.remove(node);
if (allNodes.size() == 0) {
this.nodes.remove(i);
i--;
Modified: labs/magma/trunk/foundation-beans-view/src/test/java/org/apache/magma/view/BeanViewTreeTest.java
URL: http://svn.apache.org/viewvc/labs/magma/trunk/foundation-beans-view/src/test/java/org/apache/magma/view/BeanViewTreeTest.java?rev=708562&r1=708561&r2=708562&view=diff
==============================================================================
--- labs/magma/trunk/foundation-beans-view/src/test/java/org/apache/magma/view/BeanViewTreeTest.java (original)
+++ labs/magma/trunk/foundation-beans-view/src/test/java/org/apache/magma/view/BeanViewTreeTest.java Tue Oct 28 06:05:22 2008
@@ -75,8 +75,8 @@
if (property.getViewSettings() != null) {
Node propnode = root.findById(property.getName());
if (property.getName().equals("age")) {
- assertNotNull("Age should be here, even if filtered", propnode);
- assertEquals(HiddenNode.class, propnode.getClass());
+ assertNull("Age should not be here, cause is filtered", propnode);
+ //assertEquals(HiddenNode.class, propnode.getClass());
} else if (property.getName().equals("another")) {
assertNotNull(propnode);
simpleCheck(data, propnode, "another");
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@labs.apache.org
For additional commands, e-mail: commits-help@labs.apache.org