Return false on more server issues regarding domains control #1513

the-djmaze · Mar 24, 2024 · 2a11661 · 2a11661
1 parent 6df5802
commit 2a11661
Showing 1 changed file with 13 additions and 5 deletions.
diff --git a/snappymail/v/0.0.0/app/libraries/RainLoop/Providers/Domain/DefaultDomain.php b/snappymail/v/0.0.0/app/libraries/RainLoop/Providers/Domain/DefaultDomain.php
@@ -123,20 +123,27 @@ public function Load(string $sName, bool $bFindWithWildCard = false, bool $bChec
 
 	public function Save(\RainLoop\Model\Domain $oDomain) : bool
 	{
-		$this->Delete($oDomain->Name());
 		$sRealFileName = static::encodeFileName($oDomain->Name());
-		\RainLoop\Utils::saveFile($this->sDomainPath.'/'.$sRealFileName.'.json', \json_encode($oDomain, \JSON_PRETTY_PRINT));
+		if (!$sRealFileName) {
+			return false;
+		}
+		$this->Delete($oDomain->Name());
+		\RainLoop\Utils::saveFile("{$this->sDomainPath}/{$sRealFileName}.json", \json_encode($oDomain, \JSON_PRETTY_PRINT));
 		$this->oCacher && $this->oCacher->Delete(static::CACHE_KEY);
 		return true;
 	}
 
 	public function SaveAlias(string $sName, string $sTarget) : bool
 	{
-//		$this->Delete($sName);
 //		$sTarget = \SnappyMail\IDN::toAscii($sTarget);
+//		$sTarget = static::encodeFileName($sTarget);
 		$sTarget = \strtolower(\idn_to_ascii($sTarget));
 		$sRealFileName = static::encodeFileName($sName);
-		\RainLoop\Utils::saveFile($this->sDomainPath.'/'.$sRealFileName.'.alias', $sTarget);
+		if (!$sRealFileName || !$sTarget/* || !\is_readable("{$this->sDomainPath}/{$sTarget}.json")*/) {
+			return false;
+		}
+//		$this->Delete($sName);
+		\RainLoop\Utils::saveFile("{$this->sDomainPath}/{$sRealFileName}.alias", $sTarget);
 		$this->oCacher && $this->oCacher->Delete(static::CACHE_KEY);
 		return true;
 	}
@@ -170,7 +177,8 @@ public function Disable(string $sName, bool $bDisable) : bool
 			} else {
 				$aResult = \array_filter($aResult, fn($v) => $v !== $sName);
 			}
-			\RainLoop\Utils::saveFile($this->sDomainPath.'/disabled', \implode("\n", \array_unique($aResult)));
+			$aResult = \array_unique($aResult);
+			\RainLoop\Utils::saveFile($this->sDomainPath.'/disabled', \implode("\n", $aResult));
 			return $this->getDisabled() === $aResult;
 		}
 		return false;