[Tikiwiki-cvs/svn] SF.net SVN: tikiwiki:[37389] trunk

[<jy...@us...>]

Revision: 37389
          http://tikiwiki.svn.sourceforge.net/tikiwiki/?rev=37389&view=rev
Author:   jyhem
Date:     2011-09-18 17:46:09 +0000 (Sun, 18 Sep 2011)
Log Message:
-----------
[FIX] Fix comments moderation buttons which were broken on recent FireFox & IE following change of browser behaviour due to stricter implementation of image input buttons

Modified Paths:
--------------
    trunk/templates/tiki-list_comments.tpl
    trunk/tiki-list_comments.php

Modified: trunk/templates/tiki-list_comments.tpl
===================================================================
--- trunk/templates/tiki-list_comments.tpl	2011-09-18 17:37:04 UTC (rev 37388)
+++ trunk/templates/tiki-list_comments.tpl	2011-09-18 17:46:09 UTC (rev 37389)
@@ -78,7 +78,7 @@
 		{/capture}
 
 		<tr class="{cycle}{if $prefs.feature_comments_moderation eq 'y'} post-approved-{$comments[ix].approved}{/if}">
-			<td class="checkbox"><input type="checkbox" name="checked[]" value="{$id}"/></td>
+			<td class="checkbox"><input type="checkbox" name="checked[]" value="{$id}" {if isset($rejected[$id]) }checked="checked"{/if}/></td>
 			<td class="action">
 				<a title="{tr}Actions{/tr}" href="#" {popup trigger="onClick" sticky=1 mouseoff=1 fullhtml="1" center=true text=$smarty.capture.over_actions|escape:"javascript"|escape:"html"} style="padding:0; margin:0; border:0">{icon _id='wrench' alt="{tr}Actions{/tr}"}</a>
 			</td>
@@ -115,7 +115,7 @@
 				<td class="approval">
 					{if $comments[ix].approved eq 'n'}
 						{self_link approve='y' checked=$id _icon='comment_approve'}{tr}Approve{/tr}{/self_link}
-						{self_link approve='r' checked=$id _icon='comment_reject'}{tr}Reject{/tr}{/self_link}
+						{self_link reject='r' checked=$id _icon='comment_reject'}{tr}Reject{/tr}{/self_link}
 					{elseif $comments[ix].approved eq 'y'}
 						&nbsp;{tr}Approved{/tr}&nbsp;
 					{elseif $comments[ix].approved eq 'r'}
@@ -153,7 +153,7 @@
 		{/if}
 		{if $tiki_p_admin_comments eq 'y' and $prefs.feature_comments_moderation eq 'y'}
 			{icon _id='comment_approve' _tag='input_image' name='approve' value='y' alt="{tr}Approve{/tr}"}
-			{icon _id='comment_reject' _tag='input_image' name='approve' value='r' alt="{tr}Reject{/tr}"}
+			{icon _id='comment_reject' _tag='input_image' name='reject' value='r' alt="{tr}Reject{/tr}"}
 		{/if}
 		{if $tiki_p_admin_comments eq 'y' and $prefs.comments_archive eq 'y'}
 			{icon _id='folder' _tag='input_image' name='archive' value='archive' alt="{tr}Archive{/tr}"}

Modified: trunk/tiki-list_comments.php
===================================================================
--- trunk/tiki-list_comments.php	2011-09-18 17:37:04 UTC (rev 37388)
+++ trunk/tiki-list_comments.php	2011-09-18 17:46:09 UTC (rev 37389)
@@ -102,13 +102,22 @@
 		exit;
 	}
 
-	// Approve/Reject comment(s)
-	if ($prefs['feature_comments_moderation'] == 'y' && isset($_REQUEST['approve']) && in_array($_REQUEST['approve'], array('y', 'n', 'r'))) {
+	// Approve comment(s)
+	if ($prefs['feature_comments_moderation'] == 'y' && isset($_REQUEST['approve_x']) ) {
 		foreach($checked as $id) {
-			$commentslib->approve_comment($id, $_REQUEST['approve']);
+			$commentslib->approve_comment($id, 'y');
 		}
 	}
-	
+
+	// Reject comment(s)
+	if ($prefs['feature_comments_moderation'] == 'y' && isset($_REQUEST['reject_x'])) {
+		foreach($checked as $id) {
+			$commentslib->approve_comment($id, 'r');
+			$rejected[$id] = true;
+		}
+		$smarty->assign_by_ref('rejected', $rejected);
+	}
+
 	// Archive/unarchive comment(s)
 	if ($prefs['comments_archive'] == 'y' && isset($_REQUEST['archive']) && in_array($_REQUEST['archive'], array('archive', 'unarchive'))) {
 		foreach($checked as $id) {

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.