Skip to content

Commit 1782d52

Browse files
committed
Dependency updates, new rule update to phase out non-mb_* function calls
1 parent 1838b93 commit 1782d52

File tree

2 files changed

+38
-12
lines changed

2 files changed

+38
-12
lines changed

composer.lock

Lines changed: 11 additions & 11 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

ruleset.xml

Lines changed: 27 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,33 @@
4949
<rule ref="Generic.PHP.CharacterBeforePHPOpeningTag" />
5050
<rule ref="Generic.PHP.DeprecatedFunctions" />
5151
<rule ref="Generic.PHP.DisallowShortOpenTag" />
52-
<rule ref="Generic.PHP.ForbiddenFunctions" />
52+
<rule ref="Generic.PHP.ForbiddenFunctions">
53+
<type>warning</type>
54+
<properties>
55+
<property name="forbiddenFunctions" type="array">
56+
<!-- default exclusions -->
57+
<element key="sizeof" value="count"/>
58+
<element key="delete" value="unset"/>
59+
60+
<!-- mb_* functions should be used instead -->
61+
<element key="strlen" value="mb_strlen"/>
62+
<element key="substr" value="mb_substr"/>
63+
<element key="strpos" value="mb_strpos"/>
64+
<element key="strrpos" value="mb_strrpos"/>
65+
<element key="stripos" value="mb_stripos"/>
66+
<element key="strripos" value="mb_strripos"/>
67+
<element key="strtolower" value="mb_strtolower"/>
68+
<element key="strtoupper" value="mb_strtoupper"/>
69+
<element key="str_ireplace" value="mb_str_ireplace"/>
70+
<element key="parse_str" value="mb_parse_str"/>
71+
<element key="quoted_printable_decode" value="mb_quoted_printable_decode"/>
72+
<element key="str_pad" value="mb_str_pad"/>
73+
<element key="str_split" value="mb_str_split"/>
74+
<element key="str_word_count" value="mb_str_word_count"/>
75+
<element key="wordwrap" value="mb_wordwrap"/>
76+
</property>
77+
</properties>
78+
</rule>
5379
<rule ref="Generic.PHP.LowerCaseConstant" />
5480
<rule ref="Generic.PHP.LowerCaseKeyword" />
5581

0 commit comments

Comments
 (0)