Passed
Push — master ( 49de82...9f1fc0 )
by James
35:45 queued 23:44
created
app/Http/Controllers/DebugController.php 1 patch
Indentation   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -175,12 +175,12 @@  discard block
 block discarded – undo
175 175
 
176 176
         return view(
177 177
             'debug', compact(
178
-                       'phpVersion', 'extensions', 'localeAttempts', 'appEnv', 'appDebug', 'logChannel', 'appLogLevel', 'now', 'drivers',
179
-                       'currentDriver', 'loginProvider', 'storageDisks',
180
-                       'userAgent', 'displayErrors', 'errorReporting', 'phpOs', 'interface', 'logContent', 'cacheDriver', 'isSandstorm',
181
-                       'trustedProxies',
182
-                       'toSandbox'
183
-                   )
178
+                        'phpVersion', 'extensions', 'localeAttempts', 'appEnv', 'appDebug', 'logChannel', 'appLogLevel', 'now', 'drivers',
179
+                        'currentDriver', 'loginProvider', 'storageDisks',
180
+                        'userAgent', 'displayErrors', 'errorReporting', 'phpOs', 'interface', 'logContent', 'cacheDriver', 'isSandstorm',
181
+                        'trustedProxies',
182
+                        'toSandbox'
183
+                    )
184 184
         );
185 185
     }
186 186
 
@@ -193,16 +193,16 @@  discard block
 block discarded – undo
193 193
     {
194 194
         $set    = RouteFacade::getRoutes();
195 195
         $ignore = ['chart.', 'javascript.', 'json.', 'report-data.', 'popup.', 'debugbar.', 'attachments.download', 'attachments.preview',
196
-                   'bills.rescan', 'budgets.income', 'currencies.def', 'error', 'flush', 'help.show', 'import.file',
197
-                   'login', 'logout', 'password.reset', 'profile.confirm-email-change', 'profile.undo-email-change',
198
-                   'register', 'report.options', 'routes', 'rule-groups.down', 'rule-groups.up', 'rules.up', 'rules.down',
199
-                   'rules.select', 'search.search', 'test-flash', 'transactions.link.delete', 'transactions.link.switch',
200
-                   'two-factor.lost', 'reports.options', 'debug', 'import.create-job', 'import.download', 'import.start', 'import.status.json',
201
-                   'preferences.delete-code', 'rules.test-triggers', 'piggy-banks.remove-money', 'piggy-banks.add-money',
202
-                   'accounts.reconcile.transactions', 'accounts.reconcile.overview',
203
-                   'transactions.clone', 'two-factor.index', 'api.v1', 'installer.', 'attachments.view', 'import.create',
204
-                   'import.job.download', 'import.job.start', 'import.job.status.json', 'import.job.store', 'recurring.events',
205
-                   'recurring.suggest',
196
+                    'bills.rescan', 'budgets.income', 'currencies.def', 'error', 'flush', 'help.show', 'import.file',
197
+                    'login', 'logout', 'password.reset', 'profile.confirm-email-change', 'profile.undo-email-change',
198
+                    'register', 'report.options', 'routes', 'rule-groups.down', 'rule-groups.up', 'rules.up', 'rules.down',
199
+                    'rules.select', 'search.search', 'test-flash', 'transactions.link.delete', 'transactions.link.switch',
200
+                    'two-factor.lost', 'reports.options', 'debug', 'import.create-job', 'import.download', 'import.start', 'import.status.json',
201
+                    'preferences.delete-code', 'rules.test-triggers', 'piggy-banks.remove-money', 'piggy-banks.add-money',
202
+                    'accounts.reconcile.transactions', 'accounts.reconcile.overview',
203
+                    'transactions.clone', 'two-factor.index', 'api.v1', 'installer.', 'attachments.view', 'import.create',
204
+                    'import.job.download', 'import.job.start', 'import.job.status.json', 'import.job.store', 'recurring.events',
205
+                    'recurring.suggest',
206 206
         ];
207 207
         $return = ' ';
208 208
         /** @var Route $route */
Please login to merge, or discard this patch.
app/Http/Controllers/Report/BudgetController.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
     {
65 65
         parent::__construct();
66 66
         $this->middleware(
67
-            function ($request, $next) {
67
+            function($request, $next) {
68 68
                 $this->opsRepository = app(OperationsRepositoryInterface::class);
69 69
                 $this->repository    = app(BudgetRepositoryInterface::class);
70 70
                 $this->blRepository  = app(BudgetLimitRepositoryInterface::class);
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
                     $sourceAccountId = $journal['source_account_id'];
108 108
 
109 109
 
110
-                    $report[$sourceAccountId]['currencies'][$currencyId]                           = $report[$sourceAccountId]['currencies'][$currencyId] ?? [
110
+                    $report[$sourceAccountId]['currencies'][$currencyId] = $report[$sourceAccountId]['currencies'][$currencyId] ?? [
111 111
                             'currency_id'             => $currency['currency_id'],
112 112
                             'currency_symbol'         => $currency['currency_symbol'],
113 113
                             'currency_name'           => $currency['currency_name'],
@@ -272,7 +272,7 @@  discard block
 block discarded – undo
272 272
 
273 273
                 foreach ($budget['transaction_journals'] as $journal) {
274 274
                     // add currency info to report array:
275
-                    $report[$budgetId]['currencies'][$currencyId]        = $report[$budgetId]['currencies'][$currencyId] ?? [
275
+                    $report[$budgetId]['currencies'][$currencyId] = $report[$budgetId]['currencies'][$currencyId] ?? [
276 276
                             'sum'                     => '0',
277 277
                             'sum_pct'                 => '0',
278 278
                             'currency_id'             => $currency['currency_id'],
@@ -316,7 +316,7 @@  discard block
 block discarded – undo
316 316
      */
317 317
     public function general(Collection $accounts, Carbon $start, Carbon $end)
318 318
     {
319
-        $report          = [
319
+        $report = [
320 320
             'budgets' => [],
321 321
             'sums'    => [],
322 322
         ];
@@ -387,7 +387,7 @@  discard block
 block discarded – undo
387 387
                 'no_budget'     => true,
388 388
                 'budget_limits' => [],
389 389
             ];
390
-        $noBudget             = $this->nbRepository->sumExpenses($start, $end);
390
+        $noBudget = $this->nbRepository->sumExpenses($start, $end);
391 391
         foreach ($noBudget as $noBudgetEntry) {
392 392
             $report['budgets'][0]['budget_limits'][] = [
393 393
                 'budget_limit_id'         => null,
Please login to merge, or discard this patch.
app/Repositories/Account/AccountRepository.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -308,7 +308,7 @@  discard block
 block discarded – undo
308 308
     {
309 309
         /** @var Collection $result */
310 310
         $query = $this->user->accounts()->with(
311
-            ['accountmeta' => function (HasMany $query) {
311
+            ['accountmeta' => function(HasMany $query) {
312 312
                 $query->where('name', 'account_role');
313 313
             }]
314 314
         );
@@ -615,7 +615,7 @@  discard block
 block discarded – undo
615 615
     {
616 616
         /** @var Collection $result */
617 617
         $query = $this->user->accounts()->with(
618
-            ['accountmeta' => function (HasMany $query) {
618
+            ['accountmeta' => function(HasMany $query) {
619 619
                 $query->where('name', 'account_role');
620 620
             }]
621 621
         );
Please login to merge, or discard this patch.
app/Http/Controllers/Account/IndexController.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
 
52 52
         // translations:
53 53
         $this->middleware(
54
-            function ($request, $next) {
54
+            function($request, $next) {
55 55
                 app('view')->share('mainTitleIcon', 'fa-credit-card');
56 56
                 app('view')->share('title', (string)trans('firefly.accounts'));
57 57
 
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
         $activities    = app('steam')->getLastActivities($ids);
94 94
 
95 95
         $accounts->each(
96
-            function (Account $account) use ($activities, $startBalances, $endBalances) {
96
+            function(Account $account) use ($activities, $startBalances, $endBalances) {
97 97
                 $account->lastActivityDate  = $this->isInArray($activities, $account->id);
98 98
                 $account->startBalance      = $this->isInArray($startBalances, $account->id);
99 99
                 $account->endBalance        = $this->isInArray($endBalances, $account->id);
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
         $accounts = new LengthAwarePaginator($accounts, $total, $pageSize, $page);
109 109
         $accounts->setPath(route('accounts.inactive.index', [$objectType]));
110 110
 
111
-        return view('accounts.index', compact('objectType','inactivePage', 'subTitleIcon', 'subTitle', 'page', 'accounts'));
111
+        return view('accounts.index', compact('objectType', 'inactivePage', 'subTitleIcon', 'subTitle', 'page', 'accounts'));
112 112
 
113 113
     }
114 114
 
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
         $activities    = app('steam')->getLastActivities($ids);
148 148
 
149 149
         $accounts->each(
150
-            function (Account $account) use ($activities, $startBalances, $endBalances) {
150
+            function(Account $account) use ($activities, $startBalances, $endBalances) {
151 151
                 $account->lastActivityDate  = $this->isInArray($activities, $account->id);
152 152
                 $account->startBalance      = $this->isInArray($startBalances, $account->id);
153 153
                 $account->endBalance        = $this->isInArray($endBalances, $account->id);
Please login to merge, or discard this patch.
app/Api/V1/Controllers/AttachmentController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
     {
60 60
         parent::__construct();
61 61
         $this->middleware(
62
-            function ($request, $next) {
62
+            function($request, $next) {
63 63
                 /** @var User $user */
64 64
                 $user             = auth()->user();
65 65
                 $this->repository = app(AttachmentRepositoryInterface::class);
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
             if ('' === $content) {
109 109
                 throw new FireflyException('No file has been uploaded for this attachment (yet).');
110 110
             }
111
-            $quoted  = sprintf('"%s"', addcslashes(basename($attachment->filename), '"\\'));
111
+            $quoted = sprintf('"%s"', addcslashes(basename($attachment->filename), '"\\'));
112 112
 
113 113
             /** @var LaravelResponse $response */
114 114
             $response = response($content);
Please login to merge, or discard this patch.
app/Repositories/TransactionGroup/TransactionGroupRepository.php 1 patch
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -96,9 +96,9 @@
 block discarded – undo
96 96
     {
97 97
         $journals = $group->transactionJournals->pluck('id')->toArray();
98 98
         $set      = Attachment::whereIn('attachable_id', $journals)
99
-                              ->where('attachable_type', TransactionJournal::class)
100
-                              ->where('uploaded', 1)
101
-                              ->whereNull('deleted_at')->get();
99
+                                ->where('attachable_type', TransactionJournal::class)
100
+                                ->where('uploaded', 1)
101
+                                ->whereNull('deleted_at')->get();
102 102
 
103 103
         $result = [];
104 104
         /** @var Attachment $attachment */
Please login to merge, or discard this patch.