diff --git a/app/Console/Commands/CheckData.php b/app/Console/Commands/CheckData.php index 24b869e48374..05b3db324400 100644 --- a/app/Console/Commands/CheckData.php +++ b/app/Console/Commands/CheckData.php @@ -294,7 +294,7 @@ class CheckData extends Command $wrong_balances = 0; $wrong_paid_to_dates = 0; - foreach (Client::cursor() as $client) { + foreach (Client::where('is_deleted', 0)->cursor() as $client) { $invoice_balance = $client->invoices->where('is_deleted', false)->where('status_id', '>', 1)->sum('balance'); $credit_balance = $client->credits->where('is_deleted', false)->sum('balance'); @@ -318,7 +318,7 @@ class CheckData extends Command $wrong_paid_to_dates = 0; $credit_total_applied = 0; - Client::withTrashed()->cursor()->each(function ($client) use ($wrong_paid_to_dates, $credit_total_applied) { + Client::withTrashed()->where('is_deleted', 0)->cursor()->each(function ($client) use ($wrong_paid_to_dates, $credit_total_applied) { $total_invoice_payments = 0; foreach ($client->invoices->where('is_deleted', false)->where('status_id', '>', 1) as $invoice) { @@ -358,7 +358,7 @@ class CheckData extends Command $wrong_balances = 0; $wrong_paid_to_dates = 0; - Client::cursor()->each(function ($client) use ($wrong_balances) { + Client::cursor()->where('is_deleted', 0)->each(function ($client) use ($wrong_balances) { $client->invoices->where('is_deleted', false)->whereIn('status_id', '!=', Invoice::STATUS_DRAFT)->each(function ($invoice) use ($wrong_balances, $client) { $total_amount = $invoice->payments->whereIn('status_id', [Payment::STATUS_PAID, Payment:: STATUS_PENDING, Payment::STATUS_PARTIALLY_REFUNDED])->sum('pivot.amount'); $total_refund = $invoice->payments->whereIn('status_id', [Payment::STATUS_PAID, Payment:: STATUS_PENDING, Payment::STATUS_PARTIALLY_REFUNDED])->sum('pivot.refunded'); @@ -385,7 +385,7 @@ class CheckData extends Command $wrong_balances = 0; $wrong_paid_to_dates = 0; - foreach (Client::cursor() as $client) { + foreach (Client::cursor()->where('is_deleted', 0) as $client) { //$invoice_balance = $client->invoices->where('is_deleted', false)->where('status_id', '>', 1)->sum('balance'); $invoice_balance = Invoice::where('client_id', $client->id)->where('is_deleted', false)->where('status_id', '>', 1)->withTrashed()->sum('balance'); $client_balance = Credit::where('client_id', $client->id)->where('is_deleted', false)->withTrashed()->sum('balance'); diff --git a/app/Http/Requests/Project/UpdateProjectRequest.php b/app/Http/Requests/Project/UpdateProjectRequest.php index 66e3c72f5887..20a951f06c44 100644 --- a/app/Http/Requests/Project/UpdateProjectRequest.php +++ b/app/Http/Requests/Project/UpdateProjectRequest.php @@ -13,6 +13,7 @@ namespace App\Http\Requests\Project; use App\Http\Requests\Request; use App\Utils\Traits\ChecksEntityStatus; +use Illuminate\Validation\Rule; class UpdateProjectRequest extends Request { diff --git a/app/Mail/Engine/BaseEmailEngine.php b/app/Mail/Engine/BaseEmailEngine.php index 37c13c1f263b..f1b8a7b6cc53 100644 --- a/app/Mail/Engine/BaseEmailEngine.php +++ b/app/Mail/Engine/BaseEmailEngine.php @@ -25,7 +25,7 @@ class BaseEmailEngine implements EngineInterface public $template_style; - public $attachments; + public $attachments = []; public $link; @@ -83,7 +83,8 @@ class BaseEmailEngine implements EngineInterface public function setAttachments($attachments) { - $this->attachments = $attachments; + + $this->attachments = array_merge($this->getAttachments(), $attachments); return $this; } diff --git a/app/Mail/Engine/CreditEmailEngine.php b/app/Mail/Engine/CreditEmailEngine.php index 7cbcbc0ec44b..71a2e01e15e3 100644 --- a/app/Mail/Engine/CreditEmailEngine.php +++ b/app/Mail/Engine/CreditEmailEngine.php @@ -88,7 +88,7 @@ class CreditEmailEngine extends BaseEmailEngine ->setViewText(ctrans('texts.view_credit')); if ($this->client->getSetting('pdf_email_attachment') !== false) { - $this->setAttachments($invitation->pdf_file_path()); + $this->setAttachments([$this->credit->pdf_file_path()]); } return $this; diff --git a/app/Mail/Engine/InvoiceEmailEngine.php b/app/Mail/Engine/InvoiceEmailEngine.php index af72ac485fe9..711b4bdc6dc7 100644 --- a/app/Mail/Engine/InvoiceEmailEngine.php +++ b/app/Mail/Engine/InvoiceEmailEngine.php @@ -97,7 +97,7 @@ class InvoiceEmailEngine extends BaseEmailEngine ->setViewText(ctrans('texts.view_invoice')); if ($this->client->getSetting('pdf_email_attachment') !== false) { - $this->setAttachments($this->invoice->pdf_file_path()); + $this->setAttachments([$this->invoice->pdf_file_path()]); } return $this; diff --git a/app/Mail/Engine/QuoteEmailEngine.php b/app/Mail/Engine/QuoteEmailEngine.php index eeea79ccc26a..02ca6150beb4 100644 --- a/app/Mail/Engine/QuoteEmailEngine.php +++ b/app/Mail/Engine/QuoteEmailEngine.php @@ -88,7 +88,7 @@ class QuoteEmailEngine extends BaseEmailEngine ->setViewText(ctrans('texts.view_quote')); if ($this->client->getSetting('pdf_email_attachment') !== false) { - $this->setAttachments($this->invitation->pdf_file_path()); + $this->setAttachments([$this->invitation->pdf_file_path()]); } return $this;