Client portal improvements: (#3595)

-  Hide Create Payment Method if no gateways exist
- Payment Screen, add $amount
- Fix breadcrumbs

app/Http/Controllers/ClientPortal/PaymentMethodController.php

@@ -13,6 +13,7 @@ namespace App\Http\Controllers\ClientPortal;
 
 use App\Events\Payment\Methods\MethodDeleted;
 use App\Http\Controllers\Controller;
+use App\Http\Requests\ClientPortal\CreatePaymentMethodRequest;
 use App\Models\ClientGatewayToken;
 use App\Utils\Traits\MakesDates;
 use Illuminate\Http\Request;
@@ -45,7 +46,7 @@ class PaymentMethodController extends Controller
      *
      * @return \Illuminate\Http\Response
      */
-    public function create()
+    public function create(CreatePaymentMethodRequest $request)
     {
         $gateway = auth()->user()->client->getCreditCardGateway();
 

app/Http/Requests/ClientPortal/CreatePaymentMethodRequest.php

@@ -0,0 +1,30 @@
+<?php
+
+namespace App\Http\Requests\ClientPortal;
+
+use Illuminate\Foundation\Http\FormRequest;
+
+class CreatePaymentMethodRequest extends FormRequest
+{
+    /**
+     * Determine if the user is authorized to make this request.
+     *
+     * @return bool
+     */
+    public function authorize()
+    {
+        return auth()->user()->client->getCreditCardGateway() ? true : false;
+    }
+
+    /**
+     * Get the validation rules that apply to the request.
+     *
+     * @return array
+     */
+    public function rules()
+    {
+        return [
+            //
+        ];
+    }
+}

resources/lang/en/texts.php

@@ -3194,4 +3194,5 @@ return [
 
     'open_in_new_tab' => 'Open in new tab',
     'complete_your_payment' => 'Complete payment',
+    'authorize_for_future_use' => 'Authorize payment method for future use',
 ];

resources/views/portal/ninja2020/components/general/sidebar/desktop.blade.php

@@ -3,7 +3,7 @@
         <div class="flex items-center h-16 flex-shrink-0 px-4 bg-blue-900">
             <a href="{{ route('client.dashboard') }}">
                 <img class="h-6 w-auto"
-                     src="{!! $settings->company_logo ?: 'https://www.invoiceninja.com/wp-content/themes/invoice-ninja/images/logo.png' !!}"
+                     src="{!! $settings->company_logo ?: asset('images/invoiceninja-white-logo.png') !!}"
                      alt="{{ config('app.name') }}"/>
             </a>
         </div>

resources/views/portal/ninja2020/components/general/sidebar/mobile.blade.php

@@ -15,7 +15,7 @@
         </div>
         <div class="flex-shrink-0 flex items-center px-4">
             <img class="h-6 w-auto"
-                 src="{!! $settings->company_logo ?: 'https://www.invoiceninja.com/wp-content/themes/invoice-ninja/images/logo.png' !!}"
+                 src="{!! $settings->company_logo ?: asset('images/invoiceninja-white-logo.png') !!}"
                  alt="{{ config('app.name') }}"/>
         </div>
         <div class="mt-5 flex-1 h-0 overflow-y-auto">

resources/views/portal/ninja2020/gateways/stripe/credit_card.blade.php

@@ -7,7 +7,7 @@
 @endpush
 
 @section('header')
-    Insert breadcrumbs..
+    {{ Breadcrumbs::render('invoices.pay_now') }}
 @endsection
 
 @section('body')

resources/views/portal/ninja2020/invoices/payment.blade.php

@@ -99,6 +99,14 @@
                                         @endif
                                     </dd>
                                 </div>
+                                <div class="bg-white px-4 py-5 sm:grid sm:grid-cols-3 sm:gap-4 sm:px-6">
+                                    <dt class="text-sm leading-5 font-medium text-gray-500">
+                                        {{ ctrans('texts.amount') }}
+                                    </dt>
+                                    <dd class="mt-1 text-sm leading-5 text-gray-900 sm:mt-0 sm:col-span-2">
+                                        {{ App\Utils\Number::formatMoney($invoice->amount, $invoice->client) }}
+                                    </dd>
+                                </div>
                             </dl>
                         </div>
                     </div>

resources/views/portal/ninja2020/payment_methods/index.blade.php

@@ -21,7 +21,9 @@
                     <div class="inline-flex rounded-md shadow-sm">
                         <input type="hidden" name="hashed_ids">
                         <input type="hidden" name="action" value="payment">
-                        <a href="{{ route('client.payment_methods.create') }}" class="button button-primary">@lang('texts.add_payment_method')</a>
+                        @if(auth()->user()->client->getCreditCardGateway())
+                            <a href="{{ route('client.payment_methods.create') }}" class="button button-primary">@lang('texts.add_payment_method')</a>
+                        @endif
                     </div>
                 </div>
             </div>

routes/breadcrumbs.php

@@ -89,6 +89,12 @@ Breadcrumbs::for('credits.show', function ($trail, $credit) {
     $trail->push(sprintf('%s: %s', ctrans('texts.credits'), $credit->hashed_id), route('client.credits.index', $credit->hashed_id));
 });
 
+// Invoices > Payment
+Breadcrumbs::for('invoices.pay_now', function ($trail) {
+    $trail->parent('invoices');
+    $trail->push(ctrans('texts.pay_now'));
+});
+
 // Dashboard > Client
 Breadcrumbs::for('clients', function ($trail) {
     $trail->parent('dashboard');
@@ -107,4 +113,4 @@ Breadcrumbs::for('clients.edit', function ($trail, $client) {
 
 Breadcrumbs::for('clients.create', function ($trail) {
     $trail->parent('clients');
-});
+});