Adding dropdown to newthread

[Yaldaram]

set_error() should be $this->set_error()

[Tom K.]

It was that before, and that didn't fix it

[Yaldaram]

Try to use: newthread_do_newthread_end hook. I used this before to collect data and insert into the db.

[pavemen]

something like this?

$plugins->add_hook("datahandler_post_validate_post", "support_thread_validateforum");
$plugins->add_hook("datahandler_post_validate_thread", "support_thread_validateforum");

function support_thread_validateforum(&$handler)
    {
        global $mybb;
        $display = false;

       // print_r($handler);
        //die();
        //get usergroups from settings
        $forumArr = explode(",","1,2");
        $groupArr = explode(",", "1,2");
        
        $post = $handler->data;

        if (in_array($post['fid'],$forumArr))
        {
            //current forum is a support forum
            //is the user required to enter a forum?
            if (!in_array($mybb->user['usergroup'],$groupArr))
            {
                //this user isn't exempt
                $display = true;            
            }
            
        }
        
        if ($display == true)
        {
            if ($post['forumsubject'] == 0)
            {
             $handler->set_error("you did not pick a forum");
             return false; 
            }
        }
        /*echo $display;
        print_r($mybb);
        print_r($mybb->input);
        die();*/
    } 

---

Try to use: newthread_do_newthread_end hook. I used this before to collect data and insert into the db.

that is too late, by then the new content is already in the DB

[Tom K.]

Aha, that did it pavemen Thanks, can I ask, how did you know it was $handler?

[Yaldaram]

Try to use: newthread_do_newthread_end hook. I used this before to collect data and insert into the db.

that is too late, by then the new content is already in the DB

Correct!

[pavemen]

because the hook passes $this from insider the Parser class (extended by the Post handler) and thus all the contents of the handler are in the "$this" variable. So then its just understanding the $this object.

You could have left it using "$post" as it's just a variable name, but to get the post content you would need to then test $post->data['fid'] instead of $post['fid'] for example.

[Tom K.]

because the hook passes $this from insider the Parser class (extended by the Post handler) and thus all the contents of the handler are in the "$this" variable. So then its just understanding the $this object.

You could have left it using "$post" as it's just a variable name, but to get the post content you would need to then test $post->data['fid'] instead of $post['fid'] for example.

Actually, one thing. This doesnt work at all Because "forumsubject" isn't passed If I print_r($handler) it doesnt contain forumsubject. It only deisplays the error because I tested if it was set to 0 (which also holds true for null)



So. Back to the drawing board.

[pavemen]

do you want this to run on any post, or only on new threads? what about on edited posts?

[Tom K.]

Only new threads