| Index: trunk/extensions/ArticleFeedback/SpecialArticleFeedback.php |
| — | — | @@ -261,9 +261,9 @@ |
| 262 | 262 | 'afshl_avg_overall', |
| 263 | 263 | 'afshl_avg_ratings' |
| 264 | 264 | ), |
| 265 | | - array( 'afshl_ts = ' . $row->afshl_ts), |
| | 265 | + array( 'afshl_ts' => $row->afshl_ts ), |
| 266 | 266 | __METHOD__, |
| 267 | | - array( "ORDER BY" => "afshl_avg_overall" ) |
| | 267 | + array( "ORDER BY" => "afshl_avg_overall" ) // FIXME unindexed |
| 268 | 268 | ); |
| 269 | 269 | $highs_lows = $this->buildHighsAndLows( $result ); |
| 270 | 270 | $wgMemc->set( $key, $highs_lows, 86400 ); |
| Index: trunk/extensions/ArticleFeedback/populateAFStatistics.php |
| — | — | @@ -71,7 +71,7 @@ |
| 72 | 72 | 'aa_rating_value', |
| 73 | 73 | 'aa_rating_id' |
| 74 | 74 | ), |
| 75 | | - array( 'aa_timestamp >= ' . $this->getLowerBoundTimestamp() ), |
| | 75 | + array( 'aa_timestamp >= ' . $this->dbr->addQuotes( $this->getLowerBoundTimestamp() ) ), |
| 76 | 76 | __METHOD__, |
| 77 | 77 | array() // better to do with limits and offsets? |
| 78 | 78 | ); |