When evaluating a name in another workbook, need to use an OperationEvaluationContext tied to that Workbook, and do so such that XSSF references work too

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1613154 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 14 insertions, 8 deletions

diff --git a/src/java/org/apache/poi/ss/formula/OperationEvaluationContext.java b/src/java/org/apache/poi/ss/formula/OperationEvaluationContext.java
index ab87208227..96eb89c9b3 100644
--- a/src/java/org/apache/poi/ss/formula/OperationEvaluationContext.java
+++ b/src/java/org/apache/poi/ss/formula/OperationEvaluationContext.java
@@ -377,25 +377,31 @@ public final class OperationEvaluationContext {
 	
     private ValueEval getExternalNameXEval(ExternalName externName, String workbookName) {
         try {
+            // Fetch the workbook this refers to, and the name as defined with that
             WorkbookEvaluator refWorkbookEvaluator = _bookEvaluator.getOtherWorkbookEvaluator(workbookName);
             EvaluationName evaluationName = refWorkbookEvaluator.getName(externName.getName(),externName.getIx()-1);
             if (evaluationName != null && evaluationName.hasFormula()){
                 if (evaluationName.getNameDefinition().length > 1) {
                     throw new RuntimeException("Complex name formulas not supported yet");
                 }
+                
+                // Need to evaluate the reference in the context of the other book
+                OperationEvaluationContext refWorkbookContext = new OperationEvaluationContext(
+                        refWorkbookEvaluator, refWorkbookEvaluator.getWorkbook(), -1, -1, -1, _tracker);
+                
                 Ptg ptg = evaluationName.getNameDefinition()[0];
                 if (ptg instanceof Ref3DPtg){
                     Ref3DPtg ref3D = (Ref3DPtg)ptg;
-                    int sheetIndex = refWorkbookEvaluator.getSheetIndexByExternIndex(ref3D.getExternSheetIndex());
-                    String sheetName = refWorkbookEvaluator.getSheetName(sheetIndex);
-                    SheetRefEvaluator sre = createExternSheetRefEvaluator(workbookName, sheetName);
-                    return new LazyRefEval(ref3D.getRow(), ref3D.getColumn(), sre);
+                    return refWorkbookContext.getRef3DEval(ref3D);
+                } else if (ptg instanceof Ref3DPxg){
+                    Ref3DPxg ref3D = (Ref3DPxg)ptg;
+                    return refWorkbookContext.getRef3DEval(ref3D);
                 } else if(ptg instanceof Area3DPtg){
                     Area3DPtg area3D = (Area3DPtg)ptg;
-                    int sheetIndex = refWorkbookEvaluator.getSheetIndexByExternIndex(area3D.getExternSheetIndex());
-                    String sheetName = refWorkbookEvaluator.getSheetName(sheetIndex);
-                    SheetRefEvaluator sre = createExternSheetRefEvaluator(workbookName, sheetName);
-                    return new LazyAreaEval(area3D.getFirstRow(), area3D.getFirstColumn(), area3D.getLastRow(), area3D.getLastColumn(), sre);
+                    return refWorkbookContext.getArea3DEval(area3D);
+                } else if(ptg instanceof Area3DPxg){
+                    Area3DPxg area3D = (Area3DPxg)ptg;
+                    return refWorkbookContext.getArea3DEval(area3D);
                 }
             }
             return ErrorEval.REF_INVALID;