Remove another ss_active_users check from SiteStats::isSane
Also add a comment to method doc to make sure, nobody will add it again. Follow-Up: Id8f1cb7035ae0058fb34a6c7bdf7bf9f6c740a69 Follow-Up: I268599be96106e1175fdf9750a2adc9468ebc93c Change-Id: Ifd6da554b531720aace4497f7fbc2f58c12edc07
This commit is contained in:
parent
f3b75a4764
commit
b058096a58
1 changed files with 2 additions and 1 deletions
|
|
@ -221,6 +221,8 @@ class SiteStats {
|
|||
/**
|
||||
* Is the provided row of site stats sane, or should it be regenerated?
|
||||
*
|
||||
* Checks only fields which are filled by SiteStatsInit::refresh.
|
||||
*
|
||||
* @param $row
|
||||
*
|
||||
* @return bool
|
||||
|
|
@ -229,7 +231,6 @@ class SiteStats {
|
|||
if ( $row === false
|
||||
|| $row->ss_total_pages < $row->ss_good_articles
|
||||
|| $row->ss_total_edits < $row->ss_total_pages
|
||||
|| $row->ss_users < $row->ss_active_users
|
||||
) {
|
||||
return false;
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in a new issue