From 30515c69f390f8481e4c607f64a45b78f30a8358 Mon Sep 17 00:00:00 2001
From: Hillel Coren <hillelcoren@gmail.com>
Date: Mon, 30 Jan 2017 21:49:42 +0200
Subject: [PATCH] php-cs-fixer

---
 .php_cs.dist                                  | 11 ++---
 app/Events/InvoiceInvitationWasEmailed.php    |  1 +
 app/Events/QuoteInvitationWasEmailed.php      |  1 +
 app/Http/Controllers/AccountController.php    |  1 +
 .../Controllers/AccountGatewayController.php  |  1 +
 app/Http/Controllers/ClientApiController.php  |  1 +
 app/Http/Controllers/ExpenseApiController.php |  1 +
 app/Http/Controllers/InvoiceApiController.php |  1 +
 app/Http/Controllers/InvoiceController.php    |  6 ++-
 .../Controllers/OnlinePaymentController.php   |  5 +-
 app/Http/Controllers/PaymentApiController.php |  1 +
 app/Http/Controllers/ProductApiController.php |  1 +
 app/Http/Controllers/TaxRateApiController.php |  1 +
 app/Http/Controllers/UserController.php       |  1 +
 app/Http/Controllers/VendorApiController.php  |  1 +
 app/Includes/parsecsv.lib.php                 | 48 +++++++++++++++++++
 app/Jobs/SendInvoiceEmail.php                 |  1 +
 app/Jobs/SendNotificationEmail.php            |  4 ++
 app/Jobs/SendPushNotification.php             |  1 +
 app/Models/Account.php                        |  5 +-
 app/Models/EntityModel.php                    |  1 +
 app/Models/Invoice.php                        |  8 ++--
 app/Models/Traits/GeneratesNumbers.php        |  2 +
 app/Models/User.php                           |  1 +
 app/Ninja/Import/BaseTransformer.php          |  2 +
 app/Ninja/Mailers/ContactMailer.php           |  1 +
 app/Ninja/Notifications/PushFactory.php       |  1 +
 app/Ninja/Presenters/TaskPresenter.php        |  1 +
 .../Repositories/DashboardRepository.php      |  3 ++
 app/Ninja/Repositories/InvoiceRepository.php  |  1 +
 app/Policies/AccountGatewayPolicy.php         |  3 +-
 app/Policies/BankAccountPolicy.php            |  3 +-
 app/Policies/DocumentPolicy.php               |  3 +-
 app/Policies/EntityPolicy.php                 |  3 +-
 app/Policies/ExpenseCategoryPolicy.php        |  3 +-
 app/Policies/ExpensePolicy.php                |  3 +-
 app/Policies/GenericEntityPolicy.php          |  1 +
 app/Policies/PaymentTermPolicy.php            |  3 +-
 app/Policies/QuotePolicy.php                  |  3 +-
 app/Policies/TaskPolicy.php                   |  3 +-
 app/Policies/TaxRatePolicy.php                |  3 +-
 app/Policies/VendorPolicy.php                 |  3 +-
 app/Services/CreditService.php                |  1 +
 app/Services/ProductService.php               |  1 +
 app/Services/ProjectService.php               |  2 +
 app/Services/PushService.php                  |  1 +
 46 files changed, 126 insertions(+), 27 deletions(-)

diff --git a/.php_cs.dist b/.php_cs.dist
index ffab5d513e..a1c34588f8 100644
--- a/.php_cs.dist
+++ b/.php_cs.dist
@@ -11,7 +11,7 @@ $finder = PhpCsFixer\Finder::create()
     ->ignoreVCS(true);
 
 return PhpCsFixer\Config::create()
-    ->setRules(array(
+    ->setRules([
         '@PSR2' => true,
         'binary_operator_spaces' => true,
         'blank_line_after_namespace' => true,
@@ -61,6 +61,7 @@ return PhpCsFixer\Config::create()
         'not_operator_with_successor_space' => true,
         'object_operator_without_whitespace' => true,
         'ordered_imports' => true,
+        'phpdoc_add_missing_param_annotation' => true,
         'phpdoc_align' => true,
         'phpdoc_indent' => true,
         'phpdoc_inline_tag' => true,
@@ -74,10 +75,8 @@ return PhpCsFixer\Config::create()
         'phpdoc_trim' => true,
         'phpdoc_types' => true,
         'phpdoc_var_without_name' => true,
-        //'psr4' => true,
         'self_accessor' => true,
         'short_scalar_cast' => true,
-        //'simplified_null_return' => true,
         'single_blank_line_at_eof' => true,
         'single_blank_line_before_namespace' => true,
         'single_import_per_statement' => true,
@@ -93,8 +92,6 @@ return PhpCsFixer\Config::create()
         'unary_operator_spaces' => true,
         'visibility_required' => true,
         'whitespace_after_comma_in_array' => true,
-        //'strict_param' => true,
         'array_syntax' => array('syntax' => 'short'),
-    ))
-    ->setFinder($finder)
-;
+    ])
+    ->setFinder($finder);
diff --git a/app/Events/InvoiceInvitationWasEmailed.php b/app/Events/InvoiceInvitationWasEmailed.php
index 99071ed325..6c10bcc119 100644
--- a/app/Events/InvoiceInvitationWasEmailed.php
+++ b/app/Events/InvoiceInvitationWasEmailed.php
@@ -26,6 +26,7 @@ class InvoiceInvitationWasEmailed extends Event
      * Create a new event instance.
      *
      * @param Invitation $invitation
+     * @param mixed      $notes
      */
     public function __construct(Invitation $invitation, $notes)
     {
diff --git a/app/Events/QuoteInvitationWasEmailed.php b/app/Events/QuoteInvitationWasEmailed.php
index 4efc069f4a..b6de1a33c7 100644
--- a/app/Events/QuoteInvitationWasEmailed.php
+++ b/app/Events/QuoteInvitationWasEmailed.php
@@ -26,6 +26,7 @@ class QuoteInvitationWasEmailed extends Event
      * Create a new event instance.
      *
      * @param Invitation $invitation
+     * @param mixed      $notes
      */
     public function __construct(Invitation $invitation, $notes)
     {
diff --git a/app/Http/Controllers/AccountController.php b/app/Http/Controllers/AccountController.php
index 708c994979..1e57391a5b 100644
--- a/app/Http/Controllers/AccountController.php
+++ b/app/Http/Controllers/AccountController.php
@@ -248,6 +248,7 @@ class AccountController extends BaseController
     /**
      * @param $entityType
      * @param $visible
+     * @param mixed $filter
      *
      * @return mixed
      */
diff --git a/app/Http/Controllers/AccountGatewayController.php b/app/Http/Controllers/AccountGatewayController.php
index 8266593daf..f1b4d4144d 100644
--- a/app/Http/Controllers/AccountGatewayController.php
+++ b/app/Http/Controllers/AccountGatewayController.php
@@ -166,6 +166,7 @@ class AccountGatewayController extends BaseController
 
     /**
      * Stores new account.
+     * @param mixed $accountGatewayPublicId
      */
     public function save($accountGatewayPublicId = false)
     {
diff --git a/app/Http/Controllers/ClientApiController.php b/app/Http/Controllers/ClientApiController.php
index d04e3eb11f..a6c2b068d4 100644
--- a/app/Http/Controllers/ClientApiController.php
+++ b/app/Http/Controllers/ClientApiController.php
@@ -124,6 +124,7 @@ class ClientApiController extends BaseAPIController
      *     description="an ""unexpected"" error"
      *   )
      * )
+     * @param mixed $publicId
      */
     public function update(UpdateClientRequest $request, $publicId)
     {
diff --git a/app/Http/Controllers/ExpenseApiController.php b/app/Http/Controllers/ExpenseApiController.php
index 8bf670348c..169fcd43be 100644
--- a/app/Http/Controllers/ExpenseApiController.php
+++ b/app/Http/Controllers/ExpenseApiController.php
@@ -103,6 +103,7 @@ class ExpenseApiController extends BaseAPIController
      *     description="an ""unexpected"" error"
      *   )
      * )
+     * @param mixed $publicId
      */
     public function update(UpdateExpenseRequest $request, $publicId)
     {
diff --git a/app/Http/Controllers/InvoiceApiController.php b/app/Http/Controllers/InvoiceApiController.php
index 21d3a940ea..f53ec553c5 100644
--- a/app/Http/Controllers/InvoiceApiController.php
+++ b/app/Http/Controllers/InvoiceApiController.php
@@ -325,6 +325,7 @@ class InvoiceApiController extends BaseAPIController
      *     description="an ""unexpected"" error"
      *   )
      * )
+     * @param mixed $publicId
      */
     public function update(UpdateInvoiceAPIRequest $request, $publicId)
     {
diff --git a/app/Http/Controllers/InvoiceController.php b/app/Http/Controllers/InvoiceController.php
index c87cffef34..f084f1b803 100644
--- a/app/Http/Controllers/InvoiceController.php
+++ b/app/Http/Controllers/InvoiceController.php
@@ -454,7 +454,8 @@ class InvoiceController extends BaseController
     /**
      * Display the specified resource.
      *
-     * @param int $id
+     * @param int   $id
+     * @param mixed $publicId
      *
      * @return Response
      */
@@ -468,7 +469,8 @@ class InvoiceController extends BaseController
     /**
      * Remove the specified resource from storage.
      *
-     * @param int $id
+     * @param int   $id
+     * @param mixed $entityType
      *
      * @return Response
      */
diff --git a/app/Http/Controllers/OnlinePaymentController.php b/app/Http/Controllers/OnlinePaymentController.php
index 4e22b008e0..e25defd2a0 100644
--- a/app/Http/Controllers/OnlinePaymentController.php
+++ b/app/Http/Controllers/OnlinePaymentController.php
@@ -60,8 +60,9 @@ class OnlinePaymentController extends BaseController
 
     /**
      * @param $invitationKey
-     * @param bool $gatewayType
-     * @param bool $sourceId
+     * @param bool  $gatewayType
+     * @param bool  $sourceId
+     * @param mixed $gatewayTypeAlias
      *
      * @return \Illuminate\Http\RedirectResponse
      */
diff --git a/app/Http/Controllers/PaymentApiController.php b/app/Http/Controllers/PaymentApiController.php
index 78caf418e4..446b55d35c 100644
--- a/app/Http/Controllers/PaymentApiController.php
+++ b/app/Http/Controllers/PaymentApiController.php
@@ -71,6 +71,7 @@ class PaymentApiController extends BaseAPIController
      *     description="an ""unexpected"" error"
      *   )
      * )
+     * @param mixed $publicId
      */
     public function update(UpdatePaymentRequest $request, $publicId)
     {
diff --git a/app/Http/Controllers/ProductApiController.php b/app/Http/Controllers/ProductApiController.php
index 39b3980087..6097f1a49d 100644
--- a/app/Http/Controllers/ProductApiController.php
+++ b/app/Http/Controllers/ProductApiController.php
@@ -107,6 +107,7 @@ class ProductApiController extends BaseAPIController
      *     description="an ""unexpected"" error"
      *   )
      * )
+     * @param mixed $publicId
      */
     public function update(UpdateProductRequest $request, $publicId)
     {
diff --git a/app/Http/Controllers/TaxRateApiController.php b/app/Http/Controllers/TaxRateApiController.php
index 128c1db522..f57f74200a 100644
--- a/app/Http/Controllers/TaxRateApiController.php
+++ b/app/Http/Controllers/TaxRateApiController.php
@@ -104,6 +104,7 @@ class TaxRateApiController extends BaseAPIController
      *     description="an ""unexpected"" error"
      *   )
      * )
+     * @param mixed $publicId
      */
     public function update(UpdateTaxRateRequest $request, $publicId)
     {
diff --git a/app/Http/Controllers/UserController.php b/app/Http/Controllers/UserController.php
index e714bf71b2..2ac481074b 100644
--- a/app/Http/Controllers/UserController.php
+++ b/app/Http/Controllers/UserController.php
@@ -151,6 +151,7 @@ class UserController extends BaseController
 
     /**
      * Stores new account.
+     * @param mixed $userPublicId
      */
     public function save($userPublicId = false)
     {
diff --git a/app/Http/Controllers/VendorApiController.php b/app/Http/Controllers/VendorApiController.php
index 24ea760116..392f1721f0 100644
--- a/app/Http/Controllers/VendorApiController.php
+++ b/app/Http/Controllers/VendorApiController.php
@@ -109,6 +109,7 @@ class VendorApiController extends BaseAPIController
      *     description="an ""unexpected"" error"
      *   )
      * )
+     * @param mixed $publicId
      */
     public function update(UpdateVendorRequest $request, $publicId)
     {
diff --git a/app/Includes/parsecsv.lib.php b/app/Includes/parsecsv.lib.php
index c575066469..232452ec35 100644
--- a/app/Includes/parsecsv.lib.php
+++ b/app/Includes/parsecsv.lib.php
@@ -148,6 +148,10 @@ class parseCSV
      * Constructor.
      *
      * @param   input   CSV file or string
+     * @param null|mixed $input
+     * @param null|mixed $offset
+     * @param null|mixed $limit
+     * @param null|mixed $conditions
      *
      * @return nothing
      */
@@ -175,6 +179,10 @@ class parseCSV
      * Parse CSV file or string.
      *
      * @param   input   CSV file or string
+     * @param null|mixed $input
+     * @param null|mixed $offset
+     * @param null|mixed $limit
+     * @param null|mixed $conditions
      *
      * @return nothing
      */
@@ -211,6 +219,10 @@ class parseCSV
      * @param   data     2D array with data
      * @param   append   append current data to end of target CSV if exists
      * @param   fields   field names
+     * @param null|mixed $file
+     * @param mixed      $data
+     * @param mixed      $append
+     * @param mixed      $fields
      *
      * @return true or false
      */
@@ -233,6 +245,11 @@ class parseCSV
      * @param   data        2D array with data
      * @param   fields      field names
      * @param   delimiter   delimiter used to separate data
+     * @param mixed      $output
+     * @param null|mixed $filename
+     * @param mixed      $data
+     * @param mixed      $fields
+     * @param null|mixed $delimiter
      *
      * @return CSV data using delimiter of choice, or default
      */
@@ -259,6 +276,8 @@ class parseCSV
      *
      * @param   input    input character encoding, uses default if left blank
      * @param   output   output character encoding, uses default if left blank
+     * @param null|mixed $input
+     * @param null|mixed $output
      *
      * @return nothing
      */
@@ -282,6 +301,11 @@ class parseCSV
      * @param   search_depth   number of rows to analyze
      * @param   preferred      preferred delimiter characters
      * @param   enclosure      enclosure character, default is double quote (").
+     * @param null|mixed $file
+     * @param mixed      $parse
+     * @param null|mixed $search_depth
+     * @param null|mixed $preferred
+     * @param null|mixed $enclosure
      *
      * @return delimiter character
      */
@@ -382,6 +406,7 @@ class parseCSV
      * Read file to string and call parse_string().
      *
      * @param   file   local CSV file
+     * @param null|mixed $file
      *
      * @return 2D array with CSV data, or false on failure
      */
@@ -401,6 +426,7 @@ class parseCSV
      * Parse CSV strings to arrays.
      *
      * @param   data   CSV string
+     * @param null|mixed $data
      *
      * @return 2D array with CSV data, or false on failure
      */
@@ -506,6 +532,11 @@ class parseCSV
      * @param   is_php      if a php die() call should be put on the first
      *                      line of the file, this is later ignored when read.
      * @param   delimiter   field delimiter to use
+     * @param mixed      $data
+     * @param mixed      $fields
+     * @param mixed      $append
+     * @param mixed      $is_php
+     * @param null|mixed $delimiter
      *
      * @return CSV data (text string)
      */
@@ -549,6 +580,7 @@ class parseCSV
      * Load local file or string.
      *
      * @param   input   local CSV file
+     * @param null|mixed $input
      *
      * @return true or false
      */
@@ -593,6 +625,8 @@ class parseCSV
      *
      * @param   row          array with values from a row
      * @param   conditions   specified conditions that the row must match
+     * @param mixed      $row
+     * @param null|mixed $conditions
      *
      * @return true of false
      */
@@ -629,6 +663,8 @@ class parseCSV
      *
      * @param   row          array with values from a row
      * @param   condition   specified condition that the row must match
+     * @param mixed $row
+     * @param mixed $condition
      *
      * @return true of false
      */
@@ -692,6 +728,7 @@ class parseCSV
      * Validates if the row is within the offset or not if sorting is disabled.
      *
      * @param   current_row   the current row number being processed
+     * @param mixed $current_row
      *
      * @return true of false
      */
@@ -709,6 +746,7 @@ class parseCSV
      *  - only used by unparse().
      *
      * @param   value   string to process
+     * @param null|mixed $value
      *
      * @return Processed value
      */
@@ -730,6 +768,7 @@ class parseCSV
      * Check file data.
      *
      * @param   file   local filename
+     * @param null|mixed $file
      *
      * @return true or false
      */
@@ -751,6 +790,10 @@ class parseCSV
      *  - only used by find_delimiter().
      *
      * @return special string used for delimiter selection, or false
+     * @param  mixed   $char
+     * @param  mixed   $array
+     * @param  mixed   $depth
+     * @param  mixed   $preferred
      */
     public function _check_count($char, $array, $depth, $preferred)
     {
@@ -786,6 +829,7 @@ class parseCSV
      * Read local file.
      *
      * @param   file   local filename
+     * @param null|mixed $file
      *
      * @return Data from file, or false on failure
      */
@@ -811,6 +855,10 @@ class parseCSV
      * @param   string   data to write to file
      * @param   mode     fopen() mode
      * @param   lock     flock() mode
+     * @param mixed $file
+     * @param mixed $string
+     * @param mixed $mode
+     * @param mixed $lock
      *
      * @return true or false
      */
diff --git a/app/Jobs/SendInvoiceEmail.php b/app/Jobs/SendInvoiceEmail.php
index 2b8f77573a..e1ea2c3990 100644
--- a/app/Jobs/SendInvoiceEmail.php
+++ b/app/Jobs/SendInvoiceEmail.php
@@ -37,6 +37,7 @@ class SendInvoiceEmail extends Job implements ShouldQueue
      * @param Invoice $invoice
      * @param string  $pdf
      * @param bool    $reminder
+     * @param mixed   $pdfString
      */
     public function __construct(Invoice $invoice, $reminder = false, $pdfString = false)
     {
diff --git a/app/Jobs/SendNotificationEmail.php b/app/Jobs/SendNotificationEmail.php
index fbb6904bb8..bd8e893967 100644
--- a/app/Jobs/SendNotificationEmail.php
+++ b/app/Jobs/SendNotificationEmail.php
@@ -41,6 +41,10 @@ class SendNotificationEmail extends Job implements ShouldQueue
      * @param UserMailer    $userMailer
      * @param ContactMailer $contactMailer
      * @param PushService   $pushService
+     * @param mixed         $user
+     * @param mixed         $invoice
+     * @param mixed         $type
+     * @param mixed         $payment
      */
     public function __construct($user, $invoice, $type, $payment)
     {
diff --git a/app/Jobs/SendPushNotification.php b/app/Jobs/SendPushNotification.php
index e6eba575e8..88a1e59e17 100644
--- a/app/Jobs/SendPushNotification.php
+++ b/app/Jobs/SendPushNotification.php
@@ -29,6 +29,7 @@ class SendPushNotification extends Job implements ShouldQueue
      * Create a new job instance.
 
      * @param Invoice $invoice
+     * @param mixed   $type
      */
     public function __construct($invoice, $type)
     {
diff --git a/app/Models/Account.php b/app/Models/Account.php
index 377da06d69..56d17004d2 100644
--- a/app/Models/Account.php
+++ b/app/Models/Account.php
@@ -492,8 +492,9 @@ class Account extends Eloquent
 
     /**
      * @param $amount
-     * @param null $client
-     * @param bool $hideSymbol
+     * @param null  $client
+     * @param bool  $hideSymbol
+     * @param mixed $decorator
      *
      * @return string
      */
diff --git a/app/Models/EntityModel.php b/app/Models/EntityModel.php
index 02d90df568..ef6322f9ab 100644
--- a/app/Models/EntityModel.php
+++ b/app/Models/EntityModel.php
@@ -253,6 +253,7 @@ class EntityModel extends Eloquent
     /**
      * @param $data
      * @param $entityType
+     * @param mixed $entity
      *
      * @return bool|string
      */
diff --git a/app/Models/Invoice.php b/app/Models/Invoice.php
index fad799d06d..3d01bf6df1 100644
--- a/app/Models/Invoice.php
+++ b/app/Models/Invoice.php
@@ -466,7 +466,8 @@ class Invoice extends EntityModel implements BalanceAffecting
     }
 
     /**
-     * @param bool $notify
+     * @param bool  $notify
+     * @param mixed $reminder
      */
     public function markInvitationsSent($notify = false, $reminder = false)
     {
@@ -496,8 +497,9 @@ class Invoice extends EntityModel implements BalanceAffecting
 
     /**
      * @param $invitation
-     * @param bool $messageId
-     * @param bool $notify
+     * @param bool  $messageId
+     * @param bool  $notify
+     * @param mixed $notes
      */
     public function markInvitationSent($invitation, $messageId = false, $notify = true, $notes = false)
     {
diff --git a/app/Models/Traits/GeneratesNumbers.php b/app/Models/Traits/GeneratesNumbers.php
index d7f67fbb78..6a142fbefd 100644
--- a/app/Models/Traits/GeneratesNumbers.php
+++ b/app/Models/Traits/GeneratesNumbers.php
@@ -117,6 +117,7 @@ trait GeneratesNumbers
 
     /**
      * @param $entityType
+     * @param mixed $invoice
      *
      * @return string
      */
@@ -129,6 +130,7 @@ trait GeneratesNumbers
 
     /**
      * @param $entity
+     * @param mixed $counter
      *
      * @return bool|mixed
      */
diff --git a/app/Models/User.php b/app/Models/User.php
index 94833fdfb7..5c1196c488 100644
--- a/app/Models/User.php
+++ b/app/Models/User.php
@@ -342,6 +342,7 @@ class User extends Authenticatable
      *
      * @param mixed $permission Either a single permission or an array of possible permissions
      * @param bool True to require all permissions, false to require only one
+     * @param mixed $requireAll
      *
      * @return bool
      */
diff --git a/app/Ninja/Import/BaseTransformer.php b/app/Ninja/Import/BaseTransformer.php
index 3d47bde154..0d672a0c2d 100644
--- a/app/Ninja/Import/BaseTransformer.php
+++ b/app/Ninja/Import/BaseTransformer.php
@@ -146,6 +146,8 @@ class BaseTransformer extends TransformerAbstract
     /**
      * @param $date
      * @param string $format
+     * @param mixed  $data
+     * @param mixed  $field
      *
      * @return null
      */
diff --git a/app/Ninja/Mailers/ContactMailer.php b/app/Ninja/Mailers/ContactMailer.php
index 57540c960c..46d5d3955e 100644
--- a/app/Ninja/Mailers/ContactMailer.php
+++ b/app/Ninja/Mailers/ContactMailer.php
@@ -143,6 +143,7 @@ class ContactMailer extends Mailer
      * @param $subject
      * @param $pdfString
      * @param $documentStrings
+     * @param mixed $reminder
      *
      * @throws \Laracasts\Presenter\Exceptions\PresenterException
      *
diff --git a/app/Ninja/Notifications/PushFactory.php b/app/Ninja/Notifications/PushFactory.php
index 86c85548bc..1fa973fa57 100644
--- a/app/Ninja/Notifications/PushFactory.php
+++ b/app/Ninja/Notifications/PushFactory.php
@@ -45,6 +45,7 @@ class PushFactory
      *
      * @param $token - device token
      * @param $message - user specific message
+     * @param mixed $device
      *
      * @return void
      */
diff --git a/app/Ninja/Presenters/TaskPresenter.php b/app/Ninja/Presenters/TaskPresenter.php
index 2a2f0314eb..8113c634d6 100644
--- a/app/Ninja/Presenters/TaskPresenter.php
+++ b/app/Ninja/Presenters/TaskPresenter.php
@@ -35,6 +35,7 @@ class TaskPresenter extends EntityPresenter
 
     /**
      * @param $account
+     * @param mixed $showProject
      *
      * @return mixed
      */
diff --git a/app/Ninja/Repositories/DashboardRepository.php b/app/Ninja/Repositories/DashboardRepository.php
index 5703b94d78..d5cf162df4 100644
--- a/app/Ninja/Repositories/DashboardRepository.php
+++ b/app/Ninja/Repositories/DashboardRepository.php
@@ -16,6 +16,9 @@ class DashboardRepository
      * @param $groupBy
      * @param $startDate
      * @param $endDate
+     * @param mixed $account
+     * @param mixed $currencyId
+     * @param mixed $includeExpenses
      *
      * @return array
      */
diff --git a/app/Ninja/Repositories/InvoiceRepository.php b/app/Ninja/Repositories/InvoiceRepository.php
index f9a87902d9..38e6302ee4 100644
--- a/app/Ninja/Repositories/InvoiceRepository.php
+++ b/app/Ninja/Repositories/InvoiceRepository.php
@@ -842,6 +842,7 @@ class InvoiceRepository extends BaseRepository
 
     /**
      * @param $clientId
+     * @param mixed $entityType
      *
      * @return mixed
      */
diff --git a/app/Policies/AccountGatewayPolicy.php b/app/Policies/AccountGatewayPolicy.php
index 48f73f9b30..4a6388b226 100644
--- a/app/Policies/AccountGatewayPolicy.php
+++ b/app/Policies/AccountGatewayPolicy.php
@@ -21,7 +21,8 @@ class AccountGatewayPolicy extends EntityPolicy
     }
 
     /**
-     * @param User $user
+     * @param User  $user
+     * @param mixed $item
      *
      * @return bool
      */
diff --git a/app/Policies/BankAccountPolicy.php b/app/Policies/BankAccountPolicy.php
index f7cb9f3f39..335c14e21d 100644
--- a/app/Policies/BankAccountPolicy.php
+++ b/app/Policies/BankAccountPolicy.php
@@ -21,7 +21,8 @@ class BankAccountPolicy extends EntityPolicy
     }
 
     /**
-     * @param User $user
+     * @param User  $user
+     * @param mixed $item
      *
      * @return bool
      */
diff --git a/app/Policies/DocumentPolicy.php b/app/Policies/DocumentPolicy.php
index 55fb67259f..1a668c77b2 100644
--- a/app/Policies/DocumentPolicy.php
+++ b/app/Policies/DocumentPolicy.php
@@ -10,7 +10,8 @@ use App\Models\User;
 class DocumentPolicy extends EntityPolicy
 {
     /**
-     * @param User $user
+     * @param User  $user
+     * @param mixed $item
      *
      * @return bool
      */
diff --git a/app/Policies/EntityPolicy.php b/app/Policies/EntityPolicy.php
index 67b6e58b14..19e2cf5ac9 100644
--- a/app/Policies/EntityPolicy.php
+++ b/app/Policies/EntityPolicy.php
@@ -13,7 +13,8 @@ class EntityPolicy
     use HandlesAuthorization;
 
     /**
-     * @param User $user
+     * @param User  $user
+     * @param mixed $item
      *
      * @return bool
      */
diff --git a/app/Policies/ExpenseCategoryPolicy.php b/app/Policies/ExpenseCategoryPolicy.php
index 405897314d..da7b9f0948 100644
--- a/app/Policies/ExpenseCategoryPolicy.php
+++ b/app/Policies/ExpenseCategoryPolicy.php
@@ -7,7 +7,8 @@ use App\Models\User;
 class ExpenseCategoryPolicy extends EntityPolicy
 {
     /**
-     * @param User $user
+     * @param User  $user
+     * @param mixed $item
      *
      * @return bool
      */
diff --git a/app/Policies/ExpensePolicy.php b/app/Policies/ExpensePolicy.php
index df79cba7d7..cbf9bf9b60 100644
--- a/app/Policies/ExpensePolicy.php
+++ b/app/Policies/ExpensePolicy.php
@@ -7,7 +7,8 @@ use App\Models\User;
 class ExpensePolicy extends EntityPolicy
 {
     /**
-     * @param User $user
+     * @param User  $user
+     * @param mixed $item
      *
      * @return bool
      */
diff --git a/app/Policies/GenericEntityPolicy.php b/app/Policies/GenericEntityPolicy.php
index 527f6b216b..24cfad3617 100644
--- a/app/Policies/GenericEntityPolicy.php
+++ b/app/Policies/GenericEntityPolicy.php
@@ -35,6 +35,7 @@ class GenericEntityPolicy
      * @param User $user
      * @param $entityTypee
      * @param $ownerUserId
+     * @param mixed $entityType
      *
      * @return bool|mixed
      */
diff --git a/app/Policies/PaymentTermPolicy.php b/app/Policies/PaymentTermPolicy.php
index 83d882e0f2..b1b9c12a44 100644
--- a/app/Policies/PaymentTermPolicy.php
+++ b/app/Policies/PaymentTermPolicy.php
@@ -21,7 +21,8 @@ class PaymentTermPolicy extends EntityPolicy
     }
 
     /**
-     * @param User $user
+     * @param User  $user
+     * @param mixed $item
      *
      * @return bool
      */
diff --git a/app/Policies/QuotePolicy.php b/app/Policies/QuotePolicy.php
index e23612a2b4..55b39ddd46 100644
--- a/app/Policies/QuotePolicy.php
+++ b/app/Policies/QuotePolicy.php
@@ -7,7 +7,8 @@ use App\Models\User;
 class QuotePolicy extends EntityPolicy
 {
     /**
-     * @param User $user
+     * @param User  $user
+     * @param mixed $item
      *
      * @return bool
      */
diff --git a/app/Policies/TaskPolicy.php b/app/Policies/TaskPolicy.php
index 455418c323..a5d6350902 100644
--- a/app/Policies/TaskPolicy.php
+++ b/app/Policies/TaskPolicy.php
@@ -7,7 +7,8 @@ use App\Models\User;
 class TaskPolicy extends EntityPolicy
 {
     /**
-     * @param User $user
+     * @param User  $user
+     * @param mixed $item
      *
      * @return bool
      */
diff --git a/app/Policies/TaxRatePolicy.php b/app/Policies/TaxRatePolicy.php
index 8f344b13d4..0cfb459634 100644
--- a/app/Policies/TaxRatePolicy.php
+++ b/app/Policies/TaxRatePolicy.php
@@ -18,7 +18,8 @@ class TaxRatePolicy extends EntityPolicy
     }
 
     /**
-     * @param User $user
+     * @param User  $user
+     * @param mixed $item
      *
      * @return bool
      */
diff --git a/app/Policies/VendorPolicy.php b/app/Policies/VendorPolicy.php
index c01a672740..0e679b3de3 100644
--- a/app/Policies/VendorPolicy.php
+++ b/app/Policies/VendorPolicy.php
@@ -7,7 +7,8 @@ use App\Models\User;
 class VendorPolicy extends EntityPolicy
 {
     /**
-     * @param User $user
+     * @param User  $user
+     * @param mixed $item
      *
      * @return bool
      */
diff --git a/app/Services/CreditService.php b/app/Services/CreditService.php
index 6058e9d36e..b06a4237bb 100644
--- a/app/Services/CreditService.php
+++ b/app/Services/CreditService.php
@@ -44,6 +44,7 @@ class CreditService extends BaseService
 
     /**
      * @param $data
+     * @param null|mixed $credit
      *
      * @return mixed|null
      */
diff --git a/app/Services/ProductService.php b/app/Services/ProductService.php
index ee692035b7..35a49830d7 100644
--- a/app/Services/ProductService.php
+++ b/app/Services/ProductService.php
@@ -41,6 +41,7 @@ class ProductService extends BaseService
 
     /**
      * @param $accountId
+     * @param mixed $search
      *
      * @return \Illuminate\Http\JsonResponse
      */
diff --git a/app/Services/ProjectService.php b/app/Services/ProjectService.php
index 5066cae75e..dff9ec4de3 100644
--- a/app/Services/ProjectService.php
+++ b/app/Services/ProjectService.php
@@ -43,6 +43,7 @@ class ProjectService extends BaseService
 
     /**
      * @param $data
+     * @param mixed $project
      *
      * @return mixed|null
      */
@@ -58,6 +59,7 @@ class ProjectService extends BaseService
     /**
      * @param $clientPublicId
      * @param $search
+     * @param mixed $userId
      *
      * @return \Illuminate\Http\JsonResponse
      */
diff --git a/app/Services/PushService.php b/app/Services/PushService.php
index b8766666aa..1fd50d1a3a 100644
--- a/app/Services/PushService.php
+++ b/app/Services/PushService.php
@@ -55,6 +55,7 @@ class PushService
      * @param Invoice $invoice
      * @param $token
      * @param $type
+     * @param mixed $device
      */
     private function pushMessage(Invoice $invoice, $token, $type, $device)
     {