Legoktm has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/248118

Change subject: Run contentlength filter first, as it's the fastest
......................................................................

Run contentlength filter first, as it's the fastest

We should try to fail early and skip complex rules if possible. The
contentlength filter is extremely simple, so run it first.

Change-Id: Ia41f0d185def7d94b85c793f3b362b587c47518f
---
M container.php
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Flow 
refs/changes/18/248118/1

diff --git a/container.php b/container.php
index d5c471a..3e632c4 100644
--- a/container.php
+++ b/container.php
@@ -838,12 +838,12 @@
 
 $c['controller.spamfilter'] = function( $c ) {
        return new Flow\SpamFilter\Controller(
+               $c['controller.contentlength'],
                $c['controller.spamregex'],
                $c['controller.ratelimits'],
                $c['controller.spamblacklist'],
                $c['controller.abusefilter'],
-               $c['controller.confirmedit'],
-               $c['controller.contentlength']
+               $c['controller.confirmedit']
        );
 };
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia41f0d185def7d94b85c793f3b362b587c47518f
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Legoktm <legoktm.wikipe...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to