Minor Refactoring
This commit is contained in:
parent
60023e48f2
commit
1272224164
|
|
@ -69,7 +69,10 @@ class DefinedNames
|
||||||
$this->objWriter->startElement('definedName');
|
$this->objWriter->startElement('definedName');
|
||||||
$this->objWriter->writeAttribute('name', $pDefinedName->getName());
|
$this->objWriter->writeAttribute('name', $pDefinedName->getName());
|
||||||
if ($pDefinedName->getLocalOnly() && $pDefinedName->getScope() !== null) {
|
if ($pDefinedName->getLocalOnly() && $pDefinedName->getScope() !== null) {
|
||||||
$this->objWriter->writeAttribute('localSheetId', $pDefinedName->getScope()->getParent()->getIndex($pDefinedName->getScope()));
|
$this->objWriter->writeAttribute(
|
||||||
|
'localSheetId',
|
||||||
|
$pDefinedName->getScope()->getParent()->getIndex($pDefinedName->getScope())
|
||||||
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
$definedRange = $this->getDefinedRange($pDefinedName);
|
$definedRange = $this->getDefinedRange($pDefinedName);
|
||||||
|
|
@ -207,16 +210,11 @@ class DefinedNames
|
||||||
} else {
|
} else {
|
||||||
$worksheet = str_replace("''", "'", trim($worksheet, "'"));
|
$worksheet = str_replace("''", "'", trim($worksheet, "'"));
|
||||||
}
|
}
|
||||||
|
|
||||||
if (!empty($worksheet)) {
|
if (!empty($worksheet)) {
|
||||||
$newRange = "'" . str_replace("'", "''", $worksheet) . "'!";
|
$newRange = "'" . str_replace("'", "''", $worksheet) . "'!";
|
||||||
}
|
}
|
||||||
|
$newRange = "{$newRange}{$column}{$row}";
|
||||||
if (!empty($column)) {
|
|
||||||
$newRange .= $column;
|
|
||||||
}
|
|
||||||
if (!empty($row)) {
|
|
||||||
$newRange .= $row;
|
|
||||||
}
|
|
||||||
|
|
||||||
$definedRange = substr($definedRange, 0, $offset) . $newRange . substr($definedRange, $offset + $length);
|
$definedRange = substr($definedRange, 0, $offset) . $newRange . substr($definedRange, $offset + $length);
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue