Added ability for clients to approve a quote, converting it to an invoice

app/controllers/InvoiceController.php

@@ -133,23 +133,13 @@ class InvoiceController extends \BaseController
     public function view($invitationKey)
     {
         $invitation = Invitation::where('invitation_key', '=', $invitationKey)->firstOrFail();
-
         $invoice = $invitation->invoice;
 
         if (!$invoice || $invoice->is_deleted) {
             return View::make('invoices.deleted');
         }
 
-        if ($invoice->is_quote && $invoice->quote_invoice_id) {
-            $invoice = Invoice::scope($invoice->quote_invoice_id, $invoice->account_id)->firstOrFail();
-
-            if (!$invoice || $invoice->is_deleted) {
-                return View::make('invoices.deleted');
-            }
-        }
-
         $invoice->load('user', 'invoice_items', 'invoice_design', 'account.country', 'client.contacts', 'client.country');
-
         $client = $invoice->client;
 
         if (!$client || $client->is_deleted) {
@@ -169,7 +159,7 @@ class InvoiceController extends \BaseController
 
         $invoice->invoice_date = Utils::fromSqlDate($invoice->invoice_date);
         $invoice->due_date = Utils::fromSqlDate($invoice->due_date);
-        $invoice->is_pro = $client->account->isPro();
+        $invoice->is_pro = $client->account->isPro();        
 
         $contact = $invitation->contact;
         $contact->setVisible([
@@ -179,6 +169,7 @@ class InvoiceController extends \BaseController
             'phone', ]);
 
         $data = array(
+            'isConverted' => $invoice->quote_invoice_id ? true : false,
             'showClientHeader' => true,
             'showBreadcrumbs' => false,
             'hideLogo' => $client->account->isWhiteLabel(),

app/controllers/PaymentController.php

@@ -614,10 +614,6 @@ class PaymentController extends \BaseController
             $account->save();
         }
 
-        if ($invoice->is_quote) {
-            $invoice = $this->invoiceRepo->cloneInvoice($invoice, $invoice->id);
-        }
-
         $payment = Payment::createNew($invitation);
         $payment->invitation_id = $invitation->id;
         $payment->account_gateway_id = $accountGateway->id;

app/controllers/QuoteController.php

@@ -141,7 +141,7 @@ class QuoteController extends \BaseController
             $clone = $this->invoiceRepo->cloneInvoice($invoice, $invoice->id);
 
             Session::flash('message', trans('texts.converted_to_invoice'));
-            return Redirect::to('invoices/'.$clone->public_id);                        
+            return Redirect::to('invoices/'.$clone->public_id);
         }
 
         $statusId = Input::get('statusId');
@@ -156,4 +156,24 @@ class QuoteController extends \BaseController
 
         return Redirect::to('quotes');
     }
+
+    public function approve($invitationKey)
+    {
+        $invitation = Invitation::with('invoice.invoice_items', 'invoice.invitations')->where('invitation_key', '=', $invitationKey)->firstOrFail();
+        $invoice = $invitation->invoice;
+
+        if ($invoice->is_quote && !$invoice->quote_invoice_id) {
+            Activity::approveQuote($invitation);
+            $invoice = $this->invoiceRepo->cloneInvoice($invoice, $invoice->id);
+            Session::flash('message', trans('texts.converted_to_invoice'));
+
+            foreach ($invoice->invitations as $invitationClone) {
+                if ($invitation->contact_id == $invitationClone->contact_id) {
+                    $invitationKey = $invitationClone->invitation_key;
+                }
+            }
+        }
+
+        return Redirect::to("view/{$invitationKey}");
+    }
 }

app/lang/da/texts.php

@@ -510,6 +510,6 @@ return array(
   'payment_email' => 'Payment Email',
   'quote_email' => 'Quote Email',
   'reset_all' => 'Reset All',
+  'approve' => 'Approve',  
 
-  
 );

app/lang/de/texts.php

@@ -500,6 +500,6 @@ return array(
   'payment_email' => 'Payment Email',
   'quote_email' => 'Quote Email',
   'reset_all' => 'Reset All',
-
   
+  'approve' => 'Approve',
 );

app/lang/en/texts.php

@@ -508,5 +508,6 @@ return array(
   'payment_email' => 'Payment Email',
   'quote_email' => 'Quote Email',
   'reset_all' => 'Reset All',
+  'approve' => 'Approve',
 
 );

app/lang/es/texts.php

@@ -480,6 +480,6 @@ return array(
   'payment_email' => 'Payment Email',
   'quote_email' => 'Quote Email',
   'reset_all' => 'Reset All',
-
+  'approve' => 'Approve',  
 
 );

app/lang/fr/texts.php

@@ -501,6 +501,6 @@ return array(
   'payment_email' => 'Payment Email',
   'quote_email' => 'Quote Email',
   'reset_all' => 'Reset All',
-
+  'approve' => 'Approve',  
   
 );

app/lang/it/texts.php

@@ -503,6 +503,6 @@ return array(
   'payment_email' => 'Payment Email',
   'quote_email' => 'Quote Email',
   'reset_all' => 'Reset All',
-
+  'approve' => 'Approve',
     
 );

app/lang/lt/texts.php

@@ -511,6 +511,7 @@ return array(
   'payment_email' => 'Payment Email',
   'quote_email' => 'Quote Email',
   'reset_all' => 'Reset All',
-  
+  'approve' => 'Approve',
+    
 );
 

app/lang/nb_NO/texts.php

@@ -509,7 +509,7 @@ return array(
   'payment_email' => 'Payment Email',
   'quote_email' => 'Quote Email',
   'reset_all' => 'Reset All',
-
+  'approve' => 'Approve',
     
   
 );

app/lang/nl/texts.php

@@ -504,7 +504,7 @@ return array(
   'payment_email' => 'Payment Email',
   'quote_email' => 'Quote Email',
   'reset_all' => 'Reset All',
-
+  'approve' => 'Approve',
       
   
 );

app/lang/pt_BR/texts.php

@@ -491,7 +491,7 @@ return array(
   'payment_email' => 'Payment Email',
   'quote_email' => 'Quote Email',
   'reset_all' => 'Reset All',
-
+  'approve' => 'Approve',
     
   
 );

app/models/Activity.php

@@ -231,6 +231,19 @@ class Activity extends Eloquent
         $activity->save();
     }
 
+    public static function approveQuote($invitation) {
+
+        $activity = Activity::getBlank($invitation);
+        $activity->client_id = $invitation->invoice->client_id;
+        $activity->invitation_id = $invitation->id;
+        $activity->contact_id = $invitation->contact_id;
+        $activity->invoice_id = $invitation->invoice_id;
+        $activity->activity_type_id = ACTIVITY_TYPE_APPROVE_QUOTE;
+        $activity->message = Utils::encodeActivity($invitation->contact, 'approved', $invitation->invoice);
+        $activity->balance = $invitation->invoice->client->balance;
+        $activity->save();
+    }
+
     public static function createPayment($payment)
     {
         $client = $payment->client;

app/ninja/repositories/InvoiceRepository.php

@@ -80,7 +80,6 @@ class InvoiceRepository
         $query = \DB::table('invitations')
           ->join('invoices', 'invoices.id', '=', 'invitations.invoice_id')
           ->join('clients', 'clients.id', '=', 'invoices.client_id')
-          //->join('contacts', 'contacts.client_id', '=', 'clients.id')
           ->where('invitations.contact_id', '=', $contactId)
           ->where('invitations.deleted_at', '=', null)
           ->where('invoices.is_quote', '=', $entityType == ENTITY_QUOTE)
@@ -99,7 +98,6 @@ class InvoiceRepository
         }
 
         return $table->addColumn('due_date', function ($model) { return Utils::fromSqlDate($model->due_date); })
-            //->addColumn('invoice_status_name', function($model) { return $model->invoice_status_name; })
             ->make();
     }
 

app/routes.php

@@ -46,6 +46,7 @@ Route::post('get_started', 'AccountController@getStarted');
 
 // Client visible pages
 Route::get('view/{invitation_key}', 'InvoiceController@view');
+Route::get('approve/{invitation_key}', 'QuoteController@approve');
 Route::get('payment/{invitation_key}', 'PaymentController@show_payment');
 Route::post('payment/{invitation_key}', 'PaymentController@do_payment');
 Route::get('complete', 'PaymentController@offsite_payment');
@@ -228,6 +229,7 @@ define("ACTIVITY_TYPE_RESTORE_INVOICE", 25);
 define("ACTIVITY_TYPE_RESTORE_CLIENT", 26);
 define("ACTIVITY_TYPE_RESTORE_PAYMENT", 27);
 define("ACTIVITY_TYPE_RESTORE_CREDIT", 28);
+define("ACTIVITY_TYPE_APPROVE_QUOTE", 29);
 
 define('DEFAULT_INVOICE_NUMBER', '0001');
 define('RECENTLY_VIEWED_LIMIT', 8);

app/views/invoices/pdf.blade.php

@@ -81,6 +81,7 @@
   var needsRefresh = false;
 
   function refreshPDF() {
+    PDFJS.disableWorker = true;
     if ({{ Auth::check() && Auth::user()->force_pdfjs ? 'false' : 'true' }} && (isFirefox || (isChrome && !isChromium))) {
       var string = getPDFString();
       if (!string) return;

app/views/invoices/view.blade.php

@@ -22,18 +22,20 @@
 	<div class="container">
 
 		<p>&nbsp;</p>
-
+        <div class="pull-right" style="text-align:right">
-		@if ($invoice->client->account->isGatewayConfigured() && !$invoice->isPaid() && !$invoice->is_recurring)
+        @if ($invoice->is_quote)            
-			<div class="pull-right" style="text-align:right">
+            {{ Button::normal(trans('texts.download_pdf'), array('onclick' => 'onDownloadClick()', 'class' => 'btn-lg')) }}&nbsp;&nbsp;
-				{{ Button::normal(trans('texts.download_pdf'), array('onclick' => 'onDownloadClick()', 'class' => 'btn-lg')) }}&nbsp;&nbsp;
+            @if (!$isConverted)
-				{{ Button::success_link(URL::to('payment/' . $invitation->invitation_key), trans('texts.pay_now'), array('class' => 'btn-lg')) }}
+                {{ Button::success_link(URL::to('approve/' . $invitation->invitation_key), trans('texts.approve'), array('class' => 'btn-lg')) }}
-			</div>		
+            @endif
+		@elseif ($invoice->client->account->isGatewayConfigured() && !$invoice->isPaid() && !$invoice->is_recurring)			
+			{{ Button::normal(trans('texts.download_pdf'), array('onclick' => 'onDownloadClick()', 'class' => 'btn-lg')) }}&nbsp;&nbsp;
+			{{ Button::success_link(URL::to('payment/' . $invitation->invitation_key), trans('texts.pay_now'), array('class' => 'btn-lg')) }}		
 		@else 
-			<div class="pull-right">
+			{{ Button::success('Download PDF', array('onclick' => 'onDownloadClick()', 'class' => 'btn-lg')) }}			
-				{{ Button::success('Download PDF', array('onclick' => 'onDownloadClick()', 'class' => 'btn-lg')) }}
-			</div>		
 		@endif
-		
+		</div>        
+
 		<div class="clearfix"></div><p>&nbsp;</p>
 
 		<script type="text/javascript">