-
Notifications
You must be signed in to change notification settings - Fork 163
Closed
Labels
acceptedNew rule is acceptedNew rule is acceptedevent: MageTestFest2019MageTestFest contributionsMageTestFest contributionsfalse positiveRule causes false positive findingsRule causes false positive findingsneed to discussRule requires discussionRule requires discussionnew ruleNew feature implementationNew feature implementationtechnical guidelinesThe rule is based on Magento Technical GuidelinesThe rule is based on Magento Technical Guidelines
Description
Rule
Getters SHOULD NOT change the state of an object.
Reason
Source: Magento Technical Guidelines.
Implementation
Check if method name starts with get
and find class member assignment operation inside this method.
<severity>8</severity>
<type>warning</type>
Metadata
Metadata
Assignees
Labels
acceptedNew rule is acceptedNew rule is acceptedevent: MageTestFest2019MageTestFest contributionsMageTestFest contributionsfalse positiveRule causes false positive findingsRule causes false positive findingsneed to discussRule requires discussionRule requires discussionnew ruleNew feature implementationNew feature implementationtechnical guidelinesThe rule is based on Magento Technical GuidelinesThe rule is based on Magento Technical Guidelines