php array into mysql

Question

$sql_where = ''; $exclude = '30,35,36,122,123,124,125';

if($exclude != '') 
{     
    $exclude_forums = explode(',', $exclude); 
    foreach ($exclude_forums as $id) 
    { 
        if ($id > 0) 
        { 
            $sql_where = ' AND forum_id <> ' . trim($id); 
        } 
    }
} 

$sql = 'SELECT topic_title, forum_id, topic_id, topic_type, topic_last_poster_id, topic_last_poster_name, topic_last_poster_colour, topic_last_post_time
   FROM ' . TOPICS_TABLE . '
   WHERE topic_status <> 2
      AND topic_approved = 1
      ' . $sql_where . '
   ORDER BY topic_time DESC';

The above code i use to exclude the id of forum to be displayed on sql queries.

Why doesn't it work and still display it?

Any solution

Silvio Donnini · Accepted Answer · 2010-04-10 07:15:22Z

4

You're missing the dot before =

$sql_where .= ' AND forum_id <> ' . trim($id);

answered Apr 10, 2010 at 7:15

Silvio Donnini

3,3432 gold badges30 silver badges29 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

zombat · Accepted Answer · 2010-04-10 07:13:50Z

1

You are overwriting your $sql_where variable on each loop iteration. You probably want to be using the concatenation operator (.) in the middle of your code instead:

if ($id > 0) 
{ 
    $sql_where .= ' AND forum_id <> ' . trim($id);
}

Note the change of the = to .=

answered Apr 10, 2010 at 7:13

zombat

94.5k25 gold badges158 silver badges165 bronze badges

Comments

Your Common Sense · Accepted Answer · 2010-04-10 07:22:54Z

0

if($exclude != '') $sql_where = 'AND forum_id NOT IN ($exclude);

instead of all this messy code :)

answered Apr 10, 2010 at 7:22

Your Common Sense

158k42 gold badges226 silver badges373 bronze badges

Collectives™ on Stack Overflow

php array into mysql

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related