src/Shop/Order/Event/Simple.php
Define a constant instead of duplicating this literal "Shop_Order_Event" 4 times. Open
Open
'Shop_Order_Event',- 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.
Each class must be in a namespace of at least one level (a top-level vendor name) Open
Open
class Shop_Order_Event_Simple extends Shop_Order_Event- Exclude checks
The class Shop_Order_Event_Simple is not named in CamelCase. Open
Open
class Shop_Order_Event_Simple extends Shop_Order_Event
{
/*
* Actions- Read upRead up
- Exclude checks
CamelCaseClassName
Since: 0.2
It is considered best practice to use the CamelCase notation to name classes.
Example
class class_name {
}Source
Expected 1 blank line at end of file; 2 found Open
Open
}- Exclude checks
Class name "Shop_Order_Event_Simple" is not in camel caps format Open
Open
class Shop_Order_Event_Simple extends Shop_Order_Event- Exclude checks