fix(controller): Fix false booleans in multipart/form-data

Signed-off-by: Joas Schilling <[email protected]>

lib/private/AppFramework/Http/Dispatcher.php

@@ -188,9 +188,15 @@ private function executeController(Controller $controller, string $methodName):
 			if (($type === 'bool' || $type === 'boolean') &&
 				$value === 'false' &&
 				(
-					$this->request->method === 'GET' ||
-					str_contains($this->request->getHeader('Content-Type'),
-						'application/x-www-form-urlencoded')
+					$this->request->method === 'GET'
+					|| str_contains(
+						$this->request->getHeader('Content-Type'),
+						'application/x-www-form-urlencoded',
+					)
+					|| str_contains(
+						$this->request->getHeader('Content-Type'),
+						'multipart/form-data',
+					)
 				)
 			) {
 				$value = false;

tests/lib/AppFramework/Http/DispatcherTest.php

@@ -471,6 +471,41 @@ public function testResponseTransformedByAcceptHeader(): void {
 		$this->assertEquals('{"text":[3,false,4,1]}', $response[3]);
 	}
 
+	public function testResponseTransformedBySendingMultipartFormData(): void {
+		$this->request = new Request(
+			[
+				'post' => [
+					'int' => '3',
+					'bool' => 'false',
+					'double' => 1.2,
+				],
+				'server' => [
+					'HTTP_ACCEPT' => 'application/text, test',
+					'HTTP_CONTENT_TYPE' => 'multipart/form-data'
+				],
+				'method' => 'POST'
+			],
+			$this->createMock(IRequestId::class),
+			$this->createMock(IConfig::class)
+		);
+		$this->dispatcher = new Dispatcher(
+			$this->http, $this->middlewareDispatcher, $this->reflector,
+			$this->request,
+			$this->config,
+			\OC::$server->getDatabaseConnection(),
+			$this->logger,
+			$this->eventLogger,
+			$this->container
+		);
+		$controller = new TestController('app', $this->request);
+
+		// reflector is supposed to be called once
+		$this->dispatcherPassthrough();
+		$response = $this->dispatcher->dispatch($controller, 'exec');
+
+		$this->assertEquals('{"text":[3,false,4,1]}', $response[3]);
+	}
+
 
 	public function testResponsePrimarilyTransformedByParameterFormat(): void {
 		$this->request = new Request(