Skip to content

Commit

Permalink
MONDRIAN
Browse files Browse the repository at this point in the history
       The evaluateCurrent method should not return Util.nullValue but
       rather "null" since other code depends upon a "null" being
       returned (such as the CrossJoinFunDef class' nonEmptyList method).
       This could be changed so that it returns Util.nullValue rather
       than "null" (or either) but that requires changes to other classes.

[git-p4: depot-paths = "//open/mondrian/": change = 7628]
  • Loading branch information
Richard Emberson committed Sep 12, 2006
1 parent 7a549a0 commit 71edc27
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/main/mondrian/rolap/RolapEvaluator.java
Expand Up @@ -328,7 +328,11 @@ public Object evaluateCurrent() {
evaluator.setExpanding(maxSolveMember);
final Exp exp = maxSolveMember.getExpression();
Calc calc = root.getCompiled(exp, true);
return calc.evaluate(evaluator);
Object o = calc.evaluate(evaluator);
if (o == Util.nullValue) {
o = null;
}
return o;
}

private void setExpanding(Member member) {
Expand Down

0 comments on commit 71edc27

Please sign in to comment.