From fa52e6de2b19521828c388db1480d6584d4f5fec Mon Sep 17 00:00:00 2001 From: lhg-interact <34886488+lhg-interact@users.noreply.github.com> Date: Wed, 27 Dec 2017 11:33:02 -0200 Subject: [PATCH] Fixed issue #212 --- .../analytics/ui/MemberSelectionHandler.java | 15 ++++++++++----- pivot4j-analytics/src/main/webapp/view.xhtml | 2 +- .../src/main/java/org/pivot4j/impl/Quax.java | 8 ++++---- 3 files changed, 15 insertions(+), 10 deletions(-) diff --git a/pivot4j-analytics/src/main/java/org/pivot4j/analytics/ui/MemberSelectionHandler.java b/pivot4j-analytics/src/main/java/org/pivot4j/analytics/ui/MemberSelectionHandler.java index 1afdee2c..b99e447a 100644 --- a/pivot4j-analytics/src/main/java/org/pivot4j/analytics/ui/MemberSelectionHandler.java +++ b/pivot4j-analytics/src/main/java/org/pivot4j/analytics/ui/MemberSelectionHandler.java @@ -2,6 +2,7 @@ import java.io.Serializable; import java.util.Arrays; +import java.util.Iterator; import java.util.List; import java.util.Map; import java.util.ResourceBundle; @@ -20,6 +21,7 @@ import org.olap4j.metadata.Hierarchy; import org.olap4j.metadata.Member; import org.olap4j.metadata.MetadataElement; +import org.olap4j.metadata.NamedList; import org.pivot4j.PivotModel; import org.pivot4j.analytics.component.tree.DefaultTreeNode; import org.pivot4j.analytics.component.tree.NodeFilter; @@ -416,11 +418,14 @@ public void moveDown() { public Hierarchy getHierarchy() { if (hierarchy == null && hierarchyName != null && model.isInitialized()) { - this.hierarchy = model.getCube().getHierarchies() - .get(hierarchyName); - } - - return hierarchy; + for (Hierarchy h : model.getCube().getHierarchies()) { + if (h.getUniqueName().equals(hierarchyName)) { + this.hierarchy = h; + } + } + } + + return hierarchy; } protected MemberSelection getSelection() { diff --git a/pivot4j-analytics/src/main/webapp/view.xhtml b/pivot4j-analytics/src/main/webapp/view.xhtml index 5b26eca9..96efd16b 100644 --- a/pivot4j-analytics/src/main/webapp/view.xhtml +++ b/pivot4j-analytics/src/main/webapp/view.xhtml @@ -354,7 +354,7 @@ action="#{memberSelectionHandler.show}" update=":hierarchy-form" oncomplete="PF('hierarchyConfig').show();" title="#{msg['button.hierarchyConfig.tooltip']}"> - + hierarchies) { hierarchyMap.clear(); for (Hierarchy hierarchy : hierarchies) { - hiers.add(hierarchy.getName()); - hierarchyMap.put(hierarchy.getName(), hierarchy); + hiers.add(hierarchy.getUniqueName()); + hierarchyMap.put(hierarchy.getUniqueName(), hierarchy); } } @@ -557,8 +557,8 @@ public void regeneratePosTree(List sets, boolean hiersChanged) { for (Exp set : sets) { try { Hierarchy hierarchy = quaxUtil.hierForExp(set); - hiers.add(hierarchy.getName()); - hierarchyMap.put(hierarchy.getName(), hierarchy); + hiers.add(hierarchy.getUniqueName()); + hierarchyMap.put(hierarchy.getUniqueName(), hierarchy); } catch (UnknownExpressionException e) { throw new PivotException("Unknown expression : " + e.getExpression());