modules/Accounts/ConfigTemplate.php
Define a constant instead of duplicating this literal "description" 6 times. Open
Open
'description' => 'Columns visible in Account hierarchy [$label => $columnName]',- Read upRead up
- Exclude checks
Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.
On the other hand, constants can be referenced from many places, but only need to be updated in a single place.
Noncompliant Code Example
With the default threshold of 3:
function run() {
prepare('action1'); // Non-Compliant - 'action1' is duplicated 3 times
execute('action1');
release('action1');
}
Compliant Solution
ACTION_1 = 'action1';
function run() {
prepare(ACTION_1);
execute(ACTION_1);
release(ACTION_1);
}
Exceptions
To prevent generating some false-positives, literals having less than 5 characters are excluded.
Define a constant instead of duplicating this literal "default" 6 times. Open
Open
'default' => [],- Read upRead up
- Exclude checks
Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.
On the other hand, constants can be referenced from many places, but only need to be updated in a single place.
Noncompliant Code Example
With the default threshold of 3:
function run() {
prepare('action1'); // Non-Compliant - 'action1' is duplicated 3 times
execute('action1');
release('action1');
}
Compliant Solution
ACTION_1 = 'action1';
function run() {
prepare(ACTION_1);
execute(ACTION_1);
release(ACTION_1);
}
Exceptions
To prevent generating some false-positives, literals having less than 5 characters are excluded.
Define a constant instead of duplicating this literal "validation" 5 times. Open
Open
'validation' => '\App\Validator::naturalNumber',- Read upRead up
- Exclude checks
Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.
On the other hand, constants can be referenced from many places, but only need to be updated in a single place.
Noncompliant Code Example
With the default threshold of 3:
function run() {
prepare('action1'); // Non-Compliant - 'action1' is duplicated 3 times
execute('action1');
release('action1');
}
Compliant Solution
ACTION_1 = 'action1';
function run() {
prepare(ACTION_1);
execute(ACTION_1);
release(ACTION_1);
}
Exceptions
To prevent generating some false-positives, literals having less than 5 characters are excluded.
Define a constant instead of duplicating this literal "sanitization" 3 times. Open
Open
'sanitization' => function () {- Read upRead up
- Exclude checks
Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.
On the other hand, constants can be referenced from many places, but only need to be updated in a single place.
Noncompliant Code Example
With the default threshold of 3:
function run() {
prepare('action1'); // Non-Compliant - 'action1' is duplicated 3 times
execute('action1');
release('action1');
}
Compliant Solution
ACTION_1 = 'action1';
function run() {
prepare(ACTION_1);
execute(ACTION_1);
release(ACTION_1);
}
Exceptions
To prevent generating some false-positives, literals having less than 5 characters are excluded.
Spaces must be used to indent lines; tabs are not allowed Open
Open
'validation' => '\App\Validator::naturalNumber',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'COUNT_IN_HIERARCHY' => [- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
],- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'default' => [],- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'description' => 'Columns visible in Account hierarchy [$label => $columnName]',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'description' => 'Show summary products services bookmark',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'COLUMNS_IN_HIERARCHY' => [- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
return 'List' === $arg || 'ListPreview' === $arg || 'DashBoard' === $arg || 'Tiles' === $arg;- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'description' => 'Max depth of hierarchy',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'default' => true,- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'description' => 'Count Accounts in hierarchy',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'description' => 'Default record view for list preview. Values: full or summary',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
],- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'validation' => '\App\Validator::bool',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'sanitization' => '\App\Purifier::bool',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
},- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'default' => 50,- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'validation' => '\App\Validator::bool',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'sanitization' => '\App\Purifier::bool',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'description' => 'Default module view. Values: List, ListPreview, DashBoard or Tiles, refresh menu files after you change this value',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'SHOW_SUMMARY_PRODUCTS_SERVICES' => [- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'default' => true,- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'default' => 'List',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
$arg = func_get_arg(0);- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'defaultDetailViewName' => [- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'validation' => function () {- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
],- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
$arg = func_get_arg(0);- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'sanitization' => function () {- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
return (int) func_get_arg(0);- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
},- Exclude checks
Line exceeds 120 characters; contains 142 characters Open
Open
'description' => 'Default module view. Values: List, ListPreview, DashBoard or Tiles, refresh menu files after you change this value',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'defaultViewName' => [- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
],- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
return 'full' === $arg || 'summary' === $arg;- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
],- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'default' => 'full',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'MAX_HIERARCHY_DEPTH' => [- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
],- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'validation' => function () {- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
},- Exclude checks