Merge pull request #13610 from ChristophWurst/chore/jmap-2-revert-getsource-rename

refactor(imap): revert getSource -> getRawMessage rename

Author: ChristophWurst

appinfo/routes.php

@@ -231,7 +231,7 @@
 			'verb' => 'GET'
 		],
 		[
-			'name' => 'messages#getRawMessage',
+			'name' => 'messages#getSource',
 			'url' => '/api/messages/{id}/source',
 			'verb' => 'GET'
 		],

lib/Controller/MessageApiController.php

@@ -338,7 +338,7 @@ public function getRaw(int $id): DataResponse {
 		}
 
 		try {
-			$source = $this->mailManager->getRawMessage(
+			$source = $this->mailManager->getSource(
 				$account,
 				$mailbox,
 				$message

lib/Controller/MessagesController.php

@@ -507,7 +507,7 @@ public function mdn(int $id): JSONResponse {
 	 * @throws ServiceException
 	 */
 	#[TrapError]
-	public function getRawMessage(int $id): JSONResponse {
+	public function getSource(int $id): JSONResponse {
 		if ($this->userId === null) {
 			return new JSONResponse([], Http::STATUS_UNAUTHORIZED);
 		}
@@ -521,7 +521,7 @@ public function getRawMessage(int $id): JSONResponse {
 		}
 
 		$response = new JSONResponse([
-			'source' => $this->mailManager->getRawMessage(
+			'source' => $this->mailManager->getSource(
 				$account,
 				$mailbox,
 				$message
@@ -559,7 +559,7 @@ public function export(int $id): Response {
 			return new JSONResponse([], Http::STATUS_FORBIDDEN);
 		}
 
-		$source = $this->mailManager->getRawMessage(
+		$source = $this->mailManager->getSource(
 			$account,
 			$mailbox,
 			$message
@@ -611,7 +611,7 @@ public function saveFile(int $id, string $targetPath): Response {
 			return new JSONResponse([], Http::STATUS_FORBIDDEN);
 		}
 
-		$source = $this->mailManager->getRawMessage(
+		$source = $this->mailManager->getSource(
 			$account,
 			$mailbox,
 			$message

lib/Service/MailManager.php

@@ -182,7 +182,7 @@ public function getImapMessages(Account $account, Mailbox $mailbox, bool $loadBo
 	 * @throws ClientException
 	 * @throws ServiceException
 	 */
-	public function getRawMessage(Account $account, Mailbox $mailbox, Message $message): ?string {
+	public function getSource(Account $account, Mailbox $mailbox, Message $message): ?string {
 		$raw = $this->protocolFactory
 			->messageConnector($account)
 			->fetchMessageRaw($account, $mailbox, $message);

tests/Integration/MailboxSynchronizationTest.php

@@ -282,7 +282,7 @@ public function testUnsolicitedVanishedMessage() {
 
 		// Receive unsolicited vanished uid
 		$message = $dbMessageMapper->findByUids($inbox, [$uid2])[0];
-		$mailManager->getRawMessage(
+		$mailManager->getSource(
 			new Account($this->account),
 			$inbox,
 			$message,

tests/Unit/Controller/MessagesControllerTest.php

@@ -1167,7 +1167,7 @@ public function testExport() {
 			->will($this->returnValue($this->account));
 		$source = file_get_contents(__DIR__ . '/../../data/mail-message-123.txt');
 		$this->mailManager->expects($this->exactly(1))
-			->method('getRawMessage')
+			->method('getSource')
 			->with($this->account, $mailbox, $message)
 			->willReturn($source);
 
@@ -1211,7 +1211,7 @@ public function testSaveFile() {
 			->will($this->returnValue($this->account));
 		$source = file_get_contents(__DIR__ . '/../../data/mail-message-123.txt');
 		$this->mailManager->expects($this->exactly(1))
-			->method('getRawMessage')
+			->method('getSource')
 			->with($this->account, $mailbox, $message)
 			->willReturn($source);
 		$folderNode = $this->createStub(Folder::class);