Rename account IncLogin to ImapUser and OutLogin to SmtpUser

This commit is contained in:
the-djmaze 2024-04-15 00:24:11 +02:00
parent 326d35eee7
commit f8790a3a89
3 changed files with 20 additions and 12 deletions

View file

@ -265,11 +265,11 @@ class Actions
if ($oAccount) { if ($oAccount) {
$oDomain = $oAccount->Domain(); $oDomain = $oAccount->Domain();
$sLine = \str_replace('{imap:login}', $oAccount->IncLogin(), $sLine); $sLine = \str_replace('{imap:login}', $oAccount->ImapUser(), $sLine);
$sLine = \str_replace('{imap:host}', $oDomain->ImapSettings()->host, $sLine); $sLine = \str_replace('{imap:host}', $oDomain->ImapSettings()->host, $sLine);
$sLine = \str_replace('{imap:port}', $oDomain->ImapSettings()->port, $sLine); $sLine = \str_replace('{imap:port}', $oDomain->ImapSettings()->port, $sLine);
$sLine = \str_replace('{smtp:login}', $oAccount->OutLogin(), $sLine); $sLine = \str_replace('{smtp:login}', $oAccount->SmtpUser(), $sLine);
$sLine = \str_replace('{smtp:host}', $oDomain->SmtpSettings()->host, $sLine); $sLine = \str_replace('{smtp:host}', $oDomain->SmtpSettings()->host, $sLine);
$sLine = \str_replace('{smtp:port}', $oDomain->SmtpSettings()->port, $sLine); $sLine = \str_replace('{smtp:port}', $oDomain->SmtpSettings()->port, $sLine);
} }

View file

@ -553,7 +553,7 @@ trait Messages
} }
if ($oMessage) { if ($oMessage) {
$ETag = $oMessage->ETag($this->getAccountFromToken()->IncLogin()); $ETag = $oMessage->ETag($this->getAccountFromToken()->ImapUser());
$this->verifyCacheByKey($ETag); $this->verifyCacheByKey($ETag);
$this->Plugins()->RunHook('filter.result-message', array($oMessage)); $this->Plugins()->RunHook('filter.result-message', array($oMessage));
$this->cacheByKey($ETag); $this->cacheByKey($ETag);

View file

@ -13,7 +13,7 @@ abstract class Account implements \JsonSerializable
private string $sEmail = ''; private string $sEmail = '';
private string $sLogin = ''; private string $sImapUser = '';
private ?SensitiveString $oPassword = null; private ?SensitiveString $oPassword = null;
@ -35,8 +35,12 @@ abstract class Account implements \JsonSerializable
public function IncLogin() : string public function IncLogin() : string
{ {
// return $this->oDomain->ImapSettings()->fixUsername($this->sLogin); return $this->ImapUser();
return $this->sLogin; }
public function ImapUser() : string
{
// return $this->oDomain->ImapSettings()->fixUsername($this->sImapUser);
return $this->sImapUser;
} }
public function IncPassword() : string public function IncPassword() : string
@ -45,6 +49,10 @@ abstract class Account implements \JsonSerializable
} }
public function OutLogin() : string public function OutLogin() : string
{
return $this->SmtpUser();
}
public function SmtpUser() : string
{ {
// return $this->oDomain->SmtpSettings()->fixUsername($this->sSmtpUser ?: $this->sEmail); // return $this->oDomain->SmtpSettings()->fixUsername($this->sSmtpUser ?: $this->sEmail);
return $this->sSmtpUser ?: $this->sEmail; return $this->sSmtpUser ?: $this->sEmail;
@ -59,7 +67,7 @@ abstract class Account implements \JsonSerializable
{ {
return \sha1(\implode(APP_SALT, [ return \sha1(\implode(APP_SALT, [
$this->sEmail, $this->sEmail,
$this->sLogin, $this->sImapUser,
// \json_encode($this->Domain()), // \json_encode($this->Domain()),
// $this->oPassword // $this->oPassword
])); ]));
@ -80,7 +88,7 @@ abstract class Account implements \JsonSerializable
{ {
$result = [ $result = [
'email' => $this->sEmail, 'email' => $this->sEmail,
'login' => $this->sLogin, 'login' => $this->sImapUser,
'pass' => $this->IncPassword(), 'pass' => $this->IncPassword(),
'name' => $this->sName 'name' => $this->sName
]; ];
@ -106,7 +114,7 @@ abstract class Account implements \JsonSerializable
$oAccount = new static; $oAccount = new static;
$oAccount->sEmail = \SnappyMail\IDN::emailToAscii($sEmail); $oAccount->sEmail = \SnappyMail\IDN::emailToAscii($sEmail);
$oAccount->sLogin = \SnappyMail\IDN::emailToAscii($sLogin); $oAccount->sImapUser = \SnappyMail\IDN::emailToAscii($sLogin);
$oAccount->SetPassword($oPassword); $oAccount->SetPassword($oPassword);
$oAccount->oDomain = $oDomain; $oAccount->oDomain = $oDomain;
@ -177,7 +185,7 @@ abstract class Account implements \JsonSerializable
{ {
$oSettings = $this->Domain()->ImapSettings(); $oSettings = $this->Domain()->ImapSettings();
$oSettings->timeout = \max($oSettings->timeout, (int) $oConfig->Get('imap', 'timeout', $oSettings->timeout)); $oSettings->timeout = \max($oSettings->timeout, (int) $oConfig->Get('imap', 'timeout', $oSettings->timeout));
$oSettings->username = $this->IncLogin(); $oSettings->username = $this->ImapUser();
$oSettings->expunge_all_on_delete |= !!$oConfig->Get('imap', 'use_expunge_all_on_delete', false); $oSettings->expunge_all_on_delete |= !!$oConfig->Get('imap', 'use_expunge_all_on_delete', false);
$oSettings->fast_simple_search = !(!$oSettings->fast_simple_search || !$oConfig->Get('imap', 'message_list_fast_simple_search', true)); $oSettings->fast_simple_search = !(!$oSettings->fast_simple_search || !$oConfig->Get('imap', 'message_list_fast_simple_search', true));
@ -201,7 +209,7 @@ abstract class Account implements \JsonSerializable
public function SmtpConnectAndLogin(\RainLoop\Plugins\Manager $oPlugins, \MailSo\Smtp\SmtpClient $oSmtpClient) : bool public function SmtpConnectAndLogin(\RainLoop\Plugins\Manager $oPlugins, \MailSo\Smtp\SmtpClient $oSmtpClient) : bool
{ {
$oSettings = $this->Domain()->SmtpSettings(); $oSettings = $this->Domain()->SmtpSettings();
$oSettings->username = $this->OutLogin(); $oSettings->username = $this->SmtpUser();
$oSettings->Ehlo = \MailSo\Smtp\SmtpClient::EhloHelper(); $oSettings->Ehlo = \MailSo\Smtp\SmtpClient::EhloHelper();
$oSmtpClient->Settings = $oSettings; $oSmtpClient->Settings = $oSettings;
@ -225,7 +233,7 @@ abstract class Account implements \JsonSerializable
public function SieveConnectAndLogin(\RainLoop\Plugins\Manager $oPlugins, \MailSo\Sieve\SieveClient $oSieveClient, \RainLoop\Config\Application $oConfig) public function SieveConnectAndLogin(\RainLoop\Plugins\Manager $oPlugins, \MailSo\Sieve\SieveClient $oSieveClient, \RainLoop\Config\Application $oConfig)
{ {
$oSettings = $this->Domain()->SieveSettings(); $oSettings = $this->Domain()->SieveSettings();
$oSettings->username = $this->IncLogin(); $oSettings->username = $this->ImapUser();
$oSieveClient->Settings = $oSettings; $oSieveClient->Settings = $oSettings;