Merge pull request #23161 from nextcloud/enh/noid/setup-checks2

Remove setup check for php mail

2 files changed, 0 insertions, 46 deletions

diff --git a/apps/settings/lib/Controller/CheckSetupController.php b/apps/settings/lib/Controller/CheckSetupController.php
index 6d237e04c1c..5c7bb4a85ef 100644
--- a/apps/settings/lib/Controller/CheckSetupController.php
+++ b/apps/settings/lib/Controller/CheckSetupController.php
@@ -522,10 +522,6 @@ Raw output
 		return [];
 	}
 
-	protected function isPHPMailerUsed(): bool {
-		return $this->config->getSystemValue('mail_smtpmode', 'smtp') === 'php';
-	}
-
 	protected function hasOpcacheLoaded(): bool {
 		return extension_loaded('Zend OPcache');
 	}
@@ -722,8 +718,6 @@ Raw output
 				'missingColumns' => $this->hasMissingColumns(),
 				'isSqliteUsed' => $this->isSqliteUsed(),
 				'databaseConversionDocumentation' => $this->urlGenerator->linkToDocs('admin-db-conversion'),
-				'isPHPMailerUsed' => $this->isPHPMailerUsed(),
-				'mailSettingsDocumentation' => $this->urlGenerator->getAbsoluteURL('index.php/settings/admin'),
 				'isMemoryLimitSufficient' => $this->memoryInfo->isMemoryLimitSufficient(),
 				'appDirsWithDifferentOwner' => $this->getAppDirsWithDifferentOwner(),
 				'recommendedPHPModules' => $this->hasRecommendedPHPModules(),
diff --git a/apps/settings/tests/Controller/CheckSetupControllerTest.php b/apps/settings/tests/Controller/CheckSetupControllerTest.php
index c3071de2f8e..13bb54c7cea 100644
--- a/apps/settings/tests/Controller/CheckSetupControllerTest.php
+++ b/apps/settings/tests/Controller/CheckSetupControllerTest.php
@@ -479,10 +479,6 @@ class CheckSetupControllerTest extends TestCase {
 				'relativeTime' => '2 hours ago',
 				'backgroundJobsUrl' => 'https://example.org',
 			]);
-		$this->checkSetupController
-			->expects($this->once())
-			->method('isPHPMailerUsed')
-			->willReturn(false);
 		$this->checker
 			->expects($this->once())
 			->method('hasPassedCheck')
@@ -588,8 +584,6 @@ class CheckSetupControllerTest extends TestCase {
 				'databaseConversionDocumentation' => 'http://docs.example.org/server/go.php?to=admin-db-conversion',
 				'missingIndexes' => [],
 				'missingColumns' => [],
-				'isPHPMailerUsed' => false,
-				'mailSettingsDocumentation' => 'https://server/index.php/settings/admin',
 				'isMemoryLimitSufficient' => true,
 				'appDirsWithDifferentOwner' => [],
 				'recommendedPHPModules' => [],
@@ -605,40 +599,6 @@ class CheckSetupControllerTest extends TestCase {
 		$this->assertEquals($expected, $this->checkSetupController->check());
 	}
 
-	public function testIsPHPMailerUsed() {
-		$checkSetupController = $this->getMockBuilder(CheckSetupController::class)
-			->setConstructorArgs([
-				'settings',
-				$this->request,
-				$this->config,
-				$this->clientService,
-				$this->urlGenerator,
-				$this->l10n,
-				$this->checker,
-				$this->logger,
-				$this->dispatcher,
-				$this->db,
-				$this->lockingProvider,
-				$this->dateTimeFormatter,
-				$this->memoryInfo,
-				$this->secureRandom,
-				$this->iniGetWrapper,
-			])
-			->setMethods(null)->getMock();
-
-		$this->config->expects($this->at(0))
-			->method('getSystemValue')
-			->with('mail_smtpmode', 'smtp')
-			->willReturn('php');
-		$this->config->expects($this->at(1))
-			->method('getSystemValue')
-			->with('mail_smtpmode', 'smtp')
-			->willReturn('not-php');
-
-		$this->assertTrue($this->invokePrivate($checkSetupController, 'isPHPMailerUsed'));
-		$this->assertFalse($this->invokePrivate($checkSetupController, 'isPHPMailerUsed'));
-	}
-
 	public function testGetCurlVersion() {
 		$checkSetupController = $this->getMockBuilder(CheckSetupController::class)
 			->setConstructorArgs([