From ea6cdd99b5a4d86f4f8bbbb9c4a7fe527b59ecc5 Mon Sep 17 00:00:00 2001
From: Devansh <devansh.bawari419@webkul.com>
Date: Tue, 11 Aug 2020 20:31:38 +0530
Subject: [PATCH] Just Handled Some Exceptions

---
 .../views/sales/invoices/view.blade.php       | 60 ++++++++++---------
 .../views/sales/shipments/view.blade.php      | 60 ++++++++++---------
 2 files changed, 64 insertions(+), 56 deletions(-)

diff --git a/packages/Webkul/Admin/src/Resources/views/sales/invoices/view.blade.php b/packages/Webkul/Admin/src/Resources/views/sales/invoices/view.blade.php
index 06fb088b8..0d392ac9b 100755
--- a/packages/Webkul/Admin/src/Resources/views/sales/invoices/view.blade.php
+++ b/packages/Webkul/Admin/src/Resources/views/sales/invoices/view.blade.php
@@ -107,14 +107,14 @@
                             <div class="section-content">
                                 <div class="row">
                                     <span class="title">{{ __('admin::app.sales.orders.customer-name') }}</span>
-                                    <span class="value">{{ $invoice->address->name }}</span>
+                                    <span class="value">{{ $invoice->order->customer_full_name }}</span>
                                 </div>
 
                                 {!! view_render_event('sales.invoice.customer_name.after', ['order' => $order]) !!}
 
                                 <div class="row">
                                     <span class="title">{{ __('admin::app.sales.orders.email') }}</span>
-                                    <span class="value">{{ $invoice->address->email }}</span>
+                                    <span class="value">{{ $invoice->order->customer_email }}</span>
                                 </div>
 
                                 {!! view_render_event('sales.invoice.customer_email.after', ['order' => $order]) !!}
@@ -124,36 +124,40 @@
                     </div>
                 </accordian>
 
-                <accordian :title="'{{ __('admin::app.sales.orders.address') }}'" :active="true">
-                    <div slot="body" style="display: flex; overflow:auto;">
+                @if ($order->billing_address || $order->shipping_address)
+                    <accordian :title="'{{ __('admin::app.sales.orders.address') }}'" :active="true">
+                        <div slot="body" style="display: flex; overflow:auto;">
 
-                        <div class="sale-section">
-                            <div class="secton-title" style="width: 380px;">
-                                <span>{{ __('admin::app.sales.orders.billing-address') }}</span>
-                            </div>
+                            @if ($order->billing_address)
+                                <div class="sale-section">
+                                    <div class="secton-title" style="width: 380px;">
+                                        <span>{{ __('admin::app.sales.orders.billing-address') }}</span>
+                                    </div>
 
-                            <div class="section-content" style="width: 380px;">
-                                @include ('admin::sales.address', ['address' => $order->billing_address])
+                                    <div class="section-content" style="width: 380px;">
+                                        @include ('admin::sales.address', ['address' => $order->billing_address])
 
-                                {!! view_render_event('sales.invoice.billing_address.after', ['order' => $order]) !!}
-                            </div>
+                                        {!! view_render_event('sales.invoice.billing_address.after', ['order' => $order]) !!}
+                                    </div>
+                                </div>
+                            @endif
+
+                            @if ($order->shipping_address)
+                                <div class="sale-section" style="margin: 0 0 0 300px;">
+                                    <div class="secton-title" style="width: 400px;">
+                                        <span>{{ __('admin::app.sales.orders.shipping-address') }}</span>
+                                    </div>
+
+                                    <div class="section-content" style="width: 400px;">
+                                        @include ('admin::sales.address', ['address' => $order->shipping_address])
+
+                                        {!! view_render_event('sales.invoice.shipping_address.after', ['order' => $order]) !!}
+                                    </div>
+                                </div>
+                            @endif
                         </div>
-
-                        @if ($order->shipping_address)
-                            <div class="sale-section" style="margin: 0 0 0 300px;">
-                                <div class="secton-title" style="width: 400px;">
-                                    <span>{{ __('admin::app.sales.orders.shipping-address') }}</span>
-                                </div>
-
-                                <div class="section-content" style="width: 400px;">
-                                    @include ('admin::sales.address', ['address' => $order->shipping_address])
-
-                                    {!! view_render_event('sales.invoice.shipping_address.after', ['order' => $order]) !!}
-                                </div>
-                            </div>
-                        @endif
-                    </div>
-                </accordian>
+                    </accordian>
+                @endif
 
                 <accordian :title="'{{ __('admin::app.sales.orders.products-ordered') }}'" :active="true">
                     <div slot="body">
diff --git a/packages/Webkul/Admin/src/Resources/views/sales/shipments/view.blade.php b/packages/Webkul/Admin/src/Resources/views/sales/shipments/view.blade.php
index 2d21639df..ef1db5579 100755
--- a/packages/Webkul/Admin/src/Resources/views/sales/shipments/view.blade.php
+++ b/packages/Webkul/Admin/src/Resources/views/sales/shipments/view.blade.php
@@ -87,7 +87,7 @@
                                     </span>
 
                                     <span class="value">
-                                        {{ $shipment->address->name }}
+                                        {{ $shipment->order->customer_full_name }}
                                     </span>
                                 </div>
 
@@ -97,7 +97,7 @@
                                     </span>
 
                                     <span class="value">
-                                        {{ $shipment->address->email }}
+                                        {{ $shipment->order->customer_email }}
                                     </span>
                                 </div>
                             </div>
@@ -106,37 +106,41 @@
                     </div>
                 </accordian>
 
-                <accordian :title="'{{ __('admin::app.sales.orders.address') }}'" :active="true">
-                    <div slot="body">
+                @if ($order->billing_address || $order->shipping_address)
+                    <accordian :title="'{{ __('admin::app.sales.orders.address') }}'" :active="true">
+                        <div slot="body">
 
-                        <div class="sale-section">
-                            <div class="secton-title">
-                                <span>{{ __('admin::app.sales.orders.billing-address') }}</span>
-                            </div>
+                            @if ($order->billing_address)
+                                <div class="sale-section">
+                                    <div class="secton-title">
+                                        <span>{{ __('admin::app.sales.orders.billing-address') }}</span>
+                                    </div>
 
-                            <div class="section-content">
+                                    <div class="section-content">
 
-                                @include ('admin::sales.address', ['address' => $order->billing_address])
+                                        @include ('admin::sales.address', ['address' => $order->billing_address])
+
+                                    </div>
+                                </div>
+                            @endif
+
+                            @if ($order->shipping_address)
+                                <div class="sale-section">
+                                    <div class="secton-title">
+                                        <span>{{ __('admin::app.sales.orders.shipping-address') }}</span>
+                                    </div>
+
+                                    <div class="section-content">
+
+                                        @include ('admin::sales.address', ['address' => $order->shipping_address])
+
+                                    </div>
+                                </div>
+                            @endif
 
-                            </div>
                         </div>
-
-                        @if ($order->shipping_address)
-                            <div class="sale-section">
-                                <div class="secton-title">
-                                    <span>{{ __('admin::app.sales.orders.shipping-address') }}</span>
-                                </div>
-
-                                <div class="section-content">
-
-                                    @include ('admin::sales.address', ['address' => $order->shipping_address])
-
-                                </div>
-                            </div>
-                        @endif
-
-                    </div>
-                </accordian>
+                    </accordian>
+                @endif
 
                 <accordian :title="'{{ __('admin::app.sales.orders.payment-and-shipping') }}'" :active="true">
                     <div slot="body">