Fix RequestURL check for cli commands

Fix nextcloud/files_automatedtagging#526

Fix nextcloud/groupfolders#1855

Signed-off-by: Carl Schwan <[email protected]>

Author: CarlSchwan

apps/workflowengine/lib/Check/RequestURL.php

@@ -28,7 +28,9 @@
 
 class RequestURL extends AbstractStringCheck {
 
-	/** @var string */
+	const CLI = 'cli';
+
+	/** @var ?string */
 	protected $url;
 
 	/** @var IRequest */
@@ -49,7 +51,11 @@ public function __construct(IL10N $l, IRequest $request) {
 	 * @return bool
 	 */
 	public function executeCheck($operator, $value) {
-		$actualValue = $this->getActualValue();
+		if (\OC::$CLI) {
+			$actualValue = $this->url = RequestURL::CLI;
+		} else {
+			$actualValue = $this->getActualValue();
+		}
 		if (in_array($operator, ['is', '!is'])) {
 			switch ($value) {
 				case 'webdav':
@@ -63,10 +69,7 @@ public function executeCheck($operator, $value) {
 		return $this->executeStringCheck($operator, $value, $actualValue);
 	}
 
-	/**
-	 * @return string
-	 */
-	protected function getActualValue() {
+	protected function getActualValue(): string {
 		if ($this->url !== null) {
 			return $this->url;
 		}
@@ -79,10 +82,10 @@ protected function getActualValue() {
 		return $this->url; // E.g. https://localhost/nextcloud/index.php/apps/files_texteditor/ajax/loadfile
 	}
 
-	/**
-	 * @return bool
-	 */
-	protected function isWebDAVRequest() {
+	protected function isWebDAVRequest(): bool {
+		if ($this->url === RequestURL::CLI) {
+			return false;
+		}
 		return substr($this->request->getScriptName(), 0 - strlen('/remote.php')) === '/remote.php' && (
 			$this->request->getPathInfo() === '/webdav' ||
 			strpos($this->request->getPathInfo(), '/webdav/') === 0 ||