Create a session array from Mysql query

Question

I'm not very good yet with sessions or arrays.

Here is what I have so far but I am getting an error

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource.

Here is my code:

$checkgroupadmin = $this->db->query("SELECT group_id FROM groupadmin 
  WHERE contact_id = $contactid");
while ($adminrow = mysql_fetch_array($checkgroupadmin, MYSQL_ASSOC)) {
  $_SESSION[group_admin] = array('group'=>$adminrow['group_id']);
}

Any help would be appreciated. :)

are you using a php framework like codeigniter? If so you should be using the result generation functions. — Will Sampson
– Will Sampson, Commented Oct 29, 2012 at 18:09
Add var_dump($checkgroupadmin) after the query and show the result — Vadim Ashikhman
– Vadim Ashikhman, Commented Oct 29, 2012 at 18:09
The query has failed. Always check the result if (!$checkgroupadmin) echo mysql_error(); — Michael Berkowski
– Michael Berkowski, Commented Oct 29, 2012 at 18:09
Are you using any framework ? Why $this->db->query to query and mysql_fetch_array to fetch ? — air4x
– air4x, Commented Oct 29, 2012 at 18:10

Anthony Hatzopoulos · Accepted Answer · 2012-10-29 18:15:43Z

1

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource.

You are getting your error because $checkgroupadmin is not an actual query resource that mysql_fetch_array() requires.

$checkgroupadmin = $this->db->query("SELECT group_id FROM groupadmin WHERE contact_id = $contactid");`

In other words the above object oriented call is not returning a mysql resource query.

Why are you mixing object oriented style with procedural style anyways. Secondly the above code $this>db->query should be happening inside a Class' method

answered Oct 29, 2012 at 18:15

Anthony Hatzopoulos

10.6k2 gold badges42 silver badges57 bronze badges

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

Comments

doublesharp · Accepted Answer · 2012-10-29 18:26:55Z

mysql_fetch_array() returns an associative array of the current results, but in your code you would just be overwriting the same value in the $_SESSION for each result. You likely need to create your own array of resulting IDs and then assign that to a $_SESSION variable.

// query database
$checkgroupadmin = $this->db->query("SELECT group_id FROM groupadmin WHERE contact_id = $contactid");
// array to hold each group_id
$result_array = array();
// varify that $checkgroupadmin returned results 
if ($checkgroupadmin){
    // get results into associative array
    while ($adminrow = mysql_fetch_array($checkgroupadmin, MYSQL_ASSOC)) {
        // put the group id into your results array
        $result_array[] = $adminrow['group_id'];
    }
}
// set results array to the 'group_admin' key in $_SESSION
$_SESSION['group_admin'] = $result_array;

Please note that the mysql_* extensions have been deprecated and you should be using mysqli or PDO instead.

Collectives™ on Stack Overflow

Create a session array from Mysql query

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related