Rename canInstallExists method and add new method for removal

Rename canInstallExists to shouldRemoveCanInstallFile to cover removal of this file for non-git channels and logging any failure to remove it.

Add new method to detect if this file exists during web based installation.

Signed-off-by: Alex Harpin <development@landsofshadow.co.uk>

core/Command/Maintenance/Install.php

@@ -110,7 +110,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 			$this->printErrors($output, $errors);
 			return 1;
 		}
-		if ($setupHelper->canInstallExists()) {
+		if ($setupHelper->shouldRemoveCanInstallFile()) {
 			$output->writeln('<warn>Could not remove CAN_INSTALL from the config folder. Please remove this file manually.</warn>');
 		}
 		$output->writeln("Nextcloud was successfully installed");

core/Controller/SetupController.php

@@ -64,7 +64,7 @@ public function run($post) {
 			$post['dbpass'] = $post['dbpassword'];
 		}
 
-		if (!$this->setupHelper->canInstallExists()) {
+		if (!$this->setupHelper->canInstallFileExists()) {
 			$this->displaySetupForbidden();
 			return;
 		}
@@ -112,7 +112,7 @@ private function finishSetup(bool $installRecommended) {
 		}
 		\OC::$server->getIntegrityCodeChecker()->runInstanceVerification();
 
-		if ($this->setupHelper->canInstallExists()) {
+		if ($this->setupHelper->shouldRemoveCanInstallFile()) {
 			\OC_Template::printGuestPage('', 'installation_incomplete');
 		}
 

lib/private/Setup.php

@@ -421,7 +421,7 @@ public function install($options) {
 
 			//and we are done
 			$config->setSystemValue('installed', true);
-			if (self::canInstallExists()) {
+			if (self::shouldRemoveCanInstallFile()) {
 				unlink(\OC::$configDir.'/CAN_INSTALL');
 			}
 
@@ -612,7 +612,14 @@ private function getVendor() {
 	/**
 	 * @return bool
 	 */
-	public function canInstallExists() {
+	public function shouldRemoveCanInstallFile() {
 		return \OC_Util::getChannel() !== 'git' && is_file(\OC::$configDir.'/CAN_INSTALL');
 	}
+
+	/**
+	 * @return bool
+	 */
+	public function canInstallFileExists() {
+		return is_file(\OC::$configDir.'/CAN_INSTALL');
+	}
 }