Function formatCurrencyValue has a Cognitive Complexity of 43 (exceeds 5 allowed). Consider refactoring.

Open

    private function formatCurrencyValue($value)
    {
        $currencyPattern = $this->currencyFormat;
        $curSeparator = $this->currencySeparator;
        $decSeparator = $this->decimalSeparator;

• Read up
  Read up

Method formatCurrencyValue has 110 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private function formatCurrencyValue($value)
    {
        $currencyPattern = $this->currencyFormat;
        $curSeparator = $this->currencySeparator;
        $decSeparator = $this->decimalSeparator;

The class CurrencyField has an overall complexity of 71 which is very high. The configured complexity threshold is 50.

Open

class CurrencyField
{
    private $CURRENCY_PATTERN_PLAIN = '123456789';
    private $CURRENCY_PATTERN_SINGLE_GROUPING = '123456,789';
    private $CURRENCY_PATTERN_THOUSAND_GROUPING = '123,456,789';

• Exclude checks
  • Disable engine
  • Disable check

File CurrencyField.php has 300 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php
/* +**********************************************************************************
 * The contents of this file are subject to the vtiger CRM Public License Version 1.0
 * ("License"); You may not use this file except in compliance with the License
 * The Original Code is:  vtiger CRM Open Source

Function currencyDecimalFormat has a Cognitive Complexity of 14 (exceeds 5 allowed). Consider refactoring.

Open

    public function currencyDecimalFormat($value, $user = null)
    {
        if (!$user) {
            $user = \App\User::getCurrentUserModel();
        }

• Read up
  Read up

Avoid too many return statements within this method.

Open

        return $value;

Avoid too many return statements within this method.

Open

        return $value;

The method formatCurrencyValue() has 152 lines of code. Current threshold is set to 100. Avoid really long methods.

Open

    private function formatCurrencyValue($value)
    {
        $currencyPattern = $this->currencyFormat;
        $curSeparator = $this->currencySeparator;
        $decSeparator = $this->decimalSeparator;

• Exclude checks
  • Disable engine
  • Disable check

The method formatCurrencyValue() has an NPath complexity of 474320. The configured NPath complexity threshold is 200.

Open

    private function formatCurrencyValue($value)
    {
        $currencyPattern = $this->currencyFormat;
        $curSeparator = $this->currencySeparator;
        $decSeparator = $this->decimalSeparator;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    function bar() {
        // lots of complicated code
    }
}

The method formatCurrencyValue() has a Cyclomatic Complexity of 24. The configured cyclomatic complexity threshold is 10.

Open

    private function formatCurrencyValue($value)
    {
        $currencyPattern = $this->currencyFormat;
        $curSeparator = $this->currencySeparator;
        $decSeparator = $this->decimalSeparator;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

The method currencyDecimalFormat() has a Cyclomatic Complexity of 10. The configured cyclomatic complexity threshold is 10.

Open

    public function currencyDecimalFormat($value, $user = null)
    {
        if (!$user) {
            $user = \App\User::getCurrentUserModel();
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

This function "formatCurrencyValue" has 151 lines, which is greater than the 150 lines authorized. Split it into smaller functions.

Open

    private function formatCurrencyValue($value)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

Reduce the number of returns of this function 5, down to the maximum allowed 3.

Open

    public function currencyDecimalFormat($value, $user = null)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function myFunction(){ // Noncompliant as there are 4 return statements
  if (condition1) {
    return true;
  } else {
    if (condition2) {
      return false;
    } else {
      return true;
    }
  }
  return false;
}

Reduce the number of returns of this function 5, down to the maximum allowed 3.

Open

    private function formatCurrencyValue($value)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function myFunction(){ // Noncompliant as there are 4 return statements
  if (condition1) {
    return true;
  } else {
    if (condition2) {
      return false;
    } else {
      return true;
    }
  }
  return false;
}

Refactor this function to reduce its Cognitive Complexity from 44 to the 15 allowed.

Open

    private function formatCurrencyValue($value)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

The method convertToUserFormatSymbol has a boolean flag argument $currencySymbol, which is a certain sign of a Single Responsibility Principle violation.

Open

    public static function convertToUserFormatSymbol($value, $skipConversion = false, $currencySymbol = false, $skipFormatting = false)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

Missing class import via use statement (line '457', column '14').

Open

        $id = (new \App\Db\Query())->select(['id'])->from('vtiger_currency_info')->where(['<', 'defaultid', 0])->scalar();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

The method getDisplayValue has a boolean flag argument $skipFormatting, which is a certain sign of a Single Responsibility Principle violation.

Open

    public function getDisplayValue($user = null, $skipConversion = false, $skipFormatting = false)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

The method getDBInsertedValue has a boolean flag argument $skipConversion, which is a certain sign of a Single Responsibility Principle violation.

Open

    public function getDBInsertedValue($user = null, $skipConversion = false)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

The method convertToUserFormatSymbol has a boolean flag argument $skipFormatting, which is a certain sign of a Single Responsibility Principle violation.

Open

    public static function convertToUserFormatSymbol($value, $skipConversion = false, $currencySymbol = false, $skipFormatting = false)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

The method convertToDBFormat has a boolean flag argument $skipConversion, which is a certain sign of a Single Responsibility Principle violation.

Open

    public static function convertToDBFormat($value, $user = null, $skipConversion = false)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

The method getDisplayValueWithSymbol has a boolean flag argument $skipConversion, which is a certain sign of a Single Responsibility Principle violation.

Open

    public function getDisplayValueWithSymbol($user = null, $skipConversion = false)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

The method convertToUserFormat has a boolean flag argument $skipConversion, which is a certain sign of a Single Responsibility Principle violation.

Open

    public static function convertToUserFormat($value, $user = null, $skipConversion = false, $skipFormatting = false)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

The method convertToUserFormat has a boolean flag argument $skipFormatting, which is a certain sign of a Single Responsibility Principle violation.

Open

    public static function convertToUserFormat($value, $user = null, $skipConversion = false, $skipFormatting = false)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

The method getDisplayValue has a boolean flag argument $skipConversion, which is a certain sign of a Single Responsibility Principle violation.

Open

    public function getDisplayValue($user = null, $skipConversion = false, $skipFormatting = false)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

The method convertToUserFormatSymbol has a boolean flag argument $skipConversion, which is a certain sign of a Single Responsibility Principle violation.

Open

    public static function convertToUserFormatSymbol($value, $skipConversion = false, $currencySymbol = false, $skipFormatting = false)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

The method formatCurrencyValue uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                } else {
                    $wholeNumberFirstPart = 0;
                }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class '\App\Fields\Double' in method 'currencyDecimalFormat'.

Open

                $value = \App\Fields\Double::truncateZeros($value);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\App\Fields\Currency' in method 'initialize'.

Open

        $currencyData = \App\Fields\Currency::getById($this->currencyId);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method formatCurrencyValue uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                $numericParts[0] = $wholeNumberLastPart;
            }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class '\App\User' in method 'getDisplayValue'.

Open

            $user = \App\User::getCurrentUserModel();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\App\User' in method 'currencyDecimalFormat'.

Open

            $user = \App\User::getCurrentUserModel();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\App\User' in method 'initialize'.

Open

            $user = \App\User::getCurrentUserModel();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method formatCurrencyValue uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            $sign = '';
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method formatCurrencyValue uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                $numericParts[0] = $wholeNumberLastPart;
            }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class 'App\Config' in method 'initialize'.

Open

        $defaultCharset = App\Config::main('default_charset');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'App\Purifier' in method 'currencyDecimalFormat'.

Open

            $fieldValue = explode(App\Purifier::decodeHtml($decSeparator), $value);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\App\User' in method 'getDBInsertedValue'.

Open

            $user = \App\User::getCurrentUserModel();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method initialize uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            $this->currencyId = self::getDBCurrencyId();
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method formatCurrencyValue uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                $numericParts[0] = 0;
            }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Define a constant instead of duplicating this literal "currency_decimal_separator" 3 times.

Open

            $this->decimalSeparator = str_replace("\xC2\xA0", ' ', html_entity_decode($user->getDetail('currency_decimal_separator'), ENT_QUOTES, $defaultCharset));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "no_of_currency_decimals" 3 times.

Open

        $this->numberOfDecimal = (empty($user->getDetail('no_of_currency_decimals')) && 0 !== (int) $user->getDetail('no_of_currency_decimals')) ? 2 : (int) $user->getDetail('no_of_currency_decimals');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Argument 1 (value) is false|float|int|string but \CurrencyField::__construct() takes \Number defined at /code/include/fields/CurrencyField.php:92

Open

        $self = new self($value);

• Exclude checks
  • Disable engine
  • Disable check

Parameter $user has undeclared type \App\User (Did you mean class \Tests\App\User)

Open

    public function getDisplayValue($user = null, $skipConversion = false, $skipFormatting = false)

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (currencyValue) is string but \CurrencyField::appendCurrencySymbol() takes \Number defined at /code/include/fields/CurrencyField.php:225

Open

        return self::appendCurrencySymbol($formattedValue, $this->currencySymbol, $this->currencySymbolPlacement);

• Exclude checks
  • Disable engine
  • Disable check

Call to method getDetail from undeclared class \App\User (Did you mean class \Tests\App\User)

Open

            $this->decimalSeparator = str_replace("\xC2\xA0", ' ', html_entity_decode($user->getDetail('currency_decimal_separator'), ENT_QUOTES, $defaultCharset));

• Exclude checks
  • Disable engine
  • Disable check

Call to method getCurrentUserModel from undeclared class \App\User (Did you mean class \Tests\App\User)

Open

            $currencySymbolPlacement = \App\User::getCurrentUserModel()->getDetail('currency_symbol_placement');

• Exclude checks
  • Disable engine
  • Disable check

Parameter $value has undeclared type \Number

Open

    private function formatCurrencyValue($value)

• Exclude checks
  • Disable engine
  • Disable check

Returning type 0 but convertToDBFormat() is declared to return \Number

Open

            return 0;

• Exclude checks
  • Disable engine
  • Disable check

Saw unextractable annotation for comment '* @var <type>'</type>

Open

     * @var <type>

• Exclude checks
  • Disable engine
  • Disable check

Property \CurrencyField->conversionRate has undeclared type \Number

Open

    public $conversionRate = 1;

• Exclude checks
  • Disable engine
  • Disable check

Returning type 0|float but getDBInsertedValue() is declared to return \Number

Open

        return $dbValue;

• Exclude checks
  • Disable engine
  • Disable check

Call to method getDetail from undeclared class \App\User (Did you mean class \Tests\App\User)

Open

        if (!empty($user->getDetail('currency_id'))) {

• Exclude checks
  • Disable engine
  • Disable check

Returning type float but getDBInsertedValue() is declared to return \Number

Open

        return $dbValue;

• Exclude checks
  • Disable engine
  • Disable check

Return type of convertToDBFormat() is undeclared type \Number

Open

    public static function convertToDBFormat($value, $user = null, $skipConversion = false)

• Exclude checks
  • Disable engine
  • Disable check

Parameter $user has undeclared type \App\User (Did you mean class \Tests\App\User)

Open

    public function initialize($user = null)

• Exclude checks
  • Disable engine
  • Disable check

Parameter $user has undeclared type \App\User (Did you mean class \Tests\App\User)

Open

    public static function convertToDBFormat($value, $user = null, $skipConversion = false)

• Exclude checks
  • Disable engine
  • Disable check

The phpdoc comment for @param cannot occur on a property

Open

     * @param Number $value

• Exclude checks
  • Disable engine
  • Disable check

Call to method getCurrentUserModel from undeclared class \App\User (Did you mean class \Tests\App\User)

Open

            $user = \App\User::getCurrentUserModel();

• Exclude checks
  • Disable engine
  • Disable check

Assigning int to property but \CurrencyField->conversionRate is \Number

Open

    public $conversionRate = 1;

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (str) is float|int but \substr() takes string

Open

            $value = substr($value, 1);

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (currencyValue) is string but \CurrencyField::appendCurrencySymbol() takes \Number defined at /code/include/fields/CurrencyField.php:225

Open

        $value = self::appendCurrencySymbol($formattedValue, $currencySymbol, $self->currencySymbolPlacement);

• Exclude checks
  • Disable engine
  • Disable check

Assigning int to property but \CurrencyField->maxNumberOfDecimals is \Number

Open

    public $maxNumberOfDecimals = 5;

• Exclude checks
  • Disable engine
  • Disable check

Expected @param annotation for value to be before the @param annotation for user

Open

     * @param \App\User $user

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (user) is null but \CurrencyField::getDisplayValue() takes \App\User defined at /code/include/fields/CurrencyField.php:183

Open

        $formattedValue = $self->getDisplayValue(null, $skipConversion, $skipFormatting);

• Exclude checks
  • Disable engine
  • Disable check

Parameter $user has undeclared type \App\User (Did you mean class \Tests\App\User)

Open

    public function getDisplayValueWithSymbol($user = null, $skipConversion = false)

• Exclude checks
  • Disable engine
  • Disable check

Return type of getDBInsertedValue() is undeclared type \Number

Open

    public function getDBInsertedValue($user = null, $skipConversion = false)

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (haystack) is float|int but \stripos() takes string

Open

        if (0 === stripos($value, '-')) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method getDetail from undeclared class \App\User (Did you mean class \Tests\App\User)

Open

        $this->numberOfDecimal = (empty($user->getDetail('no_of_currency_decimals')) && 0 !== (int) $user->getDetail('no_of_currency_decimals')) ? 2 : (int) $user->getDetail('no_of_currency_decimals');

• Exclude checks
  • Disable engine
  • Disable check

Parameter $user has undeclared type \App\User (Did you mean class \Tests\App\User)

Open

    public static function convertToUserFormat($value, $user = null, $skipConversion = false, $skipFormatting = false)

• Exclude checks
  • Disable engine
  • Disable check

Parameter $currencyValue has undeclared type \Number

Open

    public static function appendCurrencySymbol($currencyValue, $currencySymbol, $currencySymbolPlacement = '')

• Exclude checks
  • Disable engine
  • Disable check

Property \CurrencyField->maxNumberOfDecimals has undeclared type \Number

Open

    public $maxNumberOfDecimals = 5;

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (value) is false|string but \CurrencyField::__construct() takes \Number defined at /code/include/fields/CurrencyField.php:92

Open

        $self = new self($value);

• Exclude checks
  • Disable engine
  • Disable check

Parameter $value has undeclared type \Number

Open

    public static function convertToDBFormat($value, $user = null, $skipConversion = false)

• Exclude checks
  • Disable engine
  • Disable check

Call to method getDetail from undeclared class \App\User (Did you mean class \Tests\App\User)

Open

            $this->currencyFormat = html_entity_decode($user->getDetail('currency_grouping_pattern'), ENT_QUOTES, $defaultCharset);

• Exclude checks
  • Disable engine
  • Disable check

Call to method getDetail from undeclared class \App\User (Did you mean class \Tests\App\User)

Open

            $this->currencySeparator = str_replace("\xC2\xA0", ' ', html_entity_decode($user->getDetail('currency_grouping_separator'), ENT_QUOTES, $defaultCharset));

• Exclude checks
  • Disable engine
  • Disable check

Call to method getCurrentUserModel from undeclared class \App\User (Did you mean class \Tests\App\User)

Open

            $user = \App\User::getCurrentUserModel();

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db\Query::select

Open

        $id = (new \App\Db\Query())->select(['id'])->from('vtiger_currency_info')->where(['<', 'defaultid', 0])->scalar();

• Exclude checks
  • Disable engine
  • Disable check

Returning type null but getDBCurrencyId() is declared to return int

Open

        return null;

• Exclude checks
  • Disable engine
  • Disable check

Call to method getDetail from undeclared class \App\User (Did you mean class \Tests\App\User)

Open

        $this->currencySymbolPlacement = $user->getDetail('currency_symbol_placement');

• Exclude checks
  • Disable engine
  • Disable check

Suspicious type \Number of a variable or expression used to build a string. (Expected type to be able to cast to a string)

Open

                $returnValue = $currencyValue . ' ' . $currencySymbol;

• Exclude checks
  • Disable engine
  • Disable check

Suspicious type \Number of a variable or expression used to build a string. (Expected type to be able to cast to a string)

Open

                $returnValue = html_entity_decode($currencySymbol, ENT_COMPAT) . ' ' . $currencyValue;

• Exclude checks
  • Disable engine
  • Disable check

Call to method getCurrentUserModel from undeclared class \App\User (Did you mean class \Tests\App\User)

Open

            $user = \App\User::getCurrentUserModel();

• Exclude checks
  • Disable engine
  • Disable check

Argument 3 (subject) is \Number but \str_replace() takes array|string

Open

        $dbValue = str_replace($curSeparator, '', $this->value);

• Exclude checks
  • Disable engine
  • Disable check

Call to method getDetail from undeclared class \App\User (Did you mean class \Tests\App\User)

Open

        if (!empty($user->getDetail('currency_grouping_pattern'))) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method getDetail from undeclared class \App\User (Did you mean class \Tests\App\User)

Open

            $this->currencyId = $user->getDetail('currency_id');

• Exclude checks
  • Disable engine
  • Disable check

Parameter $user has undeclared type \App\User (Did you mean class \Tests\App\User)

Open

    public function getDBInsertedValue($user = null, $skipConversion = false)

• Exclude checks
  • Disable engine
  • Disable check

Parameter $value has undeclared type \Number

Open

    public function __construct($value)

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (number) is \Number but \number_format() takes float|int

Open

        $value = number_format($value, $currencyDecimalPlaces, '.', '');

• Exclude checks
  • Disable engine
  • Disable check

Call to method getCurrentUserModel from undeclared class \App\User (Did you mean class \Tests\App\User)

Open

            $user = \App\User::getCurrentUserModel();

• Exclude checks
  • Disable engine
  • Disable check

Avoid excessively long variable names like $currencySymbolPlacement. Keep variable name length under 20.

Open

    public $currencySymbolPlacement;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Avoid excessively long variable names like $CURRENCY_PATTERN_PLAIN. Keep variable name length under 20.

Open

    private $CURRENCY_PATTERN_PLAIN = '123456789';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Avoid excessively long variable names like $wholeNumberFirstPartElements. Keep variable name length under 20.

Open

                $wholeNumberFirstPartElements = str_split($wholeNumberFirstPart, 2);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Avoid excessively long variable names like $CURRENCY_PATTERN_MIXED_GROUPING. Keep variable name length under 20.

Open

    private $CURRENCY_PATTERN_MIXED_GROUPING = '12,34,56,789';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Avoid excessively long variable names like $currencySymbolPlacement. Keep variable name length under 20.

Open

    public static function appendCurrencySymbol($currencyValue, $currencySymbol, $currencySymbolPlacement = '')

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Avoid excessively long variable names like $currencyDecimalPlaces. Keep variable name length under 20.

Open

        $currencyDecimalPlaces = $this->numberOfDecimal;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Avoid excessively long variable names like $CURRENCY_PATTERN_SINGLE_GROUPING. Keep variable name length under 20.

Open

    private $CURRENCY_PATTERN_SINGLE_GROUPING = '123456,789';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Avoid excessively long variable names like $CURRENCY_PATTERN_THOUSAND_GROUPING. Keep variable name length under 20.

Open

    private $CURRENCY_PATTERN_THOUSAND_GROUPING = '123,456,789';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class CurrencyField

• Exclude checks
  • Disable engine
  • Disable check

The property $CURRENCY_PATTERN_MIXED_GROUPING is not named in camelCase.

Open

class CurrencyField
{
    private $CURRENCY_PATTERN_PLAIN = '123456789';
    private $CURRENCY_PATTERN_SINGLE_GROUPING = '123456,789';
    private $CURRENCY_PATTERN_THOUSAND_GROUPING = '123,456,789';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    protected $property_name;
}

The property $CURRENCY_PATTERN_SINGLE_GROUPING is not named in camelCase.

Open

class CurrencyField
{
    private $CURRENCY_PATTERN_PLAIN = '123456789';
    private $CURRENCY_PATTERN_SINGLE_GROUPING = '123456,789';
    private $CURRENCY_PATTERN_THOUSAND_GROUPING = '123,456,789';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    protected $property_name;
}

The property $CURRENCY_PATTERN_PLAIN is not named in camelCase.

Open

class CurrencyField
{
    private $CURRENCY_PATTERN_PLAIN = '123456789';
    private $CURRENCY_PATTERN_SINGLE_GROUPING = '123456,789';
    private $CURRENCY_PATTERN_THOUSAND_GROUPING = '123,456,789';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    protected $property_name;
}

The property $CURRENCY_PATTERN_THOUSAND_GROUPING is not named in camelCase.

Open

class CurrencyField
{
    private $CURRENCY_PATTERN_PLAIN = '123456789';
    private $CURRENCY_PATTERN_SINGLE_GROUPING = '123456,789';
    private $CURRENCY_PATTERN_THOUSAND_GROUPING = '123,456,789';

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    protected $property_name;
}

Avoid variables with short names like $id. Configured minimum length is 3.

Open

        $id = (new \App\Db\Query())->select(['id'])->from('vtiger_currency_info')->where(['<', 'defaultid', 0])->scalar();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Spaces must be used for alignment; tabs are not allowed

Open

    public static function convertToUserFormat($value, $user = null, $skipConversion = false, $skipFormatting = false)

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        if (0 === stripos($value, '-')) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $value = substr($value, 1);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $value = $self->getDisplayValue($user, $skipConversion, $skipFormatting);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        if (false === $skipConversion) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * Function that converts the Number into Users Currency along with currency symbol.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param Number $currencyValue

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @return bool|string Formatted Currency

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * Function that converts the Number into Users Currency.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $negative = false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $value = self::appendCurrencySymbol($formattedValue, $currencySymbol, $self->currencySymbolPlacement);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param bool      $skipConversion

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $displayValue = $this->formatCurrencyValue($displayValue);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * Static Function that appends the currency symbol to a given currency value, based on the preferred symbol placement.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            case '1.0$':

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $self = new self($value);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        // To support negative values

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $negative = true;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        if (empty($user)) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @return string Formatted Currency

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    public static function appendCurrencySymbol($currencyValue, $currencySymbol, $currencySymbolPlacement = '')

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param string $currencySymbol

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $currencySymbolPlacement = \App\User::getCurrentUserModel()->getDetail('currency_symbol_placement');

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        switch ($currencySymbolPlacement) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @deprecated        Recommend using function \App\Fields\Currency::formatToDisplay

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        if (0 === stripos($value, '-')) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $value = substr($value, 1);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $formattedValue = $self->getDisplayValue(null, $skipConversion, $skipFormatting);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $user = \App\User::getCurrentUserModel();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $displayValue = 0;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $returnValue = $currencyValue . ' ' . $currencySymbol;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param mixed     $skipFormatting

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param string $currencySymbolPlacement

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @return string Formatted Currency

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $self = new self($value);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        return ($negative) ? '-' . $value : $value;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        if (empty($displayValue)) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $displayValue = self::convertFromDollar($displayValue, $this->conversionRate);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $negative = true;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    public static function convertToUserFormatSymbol($value, $skipConversion = false, $currencySymbol = false, $skipFormatting = false)

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param \App\User $user

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @return string Formatted Currency

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        if (false === $skipFormatting) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    public function getDisplayValueWithSymbol($user = null, $skipConversion = false)

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        return self::appendCurrencySymbol($formattedValue, $this->currencySymbol, $this->currencySymbolPlacement);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * Function that formats the Number based on the User configured Pattern, Currency separator and Decimal separator.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $currencySymbol = $self->currencySymbol;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        if (!$currencySymbolPlacement) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            case '$1.0':

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    private function formatCurrencyValue($value)

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $currencyPattern = $this->currencyFormat;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        // To support negative values

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    public function getDisplayValue($user = null, $skipConversion = false, $skipFormatting = false)

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param bool      $skipConversion

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $returnValue = html_entity_decode($currencySymbol, ENT_COMPAT) . ' ' . $currencyValue;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param Number $value

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $negative = false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        if (false === $currencySymbol) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $this->initialize($user);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                break;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        return ($negative) ? '-' . $value : $value;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $displayValue = $this->value;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        return $this->currencyDecimalFormat($displayValue, $user);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @return string Currency value appended with the currency symbol

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        return $returnValue;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param \App\User $user

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $formattedValue = $this->getDisplayValue($user, $skipConversion);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            default:

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $curSeparator = $this->currencySeparator;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            if (\strlen($wholeNumber) > 3) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            // Second Part of the number (last 3 digits) which should be separated from the First part using Currency Separator

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        if ($currencyPattern == $this->CURRENCY_PATTERN_THOUSAND_GROUPING) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $wholeNumber = $numericParts[0];

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            if ($wholeNumber < 0 || $negativeNumber) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $negativeNumber = true;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            if ($wholeNumber < 0 || $negativeNumber) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            // Second Part of the number (last 3 digits) which should be separated from the First part using Currency Separator

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                // Pad the rest of the length in the number string with Leading 0, to get it to the multiples of 2

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            //if its negative number, append-back the negative symbol to the whole number part

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $numericParts[0] = '-' . $numericParts[0];

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            if (!empty($wholeNumberFirstPart)) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            // Re-create the currency value combining the whole number and the decimal part using Decimal separator

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $negativeNumber = false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $wholeNumberFirstPart = substr($wholeNumber, 0, \strlen($wholeNumber) - 3);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    $wholeNumberFirstPart = ltrim($wholeNumberFirstPart, '0');

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $sign = '';

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            // Re-create the whole number with user's configured currency separator

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $negativeNumber = true;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $negativeNumber = true;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $OddGroupLength = $numberLength % 3;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $wholeNumber = $numericParts[0];

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $wholeNumber = $positiveValues[1];

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param bool      $skipConversion

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $decSeparator = ' ';

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            return $sign . $number;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $wholeNumber = $numericParts[0];

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $wholeNumber = $positiveValues[1];

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            if ($OddGroupLength > 0) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            // Split the whole number into chunks of 3 digits

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            // Re-create the whole number with user's configured currency separator

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $numericParts[0] = ltrim($wholeNumber, '0');

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $number = implode($decSeparator, $numericParts);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $numberLength = \strlen($wholeNumberFirstPart);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                // Re-create the whole number with user's configured currency separator

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @return Number

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            // Re-create the currency value combining the whole number and the decimal part using Decimal separator

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            // Separate the numeric and decimal parts

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $gapsToBeFilled = 0;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $wholeNumberFirstPart = implode($curSeparator, $wholeNumberFirstPartElements);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $numericParts[0] = $wholeNumberFirstPart . $curSeparator . $wholeNumberLastPart;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $currencyDecimalPlaces = $this->numberOfDecimal;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $value = number_format($value, $currencyDecimalPlaces, '.', '');

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $curSeparator = ' ';

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            if ($value < 0) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $wholeNumberParts = str_split($wholeNumber, 3);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $numericParts = explode('.', $value);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            //check the whole number is negative value, then separate the negative symbol from whole number

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $wholeNumberFirstPart = str_pad($wholeNumberFirstPart, $numberLength + $gapsToBeFilled, '0', STR_PAD_LEFT);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    public function getDBInsertedValue($user = null, $skipConversion = false)

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        if ($value < 0) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        if ($currencyPattern == $this->CURRENCY_PATTERN_PLAIN) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        if ($currencyPattern == $this->CURRENCY_PATTERN_SINGLE_GROUPING) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $wholeNumberFirstPartElements = str_split($wholeNumberFirstPart, 2);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        if (empty($user)) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            // Pad the rest of the length in the number string with Leading 0, to get it to the multiples of 3

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            if ($negativeNumber) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            if ($value < 0) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    $gapsToBeFilled = 2 - $OddGroupLength;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $user = \App\User::getCurrentUserModel();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $wholeNumberLastPart = substr($wholeNumber, -3);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $numericParts[0] = $wholeNumberFirstPart . $curSeparator . $wholeNumberLastPart;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            return $sign . $number;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $numberLength = \strlen($wholeNumber);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                } else {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $number = implode($decSeparator, $numericParts);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        return $value;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * Returns the Currency value without formatting for DB Operations.

• Exclude checks
  • Disable engine
  • Disable check