Hoo man has uploaded a new change for review.
https://gerrit.wikimedia.org/r/49399
Change subject: Moodbar always assumed an AbuseFilter match
......................................................................
Moodbar always assumed an AbuseFilter match
After I72e1a6dd7dee19f889fc13b60456e9bfebd5e22b
AbuseFilter::filterAction is returning Status objects instead of
Booleans. As Moodbar kept checking for bool it always assumed a
filter was hit.
Change-Id: I8a4e84bc60555b03f9b02b3d489c9b1e68da3235
---
M include/MoodBarUtil.php
1 file changed, 3 insertions(+), 3 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MoodBar
refs/changes/99/49399/1
diff --git a/include/MoodBarUtil.php b/include/MoodBarUtil.php
index 96d85c5..330a409 100644
--- a/include/MoodBarUtil.php
+++ b/include/MoodBarUtil.php
@@ -58,8 +58,8 @@
$vars->setVar( 'old_wikitext', '' );
$vars->setVar( 'new_wikitext', $value );
$vars->addHolder( AbuseFilter::getEditVars( $title ) );
- $filter_result = AbuseFilter::filterAction( $vars,
$title );
- return $filter_result != '' && $filter_result !== true;
+ $filterStatus = AbuseFilter::filterAction( $vars,
$title );
+ return !$filterStatus->isOK();
}
return false;
@@ -207,4 +207,4 @@
return is_array( $wgMarkAsHelpfulType ) && in_array(
'mbresponse', $wgMarkAsHelpfulType );
}
-}
\ No newline at end of file
+}
--
To view, visit https://gerrit.wikimedia.org/r/49399
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8a4e84bc60555b03f9b02b3d489c9b1e68da3235
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MoodBar
Gerrit-Branch: master
Gerrit-Owner: Hoo man <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
