Skip to content

Commit

Permalink
feature 1538: "select all" (+none, +invert) on user comments validati…
Browse files Browse the repository at this point in the history
…on screen.

git-svn-id: http://piwigo.org/svn/trunk@5381 68402e56-0260-453c-a942-63ccdbb3a9ee
  • Loading branch information
plegall committed Mar 26, 2010
1 parent 0197899 commit 6d00630
Show file tree
Hide file tree
Showing 5 changed files with 88 additions and 63 deletions.
80 changes: 29 additions & 51 deletions admin/comments.php
Expand Up @@ -38,77 +38,55 @@
// | actions |
// +-----------------------------------------------------------------------+

if (isset($_POST))
if (!empty($_POST) and !is_adviser())
{
$to_validate = array();
$to_reject = array();

if (isset($_POST['submit']) and !is_adviser())
{
foreach (explode(',', $_POST['list']) as $comment_id)
{
if (isset($_POST['action-'.$comment_id]))
{
switch ($_POST['action-'.$comment_id])
{
case 'reject' :
{
array_push($to_reject, $comment_id);
break;
}
case 'validate' :
{
array_push($to_validate, $comment_id);
break;
}
}
}
}
}
else if (isset($_POST['validate-all']) and !empty($_POST['list']) and !is_adviser())
{
$to_validate = explode(',', $_POST['list']);
}
else if (isset($_POST['reject-all']) and !empty($_POST['list']) and !is_adviser())
if (empty($_POST['comments']))
{
$to_reject = explode(',', $_POST['list']);
array_push(
$page['errors'],
l10n('Select at least one comment')
);
}

if (count($to_validate) > 0)
else
{
$query = '
check_input_parameter('comments', $_POST, true, PATTERN_ID);

if (isset($_POST['validate']))
{
$query = '
UPDATE '.COMMENTS_TABLE.'
SET validated = \'true\'
, validation_date = NOW()
WHERE id IN ('.implode(',', $to_validate).')
WHERE id IN ('.implode(',', $_POST['comments']).')
;';
pwg_query($query);

array_push(
$page['infos'],
l10n_dec(
'%d user comment validated', '%d user comments validated',
count($to_validate)
count($_POST['comments'])
)
);
}
}

if (count($to_reject) > 0)
{
$query = '
if (isset($_POST['reject']))
{
$query = '
DELETE
FROM '.COMMENTS_TABLE.'
WHERE id IN ('.implode(',', $to_reject).')
WHERE id IN ('.implode(',', $_POST['comments']).')
;';
pwg_query($query);

array_push(
$page['infos'],
l10n_dec(
'%d user comment rejected', '%d user comments rejected',
count($to_reject)
)
);
pwg_query($query);

array_push(
$page['infos'],
l10n_dec(
'%d user comment rejected', '%d user comments rejected',
count($_POST['comments'])
)
);
}
}
}

Expand Down
3 changes: 3 additions & 0 deletions admin/themes/default/default-layout.css
Expand Up @@ -788,3 +788,6 @@ h2:lang(en) { text-transform:capitalize; }
}

.content .infos li, .content .errors li { list-style-type:square; }

.checkActions {text-align:left;padding:0;margin:0;}
.comment A:hover {border:none;}
58 changes: 48 additions & 10 deletions admin/themes/default/template/comments.tpl
@@ -1,3 +1,32 @@
{literal}
<script>
$(document).ready(function(){
$(".checkComment").click(function () {
var checkbox = $(this).children("input[type=checkbox]");
$(checkbox).attr('checked', !$(checkbox).is(':checked'));
});

$("#commentSelectAll").click(function () {
$(".checkComment input[type=checkbox]").attr('checked', true);
return false;
});

$("#commentSelectNone").click(function () {
$(".checkComment input[type=checkbox]").attr('checked', false);
return false;
});

$("#commentSelectInvert").click(function () {
$(".checkComment input[type=checkbox]").each(function() {
$(this).attr('checked', !$(this).is(':checked'));
});
return false;
});

});
</script>
{/literal}

<div class="titrePage">
<h2>{'Waiting'|@translate} {$TABSHEET_TITLE}</h2>
</div>
Expand All @@ -7,24 +36,33 @@
{if !empty($comments) }
<form method="post" action="{$F_ACTION}">

{foreach from=$comments item=comment}
<table width="99%">
{foreach from=$comments item=comment name=comment}
<tr valign="top" class="{if $smarty.foreach.comment.index is odd}row2{else}row1{/if}">
<td style="width:50px;" class="checkComment">
<input type="checkbox" name="comments[]" value="{$comment.ID}">
</td>
<td>
<div class="comment">
<a class="illustration" href="{$comment.U_PICTURE}"><img src="{$comment.TN_SRC}"></a>
<p class="commentHeader"><strong>{$comment.AUTHOR}</strong> - <em>{$comment.DATE}</em></p>
<blockquote>{$comment.CONTENT}</blockquote>
</div>
<ul class="actions">
<li><label><input type="radio" name="action-{$comment.ID}" value="reject">{'Reject'|@translate}</label></li>
<li><label><input type="radio" name="action-{$comment.ID}" value="validate">{'Validate'|@translate}</label></li>
</ul>
</td>
</tr>
{/foreach}
</table>

<p class="checkActions">
{'Select:'|@translate}
<a href="#" id="commentSelectAll">{'All'|@translate}</a>,
<a href="#" id="commentSelectNone">{'None'|@translate}</a>,
<a href="#" id="commentSelectInvert">{'Invert'|@translate}</a>
</p>

<p class="bottomButtons">
<input type="hidden" name="list" value="{$LIST}">
<input class="submit" type="submit" name="submit" value="{'Submit'|@translate}" {$TAG_INPUT_ENABLED}>
<input class="submit" type="submit" name="validate-all" value="{'Validate All'|@translate}" {$TAG_INPUT_ENABLED}>
<input class="submit" type="submit" name="reject-all" value="{'Reject All'|@translate}" {$TAG_INPUT_ENABLED}>
<input class="submit" type="reset" value="{'Reset'|@translate}">
<input class="submit" type="submit" name="validate" value="{'Validate'|@translate}" {$TAG_INPUT_ENABLED}>
<input class="submit" type="submit" name="reject" value="{'Reject'|@translate}" {$TAG_INPUT_ENABLED}>
</p>

</form>
Expand Down
5 changes: 4 additions & 1 deletion language/en_UK/admin.lang.php
Expand Up @@ -755,4 +755,7 @@
$lang['Installed Languages'] = 'Installed Languages';
$lang['Add New Language'] = 'Add New Language';
$lang['Language has been successfully installed'] = 'Language has been successfully installed';
?>
$lang['Select:'] = 'Select:';
$lang['None'] = 'None';
$lang['Invert'] = 'Invert';
?>
5 changes: 4 additions & 1 deletion language/fr_FR/admin.lang.php
Expand Up @@ -760,4 +760,7 @@
$lang['Add New Language'] = 'Ajouter une langue';
$lang['Language has been successfully installed'] = 'La langue a été installée avec succès';
$lang['Allow user customization'] = "Permettre la personnalisation de l'affichage";
?>
$lang['Select:'] = 'Sélectionner:';
$lang['None'] = 'Rien';
$lang['Invert'] = 'Inverser';
?>

0 comments on commit 6d00630

Please sign in to comment.