jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/401045 )

Change subject: Warn on usage of each()
......................................................................


Warn on usage of each()

foreach should be used instead.

Bug: T174354
Change-Id: I3543a1e40732ced397a929bb0e4a16a4f66cb240
---
M MediaWiki/Sniffs/Usage/ForbiddenFunctionsSniff.php
1 file changed, 3 insertions(+), 1 deletion(-)

Approvals:
  Florianschmidtwelzow: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/MediaWiki/Sniffs/Usage/ForbiddenFunctionsSniff.php 
b/MediaWiki/Sniffs/Usage/ForbiddenFunctionsSniff.php
index e3e4045..d607ea8 100644
--- a/MediaWiki/Sniffs/Usage/ForbiddenFunctionsSniff.php
+++ b/MediaWiki/Sniffs/Usage/ForbiddenFunctionsSniff.php
@@ -32,8 +32,10 @@
         */
        private $functions = [
                'is_integer' => 'is_int',
-               'create_function' => false,
                'extract' => false,
+               // Deprecated in PHP 7.2
+               'create_function' => false,
+               'each' => false,
        ];
 
        /**

-- 
To view, visit https://gerrit.wikimedia.org/r/401045
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3543a1e40732ced397a929bb0e4a16a4f66cb240
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/tools/codesniffer
Gerrit-Branch: master
Gerrit-Owner: Legoktm <[email protected]>
Gerrit-Reviewer: Addshore <[email protected]>
Gerrit-Reviewer: Florianschmidtwelzow <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to