Fata error when formula references non-existent worksheet

[seliquity]

Fatal error: Call to a member function cellExists() on a non-object in Calculation.php on line 3243

culprit line is: if ($pCellParent->getParent()->getSheetByName($matches[2])->cellExists($cellRef)) {

The sheet is expected to exist, which is not always the case for example when only loading certain named sheets using setLoadSheetsOnly()

Fix looks as simple as changing line to:

$cellSheet = $pCellParent->getParent()->getSheetByName($matches[2]);
if ($cellSheet && $cellSheet->cellExists($cellRef)) {

Not sure if preferable to throw an error (can errors be ignored?) or not instead of letting it fall through the null assignment.

[MarkBaker]

The calculation engine is going to be completely rewritten over the next few months to handle (amongst other things) array formulae and a major performance boost. I shall certainly take this into account when I do the rewrite.

My inclination is to offer a configuration option in way that it is handled: either throw an exception, or fallback to a NULL so that the formula finishes evaluating and returns a result based on that NULL.

[syslogic]

Could it just return string "#ERROR" as cell value, like Excel would?

try {
   if($ws->getCell($x.$y)->getValue() != ''){
      return $ws->getCell($x.$y)->getValue();
   }
}
catch(PHPExcel_Exception $e) {
   return '#ERROR';
   // $e->getMessage();
}

[MarkBaker]

Excel should actually return a #REF for any cell reference that doesn't exist.

[MarkBaker]

I've put in a quick fix to treat cells in non-existent worksheets as a NULL value (the same as empty cells in the current worksheet), but will leave this open as a reminder for while I'm working on the calc engine rewrite.