by trabis on 2007/12/24 16:19:31

Hello,
It's simple to do and it should be implemented in the next version of Xoops.

Just replace your root/class/commentrenderer.php with this content:

<?php
// $Id: commentrenderer.php 2 2005-11-02 18:23:29Z skalpa $
// ------------------------------------------------------------------------ //
// XOOPS - PHP Content Management System //
// Copyright (c) 2000 XOOPS.org //
// <https://xoops.org/> //
// ------------------------------------------------------------------------ //
// This program is free software; you can redistribute it and/or modify //
// it under the terms of the GNU General Public License as published by //
// the Free Software Foundation; either version 2 of the License, or //
// (at your option) any later version. //
// //
// You may not change or alter any portion of this comment or credits //
// of supporting developers from this source code or any supporting //
// source code which is considered copyrighted (c) material of the //
// original comment or credit authors. //
// //
// This program is distributed in the hope that it will be useful, //
// but WITHOUT ANY WARRANTY; without even the implied warranty of //
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the //
// GNU General Public License for more details. //
// //
// You should have received a copy of the GNU General Public License //
// along with this program; if not, write to the Free Software //
// Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA //
// ------------------------------------------------------------------------ //
// Author: Kazumi Ono (AKA onokazu) //
// URL:https://xoops.org/http://www.xoopscube.jp/http://www.myweb.ne.jp/ //
// Project: The XOOPS Project (https://xoops.org/) //
// ------------------------------------------------------------------------- //
/**
* Display comments
*
* @package kernel
* @subpackage comment
*
* @author Kazumi Ono <onokazu@xoops.org>
* @copyright (c) 2000-2003 The XOOPS Project - www.xoops.org
*/
class XoopsCommentRenderer {

/**#@+
* @access private
*/
var $_tpl;
var $_comments = null;
var $_useIcons = true;
var $_doIconCheck = false;
var $_memberHandler;
var $_statusText;
/**#@-*/

/**
* Constructor
*
* @param object &$tpl
* @param boolean $use_icons
* @param boolean $do_iconcheck
**/
function XoopsCommentRenderer(&$tpl, $use_icons = true, $do_iconcheck = false)
{
$this->_tpl =& $tpl;
$this->_useIcons = $use_icons;
$this->_doIconCheck = $do_iconcheck;
$this->_memberHandler =& xoops_gethandler('member');
$this->_statusText = array(XOOPS_COMMENT_PENDING => '<span style="text-decoration: none; font-weight: bold; color: #00ff00;">'._CM_PENDING.'</span>', XOOPS_COMMENT_ACTIVE => '<span style="text-decoration: none; font-weight: bold; color: #ff0000;">'._CM_ACTIVE.'</span>', XOOPS_COMMENT_HIDDEN => '<span style="text-decoration: none; font-weight: bold; color: #0000ff;">'._CM_HIDDEN.'</span>');
}

/**
* Access the only instance of this class
*
* @param object $tpl reference to a {@link Smarty} object
* @param boolean $use_icons
* @param boolean $do_iconcheck
* @return
**/
function &instance(&$tpl, $use_icons = true, $do_iconcheck = false)
{
static $instance;
if (!isset($instance)) {
$instance = new XoopsCommentRenderer($tpl, $use_icons, $do_iconcheck);
}
return $instance;
}

/**
* Accessor
*
* @param object &$comments_arr array of {@link XoopsComment} objects
**/
function setComments(&$comments_arr)
{
if (isset($this->_comments)) {
unset($this->_comments);
}
$this->_comments =& $comments_arr;
}

/**
* Render the comments in flat view
*
* @param boolean $admin_view
**/
function renderFlatView($admin_view = false)
{
$count = count($this->_comments);
for ($i = 0; $i < $count; $i++) {
if (false != $this->_useIcons) {
$title = $this->_getTitleIcon($this->_comments[$i]->getVar('com_icon')).'&nbsp;'.$this->_comments[$i]->getVar('com_title');
} else {
$title = $this->_comments[$i]->getVar('com_title');
}
$poster = $this->_getPosterArray($this->_comments[$i]->getVar('com_uid'));
if (false != $admin_view) {
$text = $this->_getText($this->_comments[$i]->getVar('com_text'),$this->_comments[$i]->getVar('com_pid')).'<div style="text-align:right; margin-top: 2px; margin-bottom: 0px; margin-right: 2px;">'._CM_STATUS.': '.$this->_statusText[$this->_comments[$i]->getVar('com_status')].'<br />IP: <span style="font-weight: bold;">'.$this->_comments[$i]->getVar('com_ip').'</span></div>';
} else {
// hide comments that are not active
if (XOOPS_COMMENT_ACTIVE != $this->_comments[$i]->getVar('com_status')) {
continue;
} else {
$text = $this->_getText($this->_comments[$i]->getVar('com_text'),$this->_comments[$i]->getVar('com_pid'));
}
}
$this->_tpl->append('comments', array('id' => $this->_comments[$i]->getVar('com_id'), 'title' => $title, 'text' => $text, 'date_posted' => formatTimestamp($this->_comments[$i]->getVar('com_created'), 'm'), 'date_modified' => formatTimestamp($this->_comments[$i]->getVar('com_modified'), 'm'), 'poster' => $poster));
}
}

/**
* Render the comments in thread view
*
* This method calls itself recursively
*
* @param integer $comment_id Should be "0" when called by client
* @param boolean $admin_view
* @param boolean $show_nav
**/
function renderThreadView($comment_id = 0, $admin_view = false, $show_nav = true)
{
include_once XOOPS_ROOT_PATH.'/class/tree.php';
// construct comment tree
$xot = new XoopsObjectTree($this->_comments, 'com_id', 'com_pid', 'com_rootid');
$tree =& $xot->getTree();

if (false != $this->_useIcons) {
$title = $this->_getTitleIcon($tree[$comment_id]['obj']->getVar('com_icon')).'&nbsp;'.$tree[$comment_id]['obj']->getVar('com_title');
} else {
$title = $tree[$comment_id]['obj']->getVar('com_title');
}
if (false != $show_nav && $tree[$comment_id]['obj']->getVar('com_pid') != 0) {
$this->_tpl->assign('lang_top', _CM_TOP);
$this->_tpl->assign('lang_parent', _CM_PARENT);
$this->_tpl->assign('show_threadnav', true);
} else {
$this->_tpl->assign('show_threadnav', false);
}
if (false != $admin_view) {
// admins can see all
$text = $this->_getText($tree[$comment_id]['obj']->getVar('com_text'),$tree[$comment_id]['obj']->getVar('com_pid')).'<div style="text-align:right; margin-top: 2px; margin-bottom: 0px; margin-right: 2px;">'._CM_STATUS.': '.$this->_statusText[$tree[$comment_id]['obj']->getVar('com_status')].'<br />IP: <span style="font-weight: bold;">'.$tree[$comment_id]['obj']->getVar('com_ip').'</span></div>';
} else {
// hide comments that are not active
if (XOOPS_COMMENT_ACTIVE != $tree[$comment_id]['obj']->getVar('com_status')) {
// if there are any child comments, display them as root comments
if (isset($tree[$comment_id]['child']) && !empty($tree[$comment_id]['child'])) {
foreach ($tree[$comment_id]['child'] as $child_id) {
$this->renderThreadView($child_id, $admin_view, false);
}
}
return;
} else {
$text = $this->_getText($tree[$comment_id]['obj']->getVar('com_text'),$tree[$comment_id]['obj']->getVar('com_pid'));
}
}
$replies = array();
$this->_renderThreadReplies($tree, $comment_id, $replies, '&nbsp;&nbsp;', $admin_view);
$show_replies = (count($replies) > 0) ? true : false;
$this->_tpl->append('comments', array('pid' => $tree[$comment_id]['obj']->getVar('com_pid'), 'id' => $tree[$comment_id]['obj']->getVar('com_id'), 'itemid' => $tree[$comment_id]['obj']->getVar('com_itemid'), 'rootid' => $tree[$comment_id]['obj']->getVar('com_rootid'), 'title' => $title, 'text' => $text, 'date_posted' => formatTimestamp($tree[$comment_id]['obj']->getVar('com_created'), 'm'), 'date_modified' => formatTimestamp($tree[$comment_id]['obj']->getVar('com_modified'), 'm'), 'poster' => $this->_getPosterArray($tree[$comment_id]['obj']->getVar('com_uid')), 'replies' => $replies, 'show_replies' => $show_replies));
}

/**
* Render replies to a thread
*
* @param array &$thread
* @param int $key
* @param array $replies
* @param string $prefix
* @param bool $admin_view
* @param integer $depth
* @param string $current_prefix
*
* @access private
**/
function _renderThreadReplies(&$thread, $key, &$replies, $prefix, $admin_view, $depth = 0, $current_prefix = '')
{
if ($depth > 0) {
if (false != $this->_useIcons) {
$title = $this->_getTitleIcon($thread[$key]['obj']->getVar('com_icon')).'&nbsp;'.$thread[$key]['obj']->getVar('com_title');
} else {
$title = $thread[$key]['obj']->getVar('com_title');
}
$title = (false != $admin_view) ? $title.' '.$this->_statusText[$thread[$key]['obj']->getVar('com_status')] : $title;
$replies[] = array('id' => $key, 'prefix' => $current_prefix, 'date_posted' => formatTimestamp($thread[$key]['obj']->getVar('com_created'), 'm'), 'title' => $title, 'root_id' => $thread[$key]['obj']->getVar('com_rootid'), 'status' => $this->_statusText[$thread[$key]['obj']->getVar('com_status')], 'poster' => $this->_getPosterName($thread[$key]['obj']->getVar('com_uid')));
$current_prefix .= $prefix;
}
if (isset($thread[$key]['child']) && !empty($thread[$key]['child'])) {
$depth++;
foreach ($thread[$key]['child'] as $childkey) {
if (!$admin_view && $thread[$childkey]['obj']->getVar('com_status') != XOOPS_COMMENT_ACTIVE) {
// skip this comment if it is not active and continue on processing its child comments instead
if (isset($thread[$childkey]['child']) && !empty($thread[$childkey]['child'])) {
foreach ($thread[$childkey]['child'] as $childchildkey) {
$this->_renderThreadReplies($thread, $childchildkey, $replies, $prefix, $admin_view, $depth);
}
}
} else {
$this->_renderThreadReplies($thread, $childkey, $replies, $prefix, $admin_view, $depth, $current_prefix);
}
}
}
}

/**
* Render comments in nested view
*
* Danger: Recursive!
*
* @param integer $comment_id Always "0" when called by client.
* @param boolean $admin_view
**/
function renderNestView($comment_id = 0, $admin_view = false)
{
include_once XOOPS_ROOT_PATH.'/class/tree.php';
$xot = new XoopsObjectTree($this->_comments, 'com_id', 'com_pid', 'com_rootid');
$tree =& $xot->getTree();
if (false != $this->_useIcons) {
$title = $this->_getTitleIcon($tree[$comment_id]['obj']->getVar('com_icon')).'&nbsp;'.$tree[$comment_id]['obj']->getVar('com_title');
} else {
$title = $tree[$comment_id]['obj']->getVar('com_title');
}
if (false != $admin_view) {
$text = $this->_getText($tree[$comment_id]['obj']->getVar('com_text'),$tree[$comment_id]['obj']->getVar('com_pid')).'<div style="text-align:right; margin-top: 2px; margin-bottom: 0px; margin-right: 2px;">'._CM_STATUS.': '.$this->_statusText[$tree[$comment_id]['obj']->getVar('com_status')].'<br />IP: <span style="font-weight: bold;">'.$tree[$comment_id]['obj']->getVar('com_ip').'</span></div>';
} else {
// skip this comment if it is not active and continue on processing its child comments instead
if (XOOPS_COMMENT_ACTIVE != $tree[$comment_id]['obj']->getVar('com_status')) {
// if there are any child comments, display them as root comments
if (isset($tree[$comment_id]['child']) && !empty($tree[$comment_id]['child'])) {
foreach ($tree[$comment_id]['child'] as $child_id) {
$this->renderNestView($child_id, $admin_view);
}
}
return;
} else {
$text = $this->_getText($tree[$comment_id]['obj']->getVar('com_text'),$tree[$comment_id]['obj']->getVar('com_pid'));
}
}
$replies = array();
$this->_renderNestReplies($tree, $comment_id, $replies, 25, $admin_view);
$this->_tpl->append('comments', array('pid' => $tree[$comment_id]['obj']->getVar('com_pid'), 'id' => $tree[$comment_id]['obj']->getVar('com_id'), 'itemid' => $tree[$comment_id]['obj']->getVar('com_itemid'), 'rootid' => $tree[$comment_id]['obj']->getVar('com_rootid'), 'title' => $title, 'text' => $text, 'date_posted' => formatTimestamp($tree[$comment_id]['obj']->getVar('com_created'), 'm'), 'date_modified' => formatTimestamp($tree[$comment_id]['obj']->getVar('com_modified'), 'm'), 'poster' => $this->_getPosterArray($tree[$comment_id]['obj']->getVar('com_uid')), 'replies' => $replies));
}

/**
* Render replies in nested view
*
* @param array $thread
* @param int $key
* @param array $replies
* @param string $prefix
* @param bool $admin_view
* @param integer $depth
*
* @access private
**/
function _renderNestReplies(&$thread, $key, &$replies, $prefix, $admin_view, $depth = 0)
{
if ($depth > 0) {
if (false != $this->_useIcons) {
$title = $this->_getTitleIcon($thread[$key]['obj']->getVar('com_icon')).'&nbsp;'.$thread[$key]['obj']->getVar('com_title');
} else {
$title = $thread[$key]['obj']->getVar('com_title');
}
$text = (false != $admin_view) ? $this->_getText($thread[$key]['obj']->getVar('com_text'),$thread[$key]['obj']->getVar('com_pid')).'<div style="text-align:right; margin-top: 2px; margin-right: 2px;">'._CM_STATUS.': '.$this->_statusText[$thread[$key]['obj']->getVar('com_status')].'<br />IP: <span style="font-weight: bold;">'.$thread[$key]['obj']->getVar('com_ip').'</span></div>' : $this->_getText($thread[$key]['obj']->getVar('com_text'),$thread[$key]['obj']->getVar('com_pid'));
$replies[] = array('id' => $key, 'prefix' => $prefix, 'pid' => $thread[$key]['obj']->getVar('com_pid'), 'itemid' => $thread[$key]['obj']->getVar('com_itemid'), 'rootid' => $thread[$key]['obj']->getVar('com_rootid'), 'title' => $title, 'text' => $text, 'date_posted' => formatTimestamp($thread[$key]['obj']->getVar('com_created'), 'm'), 'date_modified' => formatTimestamp($thread[$key]['obj']->getVar('com_modified'), 'm'), 'poster' => $this->_getPosterArray($thread[$key]['obj']->getVar('com_uid')));

$prefix = $prefix + 25;
}
if (isset($thread[$key]['child']) && !empty($thread[$key]['child'])) {
$depth++;
foreach ($thread[$key]['child'] as $childkey) {
if (!$admin_view && $thread[$childkey]['obj']->getVar('com_status') != XOOPS_COMMENT_ACTIVE) {
// skip this comment if it is not active and continue on processing its child comments instead
if (isset($thread[$childkey]['child']) && !empty($thread[$childkey]['child'])) {
foreach ($thread[$childkey]['child'] as $childchildkey) {
$this->_renderNestReplies($thread, $childchildkey, $replies, $prefix, $admin_view, $depth);
}
}
} else {
$this->_renderNestReplies($thread, $childkey, $replies, $prefix, $admin_view, $depth);
}
}
}
}


/**
* Get the name of the poster
*
* @param int $poster_id
* @return string
*
* @access private
**/
function _getPosterName($poster_id)
{
$poster['id'] = intval($poster_id);
if ($poster['id'] > 0) {
$com_poster =& $this->_memberHandler->getUser($poster_id);
if (is_object($com_poster)) {
$poster['uname'] = '<a href="'.XOOPS_URL.'/userinfo.php?uid='.$poster['id'].'">'.$com_poster->getVar('uname').'</a>';
return $poster;
}
}
$poster['id'] = 0; // to cope with deleted user accounts
$poster['uname'] = $GLOBALS['xoopsConfig']['anonymous'];
return $poster;
}

/**
* Get an array with info about the poster
*
* @param int $poster_id
* @return array
*
* @access private
**/
function _getPosterArray($poster_id)
{
$poster['id'] = intval($poster_id);
if ($poster['id'] > 0) {
$com_poster =& $this->_memberHandler->getUser($poster['id']);
if (is_object($com_poster)) {
$poster['uname'] = '<a href="'.XOOPS_URL.'/userinfo.php?uid='.$poster['id'].'">'.$com_poster->getVar('uname').'</a>';
$poster_rank = $com_poster->rank();
$poster['rank_image'] = ($poster_rank['image'] != '') ? $poster_rank['image'] : 'blank.gif';
$poster['rank_title'] = $poster_rank['title'];
$poster['avatar'] = $com_poster->getVar('user_avatar');
$poster['regdate'] = formatTimestamp($com_poster->getVar('user_regdate'), 's');
$poster['from'] = $com_poster->getVar('user_from');
$poster['postnum'] = $com_poster->getVar('posts');
$poster['status'] = $com_poster->isOnline() ? _CM_ONLINE : '';
return $poster;
}
}
$poster['id'] = 0; // to cope with deleted user accounts
$poster['uname'] = $GLOBALS['xoopsConfig']['anonymous'];
$poster['rank_title'] = '';
$poster['avatar'] = 'blank.gif';
$poster['regdate'] = '';
$poster['from'] = '';
$poster['postnum'] = 0;
$poster['status'] = '';
return $poster;
}

/**
* Get the IMG tag for the title icon
*
* @param string $icon_image
* @return string HTML IMG tag
*
* @access private
**/
function _getTitleIcon($icon_image)
{
$icon_image = htmlspecialchars( trim( $icon_image ) );
if ($icon_image != '') {
if (false != $this->_doIconCheck) {
if (!file_exists(XOOPS_URL.'/images/subject/'.$icon_image)) {
return '<img src="'.XOOPS_URL.'/images/icons/no_posticon.gif" alt="" />';
} else {
return '<img src="'.XOOPS_URL.'/images/subject/'.$icon_image.'" alt="" />';
}
} else {
return '<img src="'.XOOPS_URL.'/images/subject/'.$icon_image.'" alt="" />';
}
}
return '<img src="'.XOOPS_URL.'/images/icons/no_posticon.gif" alt="" />';
}

/**
* Get Text with Signature
*
* @param string $text
* @param int $uid
* @return string
*
* @access private
**/
function _getText($text='', $uid=0)
{
global $xoopsConfig, $xoopsUser;
if ( $uid != 0 ) {
$poster = new XoopsUser($uid);
if ( !$poster->isActive() ) {
$poster = 0;
}
} else {
$poster = 0;
}

if ( $poster ) {
if ( $poster->getVar("attachsig") ) {
$text .= "<p><br />_________________<br />". $poster->user_sig()."</p>";
}
}
return $text;
}

}
?>

I´ve just added an extra function to add user signature to comments in case user is using it.
It will work in all comments of all modules that use the system comments.

There is no need to touch anything else.

Of corse, please backup first and give some feedback here.