Enable adding tasks/expenses to any open invoice #1292

This commit is contained in:
Hillel Coren 2017-06-20 18:10:57 +03:00
parent 91402b2c4e
commit 0d1b9e0dc9
3 changed files with 3 additions and 9 deletions

View File

@ -105,7 +105,7 @@ class ExpenseController extends BaseController
$actions[] = ['url' => 'javascript:submitAction("invoice")', 'label' => trans('texts.invoice_expense')];
// check for any open invoices
$invoices = $expense->client_id ? $this->invoiceRepo->findOpenInvoices($expense->client_id, ENTITY_EXPENSE) : [];
$invoices = $expense->client_id ? $this->invoiceRepo->findOpenInvoices($expense->client_id) : [];
foreach ($invoices as $invoice) {
$actions[] = ['url' => 'javascript:submitAction("add_to_invoice", '.$invoice->public_id.')', 'label' => trans('texts.add_to_invoice', ['invoice' => $invoice->invoice_number])];

View File

@ -158,7 +158,7 @@ class TaskController extends BaseController
$actions[] = ['url' => 'javascript:submitAction("invoice")', 'label' => trans('texts.invoice_task')];
// check for any open invoices
$invoices = $task->client_id ? $this->invoiceRepo->findOpenInvoices($task->client_id, ENTITY_TASK) : [];
$invoices = $task->client_id ? $this->invoiceRepo->findOpenInvoices($task->client_id) : [];
foreach ($invoices as $invoice) {
$actions[] = ['url' => 'javascript:submitAction("add_to_invoice", '.$invoice->public_id.')', 'label' => trans('texts.add_to_invoice', ['invoice' => $invoice->invoice_number])];

View File

@ -962,7 +962,7 @@ class InvoiceRepository extends BaseRepository
*
* @return mixed
*/
public function findOpenInvoices($clientId, $entityType = false)
public function findOpenInvoices($clientId)
{
$query = Invoice::scope()
->invoiceType(INVOICE_TYPE_STANDARD)
@ -971,12 +971,6 @@ class InvoiceRepository extends BaseRepository
->whereDeletedAt(null)
->where('balance', '>', 0);
if ($entityType == ENTITY_TASK) {
$query->whereHasTasks(true);
} elseif ($entityType == ENTITY_EXPENSE) {
$query->whereHasTasks(false);
}
return $query->where('invoice_status_id', '<', 5)
->select(['public_id', 'invoice_number'])
->get();