Topic: sp_recent block avatar add

[gebekzor]

Hello, my question is how to add avatar to block sp_recent author of last post? As in the top posters.
This is connected with this code:

Code: [Select]

function sp_recent($parameters, $id, $return_parameters = false)
{
global $txt, $scripturl, $settings, $context, $color_profile;

$block_parameters = array(
'boards' => 'boards',
'limit' => 'int',
'type' => 'select',
'display' => 'select',
);

if ($return_parameters)
return $block_parameters;

$boards = !empty($parameters['boards']) ? explode('|', $parameters['boards']) : null;
$limit = !empty($parameters['limit']) ? (int) $parameters['limit'] : 5;
$type = 'ssi_recent' . (empty($parameters['type']) ? 'Posts' : 'Topics');
$display = empty($parameters['display']) ? 'compact' : 'full';

$items = $type($limit, null, $boards, 'array');

if (empty($items))
{
echo '
', $txt['error_sp_no_posts_found'];
return;
}
else
$items[count($items) - 1]['is_last'] = true;

$colorids = array();
foreach ($items as $item)
$colorids[] = $item['poster']['id'];

if (!empty($colorids) && sp_loadColors($colorids) !== false)
{
foreach ($items as $k => $p)
{
if (!empty($color_profile[$p['poster']['id']]['link']))
$items[$k]['poster']['link'] = $color_profile[$p['poster']['id']]['link'];
}
}


if ($display == 'compact')
{
foreach ($items as $key => $item)
echo '
<div style="padding: 0px 3px;"><span class="smalltext"><a href="', $item['href'], '">', $item['subject'], '</a></span><br /> <span class="smalltext"> ', $item['poster']['link'], $item['new'] ? '' : '<a href="' . $scripturl . '?topic=' . $item['topic'] . '.msg' . $item['new_from'] . ';topicseen#new" rel="nofollow"></a>',', ', $item['time'], '</span></div>', empty($item['is_last']) ? '<hr>' : '';

}
elseif ($display == 'full')
{
echo '
<table class="sp_fullwidth">';

foreach ($items as $item)
echo '
<tr>
<td class="sp_recent_icon sp_center">
', sp_embed_image(empty($parameters['type']) ? 'post' : 'topic'), '
</td>
<td class="sp_recent_subject">
<a href="', $item['href'], '">', $item['subject'], '</a>
', $item['new'] ? '' : '<a href="' . $scripturl . '?topic=' . $item['topic'] . '.msg' . $item['new_from'] . ';topicseen#new"><img src="' . $settings['images_url'] . '/' . $context['user']['language'] . '/new.gif" alt="' . $txt['new'] . '" border="0" /></a>', '<br />[', $item['board']['link'], ']
</td>
<td class="sp_recent_info sp_right">
', $item['poster']['link'], '<br />', $item['time'], '
</td>
</tr>';

echo '
</table>';
}
}

To look like this:


And something else, and can reduce the number of letters in the names sp_recent thread?

[AngelinaBelle]

This block uses ssi_recentPosts and ssi_recentTopics to get its information. These functions are found in SSI.php. They do not provide the avatar. 
 
If I wanted to make this change, here's what I would do:
1) Make new functions to replace ssi_recentPosts and ssi_recentTopics. These new functions would be similar to the current recentPosts and recentTopics, but would have to also get the avatar for each member_id, and return that information.
2) Change the block sp_recent to use the avatar information.  This is a bit of customization work, and involves some work with SQL statements.

[Blue]

Hum... You could try adding this to the block above after each "foreach":

Code: [Select]

global $memberContext;
loadMemberData($item['poster']['id']);
loadMemberContext($item['poster']['id']);
and to output the avatar use something like:

Code: [Select]

<img src="' . $memberContext[$item['poster']['id']]['avatar']['href'] . '" alt="" />
To reduce the number of letters you can use the php function: substr ()

[AngelinaBelle]

.... Or you could do it the easy way
This could add several queries to the page, though (depends on the # of posts in the block), and what else is going on, on the page.

[Blue]

Yes, there is always a downside if we go the easy way but if it's a small block it's ok

[gebekzor]

Hum... You could try adding this to the block above after each "foreach":

Code: [Select]

global $memberContext;
loadMemberData($item['poster']['id']);
loadMemberContext($item['poster']['id']);
and to output the avatar use something like:

Code: [Select]

<img src="' . $memberContext[$item['poster']['id']]['avatar']['href'] . '" alt="" />
To reduce the number of letters you can use the php function: substr ()

It works but when I write somebody other avatar stops appear

[AngelinaBelle]

I think this idea should work. Please show your modified code. Maybe one of us can spot the problem in an instant.
 
If not, you should be able figure it out by putting in "debug echo" statements in lots of spots, to see where it is right and where it starts to go wrong.

[gebekzor]

Here is the code:

Code: [Select]

function sp_recent($parameters, $id, $return_parameters = false)
{
global $txt, $scripturl, $settings, $context, $color_profile;
global $memberContext;

$block_parameters = array(
'boards' => 'boards',
'limit' => 'int',
'type' => 'select',
'display' => 'select',
);

if ($return_parameters)
return $block_parameters;

$boards = !empty($parameters['boards']) ? explode('|', $parameters['boards']) : null;
$limit = !empty($parameters['limit']) ? (int) $parameters['limit'] : 5;
$type = 'ssi_recent' . (empty($parameters['type']) ? 'Posts' : 'Topics');
$display = empty($parameters['display']) ? 'compact' : 'full';

$items = $type($limit, null, $boards, 'array');

if (empty($items))
{
echo '
', $txt['error_sp_no_posts_found'];
return;
}
else
$items[count($items) - 1]['is_last'] = true;

$colorids = array();
foreach ($items as $item)
$colorids[] = $item['poster']['id'];

if (!empty($colorids) && sp_loadColors($colorids) !== false)
{
foreach ($items as $k => $p)
{
if (!empty($color_profile[$p['poster']['id']]['link']))
$items[$k]['poster']['link'] = $color_profile[$p['poster']['id']]['link'];
}
}


if ($display == 'compact')
{
 loadMemberData($item['poster']['id']);
 loadMemberContext($item['poster']['id']);
 
foreach ($items as $key => $item)
echo '

<div class="userav"><img src="' . $memberContext[$item['poster']['id']]['avatar']['href'] . '" alt="" height="30px" /></div>
<div style="padding: 0px 3px;"><span class="smalltext"><a href="', $item['href'], '">', $item['subject'], '</a></span><br /> <span class="smalltext"> ', $item['poster']['link'], $item['new'] ? '' : '<a href="' . $scripturl . '?topic=' . $item['topic'] . '.msg' . $item['new_from'] . ';topicseen#new" rel="nofollow"></a>',', ', $item['time'], '</span></div>', empty($item['is_last']) ? '<hr>' : '';
}
elseif ($display == 'full')
{
echo '
<table class="sp_fullwidth">';

foreach ($items as $item)
echo '
<tr>
<td class="sp_recent_icon sp_center">
', sp_embed_image(empty($parameters['type']) ? 'post' : 'topic'), '
</td>
<td class="sp_recent_subject">
<a href="', $item['href'], '">', $item['subject'], '</a>
', $item['new'] ? '' : '<a href="' . $scripturl . '?topic=' . $item['topic'] . '.msg' . $item['new_from'] . ';topicseen#new"><img src="' . $settings['images_url'] . '/' . $context['user']['language'] . '/new.gif" alt="' . $txt['new'] . '" border="0" /></a>', '<br />[', $item['board']['link'], ']
</td>
<td class="sp_recent_info sp_right">
', $item['poster']['link'], '<br />', $item['time'], '
</td>

</tr>';

echo '
</table>';
}
}


[AngelinaBelle]

Yes.  You have told it to only load the avatar for one id. Whatever the id is before you start the foreach loop.
You need to put this inside the foreach, so it will do it for each $item in $items.
Then, this should work.

[gregorie]

I do not know that I could get ready to rewrite the code ;(

[AngelinaBelle]

You'll need to add curly brackets, of course:

Code: [Select]

foreach ($items as $key => $item)
  {
loadMemberData($item['poster']['id']);
loadMemberContext($item['poster']['id']);

echo '

<div class="userav"><img src="' . $memberContext[$item['poster']['id']]['avatar']['href'] . '" alt="" height="30px" /></div>
<div style="padding: 0px 3px;"><span class="smalltext"><a href="', $item['href'], '">', $item['subject'], '</a></span><br /> <span class="smalltext"> ', $item['poster']['link'], $item['new'] ? '' : '<a href="' . $scripturl . '?topic=' . $item['topic'] . '.msg' . $item['new_from'] . ';topicseen#new" rel="nofollow"></a>',', ', $item['time'], '</span></div>', empty($item['is_last']) ? '<hr>' : '';
  }


[Blue]

gebekzor and gregorie, have you manage to make it work?

[AngelinaBelle]

And gebekzor, of course -- the original poster.  Is this working for you?

[Blue]

And gebekzor, of course -- the original poster.  Is this working for you?

Sorry AngelinaBelle, I didn't notice there were two. Did you two manage to make it work?

[gebekzor]

And gebekzor, of course -- the original poster.  Is this working for you?

Sorry AngelinaBelle, I didn't notice there were two. Did you two manage to make it work?

Thanks, works great



But I still have one small request, could someone help me with the function substr () to block show only first 30 letters. I tried myself but good to me it did not work out.

[Blue]

find:

Code: [Select]

<a href="', $item['href'], '">', $item['subject'] , '</a>
and replace with:

Code: [Select]

<a href="', $item['href'], '">', substr ($item['subject'],0,30) , '</a>

[gebekzor]

thanks for all lock please

[Blue]

I'm glad we could help you