Merge branch 'release/4.3.2'

New Crowdin translations

.env.example

@@ -38,9 +38,16 @@ SEND_REGISTRATION_MAIL=true
 SEND_ERROR_MESSAGE=true
 SHOW_INCOMPLETE_TRANSLATIONS=false
 
+CACHE_PREFIX=firefly
+
+GOOGLE_MAPS_API_KEY=
 ANALYTICS_ID=
 SITE_OWNER=mail@example.com
 
 PUSHER_KEY=
 PUSHER_SECRET=
 PUSHER_APP_ID=
+
+DEMO_USERNAME=
+DEMO_PASSWORD=
+

.github/CONTRIBUTING

@@ -0,0 +1 @@
+If you are requesting a new feature, please check out the list of [often requested features](https://firefly-iii.github.io/requested-features/) first. Thanks!

.scrutinizer.yml

@@ -2,7 +2,50 @@
 tools:
   external_code_coverage: false
 filter:
-    excluded_paths:
-        - app/Support/Migration/*
-        - app/database/migrations/*
-        - database/migrations/*
+  paths:
+    - app/*
+    - public/js/ff/*
+  excluded_paths:
+    - "database/migrations/*"
+    - "bootstrap/*"
+    - "config/*"
+    - "docker/*"
+    - "public/js/lib/*"
+    - "public/lib/adminlte/js/*"
+    - "public/lib/bootstrap/js/*"
+    - "resources/*"
+    - "routes/*"
+    - "storage/*"
+checks:
+    php:
+        use_self_instead_of_fqcn: true
+        uppercase_constants: true
+        return_doc_comments: true
+        return_doc_comment_if_not_inferrable: true
+        remove_extra_empty_lines: true
+        parameter_doc_comments: true
+        optional_parameters_at_the_end: true
+        no_short_variable_names:
+            minimum: '3'
+        no_short_method_names:
+            minimum: '3'
+        no_long_variable_names:
+            maximum: '20'
+        no_goto: true
+        newline_at_end_of_file: true
+        encourage_single_quotes: true
+        avoid_todo_comments: true
+        avoid_perl_style_comments: true
+        avoid_fixme_comments: true
+        avoid_multiple_statements_on_same_line: true
+        align_assignments: true
+        duplication: false
+    javascript: true
+
+coding_style:
+    php:
+        spaces:
+            around_operators:
+                concatenation: true
+            other:
+                after_type_cast: false

.travis.yml

@@ -5,14 +5,13 @@ php:
 
 install:
   - phpenv config-rm xdebug.ini
-  - composer selfupdate
   - rm composer.lock
   - composer update --no-scripts
   - php artisan clear-compiled
   - php artisan optimize
   - php artisan env
-  - ./test.sh -r
-  - php artisan env
+  - cp .env.testing .env
+  - mv storage/database/databasecopy.sqlite storage/database/database.sqlite
 
 script:
   - phpunit

CHANGELOG.md

@@ -2,6 +2,71 @@
 All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/).
 
+
+## [4.3.2] - 2017-01-09
+
+An intermediate release because something in the Twig and Twigbridge libraries is broken and I have to make sure it doesn't affect you guys. But some cool features were on their way so there's that oo.
+
+### Added
+- Some code for issue #475, consistent overviews.
+- Better currency display. Make sure you have locale packages installed.
+
+### Changed
+- Uses a new version of Laravel.
+
+### Fixed
+- The password reset routine was broken.
+- Issue #522, thanks to @xpfgsyb
+- Issue #524, thanks to @worldworm
+- Issue #526, thanks to @worldworm
+- Issue #528, thanks to @skibbipl
+- Various other fixes.
+
+## [4.3.1] - 2017-01-04
+### Added
+- Support for Russian and Polish. 
+- Support for a proper demo website.
+- Support for custom decimal places in currencies (#506, suggested by @xpfgsyb).
+- Most amounts are now right-aligned (#511, suggested by @xpfgsyb).
+- German is now a "complete" language, more than 75% translated!
+
+### Changed
+- **[New Github repository!](github.com/firefly-iii/firefly-iii)**
+- Better category overview.
+- #502, thanks to @zjean
+
+### Removed
+- Removed a lot of administration functions.
+- Removed ability to activate users.
+
+### Fixed
+- #501, thanks to @zjean
+- #513, thanks to @skibbipl 
+
+### Security
+- #519, thanks to @xpfgsyb
+
+## [4.3.0] - 2015-12-26
+### Added
+- New method of keeping track of available budget, see issue #489
+- Support for Spanish
+- Firefly III now has an extended demo mode. Will expand further in the future.
+ 
+
+### Changed
+- New favicon
+- Import routine no longer gives transactions a description #483
+
+
+### Removed
+- All test data generation code.
+
+### Fixed
+- Removed import accounts from search results #478
+- Redirect after delete will no longer go back to deleted item #477
+- Cannot math #482
+- Fixed bug in virtual balance field #479
+
 ## [4.2.2] - 2016-12-18
 ### Added
 - New budget report (still a bit of a beta)
@@ -257,7 +322,7 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 
 ### Fixed
 - Bug in the mass edit routines.
-- Firefly III over a proxy will now work (see [issue #290](https://github.com/JC5/firefly-iii/issues/290)), thanks @dfiel for reporting.
+- Firefly III over a proxy will now work (see [issue #290](https://github.com/firefly-iii/firefly-iii/issues/290)), thanks @dfiel for reporting.
 - Sneaky bug in the import routine, fixed by @Bonno 
 
 ## [3.10.1] - 2016-08-25
@@ -278,7 +343,7 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 - Fixed a bug where a migration would check an empty table name.
 - Fixed various bugs in the import routine.
 - Fixed various bugs in the piggy banks pages.
-- Fixed a bug in the ``firefly:verify`` routine
+- Fixed a bug in the `firefly:verify` routine
 
 ## [3.10] - 2015-05-25
 ### Added
@@ -307,11 +372,11 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 - Bulk update problems, #280, thanks @stickgrinder
 - Fixed various problems with amount reporting of split transactions.
 
-[3.9.1]
+## [3.9.1]
 ### Fixed
 - Fixed a bug where removing money from a piggy bank would not work. See issue #265 and #269
 
-[3.9.0]
+## [3.9.0]
 ### Added
 - @zjean has added code that allows you to force "https://"-URL's.
 - @tonicospinelli has added Portuguese (Brazil) translations.

README.md

@@ -1,17 +1,17 @@
-# Firefly III [![Requires PHP7](https://img.shields.io/badge/php-7.0-red.svg)](https://secure.php.net/downloads.php#v7.0.4) [![Latest Stable Version](https://poser.pugx.org/grumpydictator/firefly-iii/v/stable)](https://packagist.org/packages/grumpydictator/firefly-iii) [![Scrutinizer Code Quality](https://scrutinizer-ci.com/g/JC5/firefly-iii/badges/quality-score.png?b=master)](https://scrutinizer-ci.com/g/JC5/firefly-iii/?branch=master)
+# Firefly III: A personal finances manager 
 
-[![Build Status](https://travis-ci.org/JC5/firefly-iii.svg?branch=master)](https://travis-ci.org/JC5/firefly-iii)
-
-## A personal finances manager
+[![Requires PHP7](https://img.shields.io/badge/php-7.0-red.svg)](https://secure.php.net/downloads.php#v7.0.4) [![Latest Stable Version](https://poser.pugx.org/grumpydictator/firefly-iii/v/stable)](https://packagist.org/packages/grumpydictator/firefly-iii) [![Scrutinizer Code Quality](https://scrutinizer-ci.com/g/firefly-iii/firefly-iii/badges/quality-score.png?b=master)](https://scrutinizer-ci.com/g/firefly-iii/firefly-iii/?branch=master) [![Build Status](https://travis-ci.org/firefly-iii/firefly-iii.svg?branch=master)](https://travis-ci.org/firefly-iii/firefly-iii) [![Donate](https://img.shields.io/badge/Donate-PayPal-green.svg)](https://www.paypal.com/cgi-bin/webscr?cmd=_s-xclick&hosted_button_id=44UKUT455HUFA) 
 
 [![The index of Firefly III](https://i.nder.be/hurdhgyg/400)](https://i.nder.be/h2b37243) [![The account overview of Firefly III](https://i.nder.be/hnkfkdpr/400)](https://i.nder.be/hv70pbwc)
 
 [![The useful financial reports of Firefly III](https://i.nder.be/h7sk6nb7/400)](https://i.nder.be/ccn0u2mp) [![Even more useful reports in Firefly III](https://i.nder.be/g237hr35/400)](https://i.nder.be/gm8hbh7z)
 
-_(You can click on the images for a better view)_
-
 "Firefly III" is a financial manager. It can help you keep track of expenses, income, budgets and everything in between. It even supports credit cards, shared  household accounts and savings accounts! It's pretty fancy. You should use it to save and organise money.
 
+## Try it out!
+
+Try out Firefly III on the [demo site](https://firefly-iii.nder.be/).
+
 ## Installation
 
 To install Firefly III, you'll need a web server (preferrably on Linux) and access to the command line. Then, please read the [installation guide](https://firefly-iii.github.io/installation-guide/).
@@ -31,4 +31,6 @@ Firefly works on the principle that if you know where you're money is going, you
 
 Firefly is pretty awesome. [You can read more about Firefly III, and its features, on the Github Pages](https://firefly-iii.github.io/).
 
+If you like Firefly and if it helps you save lots of money, why not send me [a dime for every dollar saved](https://www.paypal.com/cgi-bin/webscr?cmd=_s-xclick&hosted_button_id=44UKUT455HUFA) (this is a joke, although the Paypal form works just fine, try it!)
+
 If you want to contact me, please open an issue or [email me](mailto:thegrumpydictator@gmail.com).

app/Console/Commands/CreateImport.php

@@ -50,11 +50,10 @@ class CreateImport extends Command
     }
 
     /**
-     *
+     * @SuppressWarnings(PHPMD.ExcessiveMethodLength) // cannot be helped
      */
     public function handle()
     {
-        // find the file
         /** @var UserRepositoryInterface $userRepository */
         $userRepository = app(UserRepositoryInterface::class);
         $file           = $this->argument('file');
@@ -67,7 +66,6 @@ class CreateImport extends Command
             return;
         }
 
-        // try to parse configuration data:
         $configurationData = json_decode(file_get_contents($configuration));
         if (is_null($configurationData)) {
             $this->error(sprintf('Firefly III cannot read the contents of configuration file "%s" (working directory: "%s").', $configuration, $cwd));
@@ -82,21 +80,17 @@ class CreateImport extends Command
 
         /** @var ImportJobRepositoryInterface $jobRepository */
         $jobRepository = app(ImportJobRepositoryInterface::class, [$user]);
-
-        $job = $jobRepository->create($type);
+        $job           = $jobRepository->create($type);
         $this->line(sprintf('Created job "%s"...', $job->key));
 
-        // put the file in the proper place:
         Artisan::call('firefly:encrypt', ['file' => $file, 'key' => $job->key]);
         $this->line('Stored import data...');
 
-        // store the configuration in the job:
         $job->configuration = $configurationData;
         $job->status        = 'settings_complete';
         $job->save();
         $this->line('Stored configuration...');
 
-        // if user wants to run it, do!
         if ($this->option('start') === true) {
             $this->line('The import will start in a moment. This process is not visible...');
             Log::debug('Go for import!');
@@ -109,10 +103,10 @@ class CreateImport extends Command
 
     /**
      * @return bool
+     * @SuppressWarnings(PHPMD.CyclomaticComplexity) // it's five exactly.
      */
     private function validArguments(): bool
     {
-        // find the file
         /** @var UserRepositoryInterface $userRepository */
         $userRepository = app(UserRepositoryInterface::class);
         $file           = $this->argument('file');

app/Console/Commands/Import.php

@@ -18,6 +18,7 @@ use FireflyIII\Import\Logging\CommandHandler;
 use FireflyIII\Models\ImportJob;
 use FireflyIII\Models\TransactionJournal;
 use Illuminate\Console\Command;
+use Illuminate\Support\Collection;
 use Log;
 
 /**
@@ -69,32 +70,15 @@ class Import extends Command
         $monolog = Log::getMonolog();
         $handler = new CommandHandler($this);
         $monolog->pushHandler($handler);
+        $importProcedure = new ImportProcedure;
+        $result          = $importProcedure->runImport($job);
 
-        $result = ImportProcedure::runImport($job);
-
-
-        /**
-         * @var int                $index
-         * @var TransactionJournal $journal
-         */
-        foreach ($result as $index => $journal) {
-            if (!is_null($journal->id)) {
-                $this->line(sprintf('Line #%d has been imported as transaction #%d.', $index, $journal->id));
-                continue;
-            }
-            $this->error(sprintf('Could not store line #%d', $index));
-        }
-
+        // display result to user:
+        $this->presentResults($result);
         $this->line('The import has completed.');
 
         // get any errors from the importer:
-        $extendedStatus = $job->extended_status;
-        if (isset($extendedStatus['errors']) && count($extendedStatus['errors']) > 0) {
-            $this->line(sprintf('The following %d error(s) occured during the import:', count($extendedStatus['errors'])));
-            foreach ($extendedStatus['errors'] as $error) {
-                $this->error($error);
-            }
-        }
+        $this->presentErrors($job);
 
         return;
     }
@@ -120,4 +104,36 @@ class Import extends Command
 
         return true;
     }
+
+    /**
+     * @param ImportJob $job
+     */
+    private function presentErrors(ImportJob $job)
+    {
+        $extendedStatus = $job->extended_status;
+        if (isset($extendedStatus['errors']) && count($extendedStatus['errors']) > 0) {
+            $this->line(sprintf('The following %d error(s) occured during the import:', count($extendedStatus['errors'])));
+            foreach ($extendedStatus['errors'] as $error) {
+                $this->error($error);
+            }
+        }
+    }
+
+    /**
+     * @param Collection $result
+     */
+    private function presentResults(Collection $result)
+    {
+        /**
+         * @var int                $index
+         * @var TransactionJournal $journal
+         */
+        foreach ($result as $index => $journal) {
+            if (!is_null($journal->id)) {
+                $this->line(sprintf('Line #%d has been imported as transaction #%d.', $index, $journal->id));
+                continue;
+            }
+            $this->error(sprintf('Could not store line #%d', $index));
+        }
+    }
 }

app/Console/Commands/MoveRepository.php

@@ -1,86 +0,0 @@
-<?php
-/**
- * MoveRepository.php
- * Copyright (C) 2016 thegrumpydictator@gmail.com
- *
- * This software may be modified and distributed under the terms of the
- * Creative Commons Attribution-ShareAlike 4.0 International License.
- *
- * See the LICENSE file for details.
- */
-
-declare(strict_types = 1);
-
-namespace FireflyIII\Console\Commands;
-
-use Carbon\Carbon;
-use Illuminate\Console\Command;
-
-/**
- * Class MoveRepository
- *
- * @package FireflyIII\Console\Commands
- */
-class MoveRepository extends Command
-{
-    /**
-     * The console command description.
-     *
-     * @var string
-     */
-    protected $description = 'Alerts the user that the Github repository will move, if they are interested to know this.';
-    /**
-     * The name and signature of the console command.
-     *
-     * @var string
-     */
-    protected $signature = 'firefly:github-move';
-
-    /**
-     * Create a new command instance.
-     *
-     */
-    public function __construct()
-    {
-        parent::__construct();
-    }
-
-    /**
-     * Execute the console command.
-     */
-    public function handle()
-    {
-        $moveDate = new Carbon('2017-01-01');
-        $final    = new Carbon('2017-03-01');
-        $now      = new Carbon;
-
-        // display message before 2017-01-01
-        if ($moveDate > $now) {
-            $this->line('+------------------------------------------------------------------------------+');
-            $this->line('');
-            $this->line('The Github repository for Firefly III will MOVE');
-            $this->line('This move will be on January 1st 2017');
-            $this->line('');
-            $this->error('READ THIS WIKI PAGE FOR MORE INFORMATION');
-            $this->line('');
-            $this->info('https://github.com/firefly-iii/help/wiki/New-Github-repository');
-            $this->line('');
-            $this->line('+------------------------------------------------------------------------------+');
-        }
-
-        // display message after 2017-01-01 but before 2017-03-01
-        if ($moveDate <= $now && $now <= $final) {
-            $this->line('+------------------------------------------------------------------------------+');
-            $this->line('');
-            $this->line('The Github repository for Firefly III has MOVED');
-            $this->line('This move was on January 1st 2017!');
-            $this->line('');
-            $this->error('READ THIS WIKI PAGE FOR MORE INFORMATION');
-            $this->line('');
-            $this->info('https://github.com/firefly-iii/help/wiki/New-Github-repository');
-            $this->line('');
-            $this->line('+------------------------------------------------------------------------------+');
-        }
-
-    }
-}

app/Console/Commands/ScanAttachments.php

@@ -60,42 +60,26 @@ class ScanAttachments extends Command
         /** @var Attachment $attachment */
         foreach ($attachments as $attachment) {
             $fileName = $attachment->fileName();
-
-            // try to grab file content:
             try {
                 $content = $disk->get($fileName);
             } catch (FileNotFoundException $e) {
                 $this->error(sprintf('Could not find data for attachment #%d', $attachment->id));
                 continue;
             }
-            // try to decrypt content.
             try {
                 $decrypted = Crypt::decrypt($content);
             } catch (DecryptException $e) {
                 $this->error(sprintf('Could not decrypt data of attachment #%d', $attachment->id));
                 continue;
             }
-
-            // make temp file:
             $tmpfname = tempnam(sys_get_temp_dir(), 'FireflyIII');
-
-            // store content in temp file:
             file_put_contents($tmpfname, $decrypted);
-
-            // get md5 and mime
-            $md5  = md5_file($tmpfname);
-            $mime = mime_content_type($tmpfname);
-
-            // update attachment:
+            $md5              = md5_file($tmpfname);
+            $mime             = mime_content_type($tmpfname);
             $attachment->md5  = $md5;
             $attachment->mime = $mime;
             $attachment->save();
-
-
             $this->line(sprintf('Fixed attachment #%d', $attachment->id));
-
-            // find file:
-
         }
     }
 }

app/Console/Commands/UpgradeDatabase.php

@@ -15,6 +15,8 @@ namespace FireflyIII\Console\Commands;
 
 
 use DB;
+use FireflyIII\Models\BudgetLimit;
+use FireflyIII\Models\LimitRepetition;
 use FireflyIII\Models\Transaction;
 use FireflyIII\Models\TransactionJournal;
 use Illuminate\Console\Command;
@@ -57,8 +59,29 @@ class UpgradeDatabase extends Command
     public function handle()
     {
         $this->setTransactionIdentifier();
+        $this->migrateRepetitions();
+    }
 
-
+    private function migrateRepetitions()
+    {
+        if (!Schema::hasTable('budget_limits')) {
+            return;
+        }
+        // get all budget limits with end_date NULL
+        $set = BudgetLimit::whereNull('end_date')->get();
+        $this->line(sprintf('Found %d budget limit(s) to update', $set->count()));
+        /** @var BudgetLimit $budgetLimit */
+        foreach ($set as $budgetLimit) {
+            // get limit repetition (should be just one):
+            /** @var LimitRepetition $repetition */
+            $repetition = $budgetLimit->limitrepetitions()->first();
+            if (!is_null($repetition)) {
+                $budgetLimit->end_date = $repetition->enddate;
+                $budgetLimit->save();
+                $this->line(sprintf('Updated budget limit #%d', $budgetLimit->id));
+                $repetition->delete();
+            }
+        }
     }
 
     /**
@@ -85,42 +108,52 @@ class UpgradeDatabase extends Command
         $journalIds = array_unique($result->pluck('id')->toArray());
 
         foreach ($journalIds as $journalId) {
-            // grab all positive transactiosn from this journal that are not deleted.
-            // for each one, grab the negative opposing one which has 0 as an identifier and give it the same identifier.
-            $identifier   = 0;
-            $processed    = [];
-            $transactions = Transaction::where('transaction_journal_id', $journalId)->where('amount', '>', 0)->get();
-            /** @var Transaction $transaction */
-            foreach ($transactions as $transaction) {
-                // find opposing:
-                $amount = bcmul(strval($transaction->amount), '-1');
+            $this->updateJournal(intval($journalId));
+        }
+    }
 
-                try {
-                    /** @var Transaction $opposing */
-                    $opposing = Transaction::where('transaction_journal_id', $journalId)
-                                           ->where('amount', $amount)->where('identifier', '=', 0)
-                                           ->whereNotIn('id', $processed)
-                                           ->first();
-                } catch (QueryException $e) {
-                    Log::error($e->getMessage());
-                    $this->error('Firefly III could not find the "identifier" field in the "transactions" table.');
-                    $this->error(sprintf('This field is required for Firefly III version %s to run.', config('firefly.version')));
-                    $this->error('Please run "php artisan migrate" to add this field to the table.');
-                    $this->info('Then, run "php artisan firefly:upgrade-database" to try again.');
-                    break 2;
-                }
-                if (!is_null($opposing)) {
-                    // give both a new identifier:
-                    $transaction->identifier = $identifier;
-                    $transaction->save();
-                    $opposing->identifier = $identifier;
-                    $opposing->save();
-                    $processed[] = $transaction->id;
-                    $processed[] = $opposing->id;
-                    $this->line(sprintf('Database upgrade for journal #%d, transactions #%d and #%d', $journalId, $transaction->id, $opposing->id));
-                }
-                $identifier++;
+    /**
+     * grab all positive transactiosn from this journal that are not deleted. for each one, grab the negative opposing one
+     * which has 0 as an identifier and give it the same identifier.
+     *
+     * @param int $journalId
+     */
+    private function updateJournal(int $journalId)
+    {
+        $identifier   = 0;
+        $processed    = [];
+        $transactions = Transaction::where('transaction_journal_id', $journalId)->where('amount', '>', 0)->get();
+        /** @var Transaction $transaction */
+        foreach ($transactions as $transaction) {
+            // find opposing:
+            $amount = bcmul(strval($transaction->amount), '-1');
+
+            try {
+                /** @var Transaction $opposing */
+                $opposing = Transaction::where('transaction_journal_id', $journalId)
+                                       ->where('amount', $amount)->where('identifier', '=', 0)
+                                       ->whereNotIn('id', $processed)
+                                       ->first();
+            } catch (QueryException $e) {
+                Log::error($e->getMessage());
+                $this->error('Firefly III could not find the "identifier" field in the "transactions" table.');
+                $this->error(sprintf('This field is required for Firefly III version %s to run.', config('firefly.version')));
+                $this->error('Please run "php artisan migrate" to add this field to the table.');
+                $this->info('Then, run "php artisan firefly:upgrade-database" to try again.');
+
+                return;
             }
+            if (!is_null($opposing)) {
+                // give both a new identifier:
+                $transaction->identifier = $identifier;
+                $transaction->save();
+                $opposing->identifier = $identifier;
+                $opposing->save();
+                $processed[] = $transaction->id;
+                $processed[] = $opposing->id;
+                $this->line(sprintf('Database upgrade for journal #%d, transactions #%d and #%d', $journalId, $transaction->id, $opposing->id));
+            }
+            $identifier++;
         }
     }
 }

app/Console/Commands/UpgradeFireflyInstructions.php

@@ -63,20 +63,21 @@ class UpgradeFireflyInstructions extends Command
 
         }
 
-
         if (is_null($text)) {
             $this->line(sprintf('Thank you for installing Firefly III, v%s', $version));
             $this->info('There are no extra upgrade instructions.');
             $this->line('Firefly III should be ready for use.');
-        } else {
-            $this->line('+------------------------------------------------------------------------------+');
-            $this->line('');
-            $this->line(sprintf('Thank you for installing Firefly III, v%s', $version));
-            $this->info(wordwrap($text));
-            $this->line('');
-            $this->line('+------------------------------------------------------------------------------+');
+
+            return;
         }
 
+        $this->line('+------------------------------------------------------------------------------+');
+        $this->line('');
+        $this->line(sprintf('Thank you for installing Firefly III, v%s', $version));
+        $this->info(wordwrap($text));
+        $this->line('');
+        $this->line('+------------------------------------------------------------------------------+');
+
 
     }
 }

app/Console/Commands/VerifyDatabase.php

@@ -124,16 +124,15 @@ class VerifyDatabase extends Command
     {
         $set = Budget::leftJoin('budget_limits', 'budget_limits.budget_id', '=', 'budgets.id')
                      ->leftJoin('users', 'budgets.user_id', '=', 'users.id')
-                     ->groupBy(['budgets.id', 'budgets.name', 'budgets.user_id', 'users.email'])
+                     ->groupBy(['budgets.id', 'budgets.name', 'budgets.encrypted', 'budgets.user_id', 'users.email'])
                      ->whereNull('budget_limits.id')
-                     ->get(['budgets.id', 'budgets.name', 'budgets.user_id', 'users.email']);
+                     ->get(['budgets.id', 'budgets.name', 'budgets.user_id', 'budgets.encrypted', 'users.email']);
 
-        /** @var stdClass $entry */
+        /** @var Budget $entry */
         foreach ($set as $entry) {
-
             $line = sprintf(
                 'Notice: User #%d (%s) has budget #%d ("%s") which has no budget limits.',
-                $entry->user_id, $entry->email, $entry->id, Crypt::decrypt($entry->name)
+                $entry->user_id, $entry->email, $entry->id, $entry->name
             );
             $this->line($line);
         }
@@ -174,10 +173,8 @@ class VerifyDatabase extends Command
         $configuration = [
             // a withdrawal can not have revenue account:
             TransactionType::WITHDRAWAL => [AccountType::REVENUE],
-
             // deposit cannot have an expense account:
             TransactionType::DEPOSIT    => [AccountType::EXPENSE],
-
             // transfer cannot have either:
             TransactionType::TRANSFER   => [AccountType::EXPENSE, AccountType::REVENUE],
         ];

app/Console/Kernel.php

@@ -21,7 +21,6 @@ use FireflyIII\Console\Commands\ScanAttachments;
 use FireflyIII\Console\Commands\UpgradeDatabase;
 use FireflyIII\Console\Commands\UpgradeFireflyInstructions;
 use FireflyIII\Console\Commands\VerifyDatabase;
-use Illuminate\Console\Scheduling\Schedule;
 use Illuminate\Foundation\Console\Kernel as ConsoleKernel;
 
 /**
@@ -64,7 +63,6 @@ class Kernel extends ConsoleKernel
             EncryptFile::class,
             ScanAttachments::class,
             UpgradeDatabase::class,
-            MoveRepository::class,
         ];
 
     /**
@@ -76,15 +74,4 @@ class Kernel extends ConsoleKernel
     {
         require base_path('routes/console.php');
     }
-
-    /**
-     * Define the application's command schedule.
-     *
-     * @param  \Illuminate\Console\Scheduling\Schedule $schedule
-     *
-     * @return void
-     */
-    protected function schedule(Schedule $schedule)
-    {
-    }
 }

app/Events/BlockedBadLogin.php

@@ -1,41 +0,0 @@
-<?php
-/**
- * BlockedBadLogin.php
- * Copyright (C) 2016 thegrumpydictator@gmail.com
- *
- * This software may be modified and distributed under the terms of the
- * Creative Commons Attribution-ShareAlike 4.0 International License.
- *
- * See the LICENSE file for details.
- */
-
-declare(strict_types = 1);
-
-namespace FireflyIII\Events;
-
-use Illuminate\Queue\SerializesModels;
-
-/**
- * Class LockedOutUser
- *
- * @package FireflyIII\Events
- */
-class BlockedBadLogin extends Event
-{
-    use SerializesModels;
-
-    public $email;
-    public $ipAddress;
-
-    /**
-     * Create a new event instance. This event is triggered when a user gets themselves locked out.
-     *
-     * @param string $email
-     * @param string $ipAddress
-     */
-    public function __construct(string $email, string $ipAddress)
-    {
-        $this->email     = $email;
-        $this->ipAddress = $ipAddress;
-    }
-}

app/Events/BlockedUseOfDomain.php

@@ -1,42 +0,0 @@
-<?php
-/**
- * BlockedUseOfDomain.php
- * Copyright (C) 2016 thegrumpydictator@gmail.com
- *
- * This software may be modified and distributed under the terms of the
- * Creative Commons Attribution-ShareAlike 4.0 International License.
- *
- * See the LICENSE file for details.
- */
-
-declare(strict_types = 1);
-
-namespace FireflyIII\Events;
-
-
-use Illuminate\Queue\SerializesModels;
-
-/**
- * Class BlockedUseOfDomain
- *
- * @package FireflyIII\Events
- */
-class BlockedUseOfDomain extends Event
-{
-    use SerializesModels;
-
-    public $email;
-    public $ipAddress;
-
-    /**
-     * Create a new event instance. This event is triggered when a user tries to register with a banned domain (on blocked domain list).
-     *
-     * @param string $email
-     * @param string $ipAddress
-     */
-    public function __construct(string $email, string $ipAddress)
-    {
-        $this->email     = $email;
-        $this->ipAddress = $ipAddress;
-    }
-}

app/Events/BlockedUseOfEmail.php

@@ -1,42 +0,0 @@
-<?php
-/**
- * BlockedUseOfEmail.php
- * Copyright (C) 2016 thegrumpydictator@gmail.com
- *
- * This software may be modified and distributed under the terms of the
- * Creative Commons Attribution-ShareAlike 4.0 International License.
- *
- * See the LICENSE file for details.
- */
-
-declare(strict_types = 1);
-
-namespace FireflyIII\Events;
-
-
-use Illuminate\Queue\SerializesModels;
-
-/**
- * Class BlockedUseOfEmail
- *
- * @package FireflyIII\Events
- */
-class BlockedUseOfEmail extends Event
-{
-    use SerializesModels;
-
-    public $email;
-    public $ipAddress;
-
-    /**
-     * Create a new event instance. This event is triggered when a user tries to register with a banned email address (already used before).
-     *
-     * @param string $email
-     * @param string $ipAddress
-     */
-    public function __construct(string $email, string $ipAddress)
-    {
-        $this->email     = $email;
-        $this->ipAddress = $ipAddress;
-    }
-}

app/Events/BlockedUserLogin.php

@@ -1,42 +0,0 @@
-<?php
-/**
- * BlockedUserLogin.php
- * Copyright (C) 2016 thegrumpydictator@gmail.com
- *
- * This software may be modified and distributed under the terms of the
- * Creative Commons Attribution-ShareAlike 4.0 International License.
- *
- * See the LICENSE file for details.
- */
-
-declare(strict_types = 1);
-
-namespace FireflyIII\Events;
-
-use FireflyIII\User;
-use Illuminate\Queue\SerializesModels;
-
-/**
- * Class BlockedUserLogin
- *
- * @package FireflyIII\Events
- */
-class BlockedUserLogin extends Event
-{
-    use SerializesModels;
-
-    public $ipAddress;
-    public $user;
-
-    /**
-     * Create a new event instance. This event is triggered when a blocked user logs in.
-     *
-     * @param User   $user
-     * @param string $ipAddress
-     */
-    public function __construct(User $user, string $ipAddress)
-    {
-        $this->user      = $user;
-        $this->ipAddress = $ipAddress;
-    }
-}

app/Events/ConfirmedUser.php

@@ -1,42 +0,0 @@
-<?php
-/**
- * ConfirmedUser.php
- * Copyright (C) 2016 thegrumpydictator@gmail.com
- *
- * This software may be modified and distributed under the terms of the
- * Creative Commons Attribution-ShareAlike 4.0 International License.
- *
- * See the LICENSE file for details.
- */
-
-declare(strict_types = 1);
-
-namespace FireflyIII\Events;
-
-use FireflyIII\User;
-use Illuminate\Queue\SerializesModels;
-
-/**
- * Class ConfirmedUser
- *
- * @package FireflyIII\Events
- */
-class ConfirmedUser extends Event
-{
-    use SerializesModels;
-
-    public $ipAddress;
-    public $user;
-
-    /**
-     * Create a new event instance. This event is triggered when a user confirms their new account.
-     *
-     * @param  User  $user
-     * @param string $ipAddress
-     */
-    public function __construct(User $user, string $ipAddress)
-    {
-        $this->user      = $user;
-        $this->ipAddress = $ipAddress;
-    }
-}

app/Events/DeletedUser.php

@@ -1,38 +0,0 @@
-<?php
-/**
- * DeletedUser.php
- * Copyright (C) 2016 thegrumpydictator@gmail.com
- *
- * This software may be modified and distributed under the terms of the
- * Creative Commons Attribution-ShareAlike 4.0 International License.
- *
- * See the LICENSE file for details.
- */
-
-declare(strict_types = 1);
-
-namespace FireflyIII\Events;
-
-use Illuminate\Queue\SerializesModels;
-
-/**
- * Class DeletedUser
- *
- * @package FireflyIII\Events
- */
-class DeletedUser extends Event
-{
-    use SerializesModels;
-
-    public $email;
-
-    /**
-     * Create a new event instance. This event is triggered when a user deletes themselves.
-     *
-     * @param  string $email
-     */
-    public function __construct(string $email)
-    {
-        $this->email = $email;
-    }
-}

app/Events/LockedOutUser.php

@@ -1,41 +0,0 @@
-<?php
-/**
- * LockedOutUser.php
- * Copyright (C) 2016 thegrumpydictator@gmail.com
- *
- * This software may be modified and distributed under the terms of the
- * Creative Commons Attribution-ShareAlike 4.0 International License.
- *
- * See the LICENSE file for details.
- */
-
-declare(strict_types = 1);
-
-namespace FireflyIII\Events;
-
-use Illuminate\Queue\SerializesModels;
-
-/**
- * Class LockedOutUser
- *
- * @package FireflyIII\Events
- */
-class LockedOutUser extends Event
-{
-    use SerializesModels;
-
-    public $email;
-    public $ipAddress;
-
-    /**
-     * Create a new event instance. This event is triggered when a user gets themselves locked out.
-     *
-     * @param string $email
-     * @param string $ipAddress
-     */
-    public function __construct(string $email, string $ipAddress)
-    {
-        $this->email     = $email;
-        $this->ipAddress = $ipAddress;
-    }
-}

app/Events/RequestedNewPassword.php

@@ -43,4 +43,4 @@ class RequestedNewPassword extends Event
         $this->ipAddress = $ipAddress;
     }
 
-}
+}

app/Events/ResentConfirmation.php

@@ -1,42 +0,0 @@
-<?php
-/**
- * ResentConfirmation.php
- * Copyright (C) 2016 thegrumpydictator@gmail.com
- *
- * This software may be modified and distributed under the terms of the
- * Creative Commons Attribution-ShareAlike 4.0 International License.
- *
- * See the LICENSE file for details.
- */
-
-declare(strict_types = 1);
-
-namespace FireflyIII\Events;
-
-use FireflyIII\User;
-use Illuminate\Queue\SerializesModels;
-
-/**
- * Class ResentConfirmation
- *
- * @package FireflyIII\Events
- */
-class ResentConfirmation extends Event
-{
-    use SerializesModels;
-
-    public $ipAddress;
-    public $user;
-
-    /**
-     * Create a new event instance. This event is triggered when a users wants a new confirmation.
-     *
-     * @param  User  $user
-     * @param string $ipAddress
-     */
-    public function __construct(User $user, string $ipAddress)
-    {
-        $this->user      = $user;
-        $this->ipAddress = $ipAddress;
-    }
-}

app/Events/StoredBudgetLimit.php

@@ -1,50 +0,0 @@
-<?php
-/**
- * StoredBudgetLimit.php
- * Copyright (C) 2016 thegrumpydictator@gmail.com
- *
- * This software may be modified and distributed under the terms of the
- * Creative Commons Attribution-ShareAlike 4.0 International License.
- *
- * See the LICENSE file for details.
- */
-
-declare(strict_types = 1);
-
-namespace FireflyIII\Events;
-
-use Carbon\Carbon;
-use FireflyIII\Models\BudgetLimit;
-use Illuminate\Queue\SerializesModels;
-
-/**
- * Class StoredBudgetLimit
- *
- * @package FireflyIII\Events
- */
-class StoredBudgetLimit extends Event
-{
-
-    use SerializesModels;
-
-    /** @var  BudgetLimit */
-    public $budgetLimit;
-
-    /** @var  Carbon */
-    public $end; // the only variable we can't get from the budget limit (if necessary).
-
-    /**
-     * BudgetLimitEvents constructor.
-     *
-     * @param BudgetLimit $budgetLimit
-     * @param Carbon      $end
-     */
-    public function __construct(BudgetLimit $budgetLimit, Carbon $end)
-    {
-        //
-        $this->budgetLimit = $budgetLimit;
-        $this->end         = $end;
-
-    }
-
-}

app/Events/UpdatedBudgetLimit.php

@@ -1,50 +0,0 @@
-<?php
-/**
- * UpdatedBudgetLimit.php
- * Copyright (C) 2016 thegrumpydictator@gmail.com
- *
- * This software may be modified and distributed under the terms of the
- * Creative Commons Attribution-ShareAlike 4.0 International License.
- *
- * See the LICENSE file for details.
- */
-
-declare(strict_types = 1);
-
-namespace FireflyIII\Events;
-
-use Carbon\Carbon;
-use FireflyIII\Models\BudgetLimit;
-use Illuminate\Queue\SerializesModels;
-
-/**
- * Class UpdatedBudgetLimit
- *
- * @package FireflyIII\Events
- */
-class UpdatedBudgetLimit extends Event
-{
-
-    use SerializesModels;
-
-    /** @var  BudgetLimit */
-    public $budgetLimit;
-
-    /** @var  Carbon */
-    public $end; // the only variable we can't get from the budget limit (if necessary).
-
-    /**
-     * BudgetLimitEvents constructor.
-     *
-     * @param BudgetLimit $budgetLimit
-     * @param Carbon      $end
-     */
-    public function __construct(BudgetLimit $budgetLimit, Carbon $end)
-    {
-        //
-        $this->budgetLimit = $budgetLimit;
-        $this->end         = $end;
-
-    }
-
-}

app/Exceptions/Handler.php

@@ -21,6 +21,7 @@ use Illuminate\Database\Eloquent\ModelNotFoundException;
 use Illuminate\Foundation\Exceptions\Handler as ExceptionHandler;
 use Illuminate\Session\TokenMismatchException;
 use Illuminate\Validation\ValidationException as ValException;
+use Request;
 use Symfony\Component\HttpKernel\Exception\HttpException;
 
 /**
@@ -72,6 +73,7 @@ class Handler extends ExceptionHandler
      * This is a great spot to send exceptions to Sentry, Bugsnag, etc.
      *
      * @param  Exception $exception
+     * @SuppressWarnings(PHPMD.CyclomaticComplexity) // it's exactly five.
      *
      * @return void
      */
@@ -98,8 +100,8 @@ class Handler extends ExceptionHandler
             ];
 
             // create job that will mail.
-            $ip  = $_SERVER['REMOTE_ADDR'] ?? '0.0.0.0';
-            $job = new MailError($userData, env('SITE_OWNER', ''), $ip, $data);
+            $ipAddress = Request::ip() ?? '0.0.0.0';
+            $job       = new MailError($userData, env('SITE_OWNER', ''), $ipAddress, $data);
             dispatch($job);
         }
 

app/Export/Collector/CollectorInterface.php

@@ -35,6 +35,8 @@ interface CollectorInterface
     /**
      * @param Collection $entries
      *
+     * @return void
+     *
      */
     public function setEntries(Collection $entries);
 

app/Export/Collector/JournalExportCollector.php

@@ -287,7 +287,7 @@ class JournalExportCollector extends BasicCollector implements CollectorInterfac
     }
 
     /**
-     *
+     * @SuppressWarnings(PHPMD.ExcessiveMethodLength)
      */
     private function getWorkSet()
     {

app/Export/Entry/Entry.php

@@ -29,6 +29,7 @@ use Crypt;
  *
  *
  * Class Entry
+ * @SuppressWarnings(PHPMD.LongVariable)
  *
  * @package FireflyIII\Export\Entry
  */
@@ -71,33 +72,21 @@ final class Entry
      */
     public static function fromObject($object): Entry
     {
-        $entry = new self;
-
-        // journal information:
-        $entry->journal_id       = $object->transaction_journal_id;
-        $entry->description      = $object->journal_encrypted === 1 ? Crypt::decrypt($object->journal_description) : $object->journal_description;
-        $entry->amount           = round($object->amount, 2); // always positive
-        $entry->date             = $object->date;
-        $entry->transaction_type = $object->transaction_type;
-        $entry->currency_code    = $object->transaction_currency_code;
-
-        // source information:
-        $entry->source_account_id   = $object->account_id;
-        $entry->source_account_name = $object->account_name_encrypted === 1 ? Crypt::decrypt($object->account_name) : $object->account_name;
-
-
-        // destination information
+        $entry                           = new self;
+        $entry->journal_id               = $object->transaction_journal_id;
+        $entry->description              = self::decrypt($object->journal_encrypted, $object->journal_description);
+        $entry->amount                   = $object->amount;
+        $entry->date                     = $object->date;
+        $entry->transaction_type         = $object->transaction_type;
+        $entry->currency_code            = $object->transaction_currency_code;
+        $entry->source_account_id        = $object->account_id;
+        $entry->source_account_name      = self::decrypt($object->account_name_encrypted, $object->account_name);
         $entry->destination_account_id   = $object->opposing_account_id;
-        $entry->destination_account_name = $object->opposing_account_encrypted === 1 ? Crypt::decrypt($object->opposing_account_name)
-            : $object->opposing_account_name;
-
-
-        // category and budget
-        $entry->category_id   = $object->category_id ?? '';
-        $entry->category_name = $object->category_name ?? '';
-        $entry->budget_id     = $object->budget_id ?? '';
-        $entry->budget_name   = $object->budget_name ?? '';
-
+        $entry->destination_account_name = self::decrypt($object->opposing_account_encrypted, $object->opposing_account_name);
+        $entry->category_id              = $object->category_id ?? '';
+        $entry->category_name            = $object->category_name ?? '';
+        $entry->budget_id                = $object->budget_id ?? '';
+        $entry->budget_name              = $object->budget_name ?? '';
 
         // update description when transaction description is different:
         if (!is_null($object->description) && $object->description != $entry->description) {
@@ -107,4 +96,19 @@ final class Entry
         return $entry;
     }
 
+    /**
+     * @param int $isEncrypted
+     * @param     $value
+     *
+     * @return string
+     */
+    protected static function decrypt(int $isEncrypted, $value)
+    {
+        if ($isEncrypted === 1) {
+            return Crypt::decrypt($value);
+        }
+
+        return $value;
+    }
+
 }

app/Export/Exporter/ExporterInterface.php

@@ -40,6 +40,8 @@ interface ExporterInterface
     /**
      * @param Collection $entries
      *
+     * @return void
+     *
      */
     public function setEntries(Collection $entries);
 

app/Export/Processor.php

@@ -155,7 +155,7 @@ class Processor implements ProcessorInterface
         $zip->close();
 
         // delete the files:
-        $this->deleteFiles($disk);
+        $this->deleteFiles();
 
         return true;
     }
@@ -183,10 +183,11 @@ class Processor implements ProcessorInterface
     }
 
     /**
-     * @param FilesystemAdapter $disk
+     *
      */
-    private function deleteFiles(FilesystemAdapter $disk)
+    private function deleteFiles()
     {
+        $disk = Storage::disk('export');
         foreach ($this->getFiles() as $file) {
             $disk->delete($file);
         }

app/Export/ProcessorInterface.php

@@ -27,6 +27,7 @@ interface ProcessorInterface
      * Processor constructor.
      *
      * @param array $settings
+     *
      */
     public function __construct(array $settings);
 
@@ -64,4 +65,4 @@ interface ProcessorInterface
      * @return Collection
      */
     public function getFiles(): Collection;
-}
+}

app/Generator/Chart/Basic/ChartJsGenerator.php

@@ -31,6 +31,8 @@ class ChartJsGenerator implements GeneratorInterface
      * 0: [
      *    'label' => 'label of set',
      *    'type' => bar or line, optional
+     *    'yAxisID' => ID of yAxis, optional, will not be included when unused.
+     *    'fill' => if to fill a line? optional, will not be included when unused.
      *    'entries' =>
      *        [
      *         'label-of-entry' => 'value'
@@ -39,12 +41,15 @@ class ChartJsGenerator implements GeneratorInterface
      * 1: [
      *    'label' => 'label of another set',
      *    'type' => bar or line, optional
+     *    'yAxisID' => ID of yAxis, optional, will not be included when unused.
+     *    'fill' => if to fill a line? optional, will not be included when unused.
      *    'entries' =>
      *        [
      *         'label-of-entry' => 'value'
      *        ]
      *    ]
      *
+     * @SuppressWarnings(PHPMD.CyclomaticComplexity) // it's five.
      *
      * @param array $data
      *
@@ -54,7 +59,7 @@ class ChartJsGenerator implements GeneratorInterface
     {
         reset($data);
         $first  = current($data);
-        $labels = array_keys($first['entries']);
+        $labels = is_array($first['entries']) ? array_keys($first['entries']) : [];
 
         $chartData = [
             'count'    => count($data),
@@ -64,11 +69,19 @@ class ChartJsGenerator implements GeneratorInterface
         unset($first, $labels);
 
         foreach ($data as $set) {
-            $chartData['datasets'][] = [
+            $currentSet = [
                 'label' => $set['label'],
                 'type'  => $set['type'] ?? 'line',
                 'data'  => array_values($set['entries']),
             ];
+            if (isset($set['yAxisID'])) {
+                $currentSet['yAxisID'] = $set['yAxisID'];
+            }
+            if (isset($set['fill'])) {
+                $currentSet['fill'] = $set['fill'];
+            }
+
+            $chartData['datasets'][] = $currentSet;
         }
 
         return $chartData;
@@ -99,7 +112,7 @@ class ChartJsGenerator implements GeneratorInterface
                 $value = bcmul($value, '-1');
             }
 
-            $chartData['datasets'][0]['data'][]            = round($value, 2);
+            $chartData['datasets'][0]['data'][]            = $value;
             $chartData['datasets'][0]['backgroundColor'][] = ChartColour::getColour($index);
             $chartData['labels'][]                         = $key;
             $index++;
@@ -134,4 +147,4 @@ class ChartJsGenerator implements GeneratorInterface
 
         return $chartData;
     }
-}
+}

app/Generator/Chart/Basic/GeneratorInterface.php

@@ -70,4 +70,4 @@ interface GeneratorInterface
      */
     public function singleSet(string $setLabel, array $data): array;
 
-}
+}

app/Generator/Report/Audit/MonthReportGenerator.php

@@ -16,7 +16,7 @@ namespace FireflyIII\Generator\Report\Audit;
 
 use Carbon\Carbon;
 use FireflyIII\Generator\Report\ReportGeneratorInterface;
-use FireflyIII\Helpers\Collector\JournalCollector;
+use FireflyIII\Helpers\Collector\JournalCollectorInterface;
 use FireflyIII\Models\Account;
 use FireflyIII\Models\Transaction;
 use Illuminate\Support\Collection;
@@ -49,33 +49,8 @@ class MonthReportGenerator implements ReportGeneratorInterface
         /** @var Account $account */
         foreach ($this->accounts as $account) {
             // balance the day before:
-            $id               = $account->id;
-            $dayBeforeBalance = Steam::balance($account, $dayBefore);
-            $collector        = new JournalCollector(auth()->user());
-            $collector->setAccounts(new Collection([$account]))->setRange($this->start, $this->end);
-            $journals     = $collector->getJournals();
-            $journals     = $journals->reverse();
-            $startBalance = $dayBeforeBalance;
-
-
-            /** @var Transaction $journal */
-            foreach ($journals as $transaction) {
-                $transaction->before = $startBalance;
-                $transactionAmount   = $transaction->transaction_amount;
-                $newBalance          = bcadd($startBalance, $transactionAmount);
-                $transaction->after  = $newBalance;
-                $startBalance        = $newBalance;
-            }
-
-            /*
-             * Reverse set again.
-             */
-            $auditData[$id]['journals']         = $journals->reverse();
-            $auditData[$id]['exists']           = $journals->count() > 0;
-            $auditData[$id]['end']              = $this->end->formatLocalized(strval(trans('config.month_and_day')));
-            $auditData[$id]['endBalance']       = Steam::balance($account, $this->end);
-            $auditData[$id]['dayBefore']        = $dayBefore->formatLocalized(strval(trans('config.month_and_day')));
-            $auditData[$id]['dayBeforeBalance'] = $dayBeforeBalance;
+            $id             = $account->id;
+            $auditData[$id] = $this->getAuditReport($account, $dayBefore);
         }
 
         $defaultShow = ['icon', 'description', 'balance_before', 'amount', 'balance_after', 'date', 'to'];
@@ -153,4 +128,46 @@ class MonthReportGenerator implements ReportGeneratorInterface
 
         return $this;
     }
-}
+
+    /**
+     * @param Account $account
+     * @param Carbon  $date
+     *
+     * @return array
+     */
+    private function getAuditReport(Account $account, Carbon $date): array
+    {
+
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
+        $collector->setAccounts(new Collection([$account]))->setRange($this->start, $this->end);
+        $journals         = $collector->getJournals();
+        $journals         = $journals->reverse();
+        $dayBeforeBalance = Steam::balance($account, $date);
+        $startBalance     = $dayBeforeBalance;
+
+
+        /** @var Transaction $journal */
+        foreach ($journals as $transaction) {
+            $transaction->before = $startBalance;
+            $transactionAmount   = $transaction->transaction_amount;
+            $newBalance          = bcadd($startBalance, $transactionAmount);
+            $transaction->after  = $newBalance;
+            $startBalance        = $newBalance;
+        }
+
+        /*
+         * Reverse set again.
+         */
+        $return = [
+            'journals'         => $journals->reverse(),
+            'exists'           => $journals->count() > 0,
+            'end'              => $this->end->formatLocalized(strval(trans('config.month_and_day'))),
+            'endBalance'       => Steam::balance($account, $this->end),
+            'dayBefore'        => $date->formatLocalized(strval(trans('config.month_and_day'))),
+            'dayBeforeBalance' => $dayBeforeBalance,
+        ];
+
+        return $return;
+    }
+}

app/Generator/Report/Audit/MultiYearReportGenerator.php

@@ -24,4 +24,4 @@ class MultiYearReportGenerator extends MonthReportGenerator
     /**
      * Doesn't do anything different.
      */
-}
+}

app/Generator/Report/Audit/YearReportGenerator.php

@@ -25,4 +25,4 @@ class YearReportGenerator extends MonthReportGenerator
     /**
      * Doesn't do anything different.
      */
-}
+}

app/Generator/Report/Budget/MonthReportGenerator.php

@@ -17,7 +17,7 @@ namespace FireflyIII\Generator\Report\Budget;
 use Carbon\Carbon;
 use FireflyIII\Generator\Report\ReportGeneratorInterface;
 use FireflyIII\Generator\Report\Support;
-use FireflyIII\Helpers\Collector\JournalCollector;
+use FireflyIII\Helpers\Collector\JournalCollectorInterface;
 use FireflyIII\Models\Transaction;
 use FireflyIII\Models\TransactionType;
 use Illuminate\Support\Collection;
@@ -184,7 +184,8 @@ class MonthReportGenerator extends Support implements ReportGeneratorInterface
             return $this->expenses;
         }
 
-        $collector = new JournalCollector(auth()->user());
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
         $collector->setAccounts($this->accounts)->setRange($this->start, $this->end)
                   ->setTypes([TransactionType::WITHDRAWAL])
                   ->setBudgets($this->budgets)->withOpposingAccount()->disableFilter();
@@ -244,4 +245,4 @@ class MonthReportGenerator extends Support implements ReportGeneratorInterface
 
         return $result;
     }
-}
+}

app/Generator/Report/Budget/MultiYearReportGenerator.php

@@ -24,4 +24,4 @@ class MultiYearReportGenerator extends MonthReportGenerator
     /**
      * Doesn't do anything different.
      */
-}
+}

app/Generator/Report/Budget/YearReportGenerator.php

@@ -25,4 +25,4 @@ class YearReportGenerator extends MonthReportGenerator
     /**
      * Doesn't do anything different.
      */
-}
+}

app/Generator/Report/Category/MonthReportGenerator.php

@@ -17,7 +17,7 @@ namespace FireflyIII\Generator\Report\Category;
 use Carbon\Carbon;
 use FireflyIII\Generator\Report\ReportGeneratorInterface;
 use FireflyIII\Generator\Report\Support;
-use FireflyIII\Helpers\Collector\JournalCollector;
+use FireflyIII\Helpers\Collector\JournalCollectorInterface;
 use FireflyIII\Models\Transaction;
 use FireflyIII\Models\TransactionType;
 use Illuminate\Support\Collection;
@@ -194,7 +194,8 @@ class MonthReportGenerator extends Support implements ReportGeneratorInterface
             return $this->expenses;
         }
 
-        $collector = new JournalCollector(auth()->user());
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
         $collector->setAccounts($this->accounts)->setRange($this->start, $this->end)
                   ->setTypes([TransactionType::WITHDRAWAL, TransactionType::TRANSFER])
                   ->setCategories($this->categories)->withOpposingAccount()->disableFilter();
@@ -216,7 +217,8 @@ class MonthReportGenerator extends Support implements ReportGeneratorInterface
             return $this->income;
         }
 
-        $collector = new JournalCollector(auth()->user());
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
         $collector->setAccounts($this->accounts)->setRange($this->start, $this->end)
                   ->setTypes([TransactionType::DEPOSIT, TransactionType::TRANSFER])
                   ->setCategories($this->categories)->withOpposingAccount();
@@ -229,6 +231,7 @@ class MonthReportGenerator extends Support implements ReportGeneratorInterface
     }
 
     /**
+     * @SuppressWarnings(PHPMD.CyclomaticComplexity) // it's exactly five.
      * @param array $spent
      * @param array $earned
      *
@@ -325,4 +328,4 @@ class MonthReportGenerator extends Support implements ReportGeneratorInterface
 
         return $result;
     }
-}
+}

app/Generator/Report/Category/MultiYearReportGenerator.php

@@ -24,4 +24,4 @@ class MultiYearReportGenerator extends MonthReportGenerator
     /**
      * Doesn't do anything different.
      */
-}
+}

app/Generator/Report/Category/YearReportGenerator.php

@@ -25,4 +25,4 @@ class YearReportGenerator extends MonthReportGenerator
     /**
      * Doesn't do anything different.
      */
-}
+}

app/Generator/Report/ReportGeneratorFactory.php

@@ -57,4 +57,4 @@ class ReportGeneratorFactory
         }
         throw new FireflyException(sprintf('Cannot generate report. There is no "%s"-report for period "%s".', $type, $period));
     }
-}
+}

app/Generator/Report/ReportGeneratorInterface.php

@@ -64,4 +64,4 @@ interface ReportGeneratorInterface
      */
     public function setStartDate(Carbon $date): ReportGeneratorInterface;
 
-}
+}

app/Generator/Report/Standard/MonthReportGenerator.php

@@ -106,4 +106,4 @@ class MonthReportGenerator implements ReportGeneratorInterface
 
         return $this;
     }
-}
+}

app/Generator/Report/Standard/MultiYearReportGenerator.php

@@ -103,4 +103,4 @@ class MultiYearReportGenerator implements ReportGeneratorInterface
 
         return $this;
     }
-}
+}

app/Generator/Report/Standard/YearReportGenerator.php

@@ -103,4 +103,4 @@ class YearReportGenerator implements ReportGeneratorInterface
 
         return $this;
     }
-}
+}

app/Generator/Report/Support.php

@@ -34,27 +34,7 @@ class Support
      */
     public static function filterExpenses(Collection $collection, array $accounts): Collection
     {
-        $result = $collection->filter(
-            function (Transaction $transaction) use ($accounts) {
-                $opposing = $transaction->opposing_account_id;
-                // remove internal transfer
-                if (in_array($opposing, $accounts)) {
-                    Log::debug(sprintf('Filtered #%d because its opposite is in accounts.', $transaction->id));
-
-                    return null;
-                }
-                // remove positive amount
-                if (bccomp($transaction->transaction_amount, '0') === 1) {
-                    Log::debug(sprintf('Filtered #%d because amount is %f.', $transaction->id, $transaction->transaction_amount));
-
-                    return null;
-                }
-
-                return $transaction;
-            }
-        );
-
-        return $result;
+        return self::filterTransactions($collection, $accounts, 1);
     }
 
     /**
@@ -64,9 +44,21 @@ class Support
      * @return Collection
      */
     public static function filterIncome(Collection $collection, array $accounts): Collection
+    {
+        return self::filterTransactions($collection, $accounts, -1);
+    }
+
+    /**
+     * @param Collection $collection
+     * @param array      $accounts
+     * @param int        $modifier
+     *
+     * @return Collection
+     */
+    public static function filterTransactions(Collection $collection, array $accounts, int $modifier): Collection
     {
         $result = $collection->filter(
-            function (Transaction $transaction) use ($accounts) {
+            function (Transaction $transaction) use ($accounts, $modifier) {
                 $opposing = $transaction->opposing_account_id;
                 // remove internal transfer
                 if (in_array($opposing, $accounts)) {
@@ -75,7 +67,7 @@ class Support
                     return null;
                 }
                 // remove positive amount
-                if (bccomp($transaction->transaction_amount, '0') === -1) {
+                if (bccomp($transaction->transaction_amount, '0') === $modifier) {
                     Log::debug(sprintf('Filtered #%d because amount is %f.', $transaction->id, $transaction->transaction_amount));
 
                     return null;
@@ -88,4 +80,4 @@ class Support
         return $result;
     }
 
-}
+}

app/Handlers/Events/BudgetEventHandler.php

@@ -1,93 +0,0 @@
-<?php
-/**
- * BudgetEventHandler.php
- * Copyright (C) 2016 thegrumpydictator@gmail.com
- *
- * This software may be modified and distributed under the terms of the
- * Creative Commons Attribution-ShareAlike 4.0 International License.
- *
- * See the LICENSE file for details.
- */
-
-declare(strict_types = 1);
-
-namespace FireflyIII\Handlers\Events;
-
-
-use Carbon\Carbon;
-use FireflyIII\Events\StoredBudgetLimit;
-use FireflyIII\Events\UpdatedBudgetLimit;
-use FireflyIII\Models\BudgetLimit;
-use FireflyIII\Models\LimitRepetition;
-use Illuminate\Database\QueryException;
-use Log;
-
-/**
- * Handles budget related events.
- *
- * Class BudgetEventHandler
- *
- * @package FireflyIII\Handlers\Events
- */
-class BudgetEventHandler
-{
-    /**
-     * This method creates a new budget limit repetition when a new budget limit has been created.
-     *
-     * @param StoredBudgetLimit $budgetLimitEvent
-     *
-     * @return bool
-     */
-    public function storeRepetition(StoredBudgetLimit $budgetLimitEvent): bool
-    {
-        return $this->processRepetitionChange($budgetLimitEvent->budgetLimit, $budgetLimitEvent->end);
-    }
-
-    /**
-     * Updates, if present the budget limit repetition part of a budget limit.
-     *
-     * @param UpdatedBudgetLimit $budgetLimitEvent
-     *
-     * @return bool
-     */
-    public function updateRepetition(UpdatedBudgetLimit $budgetLimitEvent): bool
-    {
-        return $this->processRepetitionChange($budgetLimitEvent->budgetLimit, $budgetLimitEvent->end);
-    }
-
-    /**
-     * @param BudgetLimit $budgetLimit
-     * @param Carbon      $date
-     *
-     * @return bool
-     */
-    private function processRepetitionChange(BudgetLimit $budgetLimit, Carbon $date): bool
-    {
-        $set = $budgetLimit->limitrepetitions()
-                           ->where('startdate', $budgetLimit->startdate->format('Y-m-d 00:00:00'))
-                           ->where('enddate', $date->format('Y-m-d 00:00:00'))
-                           ->get();
-        if ($set->count() == 0) {
-            $repetition            = new LimitRepetition;
-            $repetition->startdate = $budgetLimit->startdate;
-            $repetition->enddate   = $date;
-            $repetition->amount    = $budgetLimit->amount;
-            $repetition->budgetLimit()->associate($budgetLimit);
-
-            try {
-                $repetition->save();
-            } catch (QueryException $e) {
-                Log::error('Trying to save new LimitRepetition failed: ' . $e->getMessage());
-            }
-        }
-
-        if ($set->count() == 1) {
-            $repetition         = $set->first();
-            $repetition->amount = $budgetLimit->amount;
-            $repetition->save();
-
-        }
-
-        return true;
-    }
-}

app/Handlers/Events/StoredJournalEventHandler.php

@@ -16,6 +16,7 @@ namespace FireflyIII\Handlers\Events;
 use FireflyIII\Events\StoredTransactionJournal;
 use FireflyIII\Models\PiggyBank;
 use FireflyIII\Models\PiggyBankEvent;
+use FireflyIII\Models\PiggyBankRepetition;
 use FireflyIII\Models\Rule;
 use FireflyIII\Models\RuleGroup;
 use FireflyIII\Models\TransactionJournal;
@@ -33,79 +34,40 @@ class StoredJournalEventHandler
     /**
      * This method connects a new transfer to a piggy bank.
      *
-     * @param StoredTransactionJournal $storedJournalEvent
+     * @param StoredTransactionJournal $event
      *
      * @return bool
      */
-    public function connectToPiggyBank(StoredTransactionJournal $storedJournalEvent): bool
+    public function connectToPiggyBank(StoredTransactionJournal $event): bool
     {
         /** @var TransactionJournal $journal */
-        $journal     = $storedJournalEvent->journal;
-        $piggyBankId = $storedJournalEvent->piggyBankId;
-
+        $journal     = $event->journal;
+        $piggyBankId = $event->piggyBankId;
         Log::debug(sprintf('Trying to connect journal %d to piggy bank %d.', $journal->id, $piggyBankId));
 
-        /** @var PiggyBank $piggyBank */
-        $piggyBank = $journal->user->piggyBanks()->where('piggy_banks.id', $piggyBankId)->first(['piggy_banks.*']);
-
-        if (is_null($piggyBank)) {
-            Log::error('No such piggy bank!');
-
-            return true;
-        }
-        Log::debug(sprintf('Found piggy bank #%d: "%s"', $piggyBank->id, $piggyBank->name));
-        // update piggy bank rep for date of transaction journal.
-        $repetition = $piggyBank->piggyBankRepetitions()->relevantOnDate($journal->date)->first();
-        if (is_null($repetition)) {
-            Log::error(sprintf('No piggy bank repetition on %s!', $journal->date->format('Y-m-d')));
+        /*
+         * Verify existence of piggy bank:
+         */
+        if (!$this->verifyExistence($event)) {
+            Log::error(sprintf('No such piggy bank or no repetition on %s', $journal->date->format('Y-m-d')));
 
             return true;
         }
 
-        $amount = TransactionJournal::amountPositive($journal);
-        Log::debug(sprintf('Will add/remove %f to piggy bank #%d ("%s")', $amount, $piggyBank->id, $piggyBank->name));
-        // if piggy account matches source account, the amount is positive
-        $sources = TransactionJournal::sourceAccountList($journal)->pluck('id')->toArray();
-        if (in_array($piggyBank->account_id, $sources)) {
-            $amount = bcmul($amount, '-1');
-            Log::debug(sprintf('Account #%d is the source, so will remove amount from piggy bank.', $piggyBank->account_id));
-        }
-
-        // if the amount is positive:
-        // make sure it fits in piggy bank:
-        if (bccomp($amount, '0') === 1) {
-            // amount is positive
-            $room = bcsub(strval($piggyBank->targetamount), strval($repetition->currentamount));
-            Log::debug(sprintf('Room in piggy bank for extra money is %f', $room));
-            if (bccomp($room, $amount) === -1) {
-                // $room is smaller than $amount
-                Log::debug(sprintf('There is NO room to add %f to piggy bank #%d ("%s")', $amount, $piggyBank->id, $piggyBank->name));
-                Log::debug(sprintf('New amount is %f', $room));
-                $amount = $room;
-            }
-        }
-
-        if (bccomp($amount, '0') === -1) {
-            // amount is negative
-            Log::debug(sprintf('Max amount to remove is %f', $repetition->currentamount));
-            $compare = bcmul($repetition->currentamount, '-1');
-            if (bccomp($compare, $amount) === 1) {
-                // $currentamount is smaller than $amount
-                Log::debug(sprintf('Cannot remove %f from piggy bank #%d ("%s")', $amount, $piggyBank->id, $piggyBank->name));
-                Log::debug(sprintf('New amount is %f', $compare));
-                $amount = $compare;
-            }
-        }
-
-
+        /*
+         * Get relevant data:
+         */
+        $piggyBank                 = $journal->user->piggyBanks()->where('piggy_banks.id', $piggyBankId)->first(['piggy_banks.*']);
+        $repetition                = $piggyBank->piggyBankRepetitions()->relevantOnDate($journal->date)->first();
+        $amount                    = $this->getExactAmount($journal, $piggyBank, $repetition);
         $repetition->currentamount = bcadd($repetition->currentamount, $amount);
         $repetition->save();
 
-        /** @var PiggyBankEvent $storedJournalEvent */
-        $storedJournalEvent = PiggyBankEvent::create(
+        /** @var PiggyBankEvent $event */
+        $event = PiggyBankEvent::create(
             ['piggy_bank_id' => $piggyBank->id, 'transaction_journal_id' => $journal->id, 'date' => $journal->date, 'amount' => $amount]
         );
-        Log::debug(sprintf('Created piggy bank event #%d', $storedJournalEvent->id));
+        Log::debug(sprintf('Created piggy bank event #%d', $event->id));
 
         return true;
     }
@@ -161,4 +123,81 @@ class StoredJournalEventHandler
 
         return true;
     }
+
+    /**
+     * @SuppressWarnings(PHPMD.CyclomaticComplexity) // it's 6 but I can live with it.
+     * @param TransactionJournal  $journal
+     * @param PiggyBank           $piggyBank
+     * @param PiggyBankRepetition $repetition
+     *
+     * @return string
+     */
+    private function getExactAmount(TransactionJournal $journal, PiggyBank $piggyBank, PiggyBankRepetition $repetition): string
+    {
+        $amount  = TransactionJournal::amountPositive($journal);
+        $sources = TransactionJournal::sourceAccountList($journal)->pluck('id')->toArray();
+        $room    = bcsub(strval($piggyBank->targetamount), strval($repetition->currentamount));
+        $compare = bcmul($repetition->currentamount, '-1');
+
+        Log::debug(sprintf('Will add/remove %f to piggy bank #%d ("%s")', $amount, $piggyBank->id, $piggyBank->name));
+
+        // if piggy account matches source account, the amount is positive
+        if (in_array($piggyBank->account_id, $sources)) {
+            $amount = bcmul($amount, '-1');
+            Log::debug(sprintf('Account #%d is the source, so will remove amount from piggy bank.', $piggyBank->account_id));
+        }
+
+
+        // if the amount is positive, make sure it fits in piggy bank:
+        if (bccomp($amount, '0') === 1 && bccomp($room, $amount) === -1) {
+            // amount is positive and $room is smaller than $amount
+            Log::debug(sprintf('Room in piggy bank for extra money is %f', $room));
+            Log::debug(sprintf('There is NO room to add %f to piggy bank #%d ("%s")', $amount, $piggyBank->id, $piggyBank->name));
+            Log::debug(sprintf('New amount is %f', $room));
+
+            return $room;
+        }
+
+        // amount is negative and $currentamount is smaller than $amount
+        if (bccomp($amount, '0') === -1 && bccomp($compare, $amount) === 1) {
+            Log::debug(sprintf('Max amount to remove is %f', $repetition->currentamount));
+            Log::debug(sprintf('Cannot remove %f from piggy bank #%d ("%s")', $amount, $piggyBank->id, $piggyBank->name));
+            Log::debug(sprintf('New amount is %f', $compare));
+
+            return $compare;
+        }
+
+        return $amount;
+    }
+
+    /**
+     * @param StoredTransactionJournal $event
+     *
+     * @return bool
+     */
+    private function verifyExistence(StoredTransactionJournal $event): bool
+    {
+        /** @var TransactionJournal $journal */
+        $journal     = $event->journal;
+        $piggyBankId = $event->piggyBankId;
+
+        /** @var PiggyBank $piggyBank */
+        $piggyBank = $journal->user->piggyBanks()->where('piggy_banks.id', $piggyBankId)->first(['piggy_banks.*']);
+
+        if (is_null($piggyBank)) {
+            Log::error('No such piggy bank!');
+
+            return false;
+        }
+        Log::debug(sprintf('Found piggy bank #%d: "%s"', $piggyBank->id, $piggyBank->name));
+        // update piggy bank rep for date of transaction journal.
+        $repetition = $piggyBank->piggyBankRepetitions()->relevantOnDate($journal->date)->first();
+        if (is_null($repetition)) {
+            Log::error(sprintf('No piggy bank repetition on %s!', $journal->date->format('Y-m-d')));
+
+            return false;
+        }
+
+        return true;
+    }
 }

app/Handlers/Events/UserEventHandler.php

@@ -13,25 +13,12 @@ declare(strict_types = 1);
 
 namespace FireflyIII\Handlers\Events;
 
-use Exception;
-use FireflyConfig;
-use FireflyIII\Events\BlockedBadLogin;
-use FireflyIII\Events\BlockedUseOfDomain;
-use FireflyIII\Events\BlockedUseOfEmail;
-use FireflyIII\Events\BlockedUserLogin;
-use FireflyIII\Events\ConfirmedUser;
-use FireflyIII\Events\DeletedUser;
-use FireflyIII\Events\LockedOutUser;
 use FireflyIII\Events\RegisteredUser;
 use FireflyIII\Events\RequestedNewPassword;
-use FireflyIII\Events\ResentConfirmation;
-use FireflyIII\Models\Configuration;
 use FireflyIII\Repositories\User\UserRepositoryInterface;
-use FireflyIII\User;
 use Illuminate\Mail\Message;
 use Log;
 use Mail;
-use Preferences;
 use Session;
 use Swift_TransportException;
 
@@ -42,7 +29,6 @@ use Swift_TransportException;
  *
  * The method name reflects what is being done. This is in the present tense.
  *
- *
  * @package FireflyIII\Handlers\Events
  */
 class UserEventHandler
@@ -82,237 +68,6 @@ class UserEventHandler
         return true;
     }
 
-    /**
-     * @param BlockedBadLogin $event
-     *
-     * @return bool
-     */
-    public function reportBadLogin(BlockedBadLogin $event)
-    {
-        $email     = $event->email;
-        $owner     = env('SITE_OWNER');
-        $ipAddress = $event->ipAddress;
-        /** @var Configuration $sendmail */
-        $sendmail = FireflyConfig::get('mail_for_bad_login', config('firefly.configuration.mail_for_bad_login'));
-        Log::debug(sprintf('Now in reportBadLogin for email address %s', $email));
-        Log::error(sprintf('User %s tried to login with bad credentials.', $email));
-        if (is_null($sendmail) || (!is_null($sendmail) && $sendmail->data === false)) {
-
-            return true;
-        }
-
-        try {
-            Mail::send(
-                ['emails.blocked-bad-creds-html', 'emails.blocked-bad-creds-text'], ['email' => $email, 'ip' => $ipAddress],
-                function (Message $message) use ($owner) {
-                    $message->to($owner, $owner)->subject('Blocked login attempt with bad credentials');
-                }
-            );
-        } catch (Swift_TransportException $e) {
-            Log::error($e->getMessage());
-        }
-
-        return true;
-    }
-
-    /**
-     * @param BlockedUserLogin $event
-     *
-     * @return bool
-     */
-    public function reportBlockedUser(BlockedUserLogin $event): bool
-    {
-        $user      = $event->user;
-        $owner     = env('SITE_OWNER');
-        $email     = $user->email;
-        $ipAddress = $event->ipAddress;
-        /** @var Configuration $sendmail */
-        $sendmail = FireflyConfig::get('mail_for_blocked_login', config('firefly.configuration.mail_for_blocked_login'));
-        Log::debug(sprintf('Now in reportBlockedUser for email address %s', $email));
-        Log::error(sprintf('User #%d (%s) has their accout blocked (blocked_code is "%s") but tried to login.', $user->id, $email, $user->blocked_code));
-        if (is_null($sendmail) || (!is_null($sendmail) && $sendmail->data === false)) {
-            return true;
-        }
-
-        // send email message:
-        try {
-            Mail::send(
-                ['emails.blocked-login-html', 'emails.blocked-login-text'],
-                [
-                    'user_id'      => $user->id,
-                    'user_address' => $email,
-                    'ip'           => $ipAddress,
-                    'code'         => $user->blocked_code,
-                ], function (Message $message) use ($owner, $user) {
-                $message->to($owner, $owner)->subject('Blocked login attempt of blocked user');
-            }
-            );
-        } catch (Swift_TransportException $e) {
-            Log::error($e->getMessage());
-        }
-
-        return true;
-    }
-
-    /**
-     * @param LockedOutUser $event
-     *
-     * @return bool
-     */
-    public function reportLockout(LockedOutUser $event): bool
-    {
-        $email     = $event->email;
-        $owner     = env('SITE_OWNER');
-        $ipAddress = $event->ipAddress;
-        /** @var Configuration $sendmail */
-        $sendmail = FireflyConfig::get('mail_for_lockout', config('firefly.configuration.mail_for_lockout'));
-        Log::debug(sprintf('Now in respondToLockout for email address %s', $email));
-        Log::error(sprintf('User %s was locked out after too many invalid login attempts.', $email));
-        if (is_null($sendmail) || (!is_null($sendmail) && $sendmail->data === false)) {
-            return true;
-        }
-
-        // send email message:
-        try {
-            Mail::send(
-                ['emails.locked-out-html', 'emails.locked-out-text'], ['email' => $email, 'ip' => $ipAddress], function (Message $message) use ($owner) {
-                $message->to($owner, $owner)->subject('User was locked out');
-            }
-            );
-        } catch (Swift_TransportException $e) {
-            Log::error($e->getMessage());
-        }
-
-        return true;
-    }
-
-    /**
-     * @param BlockedUseOfDomain $event
-     *
-     * @return bool
-     */
-    public function reportUseBlockedDomain(BlockedUseOfDomain $event): bool
-    {
-        $email     = $event->email;
-        $owner     = env('SITE_OWNER');
-        $ipAddress = $event->ipAddress;
-        $parts     = explode('@', $email);
-        /** @var Configuration $sendmail */
-        $sendmail = FireflyConfig::get('mail_for_blocked_domain', config('firefly.configuration.mail_for_blocked_domain'));
-        Log::debug(sprintf('Now in reportUseBlockedDomain for email address %s', $email));
-        Log::error(sprintf('Somebody tried to register using an email address (%s) connected to a banned domain (%s).', $email, $parts[1]));
-        if (is_null($sendmail) || (!is_null($sendmail) && $sendmail->data === false)) {
-            return true;
-        }
-
-        // send email message:
-        try {
-            Mail::send(
-                ['emails.blocked-registration-html', 'emails.blocked-registration-text'],
-                [
-                    'email_address'  => $email,
-                    'blocked_domain' => $parts[1],
-                    'ip'             => $ipAddress,
-                ], function (Message $message) use ($owner) {
-                $message->to($owner, $owner)->subject('Blocked registration attempt with blocked domain');
-            }
-            );
-        } catch (Swift_TransportException $e) {
-            Log::error($e->getMessage());
-        }
-
-        return true;
-    }
-
-    /**
-     * @param BlockedUseOfEmail $event
-     *
-     * @return bool
-     */
-    public function reportUseOfBlockedEmail(BlockedUseOfEmail $event): bool
-    {
-        $email     = $event->email;
-        $owner     = env('SITE_OWNER');
-        $ipAddress = $event->ipAddress;
-        /** @var Configuration $sendmail */
-        $sendmail = FireflyConfig::get('mail_for_blocked_email', config('firefly.configuration.mail_for_blocked_email'));
-        Log::debug(sprintf('Now in reportUseOfBlockedEmail for email address %s', $email));
-        Log::error(sprintf('Somebody tried to register using email address %s which is blocked (SHA2 hash).', $email));
-        if (is_null($sendmail) || (!is_null($sendmail) && $sendmail->data === false)) {
-            return true;
-        }
-
-        // send email message:
-        try {
-            Mail::send(
-                ['emails.blocked-email-html', 'emails.blocked-email-text'],
-                [
-                    'user_address' => $email,
-                    'ip'           => $ipAddress,
-                ], function (Message $message) use ($owner) {
-                $message->to($owner, $owner)->subject('Blocked registration attempt with blocked email address');
-            }
-            );
-        } catch (Swift_TransportException $e) {
-            Log::error($e->getMessage());
-        }
-
-        return true;
-    }
-
-    /**
-     * @param DeletedUser $event
-     *
-     * @return bool
-     */
-    public function saveEmailAddress(DeletedUser $event): bool
-    {
-        Preferences::mark();
-        $email = hash('sha256', $event->email);
-        Log::debug(sprintf('Hash of email is %s', $email));
-        /** @var Configuration $configuration */
-        $configuration = FireflyConfig::get('deleted_users', []);
-        $content       = $configuration->data;
-        if (!is_array($content)) {
-            $content = [];
-        }
-        $content[]           = $email;
-        $configuration->data = $content;
-        Log::debug('New content of deleted_users is ', $content);
-        FireflyConfig::set('deleted_users', $content);
-
-        Preferences::mark();
-
-        return true;
-    }
-
-    /**
-     * This method will send a newly registered user a confirmation message, urging him or her to activate their account.
-     *
-     * @param RegisteredUser $event
-     *
-     * @return bool
-     */
-    public function sendConfirmationMessage(RegisteredUser $event): bool
-    {
-        return $this->sendConfirmation($event->user, $event->ipAddress);
-    }
-
-    /**
-     * If the user has somehow lost his or her confirmation message, this event will send it to the user again.
-     *
-     * At the moment, this method is exactly the same as the ::sendConfirmationMessage method, but that will change.
-     *
-     * @param ResentConfirmation $event
-     *
-     * @return bool
-     */
-    function sendConfirmationMessageAgain(ResentConfirmation $event): bool
-    {
-        return $this->sendConfirmation($event->user, $event->ipAddress);
-
-    }
-
     /**
      * @param RequestedNewPassword $event
      *
@@ -372,75 +127,4 @@ class UserEventHandler
 
         return true;
     }
-
-    /**
-     * When the user is confirmed, this method stores the IP address of the user
-     * as a preference. Since this preference cannot be edited, it is effectively hidden
-     * from the user yet stored conveniently.
-     *
-     * @param ConfirmedUser $event
-     *
-     * @return bool
-     */
-    public function storeConfirmationIpAddress(ConfirmedUser $event): bool
-    {
-        Preferences::setForUser($event->user, 'confirmation_ip_address', $event->ipAddress);
-
-        return true;
-    }
-
-    /**
-     * This message stores the users IP address on registration, in much the same
-     * fashion as the previous method.
-     *
-     * @param RegisteredUser $event
-     *
-     * @return bool
-     */
-    public function storeRegistrationIpAddress(RegisteredUser $event): bool
-    {
-        Preferences::setForUser($event->user, 'registration_ip_address', $event->ipAddress);
-
-        return true;
-
-    }
-
-    /**
-     * @param User   $user
-     * @param string $ipAddress
-     *
-     * @return bool
-     */
-    private function sendConfirmation(User $user, string $ipAddress): bool
-    {
-        $mustConfirmAccount = FireflyConfig::get('must_confirm_account', config('firefly.configuration.must_confirm_account'))->data;
-        if ($mustConfirmAccount === false) {
-            Preferences::setForUser($user, 'user_confirmed', true);
-            Preferences::setForUser($user, 'user_confirmed_last_mail', 0);
-            Preferences::mark();
-
-            return true;
-        }
-        $email = $user->email;
-        $code  = str_random(16);
-        $route = route('do_confirm_account', [$code]);
-        Preferences::setForUser($user, 'user_confirmed', false);
-        Preferences::setForUser($user, 'user_confirmed_last_mail', time());
-        Preferences::setForUser($user, 'user_confirmed_code', $code);
-        try {
-            Mail::send(
-                ['emails.confirm-account-html', 'emails.confirm-account-text'], ['route' => $route, 'ip' => $ipAddress],
-                function (Message $message) use ($email) {
-                    $message->to($email, $email)->subject('Please confirm your Firefly III account');
-                }
-            );
-        } catch (Swift_TransportException $e) {
-            Log::error($e->getMessage());
-        } catch (Exception $e) {
-            Log::error($e->getMessage());
-        }
-
-        return true;
-    }
-
 }

app/Helpers/Attachments/AttachmentHelper.php

@@ -16,7 +16,6 @@ use Crypt;
 use FireflyIII\Models\Attachment;
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Support\MessageBag;
-use Input;
 use Storage;
 use Symfony\Component\HttpFoundation\File\UploadedFile;
 
@@ -81,20 +80,19 @@ class AttachmentHelper implements AttachmentHelperInterface
     }
 
     /**
-     * @param Model $model
+     * @param Model      $model
+     * @param array|null $files
      *
      * @return bool
      */
-    public function saveAttachmentsForModel(Model $model): bool
+    public function saveAttachmentsForModel(Model $model, array $files = null): bool
     {
-        $files = $this->getFiles();
-
-        if (!is_null($files) && !is_array($files)) {
-            $this->processFile($files, $model);
-        }
-
         if (is_array($files)) {
-            $this->processFiles($files, $model);
+            foreach ($files as $entry) {
+                if (!is_null($entry)) {
+                    $this->processFile($entry, $model);
+                }
+            }
         }
 
         return true;
@@ -227,37 +225,4 @@ class AttachmentHelper implements AttachmentHelperInterface
 
         return true;
     }
-
-    /**
-     * @return array|null|UploadedFile
-     */
-    private function getFiles()
-    {
-        $files = null;
-        if (Input::hasFile('attachments')) {
-            $files = Input::file('attachments');
-        }
-
-        return $files;
-    }
-
-    /**
-     * @param array $files
-     *
-     * @param Model $model
-     *
-     * @return bool
-     */
-    private function processFiles(array $files, Model $model): bool
-    {
-        foreach ($files as $entry) {
-            if (!is_null($entry)) {
-                $this->processFile($entry, $model);
-            }
-        }
-
-        return true;
-    }
-
-
 }

app/Helpers/Attachments/AttachmentHelperInterface.php

@@ -46,6 +46,6 @@ interface AttachmentHelperInterface
      *
      * @return bool
      */
-    public function saveAttachmentsForModel(Model $model): bool;
+    public function saveAttachmentsForModel(Model $model, array $files = null): bool;
 
 }

app/Helpers/Chart/MetaPieChart.php

@@ -0,0 +1,278 @@
+<?php
+/**
+ * MetaPieChart.php
+ * Copyright (c) 2017 thegrumpydictator@gmail.com
+ * This software may be modified and distributed under the terms of the Creative Commons Attribution-ShareAlike 4.0 International License.
+ *
+ * See the LICENSE file for details.
+ */
+
+declare(strict_types = 1);
+
+namespace FireflyIII\Helpers\Chart;
+
+use Carbon\Carbon;
+use FireflyIII\Generator\Report\Support;
+use FireflyIII\Helpers\Collector\JournalCollectorInterface;
+use FireflyIII\Models\Transaction;
+use FireflyIII\Models\TransactionType;
+use FireflyIII\Repositories\Account\AccountRepositoryInterface;
+use FireflyIII\Repositories\Budget\BudgetRepositoryInterface;
+use FireflyIII\Repositories\Category\CategoryRepositoryInterface;
+use FireflyIII\User;
+use Illuminate\Support\Collection;
+
+/**
+ * Class MetaPieChart
+ *
+ * @package FireflyIII\Helpers\Chart
+ */
+class MetaPieChart implements MetaPieChartInterface
+{
+    /** @var  Collection */
+    protected $accounts;
+    /** @var  Collection */
+    protected $budgets;
+    /** @var  Collection */
+    protected $categories;
+    /** @var bool */
+    protected $collectOtherObjects = false;
+    /** @var  Carbon */
+    protected $end;
+    /** @var array */
+    protected $grouping
+        = [
+            'account'  => ['opposing_account_id'],
+            'budget'   => ['transaction_journal_budget_id', 'transaction_budget_id'],
+            'category' => ['transaction_journal_category_id', 'transaction_category_id'],
+        ];
+
+    /** @var array */
+    protected $repositories
+        = [
+            'account'  => AccountRepositoryInterface::class,
+            'budget'   => BudgetRepositoryInterface::class,
+            'category' => CategoryRepositoryInterface::class,
+        ];
+
+
+    /** @var  Carbon */
+    protected $start;
+    /** @var  string */
+    protected $total = '0';
+    /** @var  User */
+    protected $user;
+
+    public function __construct()
+    {
+        $this->accounts   = new Collection;
+        $this->budgets    = new Collection;
+        $this->categories = new Collection;
+    }
+
+    /**
+     * @param string $direction
+     * @param string $group
+     *
+     * @return array
+     */
+    public function generate(string $direction, string $group): array
+    {
+        $transactions = $this->getTransactions($direction);
+        $grouped      = $this->groupByFields($transactions, $this->grouping[$group]);
+        $chartData    = $this->organizeByType($group, $grouped);
+
+        // also collect all other transactions
+        if ($this->collectOtherObjects && $direction === 'expense') {
+            /** @var JournalCollectorInterface $collector */
+            $collector = app(JournalCollectorInterface::class, [$this->user]);
+            $collector->setAccounts($this->accounts)->setRange($this->start, $this->end)->setTypes([TransactionType::WITHDRAWAL]);
+            $journals                                            = $collector->getJournals();
+            $sum                                                 = strval($journals->sum('transaction_amount'));
+            $sum                                                 = bcmul($sum, '-1');
+            $sum                                                 = bcsub($sum, $this->total);
+            $chartData[strval(trans('firefly.everything_else'))] = $sum;
+        }
+
+        if ($this->collectOtherObjects && $direction === 'income') {
+            /** @var JournalCollectorInterface $collector */
+            $collector = app(JournalCollectorInterface::class, [auth()->user()]);
+            $collector->setAccounts($this->accounts)->setRange($this->start, $this->end)->setTypes([TransactionType::DEPOSIT]);
+            $journals                                            = $collector->getJournals();
+            $sum                                                 = strval($journals->sum('transaction_amount'));
+            $sum                                                 = bcsub($sum, $this->total);
+            $chartData[strval(trans('firefly.everything_else'))] = $sum;
+        }
+
+        return $chartData;
+
+    }
+
+    /**
+     * @param Collection $accounts
+     *
+     * @return MetaPieChartInterface
+     */
+    public function setAccounts(Collection $accounts): MetaPieChartInterface
+    {
+        $this->accounts = $accounts;
+
+        return $this;
+    }
+
+    /**
+     * @param Collection $budgets
+     *
+     * @return MetaPieChartInterface
+     */
+    public function setBudgets(Collection $budgets): MetaPieChartInterface
+    {
+        $this->budgets = $budgets;
+
+        return $this;
+    }
+
+    /**
+     * @param Collection $categories
+     *
+     * @return MetaPieChartInterface
+     */
+    public function setCategories(Collection $categories): MetaPieChartInterface
+    {
+        $this->categories = $categories;
+
+        return $this;
+    }
+
+    /**
+     * @param bool $collectOtherObjects
+     *
+     * @return MetaPieChartInterface
+     */
+    public function setCollectOtherObjects(bool $collectOtherObjects): MetaPieChartInterface
+    {
+        $this->collectOtherObjects = $collectOtherObjects;
+
+        return $this;
+    }
+
+    /**
+     * @param Carbon $end
+     *
+     * @return MetaPieChartInterface
+     */
+    public function setEnd(Carbon $end): MetaPieChartInterface
+    {
+        $this->end = $end;
+
+        return $this;
+    }
+
+    /**
+     * @param Carbon $start
+     *
+     * @return MetaPieChartInterface
+     */
+    public function setStart(Carbon $start): MetaPieChartInterface
+    {
+        $this->start = $start;
+
+        return $this;
+    }
+
+    /**
+     * @param User $user
+     *
+     * @return MetaPieChartInterface
+     */
+    public function setUser(User $user): MetaPieChartInterface
+    {
+        $this->user = $user;
+
+        return $this;
+    }
+
+    protected function getTransactions(string $direction)
+    {
+        $types    = [TransactionType::DEPOSIT, TransactionType::TRANSFER];
+        $modifier = -1;
+        if ($direction === 'expense') {
+            $types    = [TransactionType::WITHDRAWAL, TransactionType::TRANSFER];
+            $modifier = 1;
+        }
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
+        $collector->setAccounts($this->accounts);
+        $collector->setRange($this->start, $this->end);
+        $collector->setTypes($types);
+        $collector->withOpposingAccount();
+
+        if ($direction === 'income') {
+            $collector->disableFilter();
+        }
+
+        if ($this->budgets->count() > 0) {
+            $collector->setBudgets($this->budgets);
+        }
+        if ($this->categories->count() > 0) {
+            $collector->setCategories($this->categories);
+        }
+
+        $accountIds   = $this->accounts->pluck('id')->toArray();
+        $transactions = $collector->getJournals();
+        $set          = Support::filterTransactions($transactions, $accountIds, $modifier);
+
+        return $set;
+    }
+
+    /**
+     * @param Collection $set
+     * @param array      $fields
+     *
+     * @return array
+     */
+    protected function groupByFields(Collection $set, array $fields)
+    {
+        $grouped = [];
+        /** @var Transaction $transaction */
+        foreach ($set as $transaction) {
+            $values = [];
+            foreach ($fields as $field) {
+                $values[] = intval($transaction->$field);
+            }
+            $value           = max($values);
+            $grouped[$value] = $grouped[$value] ?? '0';
+            $grouped[$value] = bcadd($transaction->transaction_amount, $grouped[$value]);
+        }
+
+        return $grouped;
+    }
+
+    /**
+     * @param string $type
+     * @param array  $array
+     *
+     * @return array
+     */
+    protected function organizeByType(string $type, array $array): array
+    {
+        $chartData  = [];
+        $names      = [];
+        $repository = app($this->repositories[$type], [$this->user]);
+        foreach ($array as $objectId => $amount) {
+            if (!isset($names[$objectId])) {
+                $object           = $repository->find(intval($objectId));
+                $names[$objectId] = $object->name;
+            }
+            if (bccomp($amount, '0') === -1) {
+                $amount = bcmul($amount, '-1');
+            }
+
+            $this->total                  = bcadd($this->total, $amount);
+            $chartData[$names[$objectId]] = $amount;
+        }
+
+        return $chartData;
+
+    }
+}

app/Helpers/Chart/MetaPieChartInterface.php

@@ -0,0 +1,82 @@
+<?php
+/**
+ * MetaPieChartInterface.php
+ * Copyright (c) 2017 thegrumpydictator@gmail.com
+ * This software may be modified and distributed under the terms of the Creative Commons Attribution-ShareAlike 4.0 International License.
+ *
+ * See the LICENSE file for details.
+ */
+
+declare(strict_types = 1);
+
+namespace FireflyIII\Helpers\Chart;
+
+use Carbon\Carbon;
+use FireflyIII\User;
+use Illuminate\Support\Collection;
+
+/**
+ * Interface MetaPieChartInterface
+ *
+ * @package FireflyIII\Helpers\Chart
+ */
+interface MetaPieChartInterface
+{
+    /**
+     * @param string $direction
+     * @param string $group
+     *
+     * @return array
+     */
+    public function generate(string $direction, string $group): array;
+
+    /**
+     * @param Collection $accounts
+     *
+     * @return MetaPieChartInterface
+     */
+    public function setAccounts(Collection $accounts): MetaPieChartInterface;
+
+    /**
+     * @param Collection $budgets
+     *
+     * @return MetaPieChartInterface
+     */
+    public function setBudgets(Collection $budgets): MetaPieChartInterface;
+
+    /**
+     * @param Collection $categories
+     *
+     * @return MetaPieChartInterface
+     */
+    public function setCategories(Collection $categories): MetaPieChartInterface;
+
+    /**
+     * @param bool $collectOtherObjects
+     *
+     * @return MetaPieChartInterface
+     */
+    public function setCollectOtherObjects(bool $collectOtherObjects): MetaPieChartInterface;
+
+    /**
+     * @param Carbon $end
+     *
+     * @return MetaPieChartInterface
+     */
+    public function setEnd(Carbon $end): MetaPieChartInterface;
+
+    /**
+     * @param Carbon $start
+     *
+     * @return MetaPieChartInterface
+     */
+    public function setStart(Carbon $start): MetaPieChartInterface;
+
+    /**
+     * @param User $user
+     *
+     * @return MetaPieChartInterface
+     */
+    public function setUser(User $user): MetaPieChartInterface;
+
+}

app/Helpers/Collection/BalanceLine.php

@@ -14,6 +14,7 @@ namespace FireflyIII\Helpers\Collection;
 
 use Carbon\Carbon;
 use FireflyIII\Models\Budget as BudgetModel;
+use FireflyIII\Models\BudgetLimit;
 use Illuminate\Support\Collection;
 
 /**
@@ -34,12 +35,10 @@ class BalanceLine
 
     /** @var BudgetModel */
     protected $budget;
-    /** @var  Carbon */
-    protected $endDate;
+    /** @var  BudgetLimit */
+    protected $budgetLimit;
     /** @var int */
     protected $role = self::ROLE_DEFAULTROLE;
-    /** @var  Carbon */
-    protected $startDate;
 
     /**
      *
@@ -90,20 +89,28 @@ class BalanceLine
         $this->budget = $budget;
     }
 
+    /**
+     * @return BudgetLimit
+     */
+    public function getBudgetLimit(): BudgetLimit
+    {
+        return $this->budgetLimit;
+    }
+
+    /**
+     * @param BudgetLimit $budgetLimit
+     */
+    public function setBudgetLimit(BudgetLimit $budgetLimit)
+    {
+        $this->budgetLimit = $budgetLimit;
+    }
+
     /**
      * @return Carbon
      */
     public function getEndDate()
     {
-        return $this->endDate;
-    }
-
-    /**
-     * @param Carbon $endDate
-     */
-    public function setEndDate($endDate)
-    {
-        $this->endDate = $endDate;
+        return $this->budgetLimit->end_date ?? new Carbon;
     }
 
     /**
@@ -127,18 +134,11 @@ class BalanceLine
      */
     public function getStartDate()
     {
-        return $this->startDate;
-    }
-
-    /**
-     * @param Carbon $startDate
-     */
-    public function setStartDate($startDate)
-    {
-        $this->startDate = $startDate;
+        return $this->budgetLimit->start_date ?? new Carbon;
     }
 
     /**
+     * @SuppressWarnings(PHPMD.CyclomaticComplexity)
      * @return string
      */
     public function getTitle(): string
@@ -147,13 +147,13 @@ class BalanceLine
             return $this->getBudget()->name;
         }
         if ($this->getRole() == self::ROLE_DEFAULTROLE) {
-            return trans('firefly.no_budget');
+            return strval(trans('firefly.no_budget'));
         }
         if ($this->getRole() == self::ROLE_TAGROLE) {
-            return trans('firefly.coveredWithTags');
+            return strval(trans('firefly.coveredWithTags'));
         }
         if ($this->getRole() == self::ROLE_DIFFROLE) {
-            return trans('firefly.leftUnbalanced');
+            return strval(trans('firefly.leftUnbalanced'));
         }
 
         return '';
@@ -169,7 +169,7 @@ class BalanceLine
      */
     public function leftOfRepetition(): string
     {
-        $start = $this->budget->amount ?? '0';
+        $start = $this->budgetLimit->amount ?? '0';
         /** @var BalanceEntry $balanceEntry */
         foreach ($this->getBalanceEntries() as $balanceEntry) {
             $start = bcadd($balanceEntry->getSpent(), $start);

app/Helpers/Collection/Budget.php

@@ -60,7 +60,6 @@ class Budget
      */
     public function addBudgeted(string $add): Budget
     {
-        $add            = strval(round($add, 2));
         $this->budgeted = bcadd($this->budgeted, $add);
 
         return $this;
@@ -73,7 +72,6 @@ class Budget
      */
     public function addLeft(string $add): Budget
     {
-        $add        = strval(round($add, 2));
         $this->left = bcadd($this->left, $add);
 
         return $this;
@@ -86,7 +84,6 @@ class Budget
      */
     public function addOverspent(string $add): Budget
     {
-        $add             = strval(round($add, 2));
         $this->overspent = bcadd($this->overspent, $add);
 
         return $this;
@@ -99,7 +96,6 @@ class Budget
      */
     public function addSpent(string $add): Budget
     {
-        $add         = strval(round($add, 2));
         $this->spent = bcadd($this->spent, $add);
 
         return $this;
@@ -168,7 +164,7 @@ class Budget
      */
     public function setOverspent(string $overspent): Budget
     {
-        $this->overspent = strval(round($overspent, 2));
+        $this->overspent = $overspent;
 
         return $this;
     }
@@ -188,7 +184,7 @@ class Budget
      */
     public function setSpent(string $spent): Budget
     {
-        $this->spent = strval(round($spent, 2));
+        $this->spent = $spent;
 
         return $this;
     }

app/Helpers/Collection/BudgetLine.php

@@ -13,7 +13,7 @@ declare(strict_types = 1);
 namespace FireflyIII\Helpers\Collection;
 
 use FireflyIII\Models\Budget as BudgetModel;
-use FireflyIII\Models\LimitRepetition;
+use FireflyIII\Models\BudgetLimit;
 
 /**
  *
@@ -26,14 +26,14 @@ class BudgetLine
 
     /** @var  BudgetModel */
     protected $budget;
+    /** @var  BudgetLimit */
+    protected $budgetLimit;
     /** @var string */
     protected $budgeted = '0';
     /** @var string */
     protected $left = '0';
     /** @var string */
     protected $overspent = '0';
-    /** @var  LimitRepetition */
-    protected $repetition;
     /** @var string */
     protected $spent = '0';
 
@@ -57,6 +57,26 @@ class BudgetLine
         return $this;
     }
 
+    /**
+     * @return BudgetLimit
+     */
+    public function getBudgetLimit(): BudgetLimit
+    {
+        return $this->budgetLimit ?? new BudgetLimit;
+    }
+
+    /**
+     * @param BudgetLimit $budgetLimit
+     *
+     * @return BudgetLimit
+     */
+    public function setBudgetLimit(BudgetLimit $budgetLimit): BudgetLine
+    {
+        $this->budgetLimit = $budgetLimit;
+
+        return $this;
+    }
+
     /**
      * @return string
      */
@@ -117,26 +137,6 @@ class BudgetLine
         return $this;
     }
 
-    /**
-     * @return LimitRepetition
-     */
-    public function getRepetition(): LimitRepetition
-    {
-        return $this->repetition ?? new LimitRepetition;
-    }
-
-    /**
-     * @param LimitRepetition $repetition
-     *
-     * @return BudgetLine
-     */
-    public function setRepetition(LimitRepetition $repetition): BudgetLine
-    {
-        $this->repetition = $repetition;
-
-        return $this;
-    }
-
     /**
      * @return string
      */

app/Helpers/Collection/Category.php

@@ -55,7 +55,6 @@ class Category
      */
     public function addTotal(string $add)
     {
-        $add         = strval(round($add, 2));
         $this->total = bcadd($this->total, $add);
     }
 
@@ -79,7 +78,7 @@ class Category
      */
     public function getTotal(): string
     {
-        return strval(round($this->total, 2));
+        return $this->total;
     }
 
 

app/Helpers/Collector/JournalCollector.php

@@ -60,6 +60,7 @@ class JournalCollector implements JournalCollectorInterface
             'transaction_types.type as transaction_type_type',
             'transaction_journals.bill_id',
             'bills.name as bill_name',
+            'bills.name_encrypted as bill_name_encrypted',
             'transactions.id as id',
             'transactions.amount as transaction_amount',
             'transactions.description as transaction_description',
@@ -180,10 +181,12 @@ class JournalCollector implements JournalCollectorInterface
         $set->each(
             function (Transaction $transaction) {
                 $transaction->date        = new Carbon($transaction->date);
-                $transaction->description = intval($transaction->encrypted) === 1 ? Crypt::decrypt($transaction->description) : $transaction->description;
-                $transaction->bill_name   = !is_null($transaction->bill_name) ? Crypt::decrypt($transaction->bill_name) : '';
+                $transaction->description = $transaction->encrypted ? Crypt::decrypt($transaction->description) : $transaction->description;
+
+                if (!is_null($transaction->bill_name)) {
+                    $transaction->bill_name = $transaction->bill_name_encrypted ? Crypt::decrypt($transaction->bill_name) : $transaction->bill_name;
+                }
 
-                // optionally decrypted:
                 try {
                     $transaction->opposing_account_name = Crypt::decrypt($transaction->opposing_account_name);
                 } catch (DecryptException $e) {
@@ -560,7 +563,7 @@ class JournalCollector implements JournalCollectorInterface
             return $set;
         }
         if ($this->joinedOpposing === false) {
-            Log::error('Cannot filter internal transfers because no opposing information is present.');
+            Log::info('Cannot filter internal transfers because no opposing information is present.');
 
             return $set;
         }
@@ -646,9 +649,17 @@ class JournalCollector implements JournalCollectorInterface
             // join some extra tables:
             $this->joinedBudget = true;
             $this->query->leftJoin('budget_transaction_journal', 'budget_transaction_journal.transaction_journal_id', '=', 'transaction_journals.id');
+            $this->query->leftJoin('budgets as transaction_journal_budgets', 'transaction_journal_budgets.id', '=', 'budget_transaction_journal.budget_id');
             $this->query->leftJoin('budget_transaction', 'budget_transaction.transaction_id', '=', 'transactions.id');
+            $this->query->leftJoin('budgets as transaction_budgets', 'transaction_budgets.id', '=', 'budget_transaction.budget_id');
+
             $this->fields[] = 'budget_transaction_journal.budget_id as transaction_journal_budget_id';
+            $this->fields[] = 'transaction_journal_budgets.encrypted as transaction_journal_budget_encrypted';
+            $this->fields[] = 'transaction_journal_budgets.name as transaction_journal_budget_name';
+
             $this->fields[] = 'budget_transaction.budget_id as transaction_budget_id';
+            $this->fields[] = 'transaction_budgets.encrypted as transaction_budget_encrypted';
+            $this->fields[] = 'transaction_budgets.name as transaction_budget_name';
         }
     }
 
@@ -661,12 +672,26 @@ class JournalCollector implements JournalCollectorInterface
             // join some extra tables:
             $this->joinedCategory = true;
             $this->query->leftJoin('category_transaction_journal', 'category_transaction_journal.transaction_journal_id', '=', 'transaction_journals.id');
+            $this->query->leftJoin(
+                'categories as transaction_journal_categories', 'transaction_journal_categories.id', '=', 'category_transaction_journal.category_id'
+            );
+
             $this->query->leftJoin('category_transaction', 'category_transaction.transaction_id', '=', 'transactions.id');
+            $this->query->leftJoin('categories as transaction_categories', 'transaction_categories.id', '=', 'category_transaction.category_id');
+
             $this->fields[] = 'category_transaction_journal.category_id as transaction_journal_category_id';
+            $this->fields[] = 'transaction_journal_categories.encrypted as transaction_journal_category_encrypted';
+            $this->fields[] = 'transaction_journal_categories.name as transaction_journal_category_name';
+
             $this->fields[] = 'category_transaction.category_id as transaction_category_id';
+            $this->fields[] = 'transaction_categories.encrypted as transaction_category_encrypted';
+            $this->fields[] = 'transaction_categories.name as transaction_category_name';
         }
     }
 
+    /**
+     *
+     */
     private function joinOpposingTables()
     {
         if (!$this->joinedOpposing) {
@@ -726,4 +751,4 @@ class JournalCollector implements JournalCollectorInterface
         return $query;
 
     }
-}
+}

app/Helpers/Collector/JournalCollectorInterface.php

@@ -173,4 +173,4 @@ interface JournalCollectorInterface
      * @return JournalCollectorInterface
      */
     public function withoutCategory(): JournalCollectorInterface;
-}
+}

app/Helpers/Help/Help.php

@@ -74,12 +74,8 @@ class Help implements HelpInterface
             $converter = new CommonMarkConverter();
             $content   = $converter->convertToHtml($content);
         }
-        if (strlen($content) === 0) {
-            Log::warning('Raw content length is zero.');
-        }
 
         return $content;
-
     }
 
     /**

app/Helpers/Report/BalanceReportHelper.php

@@ -19,17 +19,18 @@ use FireflyIII\Helpers\Collection\Balance;
 use FireflyIII\Helpers\Collection\BalanceEntry;
 use FireflyIII\Helpers\Collection\BalanceHeader;
 use FireflyIII\Helpers\Collection\BalanceLine;
-use FireflyIII\Models\Budget;
-use FireflyIII\Models\LimitRepetition;
+use FireflyIII\Models\BudgetLimit;
 use FireflyIII\Models\Tag;
 use FireflyIII\Models\TransactionType;
 use FireflyIII\Repositories\Budget\BudgetRepositoryInterface;
 use Illuminate\Database\Query\JoinClause;
 use Illuminate\Support\Collection;
+use Log;
 
 /**
  * Class BalanceReportHelper
  *
+ * @SuppressWarnings(PHPMD.CouplingBetweenObjects) // I can't really help it.
  * @package FireflyIII\Helpers\Report
  */
 class BalanceReportHelper implements BalanceReportHelperInterface
@@ -59,19 +60,21 @@ class BalanceReportHelper implements BalanceReportHelperInterface
      */
     public function getBalanceReport(Collection $accounts, Carbon $start, Carbon $end): Balance
     {
-        $balance          = new Balance;
-        $header           = new BalanceHeader;
-        $limitRepetitions = $this->budgetRepository->getAllBudgetLimitRepetitions($start, $end);
+        Log::debug('Start of balance report');
+        $balance      = new Balance;
+        $header       = new BalanceHeader;
+        $budgetLimits = $this->budgetRepository->getAllBudgetLimits($start, $end);
         foreach ($accounts as $account) {
+            Log::debug(sprintf('Add account %s to headers.', $account->name));
             $header->addAccount($account);
         }
 
-        /** @var LimitRepetition $repetition */
-        foreach ($limitRepetitions as $repetition) {
-            $budget = $this->budgetRepository->find($repetition->budget_id);
-            $line   = $this->createBalanceLine($budget, $repetition, $accounts);
+        /** @var BudgetLimit $budgetLimit */
+        foreach ($budgetLimits as $budgetLimit) {
+            $line = $this->createBalanceLine($budgetLimit, $accounts);
             $balance->addBalanceLine($line);
         }
+        Log::debug('Create rest of the things.');
         $noBudgetLine       = $this->createNoBudgetLine($accounts, $start, $end);
         $coveredByTagLine   = $this->createTagsBalanceLine($accounts, $start, $end);
         $leftUnbalancedLine = $this->createLeftUnbalancedLine($noBudgetLine, $coveredByTagLine);
@@ -81,9 +84,12 @@ class BalanceReportHelper implements BalanceReportHelperInterface
         $balance->addBalanceLine($leftUnbalancedLine);
         $balance->setBalanceHeader($header);
 
+        Log::debug('Clear unused budgets.');
         // remove budgets without expenses from balance lines:
         $balance = $this->removeUnusedBudgets($balance);
 
+        Log::debug('Return report.');
+
         return $balance;
     }
 
@@ -137,27 +143,22 @@ class BalanceReportHelper implements BalanceReportHelperInterface
 
 
     /**
-     * @param Budget          $budget
-     * @param LimitRepetition $repetition
-     * @param Collection      $accounts
+     * @param BudgetLimit $budgetLimit
+     * @param Collection  $accounts
      *
      * @return BalanceLine
      */
-    private function createBalanceLine(Budget $budget, LimitRepetition $repetition, Collection $accounts): BalanceLine
+    private function createBalanceLine(BudgetLimit $budgetLimit, Collection $accounts): BalanceLine
     {
-        $line           = new BalanceLine;
-        $budget->amount = $repetition->amount;
-        $line->setBudget($budget);
-        $line->setStartDate($repetition->startdate);
-        $line->setEndDate($repetition->enddate);
+        $line = new BalanceLine;
+        $line->setBudget($budgetLimit->budget);
+        $line->setBudgetLimit($budgetLimit);
 
         // loop accounts:
         foreach ($accounts as $account) {
             $balanceEntry = new BalanceEntry;
             $balanceEntry->setAccount($account);
-            $spent = $this->budgetRepository->spentInPeriod(
-                new Collection([$budget]), new Collection([$account]), $repetition->startdate, $repetition->enddate
-            );
+            $spent = $this->budgetRepository->spentInPeriod(new Collection([$budgetLimit->budget]), new Collection([$account]), $budgetLimit->start_date, $budgetLimit->end_date);
             $balanceEntry->setSpent($spent);
             $line->addBalanceEntry($balanceEntry);
         }
@@ -212,7 +213,7 @@ class BalanceReportHelper implements BalanceReportHelperInterface
         $empty = new BalanceLine;
 
         foreach ($accounts as $account) {
-            $spent = $this->budgetRepository->spentInPeriodWithoutBudget(new Collection([$account]), $start, $end);
+            $spent = $this->budgetRepository->spentInPeriodWoBudget(new Collection([$account]), $start, $end);
             // budget
             $budgetEntry = new BalanceEntry;
             $budgetEntry->setAccount($account);

app/Helpers/Report/BudgetReportHelper.php

@@ -18,7 +18,7 @@ use Carbon\Carbon;
 use FireflyIII\Helpers\Collection\Budget as BudgetCollection;
 use FireflyIII\Helpers\Collection\BudgetLine;
 use FireflyIII\Models\Budget;
-use FireflyIII\Models\LimitRepetition;
+use FireflyIII\Models\BudgetLimit;
 use FireflyIII\Repositories\Budget\BudgetRepositoryInterface;
 use Illuminate\Support\Collection;
 
@@ -43,6 +43,7 @@ class BudgetReportHelper implements BudgetReportHelperInterface
     }
 
     /**
+     * @SuppressWarnings(PHPMD.CyclomaticComplexity) // it's exactly 5.
      * @param Carbon     $start
      * @param Carbon     $end
      * @param Collection $accounts
@@ -56,13 +57,9 @@ class BudgetReportHelper implements BudgetReportHelperInterface
 
         /** @var Budget $budget */
         foreach ($set as $budget) {
-            $repetitions = $budget->limitrepetitions()->before($end)->after($start)->get();
-
-            // no repetition(s) for this budget:
-            if ($repetitions->count() == 0) {
-                // spent for budget in time range:
-                $spent = $this->repository->spentInPeriod(new Collection([$budget]), $accounts, $start, $end);
-
+            $budgetLimits = $this->repository->getBudgetLimits($budget, $start, $end);
+            if ($budgetLimits->count() == 0) { // no budget limit(s) for this budget
+                $spent = $this->repository->spentInPeriod(new Collection([$budget]), $accounts, $start, $end);// spent for budget in time range
                 if ($spent > 0) {
                     $budgetLine = new BudgetLine;
                     $budgetLine->setBudget($budget)->setOverspent($spent);
@@ -70,26 +67,20 @@ class BudgetReportHelper implements BudgetReportHelperInterface
                 }
                 continue;
             }
-            // one or more repetitions for budget:
-            /** @var LimitRepetition $repetition */
-            foreach ($repetitions as $repetition) {
-                $data = $this->calculateExpenses($budget, $repetition, $accounts);
-
+            /** @var BudgetLimit $budgetLimit */
+            foreach ($budgetLimits as $budgetLimit) { // one or more repetitions for budget
+                $data       = $this->calculateExpenses($budget, $budgetLimit, $accounts);
                 $budgetLine = new BudgetLine;
-                $budgetLine->setBudget($budget)->setRepetition($repetition)
+                $budgetLine->setBudget($budget)->setBudgetLimit($budgetLimit)
                            ->setLeft($data['left'])->setSpent($data['expenses'])->setOverspent($data['overspent'])
-                           ->setBudgeted(strval($repetition->amount));
+                           ->setBudgeted(strval($budgetLimit->amount));
 
-                $object->addBudgeted(strval($repetition->amount))->addSpent($data['spent'])
+                $object->addBudgeted(strval($budgetLimit->amount))->addSpent($data['spent'])
                        ->addLeft($data['left'])->addOverspent($data['overspent'])->addBudgetLine($budgetLine);
 
             }
-
         }
-
-        // stuff outside of budgets:
-
-        $noBudget   = $this->repository->spentInPeriodWithoutBudget($accounts, $start, $end);
+        $noBudget   = $this->repository->spentInPeriodWoBudget($accounts, $start, $end); // stuff outside of budgets
         $budgetLine = new BudgetLine;
         $budgetLine->setOverspent($noBudget)->setSpent($noBudget);
         $object->addOverspent($noBudget)->addBudgetLine($budgetLine);
@@ -128,19 +119,19 @@ class BudgetReportHelper implements BudgetReportHelperInterface
     }
 
     /**
-     * @param Budget          $budget
-     * @param LimitRepetition $repetition
-     * @param Collection      $accounts
+     * @param Budget      $budget
+     * @param BudgetLimit $budgetLimit
+     * @param Collection  $accounts
      *
      * @return array
      */
-    private function calculateExpenses(Budget $budget, LimitRepetition $repetition, Collection $accounts): array
+    private function calculateExpenses(Budget $budget, BudgetLimit $budgetLimit, Collection $accounts): array
     {
         $array              = [];
-        $expenses           = $this->repository->spentInPeriod(new Collection([$budget]), $accounts, $repetition->startdate, $repetition->enddate);
-        $array['left']      = bccomp(bcadd($repetition->amount, $expenses), '0') === 1 ? bcadd($repetition->amount, $expenses) : '0';
-        $array['spent']     = bccomp(bcadd($repetition->amount, $expenses), '0') === 1 ? $expenses : '0';
-        $array['overspent'] = bccomp(bcadd($repetition->amount, $expenses), '0') === 1 ? '0' : bcadd($expenses, $repetition->amount);
+        $expenses           = $this->repository->spentInPeriod(new Collection([$budget]), $accounts, $budgetLimit->start_date, $budgetLimit->end_date);
+        $array['left']      = bccomp(bcadd($budgetLimit->amount, $expenses), '0') === 1 ? bcadd($budgetLimit->amount, $expenses) : '0';
+        $array['spent']     = bccomp(bcadd($budgetLimit->amount, $expenses), '0') === 1 ? $expenses : '0';
+        $array['overspent'] = bccomp(bcadd($budgetLimit->amount, $expenses), '0') === 1 ? '0' : bcadd($expenses, $budgetLimit->amount);
         $array['expenses']  = $expenses;
 
         return $array;

app/Helpers/Report/ReportHelper.php

@@ -53,6 +53,8 @@ class ReportHelper implements ReportHelperInterface
      * This method generates a full report for the given period on all
      * the users bills and their payments.
      *
+     * @SuppressWarnings(PHPMD.CyclomaticComplexity) // it's exactly 5.
+     *
      * Excludes bills which have not had a payment on the mentioned accounts.
      *
      * @param Carbon     $start
@@ -80,8 +82,6 @@ class ReportHelper implements ReportHelperInterface
             $billLine->setMin(strval($bill->amount_min));
             $billLine->setMax(strval($bill->amount_max));
             $billLine->setHit(false);
-            // is hit in period?
-
             $entry = $journals->filter(
                 function (Transaction $transaction) use ($bill) {
                     return $transaction->bill_id === $bill->id;
@@ -94,44 +94,15 @@ class ReportHelper implements ReportHelperInterface
                 $billLine->setLastHitDate($first->date);
                 $billLine->setHit(true);
             }
-
-            // bill is active, or bill is hit:
             if ($billLine->isActive() || $billLine->isHit()) {
                 $collection->addBill($billLine);
             }
         }
-
-        // do some extra filtering.
         $collection->filterBills();
 
         return $collection;
     }
 
-    /**
-     * @param Carbon     $start
-     * @param Carbon     $end
-     * @param Collection $accounts
-     *
-     * @return CategoryCollection
-     */
-    public function getCategoryReport(Collection $accounts, Carbon $start, Carbon $end): CategoryCollection
-    {
-        $object = new CategoryCollection;
-        /** @var CategoryRepositoryInterface $repository */
-        $repository = app(CategoryRepositoryInterface::class);
-        $categories = $repository->getCategories();
-
-        /** @var Category $category */
-        foreach ($categories as $category) {
-            $spent = $repository->spentInPeriod(new Collection([$category]), $accounts, $start, $end);
-            // CategoryCollection expects the amount in $spent:
-            $category->spent = $spent;
-            $object->addCategory($category);
-        }
-
-        return $object;
-    }
-
     /**
      * @param Carbon $date
      *

app/Helpers/Report/ReportHelperInterface.php

@@ -42,15 +42,6 @@ interface ReportHelperInterface
      */
     public function getBillReport(Carbon $start, Carbon $end, Collection $accounts): BillCollection;
 
-    /**
-     * @param Carbon     $start
-     * @param Carbon     $end
-     * @param Collection $accounts
-     *
-     * @return CategoryCollection
-     */
-    public function getCategoryReport(Collection $accounts, Carbon $start, Carbon $end): CategoryCollection;
-
     /**
      * @param Carbon $date
      *

app/Http/Controllers/AccountController.php

@@ -17,7 +17,6 @@ use Amount;
 use Carbon\Carbon;
 use ExpandedForm;
 use FireflyIII\Exceptions\FireflyException;
-use FireflyIII\Helpers\Collector\JournalCollector;
 use FireflyIII\Helpers\Collector\JournalCollectorInterface;
 use FireflyIII\Http\Requests\AccountFormRequest;
 use FireflyIII\Models\Account;
@@ -28,8 +27,8 @@ use FireflyIII\Repositories\Account\AccountRepositoryInterface as ARI;
 use FireflyIII\Repositories\Account\AccountTaskerInterface;
 use FireflyIII\Repositories\Currency\CurrencyRepositoryInterface;
 use FireflyIII\Support\CacheProperties;
+use Illuminate\Http\Request;
 use Illuminate\Support\Collection;
-use Input;
 use Log;
 use Navigation;
 use Preferences;
@@ -66,7 +65,7 @@ class AccountController extends Controller
     /**
      * @param string $what
      *
-     * @return View
+     * @return \Illuminate\View\View|\Illuminate\Contracts\View\Factory|View
      */
     public function create(string $what = 'asset')
     {
@@ -76,12 +75,9 @@ class AccountController extends Controller
         $defaultCurrency = Amount::getDefaultCurrency();
         $subTitleIcon    = config('firefly.subIconsByIdentifier.' . $what);
         $subTitle        = trans('firefly.make_new_' . $what . '_account');
-        Session::flash(
-            'preFilled',
-            [
-                'currency_id' => $defaultCurrency->id,
-            ]
-        );
+
+        // pre fill some data
+        Session::flash('preFilled', ['currency_id' => $defaultCurrency->id,]);
 
         // put previous url in session if not redirect from store (not "create another").
         if (session('accounts.create.fromStore') !== true) {
@@ -117,24 +113,32 @@ class AccountController extends Controller
     }
 
     /**
+     * @param Request $request
      * @param ARI     $repository
      * @param Account $account
      *
      * @return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector
      */
-    public function destroy(ARI $repository, Account $account)
+    public function destroy(Request $request, ARI $repository, Account $account)
     {
-        $type     = $account->accountType->type;
-        $typeName = config('firefly.shortNamesByFullName.' . $type);
-        $name     = $account->name;
-        $moveTo   = $repository->find(intval(Input::get('move_account_before_delete')));
+        $type      = $account->accountType->type;
+        $typeName  = config('firefly.shortNamesByFullName.' . $type);
+        $name      = $account->name;
+        $accountId = $account->id;
+        $moveTo    = $repository->find(intval($request->get('move_account_before_delete')));
 
         $repository->destroy($account, $moveTo);
 
         Session::flash('success', strval(trans('firefly.' . $typeName . '_deleted', ['name' => $name])));
         Preferences::mark();
 
-        return redirect(session('accounts.delete.url'));
+        $uri = session('accounts.delete.url');
+        if (!(strpos($uri, sprintf('accounts/show/%s', $accountId)) === false)) {
+            // uri would point back to account
+            $uri = route('accounts.index', [$typeName]);
+        }
+
+        return redirect($uri);
     }
 
     /**
@@ -192,8 +196,7 @@ class AccountController extends Controller
      */
     public function index(ARI $repository, string $what)
     {
-        $what = $what ?? 'asset';
-
+        $what         = $what ?? 'asset';
         $subTitle     = trans('firefly.' . $what . '_accounts');
         $subTitleIcon = config('firefly.subIconsByIdentifier.' . $what);
         $types        = config('firefly.accountTypesByIdentifier.' . $what);
@@ -214,6 +217,7 @@ class AccountController extends Controller
                 $account->lastActivityDate = $this->isInArray($activities, $account->id);
                 $account->startBalance     = $this->isInArray($startBalances, $account->id);
                 $account->endBalance       = $this->isInArray($endBalances, $account->id);
+                $account->difference       = bcsub($account->endBalance, $account->startBalance);
             }
         );
 
@@ -221,12 +225,13 @@ class AccountController extends Controller
     }
 
     /**
+     * @param Request                   $request
      * @param JournalCollectorInterface $collector
      * @param Account                   $account
      *
-     * @return View
+     * @return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector|View
      */
-    public function show(JournalCollectorInterface $collector, Account $account)
+    public function show(Request $request, JournalCollectorInterface $collector, Account $account)
     {
         if ($account->accountType->type === AccountType::INITIAL_BALANCE) {
             return $this->redirectToOriginalAccount($account);
@@ -235,12 +240,13 @@ class AccountController extends Controller
         $subTitleIcon = config('firefly.subIconsByIdentifier.' . $account->accountType->type);
         $subTitle     = $account->name;
         $range        = Preferences::get('viewRange', '1M')->data;
-
         $start        = session('start', Navigation::startOfPeriod(new Carbon, $range));
         $end          = session('end', Navigation::endOfPeriod(new Carbon, $range));
-        $page         = intval(Input::get('page')) === 0 ? 1 : intval(Input::get('page'));
+        $page         = intval($request->get('page')) === 0 ? 1 : intval($request->get('page'));
         $pageSize     = intval(Preferences::get('transactionPageSize', 50)->data);
         $chartUri     = route('chart.account.single', [$account->id]);
+        $accountType  = $account->accountType->type;
+
         // grab those journals:
         $collector->setAccounts(new Collection([$account]))->setRange($start, $end)->setLimit($pageSize)->setPage($page);
         $journals = $collector->getPaginatedJournals();
@@ -249,24 +255,26 @@ class AccountController extends Controller
         // generate entries for each period (and cache those)
         $entries = $this->periodEntries($account);
 
-        return view('accounts.show', compact('account', 'entries', 'subTitleIcon', 'journals', 'subTitle', 'start', 'end', 'chartUri'));
+        return view('accounts.show', compact('account', 'accountType', 'entries', 'subTitleIcon', 'journals', 'subTitle', 'start', 'end', 'chartUri'));
     }
 
     /**
+     * @param Request $request
      * @param ARI     $repository
      * @param Account $account
      *
      * @return View
      */
-    public function showAll(AccountRepositoryInterface $repository, Account $account)
+    public function showAll(Request $request, AccountRepositoryInterface $repository, Account $account)
     {
         $subTitle = sprintf('%s (%s)', $account->name, strtolower(trans('firefly.everything')));
-        $page     = intval(Input::get('page')) === 0 ? 1 : intval(Input::get('page'));
+        $page     = intval($request->get('page')) === 0 ? 1 : intval($request->get('page'));
         $pageSize = intval(Preferences::get('transactionPageSize', 50)->data);
         $chartUri = route('chart.account.all', [$account->id]);
 
         // replace with journal collector:
-        $collector = new JournalCollector(auth()->user());
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
         $collector->setAccounts(new Collection([$account]))->setLimit($pageSize)->setPage($page);
         $journals = $collector->getPaginatedJournals();
         $journals->setPath('accounts/show/' . $account->id . '/all');
@@ -280,30 +288,36 @@ class AccountController extends Controller
     }
 
     /**
+     * @param Request $request
      * @param Account $account
      * @param string  $date
      *
      * @return View
      */
-    public function showByDate(Account $account, string $date)
+    public function showByDate(Request $request, Account $account, string $date)
     {
-        $carbon   = new Carbon($date);
-        $range    = Preferences::get('viewRange', '1M')->data;
-        $start    = Navigation::startOfPeriod($carbon, $range);
-        $end      = Navigation::endOfPeriod($carbon, $range);
-        $subTitle = $account->name . ' (' . Navigation::periodShow($start, $range) . ')';
-        $page     = intval(Input::get('page')) === 0 ? 1 : intval(Input::get('page'));
-        $pageSize = intval(Preferences::get('transactionPageSize', 50)->data);
-        $chartUri = route('chart.account.period', [$account->id, $carbon->format('Y-m-d')]);
+        $carbon      = new Carbon($date);
+        $range       = Preferences::get('viewRange', '1M')->data;
+        $start       = Navigation::startOfPeriod($carbon, $range);
+        $end         = Navigation::endOfPeriod($carbon, $range);
+        $subTitle    = $account->name . ' (' . Navigation::periodShow($start, $range) . ')';
+        $page        = intval($request->get('page')) === 0 ? 1 : intval($request->get('page'));
+        $pageSize    = intval(Preferences::get('transactionPageSize', 50)->data);
+        $chartUri    = route('chart.account.period', [$account->id, $carbon->format('Y-m-d')]);
+        $accountType = $account->accountType->type;
 
         // replace with journal collector:
-        $collector = new JournalCollector(auth()->user());
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
         $collector->setAccounts(new Collection([$account]))->setRange($start, $end)->setLimit($pageSize)->setPage($page);
         $journals = $collector->getPaginatedJournals();
         $journals->setPath('accounts/show/' . $account->id . '/' . $date);
 
+        // generate entries for each period (and cache those)
+        $entries = $this->periodEntries($account);
+
         // same call, except "entries".
-        return view('accounts.show', compact('account', 'subTitleIcon', 'journals', 'subTitle', 'start', 'end', 'chartUri'));
+        return view('accounts.show', compact('account', 'accountType', 'entries', 'subTitleIcon', 'journals', 'subTitle', 'start', 'end', 'chartUri'));
     }
 
     /**
@@ -328,7 +342,7 @@ class AccountController extends Controller
             Preferences::set('frontPageAccounts', $frontPage);
         }
 
-        if (intval(Input::get('create_another')) === 1) {
+        if (intval($request->get('create_another')) === 1) {
             // set value so create routine will not overwrite URL:
             Session::put('accounts.create.fromStore', true);
 
@@ -354,7 +368,7 @@ class AccountController extends Controller
         Session::flash('success', strval(trans('firefly.updated_account', ['name' => $account->name])));
         Preferences::mark();
 
-        if (intval(Input::get('return_to_edit')) === 1) {
+        if (intval($request->get('return_to_edit')) === 1) {
             // set value so edit routine will not overwrite URL:
             Session::put('accounts.edit.fromUpdate', true);
 
@@ -379,7 +393,7 @@ class AccountController extends Controller
             return $array[$entryId];
         }
 
-        return '';
+        return '0';
     }
 
     /**
@@ -430,7 +444,7 @@ class AccountController extends Controller
             $earned     = $tasker->amountInInPeriod(new Collection([$account]), $assets, $end, $currentEnd);
             $dateStr    = $end->format('Y-m-d');
             $dateName   = Navigation::periodShow($end, $range);
-            $entries->push([$dateStr, $dateName, $spent, $earned]);
+            $entries->push([$dateStr, $dateName, $spent, $earned, clone $end]);
             $end = Navigation::subtractPeriod($end, $range, 1);
 
         }

app/Http/Controllers/Admin/ConfigurationController.php

@@ -58,23 +58,13 @@ class ConfigurationController extends Controller
 
         // all available configuration and their default value in case
         // they don't exist yet.
-        $singleUserMode     = FireflyConfig::get('single_user_mode', config('firefly.configuration.single_user_mode'))->data;
-        $mustConfirmAccount = FireflyConfig::get('must_confirm_account', config('firefly.configuration.must_confirm_account'))->data;
-        $isDemoSite         = FireflyConfig::get('is_demo_site', config('firefly.configuration.is_demo_site'))->data;
-        $siteOwner          = env('SITE_OWNER');
-
-        // email settings:
-        $sendErrorMessage = [
-            'mail_for_lockout'        => FireflyConfig::get('mail_for_lockout', config('firefly.configuration.mail_for_lockout'))->data,
-            'mail_for_blocked_domain' => FireflyConfig::get('mail_for_blocked_domain', config('firefly.configuration.mail_for_blocked_domain'))->data,
-            'mail_for_blocked_email'  => FireflyConfig::get('mail_for_blocked_email', config('firefly.configuration.mail_for_blocked_email'))->data,
-            'mail_for_bad_login'      => FireflyConfig::get('mail_for_bad_login', config('firefly.configuration.mail_for_bad_login'))->data,
-            'mail_for_blocked_login'  => FireflyConfig::get('mail_for_blocked_login', config('firefly.configuration.mail_for_blocked_login'))->data,
-        ];
+        $singleUserMode = FireflyConfig::get('single_user_mode', config('firefly.configuration.single_user_mode'))->data;
+        $isDemoSite     = FireflyConfig::get('is_demo_site', config('firefly.configuration.is_demo_site'))->data;
+        $siteOwner      = env('SITE_OWNER');
 
         return view(
             'admin.configuration.index',
-            compact('subTitle', 'subTitleIcon', 'singleUserMode', 'mustConfirmAccount', 'isDemoSite', 'sendErrorMessage', 'siteOwner')
+            compact('subTitle', 'subTitleIcon', 'singleUserMode', 'isDemoSite', 'siteOwner')
         );
 
     }
@@ -91,16 +81,8 @@ class ConfigurationController extends Controller
 
         // store config values
         FireflyConfig::set('single_user_mode', $data['single_user_mode']);
-        FireflyConfig::set('must_confirm_account', $data['must_confirm_account']);
         FireflyConfig::set('is_demo_site', $data['is_demo_site']);
 
-        // email settings
-        FireflyConfig::set('mail_for_lockout', $data['mail_for_lockout']);
-        FireflyConfig::set('mail_for_blocked_domain', $data['mail_for_blocked_domain']);
-        FireflyConfig::set('mail_for_blocked_email', $data['mail_for_blocked_email']);
-        FireflyConfig::set('mail_for_bad_login', $data['mail_for_bad_login']);
-        FireflyConfig::set('mail_for_blocked_login', $data['mail_for_blocked_login']);
-
         // flash message
         Session::flash('success', strval(trans('firefly.configuration_updated')));
         Preferences::mark();

app/Http/Controllers/Admin/DomainController.php

@@ -1,140 +0,0 @@
-<?php
-/**
- * DomainController.php
- * Copyright (C) 2016 thegrumpydictator@gmail.com
- *
- * This software may be modified and distributed under the terms of the
- * Creative Commons Attribution-ShareAlike 4.0 International License.
- *
- * See the LICENSE file for details.
- */
-
-declare(strict_types = 1);
-
-namespace FireflyIII\Http\Controllers\Admin;
-
-
-use FireflyIII\Http\Controllers\Controller;
-use FireflyIII\Repositories\User\UserRepositoryInterface;
-use FireflyIII\Support\Facades\FireflyConfig;
-use FireflyIII\User;
-use Illuminate\Http\Request;
-use Session;
-
-/**
- * Class DomainController
- *
- * @package FireflyIII\Http\Controllers\Admin
- */
-class DomainController extends Controller
-{
-
-    /**
-     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
-     */
-    public function domains()
-    {
-
-        $title         = strval(trans('firefly.administration'));
-        $mainTitleIcon = 'fa-hand-spock-o';
-        $subTitle      = strval(trans('firefly.blocked_domains'));
-        $subTitleIcon  = 'fa-exclamation-circle';
-        $domains       = FireflyConfig::get('blocked-domains', [])->data;
-
-        // known domains
-        $knownDomains = $this->getKnownDomains();
-
-        return view('admin.domains.index', compact('title', 'mainTitleIcon', 'knownDomains', 'subTitle', 'subTitleIcon', 'domains'));
-    }
-
-
-    /**
-     * @param Request $request
-     *
-     * @return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector
-     */
-    public function manual(Request $request)
-    {
-        if (strlen($request->get('domain')) === 0) {
-            Session::flash('error', trans('firefly.no_domain_filled_in'));
-
-            return redirect(route('admin.users.domains'));
-        }
-
-        $domain  = strtolower($request->get('domain'));
-        $blocked = FireflyConfig::get('blocked-domains', [])->data;
-
-        if (in_array($domain, $blocked)) {
-            Session::flash('error', trans('firefly.domain_already_blocked', ['domain' => $domain]));
-
-            return redirect(route('admin.users.domains'));
-        }
-        $blocked[] = $domain;
-        FireflyConfig::set('blocked-domains', $blocked);
-
-        Session::flash('success', trans('firefly.domain_is_now_blocked', ['domain' => $domain]));
-
-        return redirect(route('admin.users.domains'));
-    }
-
-    /**
-     * @param string $domain
-     *
-     * @return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector
-     */
-    public function toggleDomain(string $domain)
-    {
-        $domain  = strtolower($domain);
-        $blocked = FireflyConfig::get('blocked-domains', [])->data;
-
-        if (in_array($domain, $blocked)) {
-            $key = array_search($domain, $blocked);
-            unset($blocked[$key]);
-            sort($blocked);
-
-            FireflyConfig::set('blocked-domains', $blocked);
-            Session::flash('message', trans('firefly.domain_now_unblocked', ['domain' => $domain]));
-
-
-            return redirect(route('admin.users.domains'));
-
-        }
-
-        $blocked[] = $domain;
-
-        FireflyConfig::set('blocked-domains', $blocked);
-        Session::flash('message', trans('firefly.domain_now_blocked', ['domain' => $domain]));
-
-        return redirect(route('admin.users.domains'));
-    }
-
-    /**
-     * @return array
-     */
-    private function getKnownDomains(): array
-    {
-        /** @var UserRepositoryInterface $repository */
-        $repository = app(UserRepositoryInterface::class);
-        $users      = $repository->all();
-        $set        = [];
-        $filtered   = [];
-        /** @var User $user */
-        foreach ($users as $user) {
-            $email = $user->email;
-            $parts = explode('@', $email);
-            $set[] = strtolower($parts[1]);
-        }
-        $set = array_unique($set);
-        // filter for already banned domains:
-        $blocked = FireflyConfig::get('blocked-domains', [])->data;
-
-        foreach ($set as $domain) {
-            // in the block array? ignore it.
-            if (!in_array($domain, $blocked)) {
-                $filtered[] = $domain;
-            }
-        }
-
-        return $filtered;
-    }
-}

app/Http/Controllers/Admin/UserController.php

@@ -14,7 +14,6 @@ declare(strict_types = 1);
 namespace FireflyIII\Http\Controllers\Admin;
 
 
-use FireflyConfig;
 use FireflyIII\Http\Controllers\Controller;
 use FireflyIII\Http\Requests\UserFormRequest;
 use FireflyIII\Repositories\User\UserRepositoryInterface;
@@ -81,30 +80,20 @@ class UserController extends Controller
      */
     public function index(UserRepositoryInterface $repository)
     {
-        $subTitle           = strval(trans('firefly.user_administration'));
-        $subTitleIcon       = 'fa-users';
-        $mustConfirmAccount = FireflyConfig::get('must_confirm_account', config('firefly.configuration.must_confirm_account'))->data;
-        $users              = $repository->all();
+        $subTitle     = strval(trans('firefly.user_administration'));
+        $subTitleIcon = 'fa-users';
+        $users        = $repository->all();
 
         // add meta stuff.
         $users->each(
-            function (User $user) use ($mustConfirmAccount) {
-                $list        = ['user_confirmed', 'twoFactorAuthEnabled', 'twoFactorAuthSecret', 'registration_ip_address', 'confirmation_ip_address'];
-                $preferences = Preferences::getArrayForUser($user, $list);
-
-                $user->activated = true;
-                if (!($preferences['user_confirmed'] === true) && $mustConfirmAccount === true) {
-                    $user->activated = false;
-                }
-
+            function (User $user) {
+                $list          = ['twoFactorAuthEnabled', 'twoFactorAuthSecret'];
+                $preferences   = Preferences::getArrayForUser($user, $list);
                 $user->isAdmin = $user->hasRole('owner');
                 $is2faEnabled  = $preferences['twoFactorAuthEnabled'] === true;
                 $has2faSecret  = !is_null($preferences['twoFactorAuthSecret']);
-                $user->has2FA  = false;
-                if ($is2faEnabled && $has2faSecret) {
-                    $user->has2FA = true;
-                }
-                $user->prefs = $preferences;
+                $user->has2FA  = ($is2faEnabled && $has2faSecret) ? true : false;
+                $user->prefs   = $preferences;
             }
         );
 
@@ -125,37 +114,12 @@ class UserController extends Controller
         $mainTitleIcon = 'fa-hand-spock-o';
         $subTitle      = strval(trans('firefly.single_user_administration', ['email' => $user->email]));
         $subTitleIcon  = 'fa-user';
-
-        // get IP info:
-        $defaultIp    = '0.0.0.0';
-        $regPref      = Preferences::getForUser($user, 'registration_ip_address');
-        $registration = $defaultIp;
-        $conPref      = Preferences::getForUser($user, 'confirmation_ip_address');
-        $confirmation = $defaultIp;
-        if (!is_null($regPref)) {
-            $registration = $regPref->data;
-        }
-        if (!is_null($conPref)) {
-            $confirmation = $conPref->data;
-        }
-
-        $registrationHost = '';
-        $confirmationHost = '';
-
-        if ($registration != $defaultIp) {
-            $registrationHost = gethostbyaddr($registration);
-        }
-        if ($confirmation != $defaultIp) {
-            $confirmationHost = gethostbyaddr($confirmation);
-        }
-
-        $information = $repository->getUserData($user);
+        $information   = $repository->getUserData($user);
 
         return view(
             'admin.users.show',
             compact(
-                'title', 'mainTitleIcon', 'subTitle', 'subTitleIcon', 'information',
-                'user', 'registration', 'confirmation', 'registrationHost', 'confirmationHost'
+                'title', 'mainTitleIcon', 'subTitle', 'subTitleIcon', 'information', 'user'
             )
         );
     }

app/Http/Controllers/AttachmentController.php

@@ -13,24 +13,22 @@ declare(strict_types = 1);
 
 namespace FireflyIII\Http\Controllers;
 
-use Crypt;
 use File;
 use FireflyIII\Exceptions\FireflyException;
 use FireflyIII\Http\Requests\AttachmentFormRequest;
 use FireflyIII\Models\Attachment;
 use FireflyIII\Repositories\Attachment\AttachmentRepositoryInterface;
-use Input;
-use Log;
 use Preferences;
 use Response;
 use Session;
-use Storage;
 use URL;
 use View;
 
 /**
  * Class AttachmentController
  *
+ * @SuppressWarnings(PHPMD.CouplingBetweenObjects) // it's 13.
+ *
  * @package FireflyIII\Http\Controllers
  */
 class AttachmentController extends Controller
@@ -57,7 +55,7 @@ class AttachmentController extends Controller
     /**
      * @param Attachment $attachment
      *
-     * @return \Illuminate\View\View|\Illuminate\Contracts\View\Factory
+     * @return View
      */
     public function delete(Attachment $attachment)
     {
@@ -90,23 +88,17 @@ class AttachmentController extends Controller
     }
 
     /**
-     * @param Attachment $attachment
+     * @param AttachmentRepositoryInterface $repository
+     * @param Attachment                    $attachment
      *
+     * @return mixed
      * @throws FireflyException
-     *
      */
-    public function download(Attachment $attachment)
+    public function download(AttachmentRepositoryInterface $repository, Attachment $attachment)
     {
-        // create a disk.
-        $disk = Storage::disk('upload');
-        $file = $attachment->fileName();
-
-        if ($disk->exists($file)) {
-
+        if ($repository->exists($attachment)) {
+            $content = $repository->getContent($attachment);
             $quoted  = sprintf('"%s"', addcslashes(basename($attachment->filename), '"\\'));
-            $content = Crypt::decrypt($disk->get($file));
-
-            Log::debug('Send file to user', ['file' => $quoted, 'size' => strlen($content)]);
 
             return response($content, 200)
                 ->header('Content-Description', 'File Transfer')
@@ -118,7 +110,6 @@ class AttachmentController extends Controller
                 ->header('Cache-Control', 'must-revalidate, post-check=0, pre-check=0')
                 ->header('Pragma', 'public')
                 ->header('Content-Length', strlen($content));
-
         }
         throw new FireflyException('Could not find the indicated attachment. The file is no longer there.');
     }
@@ -151,7 +142,6 @@ class AttachmentController extends Controller
     {
         $image = 'images/page_green.png';
 
-
         if ($attachment->mime == 'application/pdf') {
             $image = 'images/page_white_acrobat.png';
         }
@@ -178,7 +168,7 @@ class AttachmentController extends Controller
         Session::flash('success', strval(trans('firefly.attachment_updated', ['name' => $attachment->filename])));
         Preferences::mark();
 
-        if (intval(Input::get('return_to_edit')) === 1) {
+        if (intval($request->get('return_to_edit')) === 1) {
             // set value so edit routine will not overwrite URL:
             Session::put('attachments.edit.fromUpdate', true);
 

app/Http/Controllers/Auth/ConfirmationController.php

@@ -1,90 +0,0 @@
-<?php
-/**
- * ConfirmationController.php
- * Copyright (C) 2016 thegrumpydictator@gmail.com
- *
- * This software may be modified and distributed under the terms of the
- * Creative Commons Attribution-ShareAlike 4.0 International License.
- *
- * See the LICENSE file for details.
- */
-
-declare(strict_types = 1);
-
-namespace FireflyIII\Http\Controllers\Auth;
-
-use FireflyIII\Events\ConfirmedUser;
-use FireflyIII\Events\ResentConfirmation;
-use FireflyIII\Exceptions\FireflyException;
-use FireflyIII\Http\Controllers\Controller;
-use Illuminate\Http\Request;
-use Preferences;
-use Session;
-
-/**
- * Class ConfirmationController
- *
- * @package FireflyIII\Http\Controllers\Auth
- */
-class ConfirmationController extends Controller
-{
-
-    /**
-     * @return mixed
-     */
-    public function confirmationError()
-    {
-        return view('auth.confirmation.error');
-    }
-
-    /**
-     * @param Request $request
-     * @param string  $code
-     *
-     * @return mixed
-     * @throws FireflyException
-     */
-    public function doConfirmation(Request $request, string $code)
-    {
-        // check user_confirmed_last_mail
-
-        $database = Preferences::get('user_confirmed_code')->data;
-        $time     = Preferences::get('user_confirmed_last_mail', 0)->data;
-        $now      = time();
-        $maxDiff  = config('firefly.confirmation_age');
-
-        if ($database === $code && ($now - $time <= $maxDiff)) {
-
-            // trigger user registration event:
-            event(new ConfirmedUser(auth()->user(), $request->ip()));
-
-            Preferences::setForUser(auth()->user(), 'user_confirmed', true);
-            Preferences::setForUser(auth()->user(), 'user_confirmed_confirmed', time());
-            Session::flash('success', strval(trans('firefly.account_is_confirmed')));
-
-            return redirect(route('home'));
-        }
-        throw new FireflyException(trans('firefly.invalid_activation_code'));
-    }
-
-    /**
-     * @param Request $request
-     *
-     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
-     */
-    public function resendConfirmation(Request $request)
-    {
-        $time    = Preferences::get('user_confirmed_last_mail', 0)->data;
-        $now     = time();
-        $maxDiff = config('firefly.resend_confirmation');
-        $owner   = env('SITE_OWNER', 'mail@example.com');
-        $view    = 'auth.confirmation.no-resent';
-        if ($now - $time > $maxDiff) {
-            event(new ResentConfirmation(auth()->user(), $request->ip()));
-            $view = 'auth.confirmation.resent';
-        }
-
-        return view($view, ['owner' => $owner]);
-    }
-
-}

app/Http/Controllers/Auth/ForgotPasswordController.php

@@ -13,7 +13,10 @@ declare(strict_types = 1);
 namespace FireflyIII\Http\Controllers\Auth;
 
 use FireflyIII\Http\Controllers\Controller;
+use FireflyIII\User;
 use Illuminate\Foundation\Auth\SendsPasswordResetEmails;
+use Illuminate\Http\Request;
+use Password;
 
 /**
  * Class ForgotPasswordController
@@ -33,4 +36,39 @@ class ForgotPasswordController extends Controller
         parent::__construct();
         $this->middleware('guest');
     }
+
+    /**
+     * Send a reset link to the given user.
+     *
+     * @param  Request $request
+     *
+     * @return \Illuminate\Http\RedirectResponse
+     */
+    public function sendResetLinkEmail(Request $request)
+    {
+        $this->validate($request, ['email' => 'required|email']);
+
+        // verify if the user is not a demo user. If so, we give him back an error.
+        $user = User::where('email', $request->get('email'))->first();
+        if (!is_null($user) && $user->hasRole('demo')) {
+            return back()->withErrors(
+                ['email' => trans('firefly.cannot_reset_demo_user')]
+            );
+        }
+
+        $response = $this->broker()->sendResetLink(
+            $request->only('email')
+        );
+
+        if ($response === Password::RESET_LINK_SENT) {
+            return back()->with('status', trans($response));
+        }
+
+        // If an error was returned by the password broker, we will get this message
+        // translated so we can notify a user of the problem. We'll redirect back
+        // to where the users came from so they can attempt this process again.
+        return back()->withErrors(
+            ['email' => trans($response)]
+        );
+    }
 }

app/Http/Controllers/Auth/LoginController.php

@@ -14,9 +14,6 @@ namespace FireflyIII\Http\Controllers\Auth;
 
 use Config;
 use FireflyConfig;
-use FireflyIII\Events\BlockedBadLogin;
-use FireflyIII\Events\BlockedUserLogin;
-use FireflyIII\Events\LockedOutUser;
 use FireflyIII\Http\Controllers\Controller;
 use FireflyIII\User;
 use Illuminate\Foundation\Auth\AuthenticatesUsers;
@@ -53,9 +50,9 @@ class LoginController extends Controller
     /**
      * Handle a login request to the application.
      *
-     * @param  \Illuminate\Http\Request $request
+     * @param Request $request
      *
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\RedirectResponse|\Illuminate\Http\Response
      */
     public function login(Request $request)
     {
@@ -64,8 +61,6 @@ class LoginController extends Controller
         if ($lockedOut) {
             $this->fireLockoutEvent($request);
 
-            event(new LockedOutUser($request->get('email'), $request->ip()));
-
             return $this->sendLockoutResponse($request);
         }
 
@@ -76,25 +71,8 @@ class LoginController extends Controller
             return $this->sendLoginResponse($request);
         }
 
-        // check if user is blocked:
-        $errorMessage = '';
-        /** @var User $foundUser */
-        $foundUser = User::where('email', $credentials['email'])->where('blocked', 1)->first();
-        if (!is_null($foundUser)) {
-            // user exists, but is blocked:
-            $code         = strlen(strval($foundUser->blocked_code)) > 0 ? $foundUser->blocked_code : 'general_blocked';
-            $errorMessage = strval(trans('firefly.' . $code . '_error', ['email' => $credentials['email']]));
-            event(new BlockedUserLogin($foundUser, $request->ip()));
-        }
+        $errorMessage = $this->getBlockedError($credentials['email']);
 
-        // simply a bad login.
-        if (is_null($foundUser)) {
-            event(new BlockedBadLogin($credentials['email'], $request->ip()));
-        }
-
-        // If the login attempt was unsuccessful we will increment the number of attempts
-        // to login and redirect the user back to the login form. Of course, when this
-        // user surpasses their maximum number of attempts they will get locked out.
         if (!$lockedOut) {
             $this->incrementLoginAttempts($request);
         }
@@ -159,4 +137,24 @@ class LoginController extends Controller
                              ]
                          );
     }
+
+    /**
+     * @param string $email
+     *
+     * @return string
+     */
+    private function getBlockedError(string $email): string
+    {
+        // check if user is blocked:
+        $errorMessage = '';
+        /** @var User $foundUser */
+        $foundUser = User::where('email', $email)->where('blocked', 1)->first();
+        if (!is_null($foundUser)) {
+            // user exists, but is blocked:
+            $code         = strlen(strval($foundUser->blocked_code)) > 0 ? $foundUser->blocked_code : 'general_blocked';
+            $errorMessage = strval(trans('firefly.' . $code . '_error', ['email' => $email]));
+        }
+
+        return $errorMessage;
+    }
 }

app/Http/Controllers/Auth/PasswordController.php

@@ -26,8 +26,8 @@ use Illuminate\Support\Facades\Password;
  *
  * @package FireflyIII\Http\Controllers\Auth
  * @method getEmailSubject()
- * @method getSendResetLinkEmailSuccessResponse()
- * @method getSendResetLinkEmailFailureResponse()
+ * @method getSendResetLinkEmailSuccessResponse(string $response)
+ * @method getSendResetLinkEmailFailureResponse(string $response)
  */
 class PasswordController extends Controller
 {
@@ -47,6 +47,7 @@ class PasswordController extends Controller
 
     /**
      * Send a reset link to the given user.
+     * @SuppressWarnings(PHPMD.CyclomaticComplexity) // it's 7 but ok
      *
      * @param  \Illuminate\Http\Request $request
      *

app/Http/Controllers/Auth/RegisterController.php

@@ -15,18 +15,12 @@ namespace FireflyIII\Http\Controllers\Auth;
 use Auth;
 use Config;
 use FireflyConfig;
-use FireflyIII\Events\BlockedUseOfDomain;
-use FireflyIII\Events\BlockedUseOfEmail;
 use FireflyIII\Events\RegisteredUser;
 use FireflyIII\Http\Controllers\Controller;
 use FireflyIII\User;
 use Illuminate\Foundation\Auth\RegistersUsers;
 use Illuminate\Http\Request;
-use Illuminate\Mail\Message;
-use Log;
-use Mail;
 use Session;
-use Swift_TransportException;
 use Validator;
 
 /**
@@ -78,30 +72,6 @@ class RegisterController extends Controller
             $this->throwValidationException($request, $validator);
         }
 
-        $data             = $request->all();
-        $data['password'] = bcrypt($data['password']);
-
-        // is user email domain blocked?
-        if ($this->isBlockedDomain($data['email'])) {
-            $validator->getMessageBag()->add('email', (string)trans('validation.invalid_domain'));
-
-            event(new BlockedUseOfDomain($data['email'], $request->ip()));
-            $this->throwValidationException($request, $validator);
-        }
-
-        // is user a deleted user?
-        $hash          = hash('sha256', $data['email']);
-        $configuration = FireflyConfig::get('deleted_users', []);
-        $set           = $configuration->data;
-        Log::debug(sprintf('Hash of email is %s', $hash));
-        Log::debug('Hashes of deleted users: ', $set);
-        if (in_array($hash, $set)) {
-            $validator->getMessageBag()->add('email', (string)trans('validation.deleted_user'));
-            event(new BlockedUseOfEmail($data['email'], $request->ip()));
-            $this->throwValidationException($request, $validator);
-        }
-
-
         $user = $this->create($request->all());
 
         // trigger user registration event:
@@ -129,9 +99,6 @@ class RegisterController extends Controller
         // is demo site?
         $isDemoSite = FireflyConfig::get('is_demo_site', Config::get('firefly.configuration.is_demo_site'))->data;
 
-        // activate account?
-        $mustConfirmAccount = FireflyConfig::get('must_confirm_account', Config::get('firefly.configuration.must_confirm_account'))->data;
-
         // is allowed to?
         $singleUserMode = FireflyConfig::get('single_user_mode', Config::get('firefly.configuration.single_user_mode'))->data;
         $userCount      = User::count();
@@ -143,7 +110,7 @@ class RegisterController extends Controller
 
         $email = $request->old('email');
 
-        return view('auth.register', compact('isDemoSite', 'email', 'mustConfirmAccount'));
+        return view('auth.register', compact('isDemoSite', 'email'));
     }
 
     /**
@@ -179,30 +146,4 @@ class RegisterController extends Controller
                  ]
         );
     }
-
-    /**
-     * @return array
-     */
-    private function getBlockedDomains()
-    {
-        return FireflyConfig::get('blocked-domains', [])->data;
-    }
-
-    /**
-     * @param string $email
-     *
-     * @return bool
-     */
-    private function isBlockedDomain(string $email)
-    {
-        $parts   = explode('@', $email);
-        $blocked = $this->getBlockedDomains();
-
-        if (isset($parts[1]) && in_array($parts[1], $blocked)) {
-            return true;
-        }
-
-        return false;
-    }
-
 }

app/Http/Controllers/Auth/TwoFactorController.php

@@ -76,6 +76,7 @@ class TwoFactorController extends Controller
 
     /**
      * @param TokenFormRequest $request
+     * @SuppressWarnings(PHPMD.UnusedFormalParameter) // it's unused but the class does some validation.
      *
      * @return mixed
      */

app/Http/Controllers/BillController.php

@@ -14,13 +14,13 @@ declare(strict_types = 1);
 namespace FireflyIII\Http\Controllers;
 
 use Carbon\Carbon;
-use FireflyIII\Helpers\Collector\JournalCollector;
+use FireflyIII\Helpers\Collector\JournalCollectorInterface;
 use FireflyIII\Http\Requests\BillFormRequest;
 use FireflyIII\Models\Bill;
 use FireflyIII\Models\TransactionJournal;
 use FireflyIII\Repositories\Bill\BillRepositoryInterface;
+use Illuminate\Http\Request;
 use Illuminate\Support\Collection;
-use Input;
 use Preferences;
 use Session;
 use URL;
@@ -99,13 +99,20 @@ class BillController extends Controller
      */
     public function destroy(BillRepositoryInterface $repository, Bill $bill)
     {
-        $name = $bill->name;
+        $name   = $bill->name;
+        $billId = $bill->id;
         $repository->destroy($bill);
 
         Session::flash('success', strval(trans('firefly.deleted_bill', ['name' => $name])));
         Preferences::mark();
 
-        return redirect(session('bills.delete.url'));
+        $uri = session('bills.delete.url');
+        if (!(strpos($uri, sprintf('bills/show/%s', $billId)) === false)) {
+            // uri would point back to bill
+            $uri = route('bills.index');
+        }
+
+        return redirect($uri);
     }
 
     /**
@@ -190,24 +197,27 @@ class BillController extends Controller
     }
 
     /**
+     * @param Request                 $request
      * @param BillRepositoryInterface $repository
      * @param Bill                    $bill
      *
      * @return View
      */
-    public function show(BillRepositoryInterface $repository, Bill $bill)
+    public function show(Request $request, BillRepositoryInterface $repository, Bill $bill)
     {
         /** @var Carbon $date */
         $date           = session('start');
         $year           = $date->year;
-        $page           = intval(Input::get('page')) == 0 ? 1 : intval(Input::get('page'));
+        $page           = intval($request->get('page')) == 0 ? 1 : intval($request->get('page'));
         $pageSize       = intval(Preferences::get('transactionPageSize', 50)->data);
         $yearAverage    = $repository->getYearAverage($bill, $date);
         $overallAverage = $repository->getOverallAverage($bill);
 
         // use collector:
-        $collector = new JournalCollector(auth()->user());
-        $collector->setAllAssetAccounts()->setBills(new Collection([$bill]))->setPage($page)->setLimit($pageSize);
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
+        $collector->setAllAssetAccounts()->setBills(new Collection([$bill]))->setLimit($pageSize)->setPage($page)->withBudgetInformation()
+                  ->withCategoryInformation();
         $journals = $collector->getPaginatedJournals();
         $journals->setPath('/bills/show/' . $bill->id);
 
@@ -231,7 +241,7 @@ class BillController extends Controller
         Session::flash('success', strval(trans('firefly.stored_new_bill', ['name' => e($bill->name)])));
         Preferences::mark();
 
-        if (intval(Input::get('create_another')) === 1) {
+        if (intval($request->get('create_another')) === 1) {
             // set value so create routine will not overwrite URL:
             Session::put('bills.create.fromStore', true);
 
@@ -258,7 +268,7 @@ class BillController extends Controller
         Session::flash('success', strval(trans('firefly.updated_bill', ['name' => e($bill->name)])));
         Preferences::mark();
 
-        if (intval(Input::get('return_to_edit')) === 1) {
+        if (intval($request->get('return_to_edit')) === 1) {
             // set value so edit routine will not overwrite URL:
             Session::put('bills.edit.fromUpdate', true);
 

app/Http/Controllers/BudgetController.php

@@ -15,19 +15,18 @@ namespace FireflyIII\Http\Controllers;
 
 use Amount;
 use Carbon\Carbon;
-use Config;
 use FireflyIII\Exceptions\FireflyException;
-use FireflyIII\Helpers\Collector\JournalCollector;
+use FireflyIII\Helpers\Collector\JournalCollectorInterface;
 use FireflyIII\Http\Requests\BudgetFormRequest;
+use FireflyIII\Http\Requests\BudgetIncomeRequest;
 use FireflyIII\Models\AccountType;
 use FireflyIII\Models\Budget;
-use FireflyIII\Models\LimitRepetition;
+use FireflyIII\Models\BudgetLimit;
 use FireflyIII\Repositories\Account\AccountRepositoryInterface;
 use FireflyIII\Repositories\Budget\BudgetRepositoryInterface;
+use FireflyIII\Support\CacheProperties;
+use Illuminate\Http\Request;
 use Illuminate\Support\Collection;
-use Input;
-use Log;
-use Navigation;
 use Preferences;
 use Response;
 use Session;
@@ -42,6 +41,9 @@ use View;
 class BudgetController extends Controller
 {
 
+    /** @var  BudgetRepositoryInterface */
+    private $repository;
+
     /**
      *
      */
@@ -55,6 +57,7 @@ class BudgetController extends Controller
             function ($request, $next) {
                 View::share('title', trans('firefly.budgets'));
                 View::share('mainTitleIcon', 'fa-tasks');
+                $this->repository = app(BudgetRepositoryInterface::class);
 
                 return $next($request);
             }
@@ -62,32 +65,25 @@ class BudgetController extends Controller
     }
 
     /**
-     * @param BudgetRepositoryInterface $repository
-     * @param Budget                    $budget
+     * @param Request $request
+     * @param Budget  $budget
      *
-     * @return \Symfony\Component\HttpFoundation\Response
+     * @return \Illuminate\Http\JsonResponse
      */
-    public function amount(BudgetRepositoryInterface $repository, Budget $budget)
+    public function amount(Request $request, Budget $budget)
     {
-        $amount = intval(Input::get('amount'));
+        $amount = intval($request->get('amount'));
         /** @var Carbon $start */
         $start = session('start', Carbon::now()->startOfMonth());
         /** @var Carbon $end */
-        $end       = session('end', Carbon::now()->endOfMonth());
-        $viewRange = Preferences::get('viewRange', '1M')->data;
-
-        // is custom view range?
-        if (session('is_custom_range') === true) {
-            $viewRange = 'custom';
-        }
-
-        $limitRepetition = $repository->updateLimitAmount($budget, $start, $end, $viewRange, $amount);
+        $end         = session('end', Carbon::now()->endOfMonth());
+        $budgetLimit = $this->repository->updateLimitAmount($budget, $start, $end, $amount);
         if ($amount == 0) {
-            $limitRepetition = null;
+            $budgetLimit = null;
         }
         Preferences::mark();
 
-        return Response::json(['name' => $budget->name, 'repetition' => $limitRepetition ? $limitRepetition->id : 0]);
+        return Response::json(['name' => $budget->name, 'limit' => $budgetLimit ? $budgetLimit->id : 0, 'amount' => $amount]);
 
     }
 
@@ -126,23 +122,28 @@ class BudgetController extends Controller
     }
 
     /**
-     * @param Budget                    $budget
-     * @param BudgetRepositoryInterface $repository
+     * @param Budget $budget
      *
      * @return \Illuminate\Http\RedirectResponse
      */
-    public function destroy(Budget $budget, BudgetRepositoryInterface $repository)
+    public function destroy(Budget $budget)
     {
 
-        $name = $budget->name;
-        $repository->destroy($budget);
+        $name     = $budget->name;
+        $budgetId = $budget->id;
+        $this->repository->destroy($budget);
 
 
         Session::flash('success', strval(trans('firefly.deleted_budget', ['name' => e($name)])));
         Preferences::mark();
 
+        $uri = session('budgets.delete.url');
+        if (!(strpos($uri, sprintf('budgets/show/%s', $budgetId)) === false)) {
+            // uri would point back to budget
+            $uri = route('budgets.index');
+        }
 
-        return redirect(session('budgets.delete.url'));
+        return redirect($uri);
     }
 
     /**
@@ -167,95 +168,42 @@ class BudgetController extends Controller
     }
 
     /**
-     * @param BudgetRepositoryInterface  $repository
-     * @param AccountRepositoryInterface $accountRepository
-     *
      * @return View
-     *
      */
-    public function index(BudgetRepositoryInterface $repository, AccountRepositoryInterface $accountRepository)
+    public function index()
     {
-        $repository->cleanupBudgets();
+        $this->repository->cleanupBudgets();
 
-        $budgets    = $repository->getActiveBudgets();
-        $inactive   = $repository->getInactiveBudgets();
-        $spent      = '0';
-        $budgeted   = '0';
-        $range      = Preferences::get('viewRange', '1M')->data;
-        $repeatFreq = Config::get('firefly.range_to_repeat_freq.' . $range);
-
-        if (session('is_custom_range') === true) {
-            $repeatFreq = 'custom';
-        }
-
-        /** @var Carbon $start */
-        $start = session('start', new Carbon);
-        /** @var Carbon $end */
+        $budgets           = $this->repository->getActiveBudgets();
+        $inactive          = $this->repository->getInactiveBudgets();
+        $start             = session('start', new Carbon);
         $end               = session('end', new Carbon);
-        $key               = 'budgetIncomeTotal' . $start->format('Ymd') . $end->format('Ymd');
-        $budgetIncomeTotal = Preferences::get($key, 1000)->data;
-        $period            = Navigation::periodShow($start, $range);
         $periodStart       = $start->formatLocalized($this->monthAndDayFormat);
         $periodEnd         = $end->formatLocalized($this->monthAndDayFormat);
-        $accounts          = $accountRepository->getAccountsByType([AccountType::DEFAULT, AccountType::ASSET, AccountType::CASH]);
-        $startAsString     = $start->format('Y-m-d');
-        $endAsString       = $end->format('Y-m-d');
-        Log::debug('Now at /budgets');
-
-        // loop the budgets:
-        /** @var Budget $budget */
-        foreach ($budgets as $budget) {
-            Log::debug(sprintf('Now at budget #%d ("%s")', $budget->id, $budget->name));
-            $budget->spent    = $repository->spentInPeriod(new Collection([$budget]), $accounts, $start, $end);
-            $allRepetitions   = $repository->getAllBudgetLimitRepetitions($start, $end);
-            $otherRepetitions = new Collection;
-
-            /** @var LimitRepetition $repetition */
-            foreach ($allRepetitions as $repetition) {
-                if ($repetition->budget_id == $budget->id) {
-                    if ($repetition->budgetLimit->repeat_freq == $repeatFreq
-                        && $repetition->startdate->format('Y-m-d') == $startAsString
-                        && $repetition->enddate->format('Y-m-d') == $endAsString
-                    ) {
-                        // do something
-                        $budget->currentRep = $repetition;
-                        continue;
-                    }
-                    $otherRepetitions->push($repetition);
-                }
-            }
-            $budget->otherRepetitions = $otherRepetitions;
-
-            if (!is_null($budget->currentRep) && !is_null($budget->currentRep->id)) {
-                $budgeted = bcadd($budgeted, $budget->currentRep->amount);
-            }
-            $spent = bcadd($spent, $budget->spent);
-
-        }
-
-
-        $defaultCurrency = Amount::getDefaultCurrency();
+        $budgetInformation = $this->collectBudgetInformation($budgets, $start, $end);
+        $defaultCurrency   = Amount::getDefaultCurrency();
+        $available         = $this->repository->getAvailableBudget($defaultCurrency, $start, $end);
+        $spent             = array_sum(array_column($budgetInformation, 'spent'));
+        $budgeted          = array_sum(array_column($budgetInformation, 'budgeted'));
 
         return view(
-            'budgets.index', compact(
-                               'periodStart', 'periodEnd',
-                               'period', 'range', 'budgetIncomeTotal',
-                               'defaultCurrency', 'inactive', 'budgets',
-                               'spent', 'budgeted'
-                           )
+            'budgets.index',
+            compact('available', 'periodStart', 'periodEnd', 'budgetInformation', 'inactive', 'budgets', 'spent', 'budgeted')
         );
     }
 
     /**
+     * @param Request $request
+     *
      * @return View
      */
-    public function noBudget()
+    public function noBudget(Request $request)
     {
         /** @var Carbon $start */
         $start = session('start', Carbon::now()->startOfMonth());
         /** @var Carbon $end */
         $end      = session('end', Carbon::now()->endOfMonth());
-        $page     = intval(Input::get('page')) == 0 ? 1 : intval(Input::get('page'));
+        $page     = intval($request->get('page')) == 0 ? 1 : intval($request->get('page'));
         $pageSize = intval(Preferences::get('transactionPageSize', 50)->data);
         $subTitle = trans(
             'firefly.without_budget_between',
@@ -263,7 +211,8 @@ class BudgetController extends Controller
         );
 
         // collector
-        $collector = new JournalCollector(auth()->user());
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
         $collector->setAllAssetAccounts()->setRange($start, $end)->setLimit($pageSize)->setPage($page)->withoutBudget();
         $journals = $collector->getPaginatedJournals();
         $journals->setPath('/budgets/list/noBudget');
@@ -274,113 +223,105 @@ class BudgetController extends Controller
     /**
      * @return \Illuminate\Http\RedirectResponse
      */
-    public function postUpdateIncome()
+    public function postUpdateIncome(BudgetIncomeRequest $request)
     {
-        $range = Preferences::get('viewRange', '1M')->data;
-        /** @var Carbon $date */
-        $date  = session('start', new Carbon);
-        $start = Navigation::startOfPeriod($date, $range);
-        $end   = Navigation::endOfPeriod($start, $range);
-        $key   = 'budgetIncomeTotal' . $start->format('Ymd') . $end->format('Ymd');
+        $start           = session('start', new Carbon);
+        $end             = session('end', new Carbon);
+        $defaultCurrency = Amount::getDefaultCurrency();
+        $amount          = $request->get('amount');
 
-        Preferences::set($key, intval(Input::get('amount')));
+        $this->repository->setAvailableBudget($defaultCurrency, $start, $end, $amount);
         Preferences::mark();
 
         return redirect(route('budgets.index'));
     }
 
     /**
-     * @param BudgetRepositoryInterface  $repository
-     * @param AccountRepositoryInterface $accountRepository
-     * @param Budget                     $budget
+     * @param Request $request
+     * @param Budget  $budget
      *
      * @return View
      */
-    public function show(BudgetRepositoryInterface $repository, AccountRepositoryInterface $accountRepository, Budget $budget)
+    public function show(Request $request, Budget $budget)
     {
         /** @var Carbon $start */
         $start      = session('first', Carbon::create()->startOfYear());
         $end        = new Carbon;
-        $page       = intval(Input::get('page')) == 0 ? 1 : intval(Input::get('page'));
+        $page       = intval($request->get('page')) == 0 ? 1 : intval($request->get('page'));
         $pageSize   = intval(Preferences::get('transactionPageSize', 50)->data);
-        $accounts   = $accountRepository->getAccountsByType([AccountType::DEFAULT, AccountType::ASSET, AccountType::CASH]);
+        $limits     = $this->getLimits($budget, $start, $end);
         $repetition = null;
         // collector:
-        $collector = new JournalCollector(auth()->user());
-        $collector->setAllAssetAccounts()->setRange($start, $end)->setBudget($budget)->setLimit($pageSize)->setPage($page);
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
+        $collector->setAllAssetAccounts()->setRange($start, $end)->setBudget($budget)->setLimit($pageSize)->setPage($page)->withCategoryInformation();
         $journals = $collector->getPaginatedJournals();
         $journals->setPath('/budgets/show/' . $budget->id);
 
 
-        $set      = $budget->limitrepetitions()->orderBy('startdate', 'DESC')->get();
         $subTitle = e($budget->name);
-        $limits   = new Collection();
-
-        /** @var LimitRepetition $entry */
-        foreach ($set as $entry) {
-            $entry->spent = $repository->spentInPeriod(new Collection([$budget]), $accounts, $entry->startdate, $entry->enddate);
-            $limits->push($entry);
-        }
 
         return view('budgets.show', compact('limits', 'budget', 'repetition', 'journals', 'subTitle'));
     }
 
     /**
-     * @param Budget          $budget
-     * @param LimitRepetition $repetition
+     * @param Request     $request
+     * @param Budget      $budget
+     * @param BudgetLimit $budgetLimit
      *
      * @return View
      * @throws FireflyException
      */
-    public function showByRepetition(Budget $budget, LimitRepetition $repetition)
+    public function showByBudgetLimit(Request $request, Budget $budget, BudgetLimit $budgetLimit)
     {
-        if ($repetition->budgetLimit->budget->id != $budget->id) {
+        if ($budgetLimit->budget->id != $budget->id) {
             throw new FireflyException('This budget limit is not part of this budget.');
         }
 
-        /** @var BudgetRepositoryInterface $repository */
-        $repository = app(BudgetRepositoryInterface::class);
         /** @var AccountRepositoryInterface $accountRepository */
         $accountRepository = app(AccountRepositoryInterface::class);
-        $start             = $repetition->startdate;
-        $end               = $repetition->enddate;
-        $page              = intval(Input::get('page')) == 0 ? 1 : intval(Input::get('page'));
+        $page              = intval($request->get('page')) == 0 ? 1 : intval($request->get('page'));
         $pageSize          = intval(Preferences::get('transactionPageSize', 50)->data);
         $subTitle          = trans(
-            'firefly.budget_in_month', ['name' => $budget->name, 'month' => $repetition->startdate->formatLocalized($this->monthFormat)]
+            'firefly.budget_in_period', [
+                                          'name'  => $budget->name,
+                                          'start' => $budgetLimit->start_date->formatLocalized($this->monthAndDayFormat),
+                                          'end'   => $budgetLimit->end_date->formatLocalized($this->monthAndDayFormat),
+                                      ]
         );
         $accounts          = $accountRepository->getAccountsByType([AccountType::DEFAULT, AccountType::ASSET, AccountType::CASH]);
 
-
         // collector:
-        $collector = new JournalCollector(auth()->user());
-        $collector->setAllAssetAccounts()->setRange($start, $end)->setBudget($budget)->setLimit($pageSize)->setPage($page);
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
+        $collector->setAllAssetAccounts()->setRange($budgetLimit->start_date, $budgetLimit->end_date)
+                  ->setBudget($budget)->setLimit($pageSize)->setPage($page)->withCategoryInformation();
         $journals = $collector->getPaginatedJournals();
-        $journals->setPath('/budgets/show/' . $budget->id . '/' . $repetition->id);
+        $journals->setPath('/budgets/show/' . $budget->id . '/' . $budgetLimit->id);
 
 
-        $repetition->spent = $repository->spentInPeriod(new Collection([$budget]), $accounts, $repetition->startdate, $repetition->enddate);
-        $limits            = new Collection([$repetition]);
+        $start  = session('first', Carbon::create()->startOfYear());
+        $end    = new Carbon;
+        $limits = $this->getLimits($budget, $start, $end);
 
-        return view('budgets.show', compact('limits', 'budget', 'repetition', 'journals', 'subTitle'));
+        return view('budgets.show', compact('limits', 'budget', 'budgetLimit', 'journals', 'subTitle'));
 
     }
 
     /**
-     * @param BudgetFormRequest         $request
-     * @param BudgetRepositoryInterface $repository
+     * @param BudgetFormRequest $request
      *
      * @return \Illuminate\Http\RedirectResponse
      */
-    public function store(BudgetFormRequest $request, BudgetRepositoryInterface $repository)
+    public function store(BudgetFormRequest $request)
     {
         $data   = $request->getBudgetData();
-        $budget = $repository->store($data);
+        $budget = $this->repository->store($data);
 
         Session::flash('success', strval(trans('firefly.stored_new_budget', ['name' => e($budget->name)])));
         Preferences::mark();
 
-        if (intval(Input::get('create_another')) === 1) {
+        if (intval($request->get('create_another')) === 1) {
             // set value so create routine will not overwrite URL:
             Session::put('budgets.create.fromStore', true);
 
@@ -393,21 +334,20 @@ class BudgetController extends Controller
     }
 
     /**
-     * @param BudgetFormRequest         $request
-     * @param BudgetRepositoryInterface $repository
-     * @param Budget                    $budget
+     * @param BudgetFormRequest $request
+     * @param Budget            $budget
      *
      * @return \Illuminate\Http\RedirectResponse
      */
-    public function update(BudgetFormRequest $request, BudgetRepositoryInterface $repository, Budget $budget)
+    public function update(BudgetFormRequest $request, Budget $budget)
     {
         $data = $request->getBudgetData();
-        $repository->update($budget, $data);
+        $this->repository->update($budget, $data);
 
         Session::flash('success', strval(trans('firefly.updated_budget', ['name' => e($budget->name)])));
         Preferences::mark();
 
-        if (intval(Input::get('return_to_edit')) === 1) {
+        if (intval($request->get('return_to_edit')) === 1) {
             // set value so edit routine will not overwrite URL:
             Session::put('budgets.edit.fromUpdate', true);
 
@@ -424,19 +364,93 @@ class BudgetController extends Controller
      */
     public function updateIncome()
     {
-        $range  = Preferences::get('viewRange', '1M')->data;
-        $format = strval(trans('config.month_and_day'));
+        $start           = session('start', new Carbon);
+        $end             = session('end', new Carbon);
+        $defaultCurrency = Amount::getDefaultCurrency();
+        $available       = $this->repository->getAvailableBudget($defaultCurrency, $start, $end);
 
-        /** @var Carbon $date */
-        $date         = session('start', new Carbon);
-        $start        = Navigation::startOfPeriod($date, $range);
-        $end          = Navigation::endOfPeriod($start, $range);
-        $key          = 'budgetIncomeTotal' . $start->format('Ymd') . $end->format('Ymd');
-        $amount       = Preferences::get($key, 1000);
-        $displayStart = $start->formatLocalized($format);
-        $displayEnd   = $end->formatLocalized($format);
 
-        return view('budgets.income', compact('amount', 'displayStart', 'displayEnd'));
+        return view('budgets.income', compact('available', 'start', 'end'));
+    }
+
+    /**
+     * @param Collection $budgets
+     * @param Carbon     $start
+     * @param Carbon     $end
+     *
+     * @return array
+     */
+    private function collectBudgetInformation(Collection $budgets, Carbon $start, Carbon $end): array
+    {
+        // get account information
+        /** @var AccountRepositoryInterface $accountRepository */
+        $accountRepository = app(AccountRepositoryInterface::class);
+        $accounts          = $accountRepository->getAccountsByType([AccountType::DEFAULT, AccountType::ASSET, AccountType::CASH]);
+        $return            = [];
+        /** @var Budget $budget */
+        foreach ($budgets as $budget) {
+            $budgetId          = $budget->id;
+            $return[$budgetId] = [
+                'spent'      => $this->repository->spentInPeriod(new Collection([$budget]), $accounts, $start, $end),
+                'budgeted'   => '0',
+                'currentRep' => false,
+            ];
+            $budgetLimits      = $this->repository->getBudgetLimits($budget, $start, $end);
+            $otherLimits       = new Collection;
+
+            // get all the budget limits relevant between start and end and examine them:
+            /** @var BudgetLimit $limit */
+            foreach ($budgetLimits as $limit) {
+                if ($limit->start_date->isSameDay($start) && $limit->end_date->isSameDay($end)
+                ) {
+                    $return[$budgetId]['currentLimit'] = $limit;
+                    $return[$budgetId]['budgeted']     = $limit->amount;
+                    continue;
+                }
+                // otherwise it's just one of the many relevant repetitions:
+                $otherLimits->push($limit);
+            }
+            $return[$budgetId]['otherLimits'] = $otherLimits;
+        }
+
+        return $return;
+    }
+
+
+    /**
+     * @param Budget $budget
+     * @param Carbon $start
+     * @param Carbon $end
+     *
+     * @return Collection
+     */
+    private function getLimits(Budget $budget, Carbon $start, Carbon $end): Collection
+    {
+        // properties for cache
+        $cache = new CacheProperties;
+        $cache->addProperty($start);
+        $cache->addProperty($end);
+        $cache->addProperty($budget->id);
+        $cache->addProperty('get-limits');
+
+        if ($cache->has()) {
+            return $cache->get();
+        }
+
+        /** @var AccountRepositoryInterface $accountRepository */
+        $accountRepository = app(AccountRepositoryInterface::class);
+        $accounts          = $accountRepository->getAccountsByType([AccountType::DEFAULT, AccountType::ASSET, AccountType::CASH]);
+        $set               = $this->repository->getBudgetLimits($budget, $start, $end);
+        $limits            = new Collection();
+
+        /** @var BudgetLimit $entry */
+        foreach ($set as $entry) {
+            $entry->spent = $this->repository->spentInPeriod(new Collection([$budget]), $accounts, $entry->start_date, $entry->end_date);
+            $limits->push($entry);
+        }
+        $cache->store($limits);
+
+        return $set;
     }
 
 }

app/Http/Controllers/CategoryController.php

@@ -14,16 +14,15 @@ declare(strict_types = 1);
 namespace FireflyIII\Http\Controllers;
 
 use Carbon\Carbon;
-use FireflyIII\Helpers\Collector\JournalCollector;
 use FireflyIII\Helpers\Collector\JournalCollectorInterface;
 use FireflyIII\Http\Requests\CategoryFormRequest;
 use FireflyIII\Models\AccountType;
 use FireflyIII\Models\Category;
 use FireflyIII\Repositories\Account\AccountRepositoryInterface;
-use FireflyIII\Repositories\Category\CategoryRepositoryInterface as CRI;
+use FireflyIII\Repositories\Category\CategoryRepositoryInterface;
 use FireflyIII\Support\CacheProperties;
+use Illuminate\Http\Request;
 use Illuminate\Support\Collection;
-use Input;
 use Navigation;
 use Preferences;
 use Session;
@@ -89,22 +88,30 @@ class CategoryController extends Controller
         return view('categories.delete', compact('category', 'subTitle'));
     }
 
+
     /**
-     * @param CRI      $repository
-     * @param Category $category
+     * @param CategoryRepositoryInterface $repository
+     * @param Category                    $category
      *
-     * @return \Illuminate\Http\RedirectResponse
+     * @return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector
      */
-    public function destroy(CRI $repository, Category $category)
+    public function destroy(CategoryRepositoryInterface $repository, Category $category)
     {
 
-        $name = $category->name;
+        $name       = $category->name;
+        $categoryId = $category->id;
         $repository->destroy($category);
 
         Session::flash('success', strval(trans('firefly.deleted_category', ['name' => e($name)])));
         Preferences::mark();
 
-        return redirect(session('categories.delete.url'));
+        $uri = session('categories.delete.url');
+        if (!(strpos($uri, sprintf('categories/show/%s', $categoryId)) === false)) {
+            // uri would point back to category
+            $uri = route('categories.index');
+        }
+
+        return redirect($uri);
     }
 
     /**
@@ -129,11 +136,11 @@ class CategoryController extends Controller
     }
 
     /**
-     * @param CRI $repository
+     * @param CategoryRepositoryInterface $repository
      *
      * @return View
      */
-    public function index(CRI $repository)
+    public function index(CategoryRepositoryInterface $repository)
     {
         $categories = $repository->getCategories();
 
@@ -157,7 +164,8 @@ class CategoryController extends Controller
         $end = session('end', Carbon::now()->startOfMonth());
 
         // new collector:
-        $collector = new JournalCollector(auth()->user());
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
         $collector->setAllAssetAccounts()->setRange($start, $end)->withoutCategory();//->groupJournals();
         $journals = $collector->getJournals();
         $subTitle = trans(
@@ -169,110 +177,102 @@ class CategoryController extends Controller
     }
 
     /**
-     * @param CRI                        $repository
-     * @param AccountRepositoryInterface $accountRepository
-     * @param Category                   $category
+     * @param Request                   $request
+     * @param JournalCollectorInterface $collector
+     * @param Category                  $category
      *
      * @return View
      */
-    public function show(CRI $repository, AccountRepositoryInterface $accountRepository, Category $category)
+    public function show(Request $request, JournalCollectorInterface $collector, Category $category)
     {
-        $range = Preferences::get('viewRange', '1M')->data;
-        /** @var Carbon $start */
-        $start = session('start', Navigation::startOfPeriod(new Carbon, $range));
-        /** @var Carbon $end */
+        $range        = Preferences::get('viewRange', '1M')->data;
+        $start        = session('start', Navigation::startOfPeriod(new Carbon, $range));
         $end          = session('end', Navigation::endOfPeriod(new Carbon, $range));
-        $accounts     = $accountRepository->getAccountsByType([AccountType::DEFAULT, AccountType::ASSET]);
         $hideCategory = true; // used in list.
-        $page         = intval(Input::get('page')) === 0 ? 1 : intval(Input::get('page'));
+        $page         = intval($request->get('page')) === 0 ? 1 : intval($request->get('page'));
         $pageSize     = intval(Preferences::get('transactionPageSize', 50)->data);
         $subTitle     = $category->name;
         $subTitleIcon = 'fa-bar-chart';
+        $entries      = $this->getGroupedEntries($category);
+        $method       = 'default';
 
-        // use journal collector
-        $collector = app(JournalCollectorInterface::class);
-        $collector->setPage($page)->setLimit($pageSize)->setAllAssetAccounts()->setRange($start, $end)->setCategory($category);
+        // get journals
+        $collector->setLimit($pageSize)->setPage($page)->setAllAssetAccounts()->setRange($start, $end)->setCategory($category)->withBudgetInformation();
         $journals = $collector->getPaginatedJournals();
         $journals->setPath('categories/show/' . $category->id);
 
-        // oldest transaction in category:
+
+        return view('categories.show', compact('category', 'method', 'journals', 'entries', 'hideCategory', 'subTitle', 'subTitleIcon', 'start', 'end'));
+    }
+
+    /**
+     * @param Request                     $request
+     * @param CategoryRepositoryInterface $repository
+     * @param Category                    $category
+     *
+     * @return View
+     */
+    public function showAll(Request $request, CategoryRepositoryInterface $repository, Category $category)
+    {
+        $range = Preferences::get('viewRange', '1M')->data;
         $start = $repository->firstUseDate($category);
         if ($start->year == 1900) {
             $start = new Carbon;
         }
-        $range   = Preferences::get('viewRange', '1M')->data;
-        $start   = Navigation::startOfPeriod($start, $range);
-        $end     = Navigation::endOfX(new Carbon, $range);
-        $entries = new Collection;
+        $end          = Navigation::endOfPeriod(new Carbon, $range);
+        $subTitle     = $category->name;
+        $subTitleIcon = 'fa-bar-chart';
+        $hideCategory = true; // used in list.
+        $page         = intval($request->get('page')) === 0 ? 1 : intval($request->get('page'));
+        $pageSize     = intval(Preferences::get('transactionPageSize', 50)->data);
+        $method       = 'all';
 
-        // chart properties for cache:
-        $cache = new CacheProperties();
-        $cache->addProperty($start);
-        $cache->addProperty($end);
-        $cache->addProperty('category-show');
-        $cache->addProperty($category->id);
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class);
+        $collector->setLimit($pageSize)->setPage($page)->setAllAssetAccounts()->setCategory($category)->withBudgetInformation();
+        $journals = $collector->getPaginatedJournals();
+        $journals->setPath('categories/show/' . $category->id . '/all');
 
-
-        if ($cache->has()) {
-            $entries = $cache->get();
-
-            return view('categories.show', compact('category', 'journals', 'entries', 'subTitleIcon', 'hideCategory', 'subTitle'));
-        }
-
-
-        $categoryCollection = new Collection([$category]);
-
-        while ($end >= $start) {
-            $end        = Navigation::startOfPeriod($end, $range);
-            $currentEnd = Navigation::endOfPeriod($end, $range);
-            $spent      = $repository->spentInPeriod($categoryCollection, $accounts, $end, $currentEnd);
-            $earned     = $repository->earnedInPeriod($categoryCollection, $accounts, $end, $currentEnd);
-            $dateStr    = $end->format('Y-m-d');
-            $dateName   = Navigation::periodShow($end, $range);
-            $entries->push([$dateStr, $dateName, $spent, $earned]);
-
-            $end = Navigation::subtractPeriod($end, $range, 1);
-
-        }
-        $cache->store($entries);
-
-        return view('categories.show', compact('category', 'journals', 'entries', 'hideCategory', 'subTitle', 'subTitleIcon'));
+        return view('categories.show', compact('category', 'method', 'journals', 'hideCategory', 'subTitle', 'subTitleIcon', 'start', 'end'));
     }
 
     /**
+     * @param Request  $request
      * @param Category $category
-     * @param          $date
+     * @param string   $date
      *
      * @return View
      */
-    public function showByDate(Category $category, string $date)
+    public function showByDate(Request $request, Category $category, string $date)
     {
         $carbon       = new Carbon($date);
         $range        = Preferences::get('viewRange', '1M')->data;
         $start        = Navigation::startOfPeriod($carbon, $range);
         $end          = Navigation::endOfPeriod($carbon, $range);
         $subTitle     = $category->name;
+        $subTitleIcon = 'fa-bar-chart';
         $hideCategory = true; // used in list.
-        $page         = intval(Input::get('page')) === 0 ? 1 : intval(Input::get('page'));
+        $page         = intval($request->get('page')) === 0 ? 1 : intval($request->get('page'));
         $pageSize     = intval(Preferences::get('transactionPageSize', 50)->data);
+        $entries      = $this->getGroupedEntries($category);
+        $method       = 'date';
 
-        // new collector:
+        /** @var JournalCollectorInterface $collector */
         $collector = app(JournalCollectorInterface::class);
-        $collector->setPage($page)->setLimit($pageSize)->setAllAssetAccounts()->setRange($start, $end)->setCategory($category);
+        $collector->setLimit($pageSize)->setPage($page)->setAllAssetAccounts()->setRange($start, $end)->setCategory($category)->withBudgetInformation();
         $journals = $collector->getPaginatedJournals();
         $journals->setPath('categories/show/' . $category->id . '/' . $date);
 
-
-        return view('categories.show-by-date', compact('category', 'journals', 'hideCategory', 'subTitle', 'carbon'));
+        return view('categories.show', compact('category', 'method', 'entries', 'journals', 'hideCategory', 'subTitle', 'subTitleIcon', 'start', 'end'));
     }
 
     /**
-     * @param CategoryFormRequest $request
-     * @param CRI                 $repository
+     * @param CategoryFormRequest         $request
+     * @param CategoryRepositoryInterface $repository
      *
-     * @return \Illuminate\Http\RedirectResponse
+     * @return $this|\Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector
      */
-    public function store(CategoryFormRequest $request, CRI $repository)
+    public function store(CategoryFormRequest $request, CategoryRepositoryInterface $repository)
     {
         $data     = $request->getCategoryData();
         $category = $repository->store($data);
@@ -280,7 +280,7 @@ class CategoryController extends Controller
         Session::flash('success', strval(trans('firefly.stored_category', ['name' => e($category->name)])));
         Preferences::mark();
 
-        if (intval(Input::get('create_another')) === 1) {
+        if (intval($request->get('create_another')) === 1) {
             Session::put('categories.create.fromStore', true);
 
             return redirect(route('categories.create'))->withInput();
@@ -291,13 +291,13 @@ class CategoryController extends Controller
 
 
     /**
-     * @param CategoryFormRequest $request
-     * @param CRI                 $repository
-     * @param Category            $category
+     * @param CategoryFormRequest         $request
+     * @param CategoryRepositoryInterface $repository
+     * @param Category                    $category
      *
-     * @return \Illuminate\Http\RedirectResponse
+     * @return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector
      */
-    public function update(CategoryFormRequest $request, CRI $repository, Category $category)
+    public function update(CategoryFormRequest $request, CategoryRepositoryInterface $repository, Category $category)
     {
         $data = $request->getCategoryData();
         $repository->update($category, $data);
@@ -305,7 +305,7 @@ class CategoryController extends Controller
         Session::flash('success', strval(trans('firefly.updated_category', ['name' => e($category->name)])));
         Preferences::mark();
 
-        if (intval(Input::get('return_to_edit')) === 1) {
+        if (intval($request->get('return_to_edit')) === 1) {
             Session::put('categories.edit.fromUpdate', true);
 
             return redirect(route('categories.edit', [$category->id]));
@@ -316,4 +316,50 @@ class CategoryController extends Controller
 
     }
 
+    /**
+     * @param Category $category
+     *
+     * @return Collection
+     */
+    private function getGroupedEntries(Category $category): Collection
+    {
+        /** @var CategoryRepositoryInterface $repository */
+        $repository = app(CategoryRepositoryInterface::class);
+        /** @var AccountRepositoryInterface $accountRepository */
+        $accountRepository = app(AccountRepositoryInterface::class);
+        $accounts          = $accountRepository->getAccountsByType([AccountType::DEFAULT, AccountType::ASSET]);
+        $first             = $repository->firstUseDate($category);
+        if ($first->year == 1900) {
+            $first = new Carbon;
+        }
+        $range   = Preferences::get('viewRange', '1M')->data;
+        $first   = Navigation::startOfPeriod($first, $range);
+        $end     = Navigation::endOfX(new Carbon, $range);
+        $entries = new Collection;
+
+        // properties for entries with their amounts.
+        $cache = new CacheProperties();
+        $cache->addProperty($first);
+        $cache->addProperty($end);
+        $cache->addProperty('categories.entries');
+        $cache->addProperty($category->id);
+
+        if ($cache->has()) {
+            return $cache->get();
+        }
+        while ($end >= $first) {
+            $end        = Navigation::startOfPeriod($end, $range);
+            $currentEnd = Navigation::endOfPeriod($end, $range);
+            $spent      = $repository->spentInPeriod(new Collection([$category]), $accounts, $end, $currentEnd);
+            $earned     = $repository->earnedInPeriod(new Collection([$category]), $accounts, $end, $currentEnd);
+            $dateStr    = $end->format('Y-m-d');
+            $dateName   = Navigation::periodShow($end, $range);
+            $entries->push([$dateStr, $dateName, $spent, $earned, clone $end]);
+            $end = Navigation::subtractPeriod($end, $range, 1);
+        }
+        $cache->store($entries);
+
+        return $entries;
+    }
+
 }

app/Http/Controllers/Chart/AccountController.php

@@ -128,7 +128,7 @@ class AccountController extends Controller
             $endBalance   = $endBalances[$id] ?? '0';
             $diff         = bcsub($endBalance, $startBalance);
             if (bccomp($diff, '0') !== 0) {
-                $chartData[$account->name] = round($diff, 2);
+                $chartData[$account->name] = $diff;
             }
         }
         arsort($chartData);
@@ -391,7 +391,7 @@ class AccountController extends Controller
             $diff         = bcsub($endBalance, $startBalance);
             $diff         = bcmul($diff, '-1');
             if (bccomp($diff, '0') !== 0) {
-                $chartData[$account->name] = round($diff, 2);
+                $chartData[$account->name] = $diff;
             }
         }
 
@@ -475,11 +475,11 @@ class AccountController extends Controller
             ];
             $currentStart = clone $start;
             $range        = Steam::balanceInRange($account, $start, clone $end);
-            $previous     = round(array_values($range)[0], 2);
+            $previous     = array_values($range)[0];
             while ($currentStart <= $end) {
                 $format   = $currentStart->format('Y-m-d');
                 $label    = $currentStart->formatLocalized(strval(trans('config.month_and_day')));
-                $balance  = isset($range[$format]) ? round($range[$format], 2) : $previous;
+                $balance  = isset($range[$format]) ? round($range[$format], 12) : $previous;
                 $previous = $balance;
                 $currentStart->addDay();
                 $currentSet['entries'][$label] = $balance;

app/Http/Controllers/Chart/BillController.php

@@ -91,40 +91,24 @@ class BillController extends Controller
             return Response::json($cache->get());
         }
 
-        $results = $collector->setAllAssetAccounts()->setBills(new Collection([$bill]))->getJournals();
-        $results = $results->sortBy(
+        $results   = $collector->setAllAssetAccounts()->setBills(new Collection([$bill]))->getJournals();
+        $results   = $results->sortBy(
             function (Transaction $transaction) {
                 return $transaction->date->format('U');
             }
         );
-
         $chartData = [
-            [
-                'type'    => 'bar',
-                'label'   => trans('firefly.min-amount'),
-                'entries' => [],
-            ],
-            [
-                'type'    => 'bar',
-                'label'   => trans('firefly.max-amount'),
-                'entries' => [],
-            ],
-            [
-                'type'    => 'line',
-                'label'   => trans('firefly.journal-amount'),
-                'entries' => [],
-            ],
+            ['type' => 'bar', 'label' => trans('firefly.min-amount'), 'entries' => [],],
+            ['type' => 'bar', 'label' => trans('firefly.max-amount'), 'entries' => [],],
+            ['type' => 'line', 'label' => trans('firefly.journal-amount'), 'entries' => [],],
         ];
 
         /** @var Transaction $entry */
         foreach ($results as $entry) {
-            $date = $entry->date->formatLocalized(strval(trans('config.month_and_day')));
-            // minimum amount of bill:
-            $chartData[0]['entries'][$date] = $bill->amount_min;
-            // maximum amount of bill:
-            $chartData[1]['entries'][$date] = $bill->amount_max;
-            // amount of journal:
-            $chartData[2]['entries'][$date] = bcmul($entry->transaction_amount, '-1');
+            $date                           = $entry->date->formatLocalized(strval(trans('config.month_and_day')));
+            $chartData[0]['entries'][$date] = $bill->amount_min; // minimum amount of bill
+            $chartData[1]['entries'][$date] = $bill->amount_max; // maximum amount of bill
+            $chartData[2]['entries'][$date] = bcmul($entry->transaction_amount, '-1'); // amount of journal
         }
 
         $data = $this->generator->multiSet($chartData);

app/Http/Controllers/Chart/BudgetController.php

@@ -14,11 +14,12 @@ declare(strict_types = 1);
 namespace FireflyIII\Http\Controllers\Chart;
 
 use Carbon\Carbon;
+use FireflyIII\Exceptions\FireflyException;
 use FireflyIII\Generator\Chart\Basic\GeneratorInterface;
-use FireflyIII\Helpers\Collector\JournalCollector;
+use FireflyIII\Helpers\Collector\JournalCollectorInterface;
 use FireflyIII\Http\Controllers\Controller;
 use FireflyIII\Models\Budget;
-use FireflyIII\Models\LimitRepetition;
+use FireflyIII\Models\BudgetLimit;
 use FireflyIII\Models\Transaction;
 use FireflyIII\Models\TransactionType;
 use FireflyIII\Repositories\Budget\BudgetRepositoryInterface;
@@ -31,6 +32,8 @@ use Response;
 /**
  * Class BudgetController
  *
+ * @SuppressWarnings(PHPMD.CouplingBetweenObjects) // can't realy be helped.
+ *
  * @package FireflyIII\Http\Controllers\Chart
  */
 class BudgetController extends Controller
@@ -39,26 +42,35 @@ class BudgetController extends Controller
     /** @var GeneratorInterface */
     protected $generator;
 
+    /** @var  BudgetRepositoryInterface */
+    protected $repository;
+
     /**
      * BudgetController constructor.
      */
     public function __construct()
     {
         parent::__construct();
-        $this->generator = app(GeneratorInterface::class);
+
+        $this->middleware(
+            function ($request, $next) {
+                $this->generator  = app(GeneratorInterface::class);
+                $this->repository = app(BudgetRepositoryInterface::class);
+
+                return $next($request);
+            }
+        );
     }
 
     /**
-     * checked
      *
-     * @param BudgetRepositoryInterface $repository
-     * @param Budget                    $budget
+     * @param Budget $budget
      *
      * @return \Symfony\Component\HttpFoundation\Response
      */
-    public function budget(BudgetRepositoryInterface $repository, Budget $budget)
+    public function budget(Budget $budget)
     {
-        $first = $repository->firstUseDate($budget);
+        $first = $this->repository->firstUseDate($budget);
         $range = Preferences::get('viewRange', '1M')->data;
         $last  = session('end', new Carbon);
 
@@ -73,7 +85,6 @@ class BudgetController extends Controller
 
         $final = clone $last;
         $final->addYears(2);
-
         $budgetCollection = new Collection([$budget]);
         $last             = Navigation::endOfX($last, $range, $final); // not to overshoot.
         $entries          = [];
@@ -84,7 +95,7 @@ class BudgetController extends Controller
             $currentEnd   = Navigation::endOfPeriod($first, $range);
             // sub another day because reasons.
             $currentEnd->subDay();
-            $spent            = $repository->spentInPeriod($budgetCollection, new Collection, $currentStart, $currentEnd);
+            $spent            = $this->repository->spentInPeriod($budgetCollection, new Collection, $currentStart, $currentEnd);
             $format           = Navigation::periodShow($first, $range);
             $entries[$format] = bcmul($spent, '-1');
             $first            = Navigation::addPeriod($first, $range, 0);
@@ -100,31 +111,36 @@ class BudgetController extends Controller
     /**
      * Shows the amount left in a specific budget limit.
      *
-     * @param BudgetRepositoryInterface $repository
-     * @param Budget                    $budget
-     * @param LimitRepetition           $repetition
+     * @SuppressWarnings(PHPMD.CyclomaticComplexity) // it's exactly five.
+     * @param Budget      $budget
+     * @param BudgetLimit $budgetLimit
      *
      * @return \Symfony\Component\HttpFoundation\Response
+     * @throws FireflyException
      */
-    public function budgetLimit(BudgetRepositoryInterface $repository, Budget $budget, LimitRepetition $repetition)
+    public function budgetLimit(Budget $budget, BudgetLimit $budgetLimit)
     {
-        $start = clone $repetition->startdate;
-        $end   = $repetition->enddate;
+        if ($budgetLimit->budget->id != $budget->id) {
+            throw new FireflyException('This budget limit is not part of this budget.');
+        }
+
+        $start = clone $budgetLimit->start_date;
+        $end   = clone $budgetLimit->end_date;
         $cache = new CacheProperties();
         $cache->addProperty($start);
         $cache->addProperty($end);
         $cache->addProperty('chart.budget.budget.limit');
-        $cache->addProperty($repetition->id);
+        $cache->addProperty($budgetLimit->id);
 
         if ($cache->has()) {
             return Response::json($cache->get());
         }
 
         $entries          = [];
-        $amount           = $repetition->amount;
+        $amount           = $budgetLimit->amount;
         $budgetCollection = new Collection([$budget]);
         while ($start <= $end) {
-            $spent            = $repository->spentInPeriod($budgetCollection, new Collection, $start, $start);
+            $spent            = $this->repository->spentInPeriod($budgetCollection, new Collection, $start, $start);
             $amount           = bcadd($amount, $spent);
             $format           = $start->formatLocalized(strval(trans('config.month_and_day')));
             $entries[$format] = $amount;
@@ -139,12 +155,12 @@ class BudgetController extends Controller
 
     /**
      * Shows a budget list with spent/left/overspent.
-     *
-     * @param BudgetRepositoryInterface $repository
+     * @SuppressWarnings(PHPMD.CyclomaticComplexity) // it's exactly five.
+     * @SuppressWarnings(PHPMD.ExcessiveMethodLength) // 46 lines, I'm fine with this.
      *
      * @return \Symfony\Component\HttpFoundation\Response
      */
-    public function frontpage(BudgetRepositoryInterface $repository)
+    public function frontpage()
     {
         $start = session('start', Carbon::now()->startOfMonth());
         $end   = session('end', Carbon::now()->endOfMonth());
@@ -156,58 +172,32 @@ class BudgetController extends Controller
         if ($cache->has()) {
             return Response::json($cache->get());
         }
-        $budgets     = $repository->getActiveBudgets();
-        $repetitions = $repository->getAllBudgetLimitRepetitions($start, $end);
-        $chartData   = [
-            [
-                'label'   => strval(trans('firefly.spent_in_budget')),
-                'entries' => [],
-                'type'    => 'bar',
-            ],
-            [
-                'label'   => strval(trans('firefly.left_to_spend')),
-                'entries' => [],
-                'type'    => 'bar',
-            ],
-            [
-                'label'   => strval(trans('firefly.overspent')),
-                'entries' => [],
-                'type'    => 'bar',
-            ],
+        $budgets   = $this->repository->getActiveBudgets();
+        $chartData = [
+            ['label' => strval(trans('firefly.spent_in_budget')), 'entries' => [], 'type' => 'bar',],
+            ['label' => strval(trans('firefly.left_to_spend')), 'entries' => [], 'type' => 'bar',],
+            ['label' => strval(trans('firefly.overspent')), 'entries' => [], 'type' => 'bar',],
         ];
 
 
         /** @var Budget $budget */
         foreach ($budgets as $budget) {
             // get relevant repetitions:
-            $reps = $this->filterRepetitions($repetitions, $budget, $start, $end);
-
-            if ($reps->count() === 0) {
-                $row = $this->spentInPeriodSingle($repository, $budget, $start, $end);
-                if (bccomp($row['spent'], '0') !== 0 || bccomp($row['repetition_left'], '0') !== 0) {
-                    $chartData[0]['entries'][$row['name']] = bcmul($row['spent'], '-1');
-                    $chartData[1]['entries'][$row['name']] = $row['repetition_left'];
-                    $chartData[2]['entries'][$row['name']] = bcmul($row['repetition_overspent'], '-1');
-                }
-                continue;
+            $limits   = $this->repository->getBudgetLimits($budget, $start, $end);
+            $expenses = $this->getExpensesForBudget($limits, $budget, $start, $end);
+            foreach ($expenses as $name => $row) {
+                $chartData[0]['entries'][$name] = $row['spent'];
+                $chartData[1]['entries'][$name] = $row['left'];
+                $chartData[2]['entries'][$name] = $row['overspent'];
             }
-            $rows = $this->spentInPeriodMulti($repository, $budget, $reps);
-            foreach ($rows as $row) {
-                if (bccomp($row['spent'], '0') !== 0 || bccomp($row['repetition_left'], '0') !== 0) {
-                    $chartData[0]['entries'][$row['name']] = bcmul($row['spent'], '-1');
-                    $chartData[1]['entries'][$row['name']] = $row['repetition_left'];
-                    $chartData[2]['entries'][$row['name']] = bcmul($row['repetition_overspent'], '-1');
-                }
-            }
-            unset($rows, $row);
-
         }
         // for no budget:
-        $row = $this->spentInPeriodWithout($start, $end);
-        if (bccomp($row['spent'], '0') !== 0 || bccomp($row['repetition_left'], '0') !== 0) {
-            $chartData[0]['entries'][$row['name']] = bcmul($row['spent'], '-1');
-            $chartData[1]['entries'][$row['name']] = $row['repetition_left'];
-            $chartData[2]['entries'][$row['name']] = bcmul($row['repetition_overspent'], '-1');
+        $spent = $this->spentInPeriodWithout($start, $end);
+        $name  = strval(trans('firefly.no_budget'));
+        if (bccomp($spent, '0') !== 0) {
+            $chartData[0]['entries'][$name] = bcmul($spent, '-1');
+            $chartData[1]['entries'][$name] = '0';
+            $chartData[2]['entries'][$name] = '0';
         }
 
         $data = $this->generator->multiSet($chartData);
@@ -218,15 +208,16 @@ class BudgetController extends Controller
 
 
     /**
-     * @param BudgetRepositoryInterface $repository
-     * @param Budget                    $budget
-     * @param Carbon                    $start
-     * @param Carbon                    $end
-     * @param Collection                $accounts
+     * @SuppressWarnings(PHPMD.CyclomaticComplexity) // it's exactly five.
+     *
+     * @param Budget     $budget
+     * @param Carbon     $start
+     * @param Carbon     $end
+     * @param Collection $accounts
      *
      * @return \Illuminate\Http\JsonResponse
      */
-    public function period(BudgetRepositoryInterface $repository, Budget $budget, Collection $accounts, Carbon $start, Carbon $end)
+    public function period(Budget $budget, Collection $accounts, Carbon $start, Carbon $end)
     {
         // chart properties for cache:
         $cache = new CacheProperties();
@@ -238,56 +229,22 @@ class BudgetController extends Controller
         if ($cache->has()) {
             return Response::json($cache->get());
         }
-
-        // get the expenses
-        $budgeted = [];
         $periods  = Navigation::listOfPeriods($start, $end);
-        $entries  = $repository->getBudgetPeriodReport(new Collection([$budget]), $accounts, $start, $end);
-        $key      = Navigation::preferredCarbonFormat($start, $end);
-        $range    = Navigation::preferredRangeFormat($start, $end);
-
-        // get the budget limits (if any)
-        $repetitions = $repository->getAllBudgetLimitRepetitions($start, $end);
-        $current     = clone $start;
-        while ($current < $end) {
-            $currentStart     = Navigation::startOfPeriod($current, $range);
-            $currentEnd       = Navigation::endOfPeriod($current, $range);
-            $reps             = $repetitions->filter(
-                function (LimitRepetition $repetition) use ($budget, $currentStart, $currentEnd) {
-                    if ($repetition->budget_id === $budget->id && $repetition->startdate >= $currentStart && $repetition->enddate <= $currentEnd) {
-                        return true;
-                    }
-
-                    return false;
-                }
-            );
-            $index            = $currentStart->format($key);
-            $budgeted[$index] = $reps->sum('amount');
-            $currentEnd->addDay();
-            $current = clone $currentEnd;
-        }
+        $entries  = $this->repository->getBudgetPeriodReport(new Collection([$budget]), $accounts, $start, $end); // get the expenses
+        $budgeted = $this->getBudgetedInPeriod($budget, $start, $end);
 
         // join them into one set of data:
         $chartData = [
-            [
-                'label'   => strval(trans('firefly.spent')),
-                'type'    => 'bar',
-                'entries' => [],
-            ],
-            [
-                'label'   => strval(trans('firefly.budgeted')),
-                'type'    => 'bar',
-                'entries' => [],
-            ],
+            ['label' => strval(trans('firefly.spent')), 'type' => 'bar', 'entries' => [],],
+            ['label' => strval(trans('firefly.budgeted')), 'type' => 'bar', 'entries' => [],],
         ];
 
         foreach (array_keys($periods) as $period) {
             $label                           = $periods[$period];
             $spent                           = isset($entries[$budget->id]['entries'][$period]) ? $entries[$budget->id]['entries'][$period] : '0';
-            $limit                           = isset($entries[$period]) ? $budgeted[$period] : 0;
-            $chartData[0]['entries'][$label] = bcmul($spent, '-1');
+            $limit                           = isset($budgeted[$period]) ? $budgeted[$period] : 0;
+            $chartData[0]['entries'][$label] = round(bcmul($spent, '-1'), 12);
             $chartData[1]['entries'][$label] = $limit;
-
         }
         $data = $this->generator->multiSet($chartData);
         $cache->store($data);
@@ -296,14 +253,13 @@ class BudgetController extends Controller
     }
 
     /**
-     * @param BudgetRepositoryInterface $repository
-     * @param Collection                $accounts
-     * @param Carbon                    $start
-     * @param Carbon                    $end
+     * @param Collection $accounts
+     * @param Carbon     $start
+     * @param Carbon     $end
      *
      * @return \Illuminate\Http\JsonResponse
      */
-    public function periodNoBudget(BudgetRepositoryInterface $repository, Collection $accounts, Carbon $start, Carbon $end)
+    public function periodNoBudget(Collection $accounts, Carbon $start, Carbon $end)
     {
         // chart properties for cache:
         $cache = new CacheProperties();
@@ -317,7 +273,7 @@ class BudgetController extends Controller
 
         // the expenses:
         $periods   = Navigation::listOfPeriods($start, $end);
-        $entries   = $repository->getNoBudgetPeriodReport($accounts, $start, $end);
+        $entries   = $this->repository->getNoBudgetPeriodReport($accounts, $start, $end);
         $chartData = [];
 
         // join them:
@@ -333,98 +289,115 @@ class BudgetController extends Controller
     }
 
     /**
-     * @param Collection $repetitions
+     * @param Budget $budget
+     * @param Carbon $start
+     * @param Carbon $end
+     *
+     * @return array
+     */
+    private function getBudgetedInPeriod(Budget $budget, Carbon $start, Carbon $end): array
+    {
+        $key      = Navigation::preferredCarbonFormat($start, $end);
+        $range    = Navigation::preferredRangeFormat($start, $end);
+        $current  = clone $start;
+        $budgeted = [];
+        while ($current < $end) {
+            $currentStart     = Navigation::startOfPeriod($current, $range);
+            $currentEnd       = Navigation::endOfPeriod($current, $range);
+            $budgetLimits     = $this->repository->getBudgetLimits($budget, $currentStart, $currentEnd);
+            $index            = $currentStart->format($key);
+            $budgeted[$index] = $budgetLimits->sum('amount');
+            $currentEnd->addDay();
+            $current = clone $currentEnd;
+        }
+
+        return $budgeted;
+    }
+
+    /**
+     *
+     * @SuppressWarnings(PHPMD.CyclomaticComplexity) // it's 6 but ok.
+     *
+     * @param Collection $limits
      * @param Budget     $budget
      * @param Carbon     $start
      * @param Carbon     $end
      *
-     * @return Collection
-     */
-    private function filterRepetitions(Collection $repetitions, Budget $budget, Carbon $start, Carbon $end): Collection
-    {
-
-        return $repetitions->filter(
-            function (LimitRepetition $repetition) use ($budget, $start, $end) {
-                if ($repetition->startdate < $end && $repetition->enddate > $start && $repetition->budget_id === $budget->id) {
-                    return true;
-                }
-
-                return false;
-            }
-        );
-    }
-
-    /**
-     * Returns an array with the following values:
-     * 0 =>
-     *   'name' => name of budget + repetition
-     *   'repetition_left' => left in budget repetition (always zero)
-     *   'repetition_overspent' => spent more than budget repetition? (always zero)
-     *   'spent' => actually spent in period for budget
-     * 1 => (etc)
-     *
-     * @param BudgetRepositoryInterface $repository
-     * @param Budget                    $budget
-     * @param Collection                $repetitions
-     *
      * @return array
      */
-    private function spentInPeriodMulti(BudgetRepositoryInterface $repository, Budget $budget, Collection $repetitions): array
+    private function getExpensesForBudget(Collection $limits, Budget $budget, Carbon $start, Carbon $end): array
     {
         $return = [];
-        $format = strval(trans('config.month_and_day'));
-        $name   = $budget->name;
-        /** @var LimitRepetition $repetition */
-        foreach ($repetitions as $repetition) {
-            $expenses = $repository->spentInPeriod(new Collection([$budget]), new Collection, $repetition->startdate, $repetition->enddate);
-
-            if ($repetitions->count() > 1) {
-                $name = $budget->name . ' ' . trans(
-                        'firefly.between_dates',
-                        ['start' => $repetition->startdate->formatLocalized($format), 'end' => $repetition->enddate->formatLocalized($format)]
-                    );
+        if ($limits->count() === 0) {
+            $spent = $this->repository->spentInPeriod(new Collection([$budget]), new Collection, $start, $end);
+            if (bccomp($spent, '0') !== 0) {
+                $return[$budget->name]['spent']     = bcmul($spent, '-1');
+                $return[$budget->name]['left']      = 0;
+                $return[$budget->name]['overspent'] = 0;
             }
-            $amount    = $repetition->amount;
-            $left      = bccomp(bcadd($amount, $expenses), '0') < 1 ? '0' : bcadd($amount, $expenses);
-            $spent     = bccomp(bcadd($amount, $expenses), '0') < 1 ? bcmul($amount, '-1') : $expenses;
-            $overspent = bccomp(bcadd($amount, $expenses), '0') < 1 ? bcadd($amount, $expenses) : '0';
-            $return[]  = [
-                'name'                 => $name,
-                'repetition_left'      => $left,
-                'repetition_overspent' => $overspent,
-                'spent'                => $spent,
-            ];
+
+            return $return;
         }
 
+        $rows = $this->spentInPeriodMulti($budget, $limits);
+        foreach ($rows as $name => $row) {
+            if (bccomp($row['spent'], '0') !== 0 || bccomp($row['left'], '0') !== 0) {
+                $return[$name]['spent']     = bcmul($row['spent'], '-1');
+                $return[$name]['left']      = $row['left'];
+                $return[$name]['overspent'] = bcmul($row['overspent'], '-1');
+            }
+        }
+        unset($rows, $row);
+
         return $return;
     }
 
     /**
+     * @SuppressWarnings(PHPMD.CyclomaticComplexity) // it's exactly five.
+     *
      * Returns an array with the following values:
-     * 'name' => name of budget
-     * 'repetition_left' => left in budget repetition (always zero)
-     * 'repetition_overspent' => spent more than budget repetition? (always zero)
-     * 'spent' => actually spent in period for budget
+     * 0 =>
+     *   'name' => name of budget + repetition
+     *   'left' => left in budget repetition (always zero)
+     *   'overspent' => spent more than budget repetition? (always zero)
+     *   'spent' => actually spent in period for budget
+     * 1 => (etc)
      *
-     *
-     * @param BudgetRepositoryInterface $repository
-     * @param Budget                    $budget
-     * @param Carbon                    $start
-     * @param Carbon                    $end
+     * @param Budget     $budget
+     * @param Collection $limits
      *
      * @return array
      */
-    private function spentInPeriodSingle(BudgetRepositoryInterface $repository, Budget $budget, Carbon $start, Carbon $end): array
+    private function spentInPeriodMulti(Budget $budget, Collection $limits): array
     {
-        $spent = $repository->spentInPeriod(new Collection([$budget]), new Collection, $start, $end);
-        $array = [
-            'name'                 => $budget->name,
-            'repetition_left'      => '0',
-            'repetition_overspent' => '0',
-            'spent'                => $spent,
-        ];
+        $return = [];
+        $format = strval(trans('config.month_and_day'));
+        $name   = $budget->name;
+        /** @var BudgetLimit $budgetLimit */
+        foreach ($limits as $budgetLimit) {
+            $expenses = $this->repository->spentInPeriod(new Collection([$budget]), new Collection, $budgetLimit->start_date, $budgetLimit->end_date);
 
-        return $array;
+            if ($limits->count() > 1) {
+                $name = $budget->name . ' ' . trans(
+                        'firefly.between_dates',
+                        [
+                            'start' => $budgetLimit->start_date->formatLocalized($format),
+                            'end'   => $budgetLimit->end_date->formatLocalized($format),
+                        ]
+                    );
+            }
+            $amount        = $budgetLimit->amount;
+            $left          = bccomp(bcadd($amount, $expenses), '0') < 1 ? '0' : bcadd($amount, $expenses);
+            $spent         = $expenses;
+            $overspent     = bccomp(bcadd($amount, $expenses), '0') < 1 ? bcadd($amount, $expenses) : '0';
+            $return[$name] = [
+                'left'      => $left,
+                'overspent' => $overspent,
+                'spent'     => $spent,
+            ];
+        }
+
+        return $return;
     }
 
     /**
@@ -437,12 +410,13 @@ class BudgetController extends Controller
      * @param Carbon $start
      * @param Carbon $end
      *
-     * @return array
+     * @return string
      */
-    private function spentInPeriodWithout(Carbon $start, Carbon $end): array
+    private function spentInPeriodWithout(Carbon $start, Carbon $end): string
     {
         // collector
-        $collector = new JournalCollector(auth()->user());
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
         $types     = [TransactionType::WITHDRAWAL];
         $collector->setAllAssetAccounts()->setTypes($types)->setRange($start, $end)->withoutBudget();
         $journals = $collector->getJournals();
@@ -451,13 +425,7 @@ class BudgetController extends Controller
         foreach ($journals as $entry) {
             $sum = bcadd($entry->transaction_amount, $sum);
         }
-        $array = [
-            'name'                 => strval(trans('firefly.no_budget')),
-            'repetition_left'      => '0',
-            'repetition_overspent' => '0',
-            'spent'                => $sum,
-        ];
 
-        return $array;
+        return $sum;
     }
 }

app/Http/Controllers/Chart/BudgetReportController.php

@@ -17,9 +17,11 @@ namespace FireflyIII\Http\Controllers\Chart;
 use Carbon\Carbon;
 use FireflyIII\Generator\Chart\Basic\GeneratorInterface;
 use FireflyIII\Generator\Report\Category\MonthReportGenerator;
-use FireflyIII\Helpers\Collector\JournalCollector;
+use FireflyIII\Helpers\Chart\MetaPieChartInterface;
+use FireflyIII\Helpers\Collector\JournalCollectorInterface;
 use FireflyIII\Http\Controllers\Controller;
 use FireflyIII\Models\Budget;
+use FireflyIII\Models\BudgetLimit;
 use FireflyIII\Models\Transaction;
 use FireflyIII\Models\TransactionType;
 use FireflyIII\Repositories\Account\AccountRepositoryInterface;
@@ -29,7 +31,6 @@ use Illuminate\Support\Collection;
 use Navigation;
 use Response;
 
-
 /**
  * Separate controller because many helper functions are shared.
  *
@@ -75,49 +76,19 @@ class BudgetReportController extends Controller
      */
     public function accountExpense(Collection $accounts, Collection $budgets, Carbon $start, Carbon $end, string $others)
     {
-        /** @var bool $others */
-        $others = intval($others) === 1;
-        $cache  = new CacheProperties;
-        $cache->addProperty('chart.budget.report.account-expense');
-        $cache->addProperty($accounts);
-        $cache->addProperty($budgets);
-        $cache->addProperty($start);
-        $cache->addProperty($end);
-        if ($cache->has()) {
-            return Response::json($cache->get());
-        }
-
-        $names     = [];
-        $set       = $this->getExpenses($accounts, $budgets, $start, $end);
-        $grouped   = $this->groupByOpposingAccount($set);
-        $chartData = [];
-        $total     = '0';
-
-        foreach ($grouped as $accountId => $amount) {
-            if (!isset($names[$accountId])) {
-                $account           = $this->accountRepository->find(intval($accountId));
-                $names[$accountId] = $account->name;
-            }
-            $amount                        = bcmul($amount, '-1');
-            $total                         = bcadd($total, $amount);
-            $chartData[$names[$accountId]] = $amount;
-        }
-
-        // also collect all transactions NOT in these budgets.
-        if ($others) {
-            $collector = new JournalCollector(auth()->user());
-            $collector->setAccounts($accounts)->setRange($start, $end)->setTypes([TransactionType::WITHDRAWAL]);
-            $journals                                            = $collector->getJournals();
-            $sum                                                 = strval($journals->sum('transaction_amount'));
-            $sum                                                 = bcmul($sum, '-1');
-            $sum                                                 = bcsub($sum, $total);
-            $chartData[strval(trans('firefly.everything_else'))] = $sum;
-        }
-
-        $data = $this->generator->pieChart($chartData);
-        $cache->store($data);
+        /** @var MetaPieChartInterface $helper */
+        $helper = app(MetaPieChartInterface::class);
+        $helper->setAccounts($accounts);
+        $helper->setBudgets($budgets);
+        $helper->setUser(auth()->user());
+        $helper->setStart($start);
+        $helper->setEnd($end);
+        $helper->setCollectOtherObjects(intval($others) === 1);
+        $chartData = $helper->generate('expense', 'account');
+        $data      = $this->generator->pieChart($chartData);
 
         return Response::json($data);
+
     }
 
     /**
@@ -131,47 +102,16 @@ class BudgetReportController extends Controller
      */
     public function budgetExpense(Collection $accounts, Collection $budgets, Carbon $start, Carbon $end, string $others)
     {
-        /** @var bool $others */
-        $others = intval($others) === 1;
-        $cache  = new CacheProperties;
-        $cache->addProperty('chart.budget.report.budget-expense');
-        $cache->addProperty($accounts);
-        $cache->addProperty($budgets);
-        $cache->addProperty($start);
-        $cache->addProperty($end);
-        if ($cache->has()) {
-            return Response::json($cache->get());
-        }
-
-        $names     = [];
-        $set       = $this->getExpenses($accounts, $budgets, $start, $end);
-        $grouped   = $this->groupByBudget($set);
-        $total     = '0';
-        $chartData = [];
-
-        foreach ($grouped as $budgetId => $amount) {
-            if (!isset($names[$budgetId])) {
-                $budget           = $this->budgetRepository->find(intval($budgetId));
-                $names[$budgetId] = $budget->name;
-            }
-            $amount                       = bcmul($amount, '-1');
-            $total                        = bcadd($total, $amount);
-            $chartData[$names[$budgetId]] = $amount;
-        }
-
-        // also collect all transactions NOT in these budgets.
-        if ($others) {
-            $collector = new JournalCollector(auth()->user());
-            $collector->setAccounts($accounts)->setRange($start, $end)->setTypes([TransactionType::WITHDRAWAL]);
-            $journals                                            = $collector->getJournals();
-            $sum                                                 = strval($journals->sum('transaction_amount'));
-            $sum                                                 = bcmul($sum, '-1');
-            $sum                                                 = bcsub($sum, $total);
-            $chartData[strval(trans('firefly.everything_else'))] = $sum;
-        }
-
-        $data = $this->generator->pieChart($chartData);
-        $cache->store($data);
+        /** @var MetaPieChartInterface $helper */
+        $helper = app(MetaPieChartInterface::class);
+        $helper->setAccounts($accounts);
+        $helper->setBudgets($budgets);
+        $helper->setUser(auth()->user());
+        $helper->setStart($start);
+        $helper->setEnd($end);
+        $helper->setCollectOtherObjects(intval($others) === 1);
+        $chartData = $helper->generate('expense', 'budget');
+        $data      = $this->generator->pieChart($chartData);
 
         return Response::json($data);
     }
@@ -195,7 +135,8 @@ class BudgetReportController extends Controller
         if ($cache->has()) {
             return Response::json($cache->get());
         }
-
+        /** @var BudgetRepositoryInterface $repository */
+        $repository   = app(BudgetRepositoryInterface::class);
         $format       = Navigation::preferredCarbonLocalizedFormat($start, $end);
         $function     = Navigation::preferredEndOfPeriod($start, $end);
         $chartData    = [];
@@ -203,13 +144,30 @@ class BudgetReportController extends Controller
 
         // prep chart data:
         foreach ($budgets as $budget) {
-            $chartData[$budget->id] = [
-                'label'   => $budget->name,
+            $chartData[$budget->id]           = [
+                'label'   => strval(trans('firefly.spent_in_specific_budget', ['budget' => $budget->name])),
                 'type'    => 'bar',
+                'yAxisID' => 'y-axis-0',
+                'entries' => [],
+            ];
+            $chartData[$budget->id . '-sum']  = [
+                'label'   => strval(trans('firefly.sum_of_expenses_in_budget', ['budget' => $budget->name])),
+                'type'    => 'line',
+                'fill'    => false,
+                'yAxisID' => 'y-axis-1',
+                'entries' => [],
+            ];
+            $chartData[$budget->id . '-left'] = [
+                'label'   => strval(trans('firefly.left_in_budget_limit', ['budget' => $budget->name])),
+                'type'    => 'bar',
+                'fill'    => false,
+                'yAxisID' => 'y-axis-0',
                 'entries' => [],
             ];
         }
-
+        $allBudgetLimits = $repository->getAllBudgetLimits($start, $end);
+        $sumOfExpenses   = [];
+        $leftOfLimits    = [];
         while ($currentStart < $end) {
             $currentEnd = clone $currentStart;
             $currentEnd = $currentEnd->$function();
@@ -218,7 +176,20 @@ class BudgetReportController extends Controller
 
             /** @var Budget $budget */
             foreach ($budgets as $budget) {
-                $chartData[$budget->id]['entries'][$label] = $expenses[$budget->id] ?? '0';
+                // get budget limit(s) for this period):
+                $budgetLimits                                       = $this->filterBudgetLimits($allBudgetLimits, $budget, $currentStart, $currentEnd);
+                $currentExpenses                                    = $expenses[$budget->id] ?? '0';
+                $sumOfExpenses[$budget->id]                         = $sumOfExpenses[$budget->id] ?? '0';
+                $sumOfExpenses[$budget->id]                         = bcadd($currentExpenses, $sumOfExpenses[$budget->id]);
+                $chartData[$budget->id]['entries'][$label]          = bcmul($currentExpenses, '-1');
+                $chartData[$budget->id . '-sum']['entries'][$label] = bcmul($sumOfExpenses[$budget->id], '-1');
+
+                if (count($budgetLimits) > 0) {
+                    $budgetLimitId                                       = $budgetLimits->first()->id;
+                    $leftOfLimits[$budgetLimitId]                        = $leftOfLimits[$budgetLimitId] ?? strval($budgetLimits->sum('amount'));
+                    $leftOfLimits[$budgetLimitId]                        = bcadd($leftOfLimits[$budgetLimitId], $currentExpenses);
+                    $chartData[$budget->id . '-left']['entries'][$label] = $leftOfLimits[$budgetLimitId];
+                }
             }
             $currentStart = clone $currentEnd;
             $currentStart->addDay();
@@ -230,6 +201,32 @@ class BudgetReportController extends Controller
         return Response::json($data);
     }
 
+    /**
+     * Returns the budget limits belonging to the given budget and valid on the given day.
+     *
+     * @param Collection $budgetLimits
+     * @param Carbon     $start
+     * @param Carbon     $end
+     *
+     * @return Collection
+     */
+    private function filterBudgetLimits(Collection $budgetLimits, Budget $budget, Carbon $start, Carbon $end): Collection
+    {
+        $set = $budgetLimits->filter(
+            function (BudgetLimit $budgetLimit) use ($budget, $start, $end) {
+                if ($budgetLimit->budget_id === $budget->id
+                    && $budgetLimit->start_date->lte($start) // start of budget limit is on or before start
+                    && $budgetLimit->end_date->gte($end) // end of budget limit is on or after end
+                ) {
+                    return $budgetLimit;
+                }
+
+                return false;
+            }
+        );
+
+        return $set;
+    }
 
     /**
      * @param Collection $accounts
@@ -241,7 +238,8 @@ class BudgetReportController extends Controller
      */
     private function getExpenses(Collection $accounts, Collection $budgets, Carbon $start, Carbon $end): Collection
     {
-        $collector = new JournalCollector(auth()->user());
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
         $collector->setAccounts($accounts)->setRange($start, $end)->setTypes([TransactionType::WITHDRAWAL, TransactionType::TRANSFER])
                   ->setBudgets($budgets)->withOpposingAccount()->disableFilter();
         $accountIds   = $accounts->pluck('id')->toArray();
@@ -289,4 +287,4 @@ class BudgetReportController extends Controller
 
         return $grouped;
     }
-}
+}

app/Http/Controllers/Chart/CategoryController.php

@@ -65,7 +65,12 @@ class CategoryController extends Controller
             return Response::json($cache->get());
         }
 
-        $start     = $repository->firstUseDate($category);
+        $start = $repository->firstUseDate($category);
+
+        if ($start->year == 1900) {
+            $start = new Carbon;
+        }
+
         $range     = Preferences::get('viewRange', '1M')->data;
         $start     = Navigation::startOfPeriod($start, $range);
         $end       = new Carbon;
@@ -143,6 +148,7 @@ class CategoryController extends Controller
                 $chartData[$category->name] = bcmul($spent, '-1');
             }
         }
+
         $chartData[strval(trans('firefly.no_category'))] = bcmul($repository->spentInPeriodWithoutCategory(new Collection, $start, $end), '-1');
 
         // sort

app/Http/Controllers/Chart/CategoryReportController.php

@@ -17,7 +17,8 @@ namespace FireflyIII\Http\Controllers\Chart;
 use Carbon\Carbon;
 use FireflyIII\Generator\Chart\Basic\GeneratorInterface;
 use FireflyIII\Generator\Report\Category\MonthReportGenerator;
-use FireflyIII\Helpers\Collector\JournalCollector;
+use FireflyIII\Helpers\Chart\MetaPieChartInterface;
+use FireflyIII\Helpers\Collector\JournalCollectorInterface;
 use FireflyIII\Http\Controllers\Controller;
 use FireflyIII\Models\Category;
 use FireflyIII\Models\Transaction;
@@ -75,47 +76,16 @@ class CategoryReportController extends Controller
      */
     public function accountExpense(Collection $accounts, Collection $categories, Carbon $start, Carbon $end, string $others)
     {
-        /** @var bool $others */
-        $others = intval($others) === 1;
-        $cache  = new CacheProperties;
-        $cache->addProperty('chart.category.report.account-expense');
-        $cache->addProperty($accounts);
-        $cache->addProperty($categories);
-        $cache->addProperty($start);
-        $cache->addProperty($end);
-        if ($cache->has()) {
-            return Response::json($cache->get());
-        }
-
-        $names     = [];
-        $set       = $this->getExpenses($accounts, $categories, $start, $end);
-        $grouped   = $this->groupByOpposingAccount($set);
-        $chartData = [];
-        $total     = '0';
-
-        foreach ($grouped as $accountId => $amount) {
-            if (!isset($names[$accountId])) {
-                $account           = $this->accountRepository->find(intval($accountId));
-                $names[$accountId] = $account->name;
-            }
-            $amount                        = bcmul($amount, '-1');
-            $total                         = bcadd($total, $amount);
-            $chartData[$names[$accountId]] = $amount;
-        }
-
-        // also collect all transactions NOT in these categories.
-        if ($others) {
-            $collector = new JournalCollector(auth()->user());
-            $collector->setAccounts($accounts)->setRange($start, $end)->setTypes([TransactionType::WITHDRAWAL]);
-            $journals                                            = $collector->getJournals();
-            $sum                                                 = strval($journals->sum('transaction_amount'));
-            $sum                                                 = bcmul($sum, '-1');
-            $sum                                                 = bcsub($sum, $total);
-            $chartData[strval(trans('firefly.everything_else'))] = $sum;
-        }
-
-        $data = $this->generator->pieChart($chartData);
-        $cache->store($data);
+        /** @var MetaPieChartInterface $helper */
+        $helper = app(MetaPieChartInterface::class);
+        $helper->setAccounts($accounts);
+        $helper->setCategories($categories);
+        $helper->setUser(auth()->user());
+        $helper->setStart($start);
+        $helper->setEnd($end);
+        $helper->setCollectOtherObjects(intval($others) === 1);
+        $chartData = $helper->generate('expense', 'account');
+        $data      = $this->generator->pieChart($chartData);
 
         return Response::json($data);
     }
@@ -131,46 +101,16 @@ class CategoryReportController extends Controller
      */
     public function accountIncome(Collection $accounts, Collection $categories, Carbon $start, Carbon $end, string $others)
     {
-        /** @var bool $others */
-        $others = intval($others) === 1;
-        $cache  = new CacheProperties;
-        $cache->addProperty('chart.category.report.account-income');
-        $cache->addProperty($accounts);
-        $cache->addProperty($categories);
-        $cache->addProperty($start);
-        $cache->addProperty($end);
-        if ($cache->has()) {
-            return Response::json($cache->get());
-        }
-
-
-        $names     = [];
-        $set       = $this->getIncome($accounts, $categories, $start, $end);
-        $grouped   = $this->groupByOpposingAccount($set);
-        $chartData = [];
-        $total     = '0';
-
-        foreach ($grouped as $accountId => $amount) {
-            if (!isset($names[$accountId])) {
-                $account           = $this->accountRepository->find(intval($accountId));
-                $names[$accountId] = $account->name;
-            }
-            $total                         = bcadd($total, $amount);
-            $chartData[$names[$accountId]] = $amount;
-        }
-
-        // also collect others?
-        if ($others) {
-            $collector = new JournalCollector(auth()->user());
-            $collector->setAccounts($accounts)->setRange($start, $end)->setTypes([TransactionType::DEPOSIT]);
-            $journals                                            = $collector->getJournals();
-            $sum                                                 = strval($journals->sum('transaction_amount'));
-            $sum                                                 = bcsub($sum, $total);
-            $chartData[strval(trans('firefly.everything_else'))] = $sum;
-        }
-
-        $data = $this->generator->pieChart($chartData);
-        $cache->store($data);
+        /** @var MetaPieChartInterface $helper */
+        $helper = app(MetaPieChartInterface::class);
+        $helper->setAccounts($accounts);
+        $helper->setCategories($categories);
+        $helper->setUser(auth()->user());
+        $helper->setStart($start);
+        $helper->setEnd($end);
+        $helper->setCollectOtherObjects(intval($others) === 1);
+        $chartData = $helper->generate('income', 'account');
+        $data      = $this->generator->pieChart($chartData);
 
         return Response::json($data);
     }
@@ -186,47 +126,16 @@ class CategoryReportController extends Controller
      */
     public function categoryExpense(Collection $accounts, Collection $categories, Carbon $start, Carbon $end, string $others)
     {
-        /** @var bool $others */
-        $others = intval($others) === 1;
-        $cache  = new CacheProperties;
-        $cache->addProperty('chart.category.report.category-expense');
-        $cache->addProperty($accounts);
-        $cache->addProperty($categories);
-        $cache->addProperty($start);
-        $cache->addProperty($end);
-        if ($cache->has()) {
-            return Response::json($cache->get());
-        }
-
-        $names     = [];
-        $set       = $this->getExpenses($accounts, $categories, $start, $end);
-        $grouped   = $this->groupByCategory($set);
-        $total     = '0';
-        $chartData = [];
-
-        foreach ($grouped as $categoryId => $amount) {
-            if (!isset($names[$categoryId])) {
-                $category           = $this->categoryRepository->find(intval($categoryId));
-                $names[$categoryId] = $category->name;
-            }
-            $amount                         = bcmul($amount, '-1');
-            $total                          = bcadd($total, $amount);
-            $chartData[$names[$categoryId]] = $amount;
-        }
-
-        // also collect all transactions NOT in these categories.
-        if ($others) {
-            $collector = new JournalCollector(auth()->user());
-            $collector->setAccounts($accounts)->setRange($start, $end)->setTypes([TransactionType::WITHDRAWAL]);
-            $journals                                            = $collector->getJournals();
-            $sum                                                 = strval($journals->sum('transaction_amount'));
-            $sum                                                 = bcmul($sum, '-1');
-            $sum                                                 = bcsub($sum, $total);
-            $chartData[strval(trans('firefly.everything_else'))] = $sum;
-        }
-
-        $data = $this->generator->pieChart($chartData);
-        $cache->store($data);
+        /** @var MetaPieChartInterface $helper */
+        $helper = app(MetaPieChartInterface::class);
+        $helper->setAccounts($accounts);
+        $helper->setCategories($categories);
+        $helper->setUser(auth()->user());
+        $helper->setStart($start);
+        $helper->setEnd($end);
+        $helper->setCollectOtherObjects(intval($others) === 1);
+        $chartData = $helper->generate('expense', 'category');
+        $data      = $this->generator->pieChart($chartData);
 
         return Response::json($data);
     }
@@ -242,44 +151,17 @@ class CategoryReportController extends Controller
      */
     public function categoryIncome(Collection $accounts, Collection $categories, Carbon $start, Carbon $end, string $others)
     {
-        /** @var bool $others */
-        $others = intval($others) === 1;
-        $cache  = new CacheProperties;
-        $cache->addProperty('chart.category.report.category-income');
-        $cache->addProperty($accounts);
-        $cache->addProperty($categories);
-        $cache->addProperty($start);
-        $cache->addProperty($end);
-        if ($cache->has()) {
-            return Response::json($cache->get());
-        }
 
-        $names     = [];
-        $set       = $this->getIncome($accounts, $categories, $start, $end);
-        $grouped   = $this->groupByCategory($set);
-        $total     = '0';
-        $chartData = [];
-
-        foreach ($grouped as $categoryId => $amount) {
-            if (!isset($names[$categoryId])) {
-                $category           = $this->categoryRepository->find(intval($categoryId));
-                $names[$categoryId] = $category->name;
-            }
-            $total                          = bcadd($total, $amount);
-            $chartData[$names[$categoryId]] = $amount;
-        }
-
-        if ($others) {
-            $collector = new JournalCollector(auth()->user());
-            $collector->setAccounts($accounts)->setRange($start, $end)->setTypes([TransactionType::DEPOSIT]);
-            $journals                                            = $collector->getJournals();
-            $sum                                                 = strval($journals->sum('transaction_amount'));
-            $sum                                                 = bcsub($sum, $total);
-            $chartData[strval(trans('firefly.everything_else'))] = $sum;
-        }
-
-        $data = $this->generator->pieChart($chartData);
-        $cache->store($data);
+        /** @var MetaPieChartInterface $helper */
+        $helper = app(MetaPieChartInterface::class);
+        $helper->setAccounts($accounts);
+        $helper->setCategories($categories);
+        $helper->setUser(auth()->user());
+        $helper->setStart($start);
+        $helper->setEnd($end);
+        $helper->setCollectOtherObjects(intval($others) === 1);
+        $chartData = $helper->generate('income', 'category');
+        $data      = $this->generator->pieChart($chartData);
 
         return Response::json($data);
     }
@@ -314,14 +196,33 @@ class CategoryReportController extends Controller
             $chartData[$category->id . '-in']  = [
                 'label'   => $category->name . ' (' . strtolower(strval(trans('firefly.income'))) . ')',
                 'type'    => 'bar',
+                'yAxisID' => 'y-axis-0',
                 'entries' => [],
             ];
             $chartData[$category->id . '-out'] = [
                 'label'   => $category->name . ' (' . strtolower(strval(trans('firefly.expenses'))) . ')',
                 'type'    => 'bar',
+                'yAxisID' => 'y-axis-0',
+                'entries' => [],
+            ];
+            // total in, total out:
+            $chartData[$category->id . '-total-in']  = [
+                'label'   => $category->name . ' (' . strtolower(strval(trans('firefly.sum_of_income'))) . ')',
+                'type'    => 'line',
+                'fill'    => false,
+                'yAxisID' => 'y-axis-1',
+                'entries' => [],
+            ];
+            $chartData[$category->id . '-total-out'] = [
+                'label'   => $category->name . ' (' . strtolower(strval(trans('firefly.sum_of_expenses'))) . ')',
+                'type'    => 'line',
+                'fill'    => false,
+                'yAxisID' => 'y-axis-1',
                 'entries' => [],
             ];
         }
+        $sumOfIncome  = [];
+        $sumOfExpense = [];
 
         while ($currentStart < $end) {
             $currentEnd = clone $currentStart;
@@ -332,17 +233,40 @@ class CategoryReportController extends Controller
 
             /** @var Category $category */
             foreach ($categories as $category) {
-                $labelIn  = $category->id . '-in';
-                $labelOut = $category->id . '-out';
-                // get sum, and get label:
-                $chartData[$labelIn]['entries'][$label]  = $income[$category->id] ?? '0';
-                $chartData[$labelOut]['entries'][$label] = $expenses[$category->id] ?? '0';
+                $labelIn        = $category->id . '-in';
+                $labelOut       = $category->id . '-out';
+                $labelSumIn     = $category->id . '-total-in';
+                $labelSumOut    = $category->id . '-total-out';
+                $currentIncome  = $income[$category->id] ?? '0';
+                $currentExpense = $expenses[$category->id] ?? '0';
+
+
+                // add to sum:
+                $sumOfIncome[$category->id]  = $sumOfIncome[$category->id] ?? '0';
+                $sumOfExpense[$category->id] = $sumOfExpense[$category->id] ?? '0';
+                $sumOfIncome[$category->id]  = bcadd($sumOfIncome[$category->id], $currentIncome);
+                $sumOfExpense[$category->id] = bcadd($sumOfExpense[$category->id], $currentExpense);
+
+                // add to chart:
+                $chartData[$labelIn]['entries'][$label]     = $currentIncome;
+                $chartData[$labelOut]['entries'][$label]    = $currentExpense;
+                $chartData[$labelSumIn]['entries'][$label]  = $sumOfIncome[$category->id];
+                $chartData[$labelSumOut]['entries'][$label] = $sumOfExpense[$category->id];
             }
             $currentStart = clone $currentEnd;
             $currentStart->addDay();
         }
-
-        $data = $this->generator->multiSet($chartData);
+        // remove all empty entries to prevent cluttering:
+        $newSet = [];
+        foreach ($chartData as $key => $entry) {
+            if (!array_sum($entry['entries']) == 0) {
+                $newSet[$key] = $chartData[$key];
+            }
+        }
+        if (count($newSet) === 0) {
+            $newSet = $chartData;
+        }
+        $data = $this->generator->multiSet($newSet);
         $cache->store($data);
 
         return Response::json($data);
@@ -359,7 +283,8 @@ class CategoryReportController extends Controller
      */
     private function getExpenses(Collection $accounts, Collection $categories, Carbon $start, Carbon $end): Collection
     {
-        $collector = new JournalCollector(auth()->user());
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
         $collector->setAccounts($accounts)->setRange($start, $end)->setTypes([TransactionType::WITHDRAWAL, TransactionType::TRANSFER])
                   ->setCategories($categories)->withOpposingAccount()->disableFilter();
         $accountIds   = $accounts->pluck('id')->toArray();
@@ -379,7 +304,8 @@ class CategoryReportController extends Controller
      */
     private function getIncome(Collection $accounts, Collection $categories, Carbon $start, Carbon $end): Collection
     {
-        $collector = new JournalCollector(auth()->user());
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
         $collector->setAccounts($accounts)->setRange($start, $end)->setTypes([TransactionType::DEPOSIT, TransactionType::TRANSFER])
                   ->setCategories($categories)->withOpposingAccount();
         $accountIds   = $accounts->pluck('id')->toArray();
@@ -427,4 +353,4 @@ class CategoryReportController extends Controller
 
         return $grouped;
     }
-}
+}

app/Http/Controllers/Controller.php

@@ -13,6 +13,7 @@ declare(strict_types = 1);
 
 namespace FireflyIII\Http\Controllers;
 
+use FireflyConfig;
 use FireflyIII\Models\AccountType;
 use FireflyIII\Models\Transaction;
 use FireflyIII\Models\TransactionJournal;
@@ -49,7 +50,10 @@ class Controller extends BaseController
         View::share('hideCategories', false);
         View::share('hideBills', false);
         View::share('hideTags', false);
-
+        $isDemoSite = FireflyConfig::get('is_demo_site', config('firefly.configuration.is_demo_site'))->data;
+        View::share('IS_DEMO_SITE', $isDemoSite);
+        View::share('DEMO_USERNAME', env('DEMO_USERNAME', ''));
+        View::share('DEMO_PASSWORD', env('DEMO_PASSWORD', ''));
 
         // translations:
 

app/Http/Controllers/CurrencyController.php

@@ -17,7 +17,6 @@ use Cache;
 use FireflyIII\Http\Requests\CurrencyFormRequest;
 use FireflyIII\Models\TransactionCurrency;
 use FireflyIII\Repositories\Currency\CurrencyRepositoryInterface;
-use Input;
 use Log;
 use Preferences;
 use Session;
@@ -170,7 +169,7 @@ class CurrencyController extends Controller
 
 
         if (!auth()->user()->hasRole('owner')) {
-            Session::flash('warning', trans('firefly.ask_site_owner', ['site_owner' => env('SITE_OWNER')]));
+            Session::flash('info', trans('firefly.ask_site_owner', ['owner' => env('SITE_OWNER')]));
         }
 
 
@@ -196,7 +195,7 @@ class CurrencyController extends Controller
         $currency = $repository->store($data);
         Session::flash('success', trans('firefly.created_currency', ['name' => $currency->name]));
 
-        if (intval(Input::get('create_another')) === 1) {
+        if (intval($request->get('create_another')) === 1) {
             Session::put('currencies.create.fromStore', true);
 
             return redirect(route('currencies.create'))->withInput();
@@ -225,7 +224,7 @@ class CurrencyController extends Controller
         Preferences::mark();
 
 
-        if (intval(Input::get('return_to_edit')) === 1) {
+        if (intval($request->get('return_to_edit')) === 1) {
             Session::put('currencies.edit.fromUpdate', true);
 
             return redirect(route('currencies.edit', [$currency->id]));

app/Http/Controllers/ExportController.php

@@ -22,10 +22,10 @@ use FireflyIII\Http\Requests\ExportFormRequest;
 use FireflyIII\Models\AccountType;
 use FireflyIII\Models\ExportJob;
 use FireflyIII\Repositories\Account\AccountRepositoryInterface;
+use FireflyIII\Repositories\ExportJob\ExportJobRepositoryInterface;
 use FireflyIII\Repositories\ExportJob\ExportJobRepositoryInterface as EJRI;
 use Preferences;
 use Response;
-use Storage;
 use View;
 
 /**
@@ -59,21 +59,22 @@ class ExportController extends Controller
      * @return \Symfony\Component\HttpFoundation\Response|\Illuminate\Contracts\Routing\ResponseFactory
      * @throws FireflyException
      */
-    public function download(ExportJob $job)
+    public function download(ExportJobRepositoryInterface $repository, ExportJob $job)
     {
-        $disk   = Storage::disk('export');
         $file   = $job->key . '.zip';
         $date   = date('Y-m-d \a\t H-i-s');
         $name   = 'Export job on ' . $date . '.zip';
         $quoted = sprintf('"%s"', addcslashes($name, '"\\'));
 
-        if (!$disk->exists($file)) {
+        if (!$repository->exists($job)) {
             throw new FireflyException('Against all expectations, zip file "' . $file . '" does not exist.');
         }
+        $content = $repository->getContent($job);
+
 
         $job->change('export_downloaded');
 
-        return response($disk->get($file), 200)
+        return response($content, 200)
             ->header('Content-Description', 'File Transfer')
             ->header('Content-Type', 'application/octet-stream')
             ->header('Content-Disposition', 'attachment; filename=' . $quoted)
@@ -82,7 +83,7 @@ class ExportController extends Controller
             ->header('Expires', '0')
             ->header('Cache-Control', 'must-revalidate, post-check=0, pre-check=0')
             ->header('Pragma', 'public')
-            ->header('Content-Length', $disk->size($file));
+            ->header('Content-Length', strlen($content));
 
     }
 
@@ -143,7 +144,7 @@ class ExportController extends Controller
             'job'                => $job,
         ];
 
-        $job->change('export_status_make_exporter');
+        $jobs->changeStatus($job, 'export_status_make_exporter');
 
         /** @var ProcessorInterface $processor */
         $processor = app(ProcessorInterface::class, [$settings]);
@@ -151,47 +152,46 @@ class ExportController extends Controller
         /*
          * Collect journals:
          */
-        $job->change('export_status_collecting_journals');
+        $jobs->changeStatus($job, 'export_status_collecting_journals');
         $processor->collectJournals();
-        $job->change('export_status_collected_journals');
+        $jobs->changeStatus($job, 'export_status_collected_journals');
         /*
          * Transform to exportable entries:
          */
-        $job->change('export_status_converting_to_export_format');
+        $jobs->changeStatus($job, 'export_status_converting_to_export_format');
         $processor->convertJournals();
-        $job->change('export_status_converted_to_export_format');
+        $jobs->changeStatus($job, 'export_status_converted_to_export_format');
         /*
          * Transform to (temporary) file:
          */
-        $job->change('export_status_creating_journal_file');
+        $jobs->changeStatus($job, 'export_status_creating_journal_file');
         $processor->exportJournals();
-        $job->change('export_status_created_journal_file');
+        $jobs->changeStatus($job, 'export_status_created_journal_file');
         /*
          *  Collect attachments, if applicable.
          */
         if ($settings['includeAttachments']) {
-            $job->change('export_status_collecting_attachments');
+            $jobs->changeStatus($job, 'export_status_collecting_attachments');
             $processor->collectAttachments();
-            $job->change('export_status_collected_attachments');
+            $jobs->changeStatus($job, 'export_status_collected_attachments');
         }
 
         /*
          * Collect old uploads
          */
         if ($settings['includeOldUploads']) {
-            $job->change('export_status_collecting_old_uploads');
+            $jobs->changeStatus($job, 'export_status_collecting_old_uploads');
             $processor->collectOldUploads();
-            $job->change('export_status_collected_old_uploads');
+            $jobs->changeStatus($job, 'export_status_collected_old_uploads');
         }
 
         /*
          * Create ZIP file:
          */
-        $job->change('export_status_creating_zip_file');
+        $jobs->changeStatus($job, 'export_status_creating_zip_file');
         $processor->createZipFile();
-        $job->change('export_status_created_zip_file');
-
-        $job->change('export_status_finished');
+        $jobs->changeStatus($job, 'export_status_created_zip_file');
+        $jobs->changeStatus($job, 'export_status_finished');
 
         return Response::json('ok');
     }

app/Http/Controllers/HomeController.php

@@ -107,7 +107,7 @@ class HomeController extends Controller
                 $journal->save();
             }
         }
-
+        Session::forget(['start', 'end', 'viewRange', 'range', 'is_custom_range']);
 
         Session::clear();
         Artisan::call('cache:clear');
@@ -174,9 +174,6 @@ class HomeController extends Controller
             'logout',
             'two-fac',
             'lost-two',
-            'confirm',
-            'resend',
-            'do_confirm',
             // test troutes
             'test-flash',
             'all-routes',

app/Http/Controllers/ImportController.php

@@ -15,7 +15,7 @@ namespace FireflyIII\Http\Controllers;
 use Crypt;
 use FireflyIII\Exceptions\FireflyException;
 use FireflyIII\Http\Requests\ImportUploadRequest;
-use FireflyIII\Import\ImportProcedure;
+use FireflyIII\Import\ImportProcedureInterface;
 use FireflyIII\Import\Setup\SetupInterface;
 use FireflyIII\Models\ImportJob;
 use FireflyIII\Repositories\ImportJob\ImportJobRepositoryInterface;
@@ -23,6 +23,7 @@ use FireflyIII\Repositories\Tag\TagRepositoryInterface;
 use Illuminate\Http\Request;
 use Log;
 use Response;
+use Session;
 use SplFileObject;
 use Storage;
 use Symfony\Component\HttpFoundation\File\UploadedFile;
@@ -315,13 +316,13 @@ class ImportController extends Controller
     }
 
     /**
-     * @param ImportJob $job
+     * @param ImportProcedureInterface $importProcedure
+     * @param ImportJob                $job
      */
-    public function start(ImportJob $job)
+    public function start(ImportProcedureInterface $importProcedure, ImportJob $job)
     {
         set_time_limit(0);
         if ($job->status == 'settings_complete') {
-            $importProcedure = new ImportProcedure;
             $importProcedure->runImport($job);
         }
     }
@@ -334,7 +335,7 @@ class ImportController extends Controller
      * @return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector|View
      */
     public function status(ImportJob $job)
-    {
+    { //
         Log::debug('Now in status()', ['job' => $job->key]);
         if (!$this->jobInCorrectStep($job, 'status')) {
             return $this->redirectToCorrectStep($job);
@@ -368,14 +369,33 @@ class ImportController extends Controller
         $content          = $uploaded->fread($uploaded->getSize());
         $contentEncrypted = Crypt::encrypt($content);
         $disk             = Storage::disk('upload');
-        $disk->put($newName, $contentEncrypted);
 
-        Log::debug('Uploaded file', ['name' => $upload->getClientOriginalName(), 'size' => $upload->getSize(), 'mime' => $upload->getClientMimeType()]);
+        // user is demo user, replace upload with prepared file.
+        if (auth()->user()->hasRole('demo')) {
+            $stubsDisk        = Storage::disk('stubs');
+            $content          = $stubsDisk->get('demo-import.csv');
+            $contentEncrypted = Crypt::encrypt($content);
+            $disk->put($newName, $contentEncrypted);
+            Log::debug('Replaced upload with demo file.');
 
-        // store configuration file's content into the job's configuration
-        // thing.
-        // otherwise, leave it empty.
-        if ($request->files->has('configuration_file')) {
+            // also set up prepared configuration.
+            $configuration      = json_decode($stubsDisk->get('demo-configuration.json'), true);
+            $job->configuration = $configuration;
+            $job->save();
+            Log::debug('Set configuration for demo user', $configuration);
+
+            // also flash info
+            Session::flash('info', trans('demo.import-configure-security'));
+        }
+        if (!auth()->user()->hasRole('demo')) {
+            // user is not demo, process original upload:
+            $disk->put($newName, $contentEncrypted);
+            Log::debug('Uploaded file', ['name' => $upload->getClientOriginalName(), 'size' => $upload->getSize(), 'mime' => $upload->getClientMimeType()]);
+        }
+
+        // store configuration file's content into the job's configuration thing. Otherwise, leave it empty.
+        // demo user's configuration upload is ignored completely.
+        if ($request->files->has('configuration_file') && !auth()->user()->hasRole('demo')) {
             /** @var UploadedFile $configFile */
             $configFile = $request->files->get('configuration_file');
             Log::debug(
@@ -394,6 +414,9 @@ class ImportController extends Controller
             }
         }
 
+        // if user is demo user, replace config with prepared config:
+
+
         return redirect(route('import.configure', [$job->key]));
 
     }

app/Http/Controllers/JsonController.php

@@ -15,7 +15,7 @@ namespace FireflyIII\Http\Controllers;
 use Amount;
 use Carbon\Carbon;
 use FireflyIII\Exceptions\FireflyException;
-use FireflyIII\Helpers\Collector\JournalCollector;
+use FireflyIII\Helpers\Collector\JournalCollectorInterface;
 use FireflyIII\Models\AccountType;
 use FireflyIII\Repositories\Account\AccountRepositoryInterface;
 use FireflyIII\Repositories\Account\AccountTaskerInterface;
@@ -23,7 +23,7 @@ use FireflyIII\Repositories\Bill\BillRepositoryInterface;
 use FireflyIII\Repositories\Category\CategoryRepositoryInterface as CRI;
 use FireflyIII\Repositories\Tag\TagRepositoryInterface;
 use FireflyIII\Support\CacheProperties;
-use Input;
+use Illuminate\Http\Request;
 use Preferences;
 use Response;
 
@@ -43,11 +43,13 @@ class JsonController extends Controller
     }
 
     /**
+     * @param Request $request
+     *
      * @return \Illuminate\Http\JsonResponse
      */
-    public function action()
+    public function action(Request $request)
     {
-        $count   = intval(Input::get('count')) > 0 ? intval(Input::get('count')) : 1;
+        $count   = intval($request->get('count')) > 0 ? intval($request->get('count')) : 1;
         $keys    = array_keys(config('firefly.rule-actions'));
         $actions = [];
         foreach ($keys as $key) {
@@ -269,18 +271,18 @@ class JsonController extends Controller
     }
 
     /**
-     * @param $what
+     * @param JournalCollectorInterface $collector
+     * @param string                    $what
      *
      * @return \Illuminate\Http\JsonResponse
      */
-    public function transactionJournals($what)
+    public function transactionJournals(JournalCollectorInterface $collector, string $what)
     {
         $descriptions = [];
         $type         = config('firefly.transactionTypesByWhat.' . $what);
         $types        = [$type];
 
         // use journal collector instead:
-        $collector = new JournalCollector(auth()->user());
         $collector->setTypes($types)->setLimit(100)->setPage(1);
         $journals = $collector->getJournals();
         foreach ($journals as $j) {
@@ -296,11 +298,13 @@ class JsonController extends Controller
     }
 
     /**
+     * @param Request $request
+     *
      * @return \Illuminate\Http\JsonResponse
      */
-    public function trigger()
+    public function trigger(Request $request)
     {
-        $count    = intval(Input::get('count')) > 0 ? intval(Input::get('count')) : 1;
+        $count    = intval($request->get('count')) > 0 ? intval($request->get('count')) : 1;
         $keys     = array_keys(config('firefly.rule-triggers'));
         $triggers = [];
         foreach ($keys as $key) {

app/Http/Controllers/NewUserController.php

@@ -117,7 +117,7 @@ class NewUserController extends Controller
             'virtualBalance'         => 0,
             'active'                 => true,
             'accountRole'            => 'defaultAsset',
-            'openingBalance'         => round($request->input('bank_balance'), 2),
+            'openingBalance'         => round($request->input('bank_balance'), 12),
             'openingBalanceDate'     => new Carbon,
             'openingBalanceCurrency' => intval($request->input('amount_currency_id_bank_balance')),
         ];
@@ -142,7 +142,7 @@ class NewUserController extends Controller
             'virtualBalance'         => 0,
             'active'                 => true,
             'accountRole'            => 'savingAsset',
-            'openingBalance'         => round($request->input('savings_balance'), 2),
+            'openingBalance'         => round($request->input('savings_balance'), 12),
             'openingBalanceDate'     => new Carbon,
             'openingBalanceCurrency' => intval($request->input('amount_currency_id_savings_balance')),
         ];
@@ -163,7 +163,7 @@ class NewUserController extends Controller
             'name'                   => 'Credit card',
             'iban'                   => null,
             'accountType'            => 'asset',
-            'virtualBalance'         => round($request->get('credit_card_limit'), 2),
+            'virtualBalance'         => round($request->get('credit_card_limit'), 12),
             'active'                 => true,
             'accountRole'            => 'ccAsset',
             'openingBalance'         => null,

app/Http/Controllers/PiggyBankController.php

@@ -20,10 +20,11 @@ use FireflyIII\Models\AccountType;
 use FireflyIII\Models\PiggyBank;
 use FireflyIII\Repositories\Account\AccountRepositoryInterface;
 use FireflyIII\Repositories\PiggyBank\PiggyBankRepositoryInterface;
+use Illuminate\Http\Request;
 use Illuminate\Support\Collection;
-use Input;
 use Log;
 use Preferences;
+use Response;
 use Session;
 use Steam;
 use URL;
@@ -149,13 +150,18 @@ class PiggyBankController extends Controller
      */
     public function destroy(PiggyBankRepositoryInterface $repository, PiggyBank $piggyBank)
     {
-
-
         Session::flash('success', strval(trans('firefly.deleted_piggy_bank', ['name' => e($piggyBank->name)])));
         Preferences::mark();
+        $piggyBankId = $piggyBank->id;
         $repository->destroy($piggyBank);
 
-        return redirect(session('piggy-banks.delete.url'));
+        $uri = session('piggy-banks.delete.url');
+        if (!(strpos($uri, sprintf('piggy-banks/show/%s', $piggyBankId)) === false)) {
+            // uri would point back to piggy bank
+            $uri = route('piggy-banks.index');
+        }
+
+        return redirect($uri);
     }
 
     /**
@@ -213,7 +219,7 @@ class PiggyBankController extends Controller
         $accounts = [];
         /** @var PiggyBank $piggyBank */
         foreach ($piggyBanks as $piggyBank) {
-            $piggyBank->savedSoFar = round($piggyBank->currentRelevantRep()->currentamount, 2);
+            $piggyBank->savedSoFar = $piggyBank->currentRelevantRep()->currentamount;
             $piggyBank->percentage = $piggyBank->savedSoFar != 0 ? intval($piggyBank->savedSoFar / $piggyBank->targetamount * 100) : 0;
             $piggyBank->leftToSave = bcsub($piggyBank->targetamount, strval($piggyBank->savedSoFar));
             $piggyBank->percentage = $piggyBank->percentage > 100 ? 100 : $piggyBank->percentage;
@@ -228,7 +234,7 @@ class PiggyBankController extends Controller
                     'balance'           => Steam::balanceIgnoreVirtual($account, $end),
                     'leftForPiggyBanks' => $piggyBank->leftOnAccount($end),
                     'sumOfSaved'        => strval($piggyBank->savedSoFar),
-                    'sumOfTargets'      => strval(round($piggyBank->targetamount, 2)),
+                    'sumOfTargets'      => $piggyBank->targetamount,
                     'leftToSave'        => $piggyBank->leftToSave,
                 ];
             } else {
@@ -242,11 +248,14 @@ class PiggyBankController extends Controller
     }
 
     /**
+     * @param Request                      $request
      * @param PiggyBankRepositoryInterface $repository
+     *
+     * @return \Illuminate\Http\JsonResponse
      */
-    public function order(PiggyBankRepositoryInterface $repository)
+    public function order(Request $request, PiggyBankRepositoryInterface $repository)
     {
-        $data = Input::get('order');
+        $data = $request->get('order');
 
         // set all users piggy banks to zero:
         $repository->reset();
@@ -257,25 +266,29 @@ class PiggyBankController extends Controller
                 $repository->setOrder(intval($id), ($order + 1));
             }
         }
+
+        return Response::json(['result' => 'ok']);
     }
 
     /**
+     * @param Request                      $request
      * @param PiggyBankRepositoryInterface $repository
      * @param PiggyBank                    $piggyBank
      *
      * @return \Illuminate\Http\RedirectResponse
      */
-    public function postAdd(PiggyBankRepositoryInterface $repository, PiggyBank $piggyBank)
+    public function postAdd(Request $request, PiggyBankRepositoryInterface $repository, PiggyBank $piggyBank)
     {
-        $amount = strval(round(Input::get('amount'), 2));
+        $amount = $request->get('amount');
+        Log::debug(sprintf('Found amount is %s', $amount));
         /** @var Carbon $date */
         $date          = session('end', Carbon::now()->endOfMonth());
         $leftOnAccount = $piggyBank->leftOnAccount($date);
         $savedSoFar    = strval($piggyBank->currentRelevantRep()->currentamount);
         $leftToSave    = bcsub($piggyBank->targetamount, $savedSoFar);
-        $maxAmount     = round(min($leftOnAccount, $leftToSave), 2);
+        $maxAmount     = strval(min(round($leftOnAccount, 12), round($leftToSave, 12)));
 
-        if ($amount <= $maxAmount) {
+        if (bccomp($amount, $maxAmount) <= 0) {
             $repetition                = $piggyBank->currentRelevantRep();
             $currentAmount             = $repetition->currentamount ?? '0';
             $repetition->currentamount = bcadd($currentAmount, $amount);
@@ -299,18 +312,19 @@ class PiggyBankController extends Controller
     }
 
     /**
+     * @param Request                      $request
      * @param PiggyBankRepositoryInterface $repository
      * @param PiggyBank                    $piggyBank
      *
      * @return \Illuminate\Http\RedirectResponse
      */
-    public function postRemove(PiggyBankRepositoryInterface $repository, PiggyBank $piggyBank)
+    public function postRemove(Request $request, PiggyBankRepositoryInterface $repository, PiggyBank $piggyBank)
     {
-        $amount = strval(round(Input::get('amount'), 2));
+        $amount = strval(round($request->get('amount'), 12));
 
         $savedSoFar = $piggyBank->currentRelevantRep()->currentamount;
 
-        if ($amount <= $savedSoFar) {
+        if (bccomp($amount, $savedSoFar) <= 0) {
             $repetition                = $piggyBank->currentRelevantRep();
             $repetition->currentamount = bcsub($repetition->currentamount, $amount);
             $repetition->save();
@@ -384,7 +398,7 @@ class PiggyBankController extends Controller
         Session::flash('success', strval(trans('firefly.stored_piggy_bank', ['name' => e($piggyBank->name)])));
         Preferences::mark();
 
-        if (intval(Input::get('create_another')) === 1) {
+        if (intval($request->get('create_another')) === 1) {
             Session::put('piggy-banks.create.fromStore', true);
 
             return redirect(route('piggy-banks.create'))->withInput();
@@ -400,7 +414,7 @@ class PiggyBankController extends Controller
      * @param PiggyBankFormRequest         $request
      * @param PiggyBank                    $piggyBank
      *
-     * @return $this
+     * @return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector
      */
     public function update(PiggyBankRepositoryInterface $repository, PiggyBankFormRequest $request, PiggyBank $piggyBank)
     {
@@ -410,7 +424,7 @@ class PiggyBankController extends Controller
         Session::flash('success', strval(trans('firefly.updated_piggy_bank', ['name' => e($piggyBank->name)])));
         Preferences::mark();
 
-        if (intval(Input::get('return_to_edit')) === 1) {
+        if (intval($request->get('return_to_edit')) === 1) {
             Session::put('piggy-banks.edit.fromUpdate', true);
 
             return redirect(route('piggy-banks.edit', [$piggyBank->id]));

app/Http/Controllers/Popup/ReportController.php

@@ -17,7 +17,7 @@ namespace FireflyIII\Http\Controllers\Popup;
 use Carbon\Carbon;
 use FireflyIII\Exceptions\FireflyException;
 use FireflyIII\Helpers\Collection\BalanceLine;
-use FireflyIII\Helpers\Collector\JournalCollector;
+use FireflyIII\Helpers\Collector\JournalCollectorInterface;
 use FireflyIII\Http\Controllers\Controller;
 use FireflyIII\Models\Transaction;
 use FireflyIII\Models\TransactionJournal;
@@ -102,7 +102,8 @@ class ReportController extends Controller
 
         switch (true) {
             case ($role === BalanceLine::ROLE_DEFAULTROLE && !is_null($budget->id)):
-                $collector = new JournalCollector(auth()->user());
+                /** @var JournalCollectorInterface $collector */
+                $collector = app(JournalCollectorInterface::class, [auth()->user()]);
                 $collector
                     ->setAccounts(new Collection([$account]))
                     ->setRange($attributes['startDate'], $attributes['endDate'])
@@ -112,8 +113,8 @@ class ReportController extends Controller
                 break;
             case ($role === BalanceLine::ROLE_DEFAULTROLE && is_null($budget->id)):
                 $budget->name = strval(trans('firefly.no_budget'));
-                // collector
-                $collector = new JournalCollector(auth()->user());
+                /** @var JournalCollectorInterface $collector */
+                $collector = app(JournalCollectorInterface::class, [auth()->user()]);
                 $collector
                     ->setAccounts(new Collection([$account]))
                     ->setTypes($types)
@@ -122,8 +123,8 @@ class ReportController extends Controller
                 $journals = $collector->getJournals();
                 break;
             case ($role === BalanceLine::ROLE_DIFFROLE):
-                // journals no budget, not corrected by a tag.
-                $collector = new JournalCollector(auth()->user());
+                /** @var JournalCollectorInterface $collector */
+                $collector = app(JournalCollectorInterface::class, [auth()->user()]);
                 $collector
                     ->setAccounts(new Collection([$account]))
                     ->setTypes($types)
@@ -133,8 +134,8 @@ class ReportController extends Controller
 
                 $budget->name = strval(trans('firefly.leftUnbalanced'));
                 $journals     = $journals->filter(
-                    function (TransactionJournal $journal) {
-                        $tags = $journal->tags()->where('tagMode', 'balancingAct')->count();
+                    function (Transaction $transaction) {
+                        $tags = $transaction->transactionJournal->tags()->where('tagMode', 'balancingAct')->count();
                         if ($tags === 0) {
                             return true;
                         }
@@ -167,7 +168,8 @@ class ReportController extends Controller
         /** @var BudgetRepositoryInterface $repository */
         $repository = app(BudgetRepositoryInterface::class);
         $budget     = $repository->find(intval($attributes['budgetId']));
-        $collector  = new JournalCollector(auth()->user());
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
 
         $collector
             ->setAccounts($attributes['accounts'])
@@ -200,8 +202,8 @@ class ReportController extends Controller
         $repository = app(CategoryRepositoryInterface::class);
         $category   = $repository->find(intval($attributes['categoryId']));
         $types      = [TransactionType::WITHDRAWAL, TransactionType::TRANSFER];
-        // get journal collector instead:
-        $collector = new JournalCollector(auth()->user());
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
         $collector->setAccounts($attributes['accounts'])->setTypes($types)
                   ->setRange($attributes['startDate'], $attributes['endDate'])
                   ->setCategory($category);
@@ -225,9 +227,10 @@ class ReportController extends Controller
         /** @var AccountRepositoryInterface $repository */
         $repository = app(AccountRepositoryInterface::class);
 
-        $account   = $repository->find(intval($attributes['accountId']));
-        $types     = [TransactionType::WITHDRAWAL, TransactionType::TRANSFER];
-        $collector = new JournalCollector(auth()->user());
+        $account = $repository->find(intval($attributes['accountId']));
+        $types   = [TransactionType::WITHDRAWAL, TransactionType::TRANSFER];
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
         $collector->setAccounts(new Collection([$account]))->setRange($attributes['startDate'], $attributes['endDate'])->setTypes($types);
         $journals = $collector->getJournals();
         $report   = $attributes['accounts']->pluck('id')->toArray(); // accounts used in this report
@@ -262,7 +265,8 @@ class ReportController extends Controller
         $repository = app(AccountRepositoryInterface::class);
         $account    = $repository->find(intval($attributes['accountId']));
         $types      = [TransactionType::DEPOSIT, TransactionType::TRANSFER];
-        $collector  = new JournalCollector(auth()->user());
+        /** @var JournalCollectorInterface $collector */
+        $collector = app(JournalCollectorInterface::class, [auth()->user()]);
         $collector->setAccounts(new Collection([$account]))->setRange($attributes['startDate'], $attributes['endDate'])->setTypes($types);
         $journals = $collector->getJournals();
         $report   = $attributes['accounts']->pluck('id')->toArray(); // accounts used in this report

app/Http/Controllers/PreferencesController.php

@@ -113,6 +113,7 @@ class PreferencesController extends Controller
      * @param TokenFormRequest $request
      *
      * @return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector
+     * @SuppressWarnings(PHPMD.UnusedFormalParameter) // it's unused but the class does some validation.
      */
     public function postCode(TokenFormRequest $request)
     {
@@ -150,7 +151,7 @@ class PreferencesController extends Controller
 
         // custom fiscal year
         $customFiscalYear = intval($request->get('customFiscalYear')) === 1;
-        $fiscalYearStart  = date('m-d', strtotime($request->get('fiscalYearStart')));
+        $fiscalYearStart  = date('m-d', strtotime(strval($request->get('fiscalYearStart'))));
         Preferences::set('customFiscalYear', $customFiscalYear);
         Preferences::set('fiscalYearStart', $fiscalYearStart);
 
@@ -166,13 +167,17 @@ class PreferencesController extends Controller
             Preferences::set('transactionPageSize', 50);
         }
 
-        // two factor auth
-        $twoFactorAuthEnabled   = intval($request->get('twoFactorAuthEnabled'));
-        $hasTwoFactorAuthSecret = !is_null(Preferences::get('twoFactorAuthSecret'));
+        $twoFactorAuthEnabled   = false;
+        $hasTwoFactorAuthSecret = false;
+        if (!auth()->user()->hasRole('demo')) {
+            // two factor auth
+            $twoFactorAuthEnabled   = intval($request->get('twoFactorAuthEnabled'));
+            $hasTwoFactorAuthSecret = !is_null(Preferences::get('twoFactorAuthSecret'));
 
-        // If we already have a secret, just set the two factor auth enabled to 1, and let the user continue with the existing secret.
-        if ($hasTwoFactorAuthSecret) {
-            Preferences::set('twoFactorAuthEnabled', $twoFactorAuthEnabled);
+            // If we already have a secret, just set the two factor auth enabled to 1, and let the user continue with the existing secret.
+            if ($hasTwoFactorAuthSecret) {
+                Preferences::set('twoFactorAuthEnabled', $twoFactorAuthEnabled);
+            }
         }
 
         // language:

app/Http/Controllers/ProfileController.php

@@ -13,6 +13,7 @@ declare(strict_types = 1);
 
 namespace FireflyIII\Http\Controllers;
 
+use FireflyIII\Exceptions\ValidationException;
 use FireflyIII\Http\Requests\DeleteAccountFormRequest;
 use FireflyIII\Http\Requests\ProfileFormRequest;
 use FireflyIII\Repositories\User\UserRepositoryInterface;
@@ -51,6 +52,12 @@ class ProfileController extends Controller
      */
     public function changePassword()
     {
+        if (auth()->user()->hasRole('demo')) {
+            Session::flash('info', strval(trans('firefly.cannot_change_demo')));
+
+            return redirect(route('profile.index'));
+        }
+
         $title        = auth()->user()->email;
         $subTitle     = strval(trans('firefly.change_your_password'));
         $subTitleIcon = 'fa-key';
@@ -63,6 +70,12 @@ class ProfileController extends Controller
      */
     public function deleteAccount()
     {
+        if (auth()->user()->hasRole('demo')) {
+            Session::flash('info', strval(trans('firefly.cannot_delete_demo')));
+
+            return redirect(route('profile.index'));
+        }
+
         $title        = auth()->user()->email;
         $subTitle     = strval(trans('firefly.delete_account'));
         $subTitleIcon = 'fa-trash';
@@ -83,29 +96,36 @@ class ProfileController extends Controller
     }
 
     /**
-     * @param ProfileFormRequest $request
+     * @param ProfileFormRequest      $request
+     * @param UserRepositoryInterface $repository
      *
-     * @return \Illuminate\Http\RedirectResponse|\Illuminate\View\View
+     * @return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector
      */
-    public function postChangePassword(ProfileFormRequest $request)
+    public function postChangePassword(ProfileFormRequest $request, UserRepositoryInterface $repository)
     {
+        if (auth()->user()->hasRole('demo')) {
+            Session::flash('info', strval(trans('firefly.cannot_change_demo')));
+
+            return redirect(route('profile.index'));
+        }
+
         // old, new1, new2
         if (!Hash::check($request->get('current_password'), auth()->user()->password)) {
             Session::flash('error', strval(trans('firefly.invalid_current_password')));
 
             return redirect(route('profile.change-password'));
         }
-        $result = $this->validatePassword($request->get('current_password'), $request->get('new_password'));
-        if (!($result === true)) {
-            Session::flash('error', $result);
+
+        try {
+            $this->validatePassword($request->get('current_password'), $request->get('new_password'));
+        } catch (ValidationException $e) {
+            Session::flash('error', $e->getMessage());
 
             return redirect(route('profile.change-password'));
         }
 
         // update the user with the new password.
-        auth()->user()->password = bcrypt($request->get('new_password'));
-        auth()->user()->save();
-
+        $repository->changePassword(auth()->user(), $request->get('new_password'));
         Session::flash('success', strval(trans('firefly.password_changed')));
 
         return redirect(route('profile.index'));
@@ -119,6 +139,12 @@ class ProfileController extends Controller
      */
     public function postDeleteAccount(UserRepositoryInterface $repository, DeleteAccountFormRequest $request)
     {
+        if (auth()->user()->hasRole('demo')) {
+            Session::flash('info', strval(trans('firefly.cannot_delete_demo')));
+
+            return redirect(route('profile.index'));
+        }
+
         // old, new1, new2
         if (!Hash::check($request->get('password'), auth()->user()->password)) {
             Session::flash('error', strval(trans('firefly.invalid_password')));
@@ -140,16 +166,16 @@ class ProfileController extends Controller
     }
 
     /**
-     *
      * @param string $old
-     * @param string $new1
+     * @param string $new
      *
-     * @return string|bool
+     * @return bool
+     * @throws ValidationException
      */
-    protected function validatePassword(string $old, string $new1)
+    protected function validatePassword(string $old, string $new): bool
     {
-        if ($new1 == $old) {
-            return trans('firefly.should_change');
+        if ($new === $old) {
+            throw new ValidationException(strval(trans('firefly.should_change')));
         }
 
         return true;

app/Http/Controllers/Report/BalanceController.php

@@ -57,4 +57,4 @@ class BalanceController extends Controller
 
         return $result;
     }
-}
+}