ENH PHP 8.1 compatibility

This commit is contained in:
Steve Boyd 2022-04-13 11:50:04 +12:00
parent c23cd84be6
commit 31fa96ef66
2 changed files with 4 additions and 4 deletions

View File

@ -142,8 +142,8 @@ class IFramePage extends Page
//whitelist allowed URL schemes //whitelist allowed URL schemes
$allowed_schemes = array('http', 'https'); $allowed_schemes = array('http', 'https');
if ($matches = parse_url($this->IFrameURL)) { if ($matches = parse_url($this->IFrameURL ?? '')) {
if (isset($matches['scheme']) && !in_array($matches['scheme'], $allowed_schemes)) { if (isset($matches['scheme']) && !in_array($matches['scheme'], $allowed_schemes ?? [])) {
$result->addError(_t(__CLASS__ . '.VALIDATION_BANNEDURLSCHEME', "This URL scheme is not allowed.")); $result->addError(_t(__CLASS__ . '.VALIDATION_BANNEDURLSCHEME', "This URL scheme is not allowed."));
} }
} }

View File

@ -16,8 +16,8 @@ class IFramePageController extends PageController
if ($desiredProtocol && $currentProtocol !== $desiredProtocol) { if ($desiredProtocol && $currentProtocol !== $desiredProtocol) {
$enforcedLocation = preg_replace( $enforcedLocation = preg_replace(
"#^${currentProtocol}#", "#^${currentProtocol}#",
$desiredProtocol, $desiredProtocol ?? '',
$this->AbsoluteLink() $this->AbsoluteLink() ?? ''
); );
return $this->redirect($enforcedLocation); return $this->redirect($enforcedLocation);
} }