From 3299188edfa6620a0ae5ea692a40fed2cdd2c4ad Mon Sep 17 00:00:00 2001
From: James Cole <thegrumpydictator@gmail.com>
Date: Fri, 28 Nov 2014 14:41:58 +0100
Subject: [PATCH] New report

---
 app/controllers/ReportController.php   | 63 ++++++++++++++++----------
 app/views/reports/unbalanced.blade.php | 58 +++++++++---------------
 2 files changed, 60 insertions(+), 61 deletions(-)

diff --git a/app/controllers/ReportController.php b/app/controllers/ReportController.php
index 98b9c4010c..2ec832c34a 100644
--- a/app/controllers/ReportController.php
+++ b/app/controllers/ReportController.php
@@ -157,7 +157,14 @@ class ReportController extends BaseController
         /** @var \FireflyIII\Database\TransactionJournal $journalRepository */
         $journalRepository = App::make('FireflyIII\Database\TransactionJournal');
 
-        $journals    = $journalRepository->getInDateRange($start, $end);
+        /*
+         * Get all journals from this month:
+         */
+        $journals = $journalRepository->getInDateRange($start, $end);
+
+        /*
+         * Filter withdrawals:
+         */
         $withdrawals = $journals->filter(
             function (TransactionJournal $journal) {
                 if ($journal->transactionType->type == 'Withdrawal' && count($journal->budgets) == 0) {
@@ -165,8 +172,19 @@ class ReportController extends BaseController
                 }
             }
         );
-
-        // filter again for transfers
+        /*
+         * Filter deposits.
+         */
+        $deposits = $journals->filter(
+            function (TransactionJournal $journal) {
+                if ($journal->transactionType->type == 'Withdrawal' && count($journal->budgets) == 0) {
+                    return $journal;
+                }
+            }
+        );
+        /*
+         * Filter transfers:
+         */
         $transfers = $journals->filter(
             function (TransactionJournal $journal) {
                 if ($journal->transactionType->type == 'Transfer') {
@@ -175,54 +193,51 @@ class ReportController extends BaseController
             }
         );
 
-        $withdrawals->each(
+        /*
+         * Filter withdrawals without a counter-transfer (into this account)
+         */
+        $withdrawals = $withdrawals->filter(
             function (TransactionJournal $journal) {
-                $collection = new Collection;
-                /** @var Transaction $transaction */
                 foreach ($journal->transactions as $transaction) {
                     if (floatval($transaction->amount) < 0) {
                         $account = $transaction->account;
                         // find counter transfer:
                         $counters = $account->transactions()->where('amount', floatval($transaction->amount) * -1)
+                                            ->where('account_id', '=', $transaction->account_id)
                                             ->leftJoin('transaction_journals', 'transaction_journals.id', '=', 'transactions.transaction_journal_id')
                                             ->where('transaction_journals.description', 'LIKE', '%' . e($journal->description) . '%')
-                                            ->get(['transactions.*']);
-                        /** @var Transaction $ct */
-                        foreach ($counters as $ct) {
-                            $collection->push($ct->transactionjournal);
+                                            ->count();
+                        if($counters == 0) {
+                            return $journal;
                         }
                     }
                 }
-                $journal->counters = $collection;
             }
         );
 
-        // same for transfers but the other way around! Yay!
-        $transfers->each(
+        /*
+         * Filter deposits without a counter-transfer (away from this account)
+         */
+        $deposits = $deposits->filter(
             function (TransactionJournal $journal) {
-                $collection = new Collection;
-                /** @var Transaction $transaction */
                 foreach ($journal->transactions as $transaction) {
                     if (floatval($transaction->amount) < 0) {
                         $account = $transaction->account;
-                        // TODO this has to be the most lame way of filtering ever.
-                        $descr = trim(str_replace('Geld voor','',$journal->description));
                         // find counter transfer:
                         $counters = $account->transactions()->where('amount', floatval($transaction->amount))
+                                            ->where('account_id', '=', $transaction->account_id)
                                             ->leftJoin('transaction_journals', 'transaction_journals.id', '=', 'transactions.transaction_journal_id')
-                                            ->where('transaction_journals.description', 'LIKE', '%' . $descr . '%')
-                                            ->get(['transactions.*']);
-                        /** @var Transaction $ct */
-                        foreach ($counters as $ct) {
-                            $collection->push($ct->transactionjournal);
+                                            ->where('transaction_journals.description', 'LIKE', '%' . e($journal->description) . '%')
+                                            ->count();
+                        if($counters == 0) {
+                            return $journal;
                         }
                     }
                 }
-                $journal->counters = $collection;
             }
         );
 
-        return View::make('reports.unbalanced', compact('start','transfers', 'end', 'title', 'subTitle', 'subTitleIcon', 'mainTitleIcon', 'withdrawals'));
+        return View::make('reports.unbalanced', compact('start', 'end', 'title', 'subTitle', 'subTitleIcon', 'mainTitleIcon', 'withdrawals','deposits'));
     }
 
     /**
diff --git a/app/views/reports/unbalanced.blade.php b/app/views/reports/unbalanced.blade.php
index 3816c935ef..e310258967 100644
--- a/app/views/reports/unbalanced.blade.php
+++ b/app/views/reports/unbalanced.blade.php
@@ -1,74 +1,58 @@
 @extends('layouts.default')
 @section('content')
+@if(count($withdrawals) == 0 && count($deposits) == 0)
 <div class="row">
     <div class="col-lg-4 col-md-4 col-sm-6">
-        <h3>Transactions</h3>
+        <p class="text-success">Everything accounted for.</p>
+    </div>
+</div>
+@endif
+@if(count($withdrawals) > 0)
+<div class="row">
+    <div class="col-lg-4 col-md-4 col-sm-6">
+        <h3>Withdrawals</h3>
     </div>
 </div>
 
 <div class="row">
     @foreach($withdrawals as $journal)
         <div class="col-lg-4 col-md-4 col-sm-6">
-            <div class="panel
-            @if($journal->counters->count() > 0)
-            panel-default
-            @else
-            panel-danger
-            @endif
-            ">
+            <div class="panel panel-danger">
                 <div class="panel-heading">
                     <a href="{{route('transactions.show',$journal->id)}}">{{{$journal->description}}}</a>
                 </div>
                 <div class="panel-body">
                     <p>Spent {{mf($journal->getAmount())}}</p>
-                @if($journal->counters->count() > 0)
+                    <p class="text-danger">No counter transaction!</p>
                 </div>
-                <table class="table">
-                @foreach($journal->counters as $counter)
-                <tr>
-                    <td><i class="fa fa-fw fa-arrows-h"></i></td>
-                    <td><a href="{{route('transactions.show',$counter->id)}}">{{$counter->description}}</a></td>
-                    <td>{{mf($counter->getAmount())}}</td>
-                </tr>
-                <!-- {{$counter}} -->
-                @endforeach
-                </table>
-                @else
-                <p class="text-danger">No counter transaction!</p>
-                </div>
-                @endif
             </div>
         </div>
     @endforeach
 </div>
+@endif
+@if(count($deposits) > 0)
 <div class="row">
     <div class="col-lg-4 col-md-4 col-sm-6">
-        <h3>Transfers</h3>
+        <h3>Deposits</h3>
     </div>
 </div>
-
 <div class="row">
-    @foreach($transfers as $journal)
+    @foreach($deposits as $journal)
         <div class="col-lg-4 col-md-4 col-sm-6">
-            <div class="panel
-            @if($journal->counters->count() > 0)
-                panel-default
-            @else
-                panel-danger
-            @endif
-            ">
+            <div class="panel panel-danger">
                 <div class="panel-heading">
-                    {{{$journal->description}}}
+                    <a href="{{route('transactions.show',$journal->id)}}">{{{$journal->description}}}</a>
                 </div>
                 <div class="panel-body">
-                    Bla bla?
+                    <p>Received {{mf($journal->getAmount())}}</p>
+                    <p class="text-danger">No counter transaction!</p>
                 </div>
-
             </div>
-
         </div>
     @endforeach
 </div>
+@endif
+