Need Help - Using newpost_complete

[wholemama]

I've got this plugin I've been trying to write for newpost_complete...but I can't get it to work...anyone see where I've gone wrong?

//if post is original author
$originals = $vbulletin->db->query_read("
			SELECT postuserid
			FROM " . TABLE_PREFIX . "thread
			WHERE threadid = '" . $newpost['threadid'] . "'");

while($original = $vbulletin->db->fetch_array($originals)) {
if($original['userid']==$newpost['userid']){

//Find who is subscribed to this thread
$subscribed = $vbulletin->db->query_read("
			SELECT emailupdate,userid
			FROM " . TABLE_PREFIX . "subscribethread
			WHERE threadid = '" . $newpost['threadid'] . "'");

while($subscribe = $vbulletin->db->fetch_array($subscribed)) {
        if ($subscribe['emailupdate'] == 5) {
        //User is subscribed to thread & wants notification when original replies

                //Get users email address and name
                $user = $vbulletin->db->query_first("
                                SELECT email,username
                                FROM " . TABLE_PREFIX . "user
                                WHERE userid = '" . $subscribe['userid'] . "' LIMIT 1");

                // Get email phrases
                eval(fetch_email_phrases('foruminstantnotify', $forum['languageid']));
        		
        		//Send the email
                vbmail($user['email'], $subject, $message);
        }
}
}}

[wholemama]

Bumping out of desperation...

[Xenium]

Replace the where clauses in your SQL statments in the following way.

change:

PHP Code:

WHERE threadid = '" . $newpost['threadid'] . "'"); 


change to:

PHP Code:

WHERE threadid = $newpost['threadid']"); 


do it like that for all of them.

[wholemama]

Thanks...that seems to do something...

Except now, I always get a "this is a duplicate of a post you did in the last 5 minutes" message. It's like it double posts.