codeigniter array variable

Question

Here is my code so far, it all works except when I try to make 'company' = $company there's something im missing, and id love to know what

if($query) // if the user's credentials validated///
    {
        $this->db->where('username');
        $this->db->select('company');
        $company = $this->db->get('user');

        $data = array(
            'username' => $this->input->post('username'),
            'company' => $company
            'is_logged_in' => true
        );
        $this->session->set_userdata($data);
        redirect('site/members_area');
    }
    else
    {
        $this->index();
    }

Phil Sturgeon · Accepted Answer · 2010-01-22 11:06:32Z

1

Both the other answers fix one of the two errors in your code, this is an addition.

if($query) {
  $username = $this->input->post('username');

  $this->db->select('company')->where('username', $username);
  $result = $this->db->get('user')->row_array();

  $data = array(
      'username' => $username,
      'company' => $row['company'],
      'is_logged_in' => true
  );
  $this->session->set_userdata($data);
  redirect('site/members_area');
} else {
  $this->index();
}

See what I did there? You don't need to use result_array() then grab $query_result[0] as row_array() does that for you. And with a little method chaining thrown in for good measure you can clean up your syntax.

answered Jan 22, 2010 at 11:06

Phil Sturgeon

30.8k12 gold badges82 silver badges118 bronze badges

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

Comments

Max Shawabkeh · Accepted Answer · 2010-01-22 01:50:06Z

1

There is a missing comma after "$company".

EDIT: Based on the discussion in the comments, I've rewritten your code to (hopefully) get the value of the company:

if($query) {
  $username = $this->input->post('username');

  $this->db->where('username', $username);
  $this->db->select('company');
  $result = $this->db->get('user');

  // Here we assume that the query succeeded.
  // You should probably double-check.
  $company = $result->result_array[0]['company'];

  $data = array(
      'username' => $username,
      'company' => $company,
      'is_logged_in' => true
  );
  $this->session->set_userdata($data);
  redirect('site/members_area');
} else {
  $this->index();
}

edited Jan 22, 2010 at 1:50

answered Jan 21, 2010 at 19:55

Max Shawabkeh

38.7k10 gold badges85 silver badges92 bronze badges

9 Comments

JonYork Over a year ago

if I follow what you are saying, I get the following error <code>A PHP Error was encountered Severity: 4096 Message: Object of class CI_DB_mysql_result could not be converted to string Filename: controllers/login.php Line Number: 24</code>

Max Shawabkeh Over a year ago

Well, that's another story. What line is 24 (that snippet is less than 24 lines)?

JonYork Over a year ago

line 24 is 'company' => $company the whole thing is 79 lines long. If I remove 'company' => $company, completely, there are no errors and run like it should (minus the company session data)

Max Shawabkeh Over a year ago

Looks to me like it's more likely the problem is happening on the "$this->session->set_userdata($data);" data since that may try (and fail) to force a cast. Constructing an array shouldn't try to do anything other than store the data.

JonYork Over a year ago

when I run it with the line 'company' => 'company', it runs fine. its only when i go to make that a variable that it messes up

|

Adi · Accepted Answer · 2010-01-22 09:38:35Z

1

result_array is a function, not a variable. Try

if($query) {
  $username = $this->input->post('username');

  $this->db->where('username', $username);
  $this->db->select('company');
  $query_result = $this->db->get('user');

  // Here we assume that the query succeeded.
  // You should probably double-check.
  $result= $query_result->result_array();

  $data = array(
      'username' => $username,
      'company' => $result[0]['company'],
      'is_logged_in' => true
  );
  $this->session->set_userdata($data);
  redirect('site/members_area');
} else {
  $this->index();
}

answered Jan 22, 2010 at 9:38

Adi

5561 gold badge3 silver badges8 bronze badges

Collectives™ on Stack Overflow

codeigniter array variable

3 Answers 3

Comments

9 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

9 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related