modules/HelpDesk/ConfigTemplate.php
Define a constant instead of duplicating this literal "sanitization" 8 times. Open
Open
'sanitization' => '\App\Purifier::bool'- 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 "\App\Validator::bool" 7 times. Open
Open
'validation' => '\App\Validator::bool',- 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 "\App\Purifier::bool" 7 times. Open
Open
'sanitization' => '\App\Purifier::bool'- 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" 9 times. Open
Open
'default' => true,- 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" 8 times. Open
Open
'validation' => '\App\Validator::bool',- 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 "description" 9 times. Open
Open
'description' => 'Check if account exists',- 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
],- 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
'validation' => '\App\Validator::bool',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'CHECK_ACCOUNT_EXISTS' => [- 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
'description' => 'Columns visible in HelpDesk hierarchy [$label => $columnName]'- 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
'description' => 'Max depth of hierarchy',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'description' => 'Check if service contracts exists',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'Status' => 'ticketstatus',- Exclude checks
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
'validation' => '\App\Validator::bool',- 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
'Priority' => 'ticketpriorities',- 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
'sanitization' => '\App\Purifier::bool'- 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
'description' => 'When closing the ticket, check if related tickets are closed',- 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
],- 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
'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
'description' => 'Show summary products and 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' => [- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'Assigned To' => 'assigned_user_id',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'description' => 'Count HelpDesk records in hierarchy',- 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
'default' => true,- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'CHECK_SERVICE_CONTRACTS_EXISTS' => [- 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
'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
'validation' => '\App\Validator::bool',- 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
'Ticket No' => 'ticket_no',- 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
'CHECK_IF_RELATED_TICKETS_ARE_CLOSED' => [- 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' => 'Check if account exists',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'CONTACTS_CHECK_EMAIL_OPTOUT' => [- 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
'FL_TOTAL_TIME_H' => 'sum_time',- 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
'validation' => '\App\Validator::bool',- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'Subject' => 'ticket_title',- 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
'CHECK_IF_RECORDS_HAS_TIME_CONTROL' => [- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
'description' => 'When closing the ticket, check if has time control',- 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 (int) func_get_arg(0);- 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' => true,- 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
'description' => 'Check email opt-out',- Exclude checks