Skip to content

Commit

Permalink
Fixed duplicate dominated set entries fixes #230
Browse files Browse the repository at this point in the history
  • Loading branch information
mksplg committed Sep 22, 2014
1 parent b5f807b commit e4b59c5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
Expand Up @@ -109,7 +109,7 @@ public List<List<String>> getDominatedSets(Aggregation aggregation) {
* index.
*
* Note: For performance reasons the sets are stored in lists. Thus
* allMeasureUris should not duplicates.
* allMeasureUris should not contain duplicates.
*
* @param allMeasureUris
* all measures that should be considered
Expand Down
6 changes: 1 addition & 5 deletions kbrowser/src/main/webapp/kbrowser.xhtml
Expand Up @@ -814,7 +814,7 @@
rendered="#{not empty kBrowser.dominatedSets}">
<div class="full-list">
<rich:dataGrid value="#{kBrowser.dominatedSets}" var="set"
id="dominatedSets2" rowKeyVar="dominatedSetIndex">
id="dominatedSets" rowKeyVar="dominatedSetIndex">

<rich:collapsiblePanel switchType="ajax" expanded="false"
toggleListener="#{kBrowser.loadDominatedSetPlans(dominatedSetIndex)}"
Expand All @@ -825,8 +825,6 @@
<ul>
<a4j:repeat value="#{set}" var="measure">
<li><h:outputText value="#{measure.name}" /></li>
<li><h:outputText value="#{measure.name}" /></li>
<li><h:outputText value="#{measure.name}" /></li>
</a4j:repeat>
</ul>
</div>
Expand All @@ -836,8 +834,6 @@
<ul>
<a4j:repeat value="#{set}" var="measure">
<li><h:outputText value="#{measure.name}" /></li>
<li><h:outputText value="#{measure.name}" /></li>
<li><h:outputText value="#{measure.name}" /></li>
</a4j:repeat>
</ul>
</div>
Expand Down

0 comments on commit e4b59c5

Please sign in to comment.