Fixes "PossiblyFalseArgument" notices.

Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com>
Signed-off-by: Faraz Samapoor <f.samapoor@gmail.com>
Signed-off-by: Faraz Samapoor <fsa@adlas.at>
pull/38604/head
Faraz Samapoor 3 years ago committed by Faraz Samapoor
parent 5db241b942
commit 9d15e6e21d
  1. 8
      apps/workflowengine/lib/Check/FileMimeType.php
  2. 4
      apps/workflowengine/lib/Check/RequestURL.php

@ -157,11 +157,11 @@ class FileMimeType extends AbstractStringCheck implements IFileCheck {
protected function isWebDAVRequest() {
return substr($this->request->getScriptName(), 0 - strlen('/remote.php')) === '/remote.php' && (
$this->request->getPathInfo() === '/webdav' ||
str_starts_with($this->request->getPathInfo(), '/webdav/') ||
str_starts_with($this->request->getPathInfo() ?? '', '/webdav/') ||
$this->request->getPathInfo() === '/dav/files' ||
str_starts_with($this->request->getPathInfo(), '/dav/files/') ||
str_starts_with($this->request->getPathInfo() ?? '', '/dav/files/') ||
$this->request->getPathInfo() === '/dav/uploads' ||
str_starts_with($this->request->getPathInfo(), '/dav/uploads/')
str_starts_with($this->request->getPathInfo() ?? '', '/dav/uploads/')
);
}
@ -171,7 +171,7 @@ class FileMimeType extends AbstractStringCheck implements IFileCheck {
protected function isPublicWebDAVRequest() {
return substr($this->request->getScriptName(), 0 - strlen('/public.php')) === '/public.php' && (
$this->request->getPathInfo() === '/webdav' ||
str_starts_with($this->request->getPathInfo(), '/webdav/')
str_starts_with($this->request->getPathInfo() ?? '', '/webdav/')
);
}

@ -90,9 +90,9 @@ class RequestURL extends AbstractStringCheck {
}
return substr($this->request->getScriptName(), 0 - strlen('/remote.php')) === '/remote.php' && (
$this->request->getPathInfo() === '/webdav' ||
str_starts_with($this->request->getPathInfo(), '/webdav/') ||
str_starts_with($this->request->getPathInfo() ?? '', '/webdav/') ||
$this->request->getPathInfo() === '/dav/files' ||
str_starts_with($this->request->getPathInfo(), '/dav/files/')
str_starts_with($this->request->getPathInfo() ?? '', '/dav/files/')
);
}
}

Loading…
Cancel
Save