mirror of
				https://github.com/invoiceninja/invoiceninja.git
				synced 2025-11-03 22:07:33 -05:00 
			
		
		
		
	Fixes for recurring
This commit is contained in:
		
							parent
							
								
									4cf56d5138
								
							
						
					
					
						commit
						e678f1d373
					
				@ -385,22 +385,16 @@ class RecurringInvoice extends BaseModel
 | 
				
			|||||||
 | 
					
 | 
				
			||||||
        $data = [];
 | 
					        $data = [];
 | 
				
			||||||
 | 
					
 | 
				
			||||||
            // $data[] = [
 | 
					 | 
				
			||||||
            //     'next_send_date' => $next_send_date->format('Y-m-d'), 
 | 
					 | 
				
			||||||
            //     'due_date' => $next_due_date->format('Y-m-d'),
 | 
					 | 
				
			||||||
            // ];
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
        for($x=0; $x<$iterations; $x++)
 | 
					        for($x=0; $x<$iterations; $x++)
 | 
				
			||||||
        {
 | 
					        {
 | 
				
			||||||
 | 
					            // we don't add the days... we calc the day of the month!!
 | 
				
			||||||
            $next_due_date = $next_send_date->copy()->addDays($this->due_date_days);
 | 
					            $next_due_date = $this->calculateDueDate($next_send_date->copy()->format('Y-m-d'));
 | 
				
			||||||
 | 
					
 | 
				
			||||||
            $next_send_date = Carbon::parse($next_send_date);
 | 
					            $next_send_date = Carbon::parse($next_send_date);
 | 
				
			||||||
            $next_due_date = Carbon::parse($next_due_date);
 | 
					            $next_due_date = Carbon::parse($next_due_date);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
            $data[] = [
 | 
					            $data[] = [
 | 
				
			||||||
                'next_send_date' => $next_send_date->format('Y-m-d'), 
 | 
					                'send_date' => $next_send_date->format('Y-m-d'), 
 | 
				
			||||||
                'due_date' => $next_due_date->format('Y-m-d'),
 | 
					                'due_date' => $next_due_date->format('Y-m-d'),
 | 
				
			||||||
            ];
 | 
					            ];
 | 
				
			||||||
 | 
					
 | 
				
			||||||
@ -442,13 +436,14 @@ class RecurringInvoice extends BaseModel
 | 
				
			|||||||
     */
 | 
					     */
 | 
				
			||||||
    public function calculateDateFromTerms($date) 
 | 
					    public function calculateDateFromTerms($date) 
 | 
				
			||||||
    {
 | 
					    {
 | 
				
			||||||
 | 
					        $new_date = Carbon::parse($date);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        $client_payment_terms = $this->client->getSetting('payment_terms');
 | 
					        $client_payment_terms = $this->client->getSetting('payment_terms');
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        if($client_payment_terms == '')//no due date! return null;
 | 
					        if($client_payment_terms == '')//no due date! return null;
 | 
				
			||||||
            return null;
 | 
					            return null;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        return $date->copy()->addDays($client_payment_terms); //add the number of days in the payment terms to the date
 | 
					        return $new_date->addDays($client_payment_terms); //add the number of days in the payment terms to the date
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    /**
 | 
					    /**
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user