Skip to content

Commit

Permalink
feature 2486: Add an admin view for rates by user (improvement)
Browse files Browse the repository at this point in the history
git-svn-id: http://piwigo.org/svn/trunk@12650 68402e56-0260-453c-a942-63ccdbb3a9ee
  • Loading branch information
rvelices committed Nov 21, 2011
1 parent 95325c6 commit 85a60f1
Show file tree
Hide file tree
Showing 2 changed files with 47 additions and 33 deletions.
59 changes: 36 additions & 23 deletions admin/rating_user.php
Expand Up @@ -36,6 +36,7 @@
$filter_min_rates = (int)$_GET['f_min_rates'];
}

// build users
global $conf;
$query = 'SELECT DISTINCT
u.'.$conf['user_fields']['id'].' AS id,
Expand All @@ -48,49 +49,45 @@
$result = pwg_query($query);
while ($row = pwg_db_fetch_assoc($result))
{
$user = array(
$users_by_id[(int)$row['id']] = array(
'name' => $row['name'],
'anon' => is_autorize_status(ACCESS_CLASSIC, $row['status']) ? false : true
);
$users_by_id[(int)$row['id']] = $user;
}

$by_rate_model = array();
$by_user_rating_model = array( 'rates' => array() );
foreach($conf['rate_items'] as $rate)
{
$by_rate_model[$rate] = array();
$by_user_rating_model['rates'][$rate] = array();
}


$by_user_rating_model = array( 'rates' => $by_rate_model);

$image_ids = array();
$by_user_ratings = array();
$query = '
SELECT * FROM '.RATE_TABLE.' ORDER by date DESC';
$result = pwg_query($query);
while ($row = pwg_db_fetch_assoc($result))
{

if (!isset($users_by_id[$row['user_id']]))
{
$users_by_id[$row['user_id']] = array('name' => '???'.$row['user_id'], 'anon' => false);
}
$user = $users_by_id[$row['user_id']];
if ($user['anon'])
$usr = $users_by_id[$row['user_id']];
if ($usr['anon'])
{
$user_key = $user['name'].'('.$row['anonymous_id'].')';
$user_key = $usr['name'].'('.$row['anonymous_id'].')';
}
else
{
$user_key = $user['name'];
$user_key = $usr['name'];
}
$rating = & $by_user_ratings[$user_key];
if ( is_null($rating) )
{
$rating = $by_user_rating_model;
$rating['uid'] = (int)$row['user_id'];
$rating['aid'] = $user['anon'] ? $row['anonymous_id'] : '';
$rating['aid'] = $usr['anon'] ? $row['anonymous_id'] : '';
}
$rating['rates'][$row['rate']][] = array(
'id' => $row['element_id'],
Expand All @@ -100,8 +97,7 @@
unset($rating);
}



// get image tn urls
$image_urls = array();
if (count($image_ids) > 0 )
{
Expand All @@ -118,24 +114,41 @@
}
}

$query='SELECT element_id,
AVG(rate) AS avg
FROM '.RATE_TABLE.'
GROUP BY element_id';
$all_img_sum = array();
$result = pwg_query($query);
while ($row = pwg_db_fetch_assoc($result))
{
$all_img_sum[(int)$row['element_id']] = array( 'avg'=>(float)$row['avg'] );
}

foreach($by_user_ratings as $id => &$rating)
{
$c=0; $s=0; $ss=0;
$c=0; $s=0; $ss=0; $consensus_dev=0;
foreach($rating['rates'] as $rate => $rates)
{
$ct = count($rates);
$c += $ct;
$s += $ct * $rate;
$ss += $ct * $rate * $rate;
foreach($rates as $id_date)
{
$consensus_dev += abs($rate - $all_img_sum[$id_date['id']]['avg']);
}
}

$consensus_dev /= $c;

$var = ($ss - $s*$s/$c)/$c;
$rating += array(
'id' => $id,
'count' => $c,
'avg' => $s/$c,
'std' => sqrt($var),
'cv' => $s==0 ? -1 : sqrt($var)/($s/$c), // http://en.wikipedia.org/wiki/Coefficient_of_variation
'cd' => $consensus_dev
);
}
unset($rating);
Expand All @@ -162,19 +175,18 @@ function count_compare($a, $b)
return ($d==0) ? 0 : ($d<0 ? -1 : 1);
}

function std_compare($a, $b)
function cv_compare($a, $b)
{
$d = $a['std'] - $b['std'];
$d = $b['cv'] - $a['cv']; //desc
return ($d==0) ? 0 : ($d<0 ? -1 : 1);
}

function cv_compare($a, $b)
function consensus_dev_compare($a, $b)
{
$d = $a['cv'] - $b['cv'];
$d = $b['cd'] - $a['cd']; //desc
return ($d==0) ? 0 : ($d<0 ? -1 : 1);
}


$order_by_index=3;
if (isset($_GET['order_by']) and is_numeric($_GET['order_by']))
{
Expand All @@ -184,8 +196,8 @@ function cv_compare($a, $b)
$available_order_by= array(
array(l10n('Average rate'), 'avg_compare'),
array(l10n('Number of rates'), 'count_compare'),
array('StDev', 'std_compare'),
array('Coeff of Variation', 'cv_compare'),
array(l10n('Variation'), 'cv_compare'),
array(l10n('Consensus deviation'), 'consensus_dev_compare'),
);

for ($i=0; $i<count($available_order_by); $i++)
Expand All @@ -205,6 +217,7 @@ function cv_compare($a, $b)
'available_rates' => $conf['rate_items'],
'ratings' => $by_user_ratings,
'image_urls' => $image_urls,
'TN_WIDTH' => 20+2*$conf['upload_form_thumb_maxwidth'],
) );
$template->set_filename('rating', 'rating_user.tpl');
$template->assign_var_from_handle('ADMIN_CONTENT', 'rating');
Expand Down
21 changes: 11 additions & 10 deletions admin/themes/default/template/rating_user.tpl
@@ -1,3 +1,5 @@
<h2>{$ratings|@count} {'Users'|@translate}</h2>

<form action="{$F_ACTION}" method="GET">
<fieldset>
<label>{'Sort by'|@translate}
Expand All @@ -18,13 +20,12 @@
{footer_script}{literal}
function del(elt,uid,aid)
{
if (!confirm('{'Are you sure?'|@translate|@escape:'javascript'}'))
if (!confirm({/literal}'{'Are you sure?'|@translate|@escape:'javascript'}'{literal}))
return false;
var tr = elt;
while ( tr.nodeName != "TR") tr = tr.parentNode;
tr = jQuery(tr).fadeTo(1000, 0.4);
var ws = new PwgWS({/literal}'{$ROOT_URL|@escape:javascript}'{literal});
ws.callService(
(new PwgWS({/literal}'{$ROOT_URL|@escape:javascript}'{literal})).callService(
'pwg.rates.delete', {user_id:uid, anonymous_id:aid},
{
method: 'POST',
Expand All @@ -40,8 +41,8 @@ function del(elt,uid,aid)
<td>{'Username'|@translate}</td>
<td>{'Number of rates'|@translate}</td>
<td>{'Average rate'|@translate}</td>
<td>StDev</td>
<td>CV</td>
<td>{'Variation'|@translate}</td>
<td>{'Consensus deviation'|@translate|@replace:' ':'<br>'}</td>
{foreach from=$available_rates item=rate}
<td>{$rate}</td>
{/foreach}
Expand All @@ -52,8 +53,8 @@ function del(elt,uid,aid)
<td>{$user}</td>
<td>{$rating.count}</td>
<td>{$rating.avg|@number_format:2}</td>
<td>{$rating.std|@number_format:3}</td>
<td>{$rating.cv|@number_format:3}</td>
<td>{$rating.cd|@number_format:3}</td>
{foreach from=$rating.rates item=rates key=rate}
<td>{if !empty($rates)}
{capture assign=rate_over}{foreach from=$rates item=rate_arr}<img src="{$image_urls[$rate_arr.id].tn}" alt="thumb-{$rate_arr.id}" title="{$rate_arr.date}"></img>
Expand All @@ -68,9 +69,9 @@ function del(elt,uid,aid)

{combine_script id='jquery.cluetip' load='footer' require='jquery' path='themes/default/js/plugins/jquery.cluetip.js'}
{footer_script require='jquery.cluetip'}
{literal}
jQuery('.cluetip').cluetip({
width: 350, splitTitle: '|'
jQuery(document).ready(function(){ldelim}
jQuery('.cluetip').cluetip({ldelim}
width: {$TN_WIDTH}, splitTitle: '|'
});
{/literal}
})
{/footer_script}

0 comments on commit 85a60f1

Please sign in to comment.