From 92e07fc20646839513bfebfdf8ff307186c06caa Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Denis=20Duli=C3=A7i?= <denisdulici@gmail.com>
Date: Sat, 6 Jun 2020 11:48:20 +0300
Subject: [PATCH] hide lines w/o money in reports

---
 .../views/partials/reports/table.blade.php    |  2 +-
 .../partials/reports/table/footer.blade.php   | 24 ++++++++++++-------
 .../partials/reports/table/rows.blade.php     | 18 +++++++-------
 .../profit_loss/table/footer.blade.php        | 24 ++++++++++++-------
 .../tax_summary/table/footer.blade.php        | 24 ++++++++++++-------
 5 files changed, 55 insertions(+), 37 deletions(-)

diff --git a/resources/views/partials/reports/table.blade.php b/resources/views/partials/reports/table.blade.php
index a3794a43a..5366b369d 100644
--- a/resources/views/partials/reports/table.blade.php
+++ b/resources/views/partials/reports/table.blade.php
@@ -9,7 +9,7 @@
             @else
                 <tr>
                     <td colspan="{{ count($class->dates) + 2 }}">
-                        <h5 class="text-center pl-0">{{ trans('general.no_records') }}</h5>
+                        <div class="text-muted pl-0">{{ trans('general.no_records') }}</div>
                     </td>
                 </tr>
             @endif
diff --git a/resources/views/partials/reports/table/footer.blade.php b/resources/views/partials/reports/table/footer.blade.php
index ba6e3260a..8443fd1ba 100644
--- a/resources/views/partials/reports/table/footer.blade.php
+++ b/resources/views/partials/reports/table/footer.blade.php
@@ -1,11 +1,17 @@
 <tfoot>
-    <tr class="row rp-border-top-1 font-size-unset px-3">
-        <th class="{{ $class->column_name_width }} text-uppercase">{{ trans_choice('general.totals', 1) }}</th>
-        @php $grand_total = 0; @endphp
-        @foreach($class->footer_totals[$table] as $total)
-            @php $grand_total += $total; @endphp
-            <th class="{{ $class->column_value_width }} text-right px-0">@money($total, setting('default.currency'), true)</th>
-        @endforeach
-        <th class="{{ $class->column_name_width }} text-right pl-0 pr-4">@money($grand_total, setting('default.currency'), true)</th>
-    </tr>
+    @if ($grand_total = array_sum($class->footer_totals[$table]))
+        <tr class="row rp-border-top-1 font-size-unset px-3">
+            <th class="{{ $class->column_name_width }} text-uppercase">{{ trans_choice('general.totals', 1) }}</th>
+            @foreach($class->footer_totals[$table] as $total)
+                <th class="{{ $class->column_value_width }} text-right px-0">@money($total, setting('default.currency'), true)</th>
+            @endforeach
+            <th class="{{ $class->column_name_width }} text-right pl-0 pr-4">@money($grand_total, setting('default.currency'), true)</th>
+        </tr>
+    @else
+        <tr>
+            <td colspan="{{ count($class->dates) + 2 }}">
+                <div class="text-muted pl-0">{{ trans('general.no_records') }}</div>
+            </td>
+        </tr>
+    @endif
 </tfoot>
diff --git a/resources/views/partials/reports/table/rows.blade.php b/resources/views/partials/reports/table/rows.blade.php
index 51832cc90..35f8c5cfc 100644
--- a/resources/views/partials/reports/table/rows.blade.php
+++ b/resources/views/partials/reports/table/rows.blade.php
@@ -1,9 +1,9 @@
-@php $row_total = 0; @endphp
-<tr class="row rp-border-top-1 font-size-unset">
-    <td class="{{ $class->column_name_width }} long-texts pr-0">{{ $class->row_names[$table][$id] }}</td>
-    @foreach($rows as $row)
-        @php $row_total += $row; @endphp
-        <td class="{{ $class->column_value_width }} text-right px-0">@money($row, setting('default.currency'), true)</td>
-    @endforeach
-    <td class="{{ $class->column_name_width }} text-right pl-0 pr-4">@money($row_total, setting('default.currency'), true)</td>
-</tr>
+@if ($row_total = array_sum($rows))
+    <tr class="row rp-border-top-1 font-size-unset">
+        <td class="{{ $class->column_name_width }} long-texts pr-0">{{ $class->row_names[$table][$id] }}</td>
+        @foreach($rows as $row)
+            <td class="{{ $class->column_value_width }} text-right px-0">@money($row, setting('default.currency'), true)</td>
+        @endforeach
+        <td class="{{ $class->column_name_width }} text-right pl-0 pr-4">@money($row_total, setting('default.currency'), true)</td>
+    </tr>
+@endif
diff --git a/resources/views/reports/profit_loss/table/footer.blade.php b/resources/views/reports/profit_loss/table/footer.blade.php
index 5cae13c0c..8554a37fb 100644
--- a/resources/views/reports/profit_loss/table/footer.blade.php
+++ b/resources/views/reports/profit_loss/table/footer.blade.php
@@ -1,11 +1,17 @@
 <tfoot>
-    <tr class="row rp-border-top-1 font-size-unset px-3 mb-3">
-        <th class="{{ $class->column_name_width }} text-uppercase">{{ trans_choice('general.totals', 1) }}</th>
-        @php $grand_total = 0; @endphp
-        @foreach($class->footer_totals[$table] as $date => $total)
-            @php $grand_total += $total; @endphp
-            <th class="{{ $class->column_value_width }} text-right px-0">@money($total, setting('default.currency'), true)</th>
-        @endforeach
-        <th class="{{ $class->column_name_width }} text-right pl-0 pr-4">@money($grand_total, setting('default.currency'), true)</th>
-    </tr>
+    @if ($grand_total = array_sum($class->footer_totals[$table]))
+        <tr class="row rp-border-top-1 font-size-unset px-3 mb-3">
+            <th class="{{ $class->column_name_width }} text-uppercase">{{ trans_choice('general.totals', 1) }}</th>
+            @foreach($class->footer_totals[$table] as $date => $total)
+                <th class="{{ $class->column_value_width }} text-right px-0">@money($total, setting('default.currency'), true)</th>
+            @endforeach
+            <th class="{{ $class->column_name_width }} text-right pl-0 pr-4">@money($grand_total, setting('default.currency'), true)</th>
+        </tr>
+    @else
+        <tr>
+            <td colspan="{{ count($class->dates) + 2 }}">
+                <div class="text-muted pl-0">{{ trans('general.no_records') }}</div>
+            </td>
+        </tr>
+    @endif
 </tfoot>
diff --git a/resources/views/reports/tax_summary/table/footer.blade.php b/resources/views/reports/tax_summary/table/footer.blade.php
index 17b2c7bd9..9804cec3f 100644
--- a/resources/views/reports/tax_summary/table/footer.blade.php
+++ b/resources/views/reports/tax_summary/table/footer.blade.php
@@ -1,11 +1,17 @@
 <tfoot>
-    <tr class="row rp-border-bottom-1 font-size-unset px-3">
-        <th class="{{ $class->column_name_width }} text-left">{{ trans('reports.net') }}</th>
-        @php $grand_total = 0; @endphp
-        @foreach($class->footer_totals[$table] as $total)
-            @php $grand_total += $total; @endphp
-            <th class="{{ $class->column_value_width }} text-right px-0">@money($total, setting('default.currency'), true)</th>
-        @endforeach
-        <th class="{{ $class->column_name_width }} text-right pl-0 pr-4">@money($grand_total, setting('default.currency'), true)</th>
-    </tr>
+    @if ($grand_total = array_sum($class->footer_totals[$table]))
+        <tr class="row rp-border-bottom-1 font-size-unset px-3">
+            <th class="{{ $class->column_name_width }} text-left">{{ trans('reports.net') }}</th>
+            @foreach($class->footer_totals[$table] as $total)
+                <th class="{{ $class->column_value_width }} text-right px-0">@money($total, setting('default.currency'), true)</th>
+            @endforeach
+            <th class="{{ $class->column_name_width }} text-right pl-0 pr-4">@money($grand_total, setting('default.currency'), true)</th>
+        </tr>
+    @else
+        <tr>
+            <td colspan="{{ count($class->dates) + 2 }}">
+                <div class="text-muted pl-0">{{ trans('general.no_records') }}</div>
+            </td>
+        </tr>
+    @endif
 </tfoot>