fix an obvious bug in SqmSelectClause

This commit is contained in:
Nathan Xu 2020-05-12 18:53:24 -04:00 committed by Steve Ebersole
parent 5c11778032
commit fa6acf4e50

View File

@ -109,14 +109,15 @@ public List<SqmSelectableNode<?>> getSelectionItems() {
final List<SqmSelectableNode<?>> subSelections = new ArrayList<>();
//TODO: this has gotta be wrong!!
if ( this.selections != null || this.selections.size() != 1 ) {
this.selections.get( 0 ).getSelectableNode().visitSubSelectableNodes( subSelections::add );
}
else {
for ( SqmSelection<?> selection : this.selections ) {
selection.getSelectableNode().visitSubSelectableNodes( subSelections::add );
if ( this.selections != null ) {
if ( this.selections.size() == 1 ) {
this.selections.get( 0 ).getSelectableNode().visitSubSelectableNodes( subSelections::add );
}
else {
for ( SqmSelection<?> selection : this.selections ) {
selection.getSelectableNode().visitSubSelectableNodes( subSelections::add );
}
}
}
return subSelections;
}