diff options
Diffstat (limited to 'site/app/models')
| -rw-r--r-- | site/app/models/addon.php | 2 | ||||
| -rw-r--r-- | site/app/models/editor_subscription.php | 94 | ||||
| -rw-r--r-- | site/app/models/review.php | 72 |
3 files changed, 167 insertions, 1 deletions
diff --git a/site/app/models/addon.php b/site/app/models/addon.php index b7a6418..aaa09c9 100644 --- a/site/app/models/addon.php +++ b/site/app/models/addon.php @@ -388,7 +388,7 @@ class Addon extends AppModel $orderby .= 'datestatuschanged'; break; case 'rated': - $orderby .= 'Addon.averagerating'; + $orderby .= 'Addon.bayesianrating'; break; case 'newest': $orderby .= 'Addon.created'; diff --git a/site/app/models/editor_subscription.php b/site/app/models/editor_subscription.php new file mode 100644 index 0000000..76dbe2f --- /dev/null +++ b/site/app/models/editor_subscription.php @@ -0,0 +1,94 @@ +<?php +/* ***** BEGIN LICENSE BLOCK ***** + * Version: MPL 1.1/GPL 2.0/LGPL 2.1 + * + * The contents of this file are subject to the Mozilla Public License Version + * 1.1 (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * http://www.mozilla.org/MPL/ + * + * Software distributed under the License is distributed on an "AS IS" basis, + * WITHOUT WARRANTY OF ANY KIND, either express or implied. See the License + * for the specific language governing rights and limitations under the + * License. + * + * The Original Code is addons.mozilla.org site. + * + * The Initial Developer of the Original Code is + * Frederic Wenzel <fwenzel@mozilla.com> + * Portions created by the Initial Developer are Copyright (C) 2009 + * the Initial Developer. All Rights Reserved. + * + * Contributor(s): + * + * Alternatively, the contents of this file may be used under the terms of + * either the GNU General Public License Version 2 or later (the "GPL"), or + * the GNU Lesser General Public License Version 2.1 or later (the "LGPL"), + * in which case the provisions of the GPL or the LGPL are applicable instead + * of those above. If you wish to allow use of your version of this file only + * under the terms of either the GPL or the LGPL, and not to allow others to + * use your version of this file under the terms of the MPL, indicate your + * decision by deleting the provisions above and replace them with the notice + * and other provisions required by the GPL or the LGPL. If you do not delete + * the provisions above, a recipient may use your version of this file under + * the terms of any one of the MPL, the GPL or the LGPL. + * + * ***** END LICENSE BLOCK ***** */ + +class EditorSubscription extends AppModel +{ + var $name = 'EditorSubscription'; + var $useTable = 'editor_subscriptions'; + var $belongsTo = array('Addon', 'User'); + + var $recursive = -1; + + /** + * Sign editor up for an email the next time an add-on is updated. + * @param int $userid User's ID to be signed up for the next update + * @param int $addonid Addon's ID to subscribe the user to + * @return void + */ + function subscribeToUpdates($userid, $addonid) { + $this->query(" + INSERT INTO editor_subscriptions + SET + user_id = '{$userid}', + addon_id = '{$addonid}';"); + } + + /** + * Do not notify editor the next time an add-on is updated. + * @param int $userid User's ID to be signed up for the next update + * @param int $addonid Addon's ID to subscribe the user to + * @return void + */ + function cancelUpdates($userid, $addonid) { + $this->query(" + DELETE FROM editor_subscriptions + WHERE + user_id = '{$userid}' AND + addon_id = '{$addonid}';"); + } + + /** + * get list of subscribers for a given add-on + * @param int $addonid ID of the add-on whose subscribers are required + * @return array user IDs for all subscribers of the add-on + */ + function getSubscribers($addonid) { + $_result = $this->query(" + SELECT user_id + FROM editor_subscriptions + WHERE addon_id = '{$addonid};'"); + if (empty($_result)) return array(); + + $_ids = array(); + foreach ($_result as &$_id) { + $_ids[] = $_id['editor_subscriptions']['user_id']; + } + + return $_ids; + } +} +?> diff --git a/site/app/models/review.php b/site/app/models/review.php index 8c53304..96ad236 100644 --- a/site/app/models/review.php +++ b/site/app/models/review.php @@ -231,6 +231,78 @@ class Review extends AppModel } return $results; } + + /** + * Update the bayesian rating (cf. bug 477343) for a single (or multiple) add-on(s). + * Also updates average rating and total review count. + * + * Note that similar code exists in the reviews/ratings sections of bin/maintenance.php + * + * @param array addonids add-on IDs whose bayesian ratings to update. + * @return boolean success + */ + function updateBayesianRating($addonids = array()) { + if (empty($addonids)) return false; + + // get average review count and average rating + $rows = $this->query(" + SELECT AVG(a.cnt) AS avg_cnt + FROM ( + SELECT COUNT(*) AS cnt + FROM reviews AS r + INNER JOIN versions AS v ON (r.version_id = v.id) + WHERE reply_to IS NULL + AND rating > 0 + GROUP BY v.addon_id + ) AS a + "); + $avg_num_votes = $rows[0][0]['avg_cnt']; + + $rows = $this->query(" + SELECT AVG(a.addon_rating) AS avg_rating + FROM ( + SELECT AVG(rating) AS addon_rating + FROM reviews AS r + INNER JOIN versions AS v ON (r.version_id = v.id) + WHERE reply_to IS NULL + AND rating > 0 + GROUP BY v.addon_id + ) AS a + "); + $avg_rating = $rows[0][0]['avg_rating']; + + // update total review count and average rating + $this->query(" + UPDATE addons AS a + INNER JOIN ( + SELECT + versions.addon_id as addon_id, + COUNT(*) as count, + AVG(rating) as avg_rating + FROM reviews + INNER JOIN versions ON reviews.version_id = versions.id + WHERE reviews.reply_to IS NULL + AND versions.addon_id IN (".(implode(',',$addonids)).") + AND reviews.rating > 0 + GROUP BY versions.addon_id + ) AS c ON (a.id = c.addon_id) + SET a.totalreviews = c.count, + a.averagerating = ROUND(c.avg_rating, 2) + WHERE a.id IN (".(implode(',',$addonids)).") + "); + + // calculate and store bayesian rating + $this->query(" + UPDATE addons AS a + SET a.bayesianrating = + IF (a.totalreviews > 0, ( + ( ({$avg_num_votes} * {$avg_rating}) + (a.totalreviews * a.averagerating) ) / + ({$avg_num_votes} + a.totalreviews) + ), 0) + WHERE a.id IN (".(implode(',',$addonids)).") + "); + return true; + } } ?> |