mirror of
				https://github.com/invoiceninja/invoiceninja.git
				synced 2025-11-03 22:17:35 -05:00 
			
		
		
		
	Merge pull request #6010 from turbo124/v5-develop
Don't delete the PDF prior to sending
This commit is contained in:
		
						commit
						046bbfd546
					
				@ -564,7 +564,7 @@ class CreditController extends BaseController
 | 
			
		||||
                // EmailCredit::dispatch($credit, $credit->company);
 | 
			
		||||
 | 
			
		||||
                $credit->invitations->load('contact.client.country', 'credit.client.country', 'credit.company')->each(function ($invitation) use ($credit) {
 | 
			
		||||
                    EmailEntity::dispatch($invitation, $credit->company, 'credit')->delay(now()->addSeconds(60));
 | 
			
		||||
                    EmailEntity::dispatch($invitation, $credit->company, 'credit');
 | 
			
		||||
                });
 | 
			
		||||
 | 
			
		||||
 | 
			
		||||
 | 
			
		||||
@ -105,7 +105,7 @@ class CreateEntityPdf implements ShouldQueue
 | 
			
		||||
        /* Set customized translations _NOW_ */
 | 
			
		||||
        $t->replace(Ninja::transformTranslations($this->entity->client->getMergedSettings()));
 | 
			
		||||
 | 
			
		||||
        $this->entity->service()->deletePdf();
 | 
			
		||||
        // $this->entity->service()->deletePdf();
 | 
			
		||||
 | 
			
		||||
        if (config('ninja.phantomjs_pdf_generation') || config('ninja.pdf_generator') == 'phantom') {
 | 
			
		||||
            return (new Phantom)->generate($this->invitation);
 | 
			
		||||
 | 
			
		||||
@ -213,7 +213,7 @@ class SendReminders implements ShouldQueue
 | 
			
		||||
            if ($this->checkSendSetting($invoice, $template) && $invoice->company->account->hasFeature(Account::FEATURE_EMAIL_TEMPLATES_REMINDERS)) {
 | 
			
		||||
                nlog("firing email");
 | 
			
		||||
 | 
			
		||||
                EmailEntity::dispatchNow($invitation, $invitation->company, $template)->delay(now()->addSeconds(60));
 | 
			
		||||
                EmailEntity::dispatchNow($invitation, $invitation->company, $template);
 | 
			
		||||
                
 | 
			
		||||
            }
 | 
			
		||||
        });
 | 
			
		||||
 | 
			
		||||
@ -96,7 +96,7 @@ class SendRecurring implements ShouldQueue
 | 
			
		||||
            if ($invitation->contact && strlen($invitation->contact->email) >=1) {
 | 
			
		||||
 | 
			
		||||
                try{
 | 
			
		||||
                    EmailEntity::dispatch($invitation, $invoice->company)->delay(now()->addSeconds(60));
 | 
			
		||||
                    EmailEntity::dispatch($invitation, $invoice->company);
 | 
			
		||||
                }
 | 
			
		||||
                catch(\Exception $e) {
 | 
			
		||||
                    nlog($e->getMessage());
 | 
			
		||||
 | 
			
		||||
@ -60,7 +60,7 @@ class ReminderJob implements ShouldQueue
 | 
			
		||||
                $invoice->service()->touchReminder($reminder_template)->save();
 | 
			
		||||
 | 
			
		||||
                $invoice->invitations->each(function ($invitation) use ($invoice, $reminder_template) {
 | 
			
		||||
                    EmailEntity::dispatch($invitation, $invitation->company, $reminder_template)->delay(now()->addSeconds(60));
 | 
			
		||||
                    EmailEntity::dispatch($invitation, $invitation->company, $reminder_template);
 | 
			
		||||
                    nlog("Firing reminder email for invoice {$invoice->number}");
 | 
			
		||||
                });
 | 
			
		||||
 | 
			
		||||
 | 
			
		||||
@ -64,7 +64,7 @@ class SendFailedEmails implements ShouldQueue
 | 
			
		||||
 | 
			
		||||
            if ($invitation->invoice) {
 | 
			
		||||
                if ($invitation->contact->send_email && $invitation->contact->email) {
 | 
			
		||||
                    EmailEntity::dispatch($invitation, $invitation->company, $job_meta_array['reminder_template'])->delay(now()->addSeconds(60));
 | 
			
		||||
                    EmailEntity::dispatch($invitation, $invitation->company, $job_meta_array['reminder_template']);
 | 
			
		||||
                }
 | 
			
		||||
            }
 | 
			
		||||
        });
 | 
			
		||||
 | 
			
		||||
@ -62,7 +62,7 @@ class TriggeredActions extends AbstractService
 | 
			
		||||
        $reminder_template = 'payment';
 | 
			
		||||
 | 
			
		||||
        $this->invoice->invitations->load('contact.client.country', 'invoice.client.country', 'invoice.company')->each(function ($invitation) use ($reminder_template) {
 | 
			
		||||
            EmailEntity::dispatch($invitation, $this->invoice->company, $reminder_template)->delay(now()->addSeconds(60));
 | 
			
		||||
            EmailEntity::dispatch($invitation, $this->invoice->company, $reminder_template);
 | 
			
		||||
        });
 | 
			
		||||
 | 
			
		||||
        if ($this->invoice->invitations->count() > 0) {
 | 
			
		||||
 | 
			
		||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user