[how to] last threads with thread text ??

[Almotmaiz.Net]

heloo every body ..

i wanna to build a new hack .. get me a last threads with thread text ..

i try do this .. (global_start)
$Alzaabi = intval(500);
$AlzaabiGetNewsAlzaabiIDs = $db->query_read("
SELECT threadid FROM thread AS thread
WHERE visible = 1 AND open != 10
ORDER BY dateline
DESC LIMIT 0, 3
");
while ($AlzaabiIDs = $db->fetch_array($AlzaabiGetNewsAlzaabiIDs))
{
$AlzaabiNewstAlzaabiIDs[] = $AlzaabiIDs['threadid'];
$AlzaabiNewspAlzaabiIDs[] = $AlzaabiIDs['firstpostid'];
}
$AlzaabiGetNews = $db->query_read("
SELECT thread.threadid, post.title, thread.replycount, postusername, postuserid, thread.dateline AS postdateline, thread.lastposter, thread.lastpost, IF(views<=thread.replycount, thread.replycount+1, views) AS views, thread.forumid, post.postid, pagetext, allowsmilie
FROM thread AS thread
LEFT JOIN post AS post ON (post.postid = thread.firstpostid)
LEFT JOIN postparsed AS postparsed ON (postparsed.postid = post.postid AND postparsed.styleid = " . intval(STYLEID) . " AND postparsed.languageid = " . intval(LANGUAGEID) . ")
WHERE thread.threadid IN(" . implode(',', $AlzaabiNewstAlzaabiIDs) . ")
GROUP BY post.postid
ORDER BY thread.dateline
DESC LIMIT 0, 3
");
while ($AlzaabiNews = $db->fetch_array($AlzaabiGetNews))
{
$AlzaabiNews['message'] = $bbcode_parser->do_parse($AlzaabiNews['pagetext']);
if(strlen($AlzaabiNews['message']) > $Alzaabi){
$AlzaabiNews['message'] = substr($AlzaabiNews['message'], 0, strrpos(substr($AlzaabiNews['message'], 0, $Alzaabi), ' ')) . '...';
}else{
$AlzaabiNews['message'] = $AlzaabiNews['message'];
}
eval('$Alzaabi_News = "' . fetch_template('Alzaabi_News') . '";');
}

templets ..

<table cellpadding="$stylevar[cellpadding]" cellspacing="$stylevar[cellspacing]" border="0" width="100%" align="center">
<tr>
<td>
<table border="0" width="100%" cellspacing="0" cellpadding="0">
<tr>
<td><img border="0" src="images/KalbaEyes_tcat_02.gif" width="88" height="25"></td>
<td style="background-image: url('images/KalbaEyes_tcat_01.gif'); background-repeat: no-repeat; background-position: right center" width="100%">
<a href="$vboptions[bburl]/showthread.php?t=$AlzaabiNews[threadid]">$AlzaabiNews[title]</a>
</td>
</tr>
</table>
</td>
</tr>
<tr>
<td>
<table class="tborder" cellpadding="$stylevar[cellpadding]" cellspacing="$stylevar[cellspacing]" border="0" width="100%" align="center">
<tr>
<td class="alt1">
<table border="0" width="100%" cellspacing="0" cellpadding="0">
<tr>
<td align="center">$AlzaabiNews[message]</td>
</tr>
<tr>
<td align="left">
<a href="$vboptions[bburl]/showthread.php?t=$AlzaabiNews[threadid]">Read more ..</a></td>
</tr>
</table>
</td>
</tr>
</table>
</td>
</tr>
</table>
<table border="0" width="100%" cellspacing="0" cellpadding="0">
<tr>
<td><img border="0" src="images/shadow.gif" width="150" height="15"></td>
<td width="100%">&nbsp;</td>
<td><img border="0" src="images/rshadow.gif" width="150" height="15"></td>
</tr>
</table>
<br />


wut's wrong with my Code ??:confused:

up !!!! ... where's Coders ???

:confused: :confused: :confused:

:confused: :confused:

:confused:

[Paul M]

If you want help then you need to say what the problem is - what should happen, what actually happens.

[WetWired]

As Paul M indicated, it's hard to tell you why it's doing an unspecified undesired behavior. Just looking at the code alone, though:
You're making a list of threadids and firstpostids, except that you didn't select firstpostids, so you're making a list of threaids and NULLs, which is probably giving you trouble in the second query. Remove the line adding firstpostids to the list; you're using the list as threadids anyway.
Your second query GROUPs BY a post field, but you're joining with the post table on the primary key, so there will never be duplicates; the GROUP BY clause is unneeded and may be causing problems
You're joining with the parsed post table, but not using it. This is a waste of mysql time.
You limit your results in both queries, which is sure to cause problems when you later increase the first limit and can't figure out why it didn't work.
Your current design cuts off a post in the middle of any tags that may be open, leaving them open for the rest of the page
When you do the template, you set the string equal to it, instead of concatenating, which means only the last result will be in your string when you're done