Merge pull request #8153 from turbo124/v5-develop

Fixes for client mailers
This commit is contained in:
David Bomba 2023-01-15 15:36:20 +11:00 committed by GitHub
commit f66a069cb3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -90,7 +90,7 @@ class NinjaMailerJob implements ShouldQueue
$this->company = Company::where('company_key', $this->nmo->company->company_key)->first();
/* If any pre conditions fail, we return early here */
if($this->preFlightChecksFail())
if(!$this->company || $this->preFlightChecksFail())
return;
/* Set the email driver */
@ -350,7 +350,10 @@ class NinjaMailerJob implements ShouldQueue
{
$sending_user = $this->nmo->settings->gmail_sending_user_id;
$user = User::find($this->decodePrimaryKey($sending_user));
if($sending_user == "0")
$user = $this->company->owner();
else
$user = User::find($this->decodePrimaryKey($sending_user));
return $user;
}
@ -406,14 +409,6 @@ class NinjaMailerJob implements ShouldQueue
private function setOfficeMailer()
{
$user = $this->resolveSendingUser();
/* Always ensure the user is set on the correct account */
// if($user->account_id != $this->company->account_id){
// $this->nmo->settings->email_sending_method = 'default';
// return $this->setMailDriver();
// }
$this->checkValidSendingUser($user);