CodeIgniter "Using $this when not in object context" in function

Question

how I can replace $this variable in this code? I know that it is incorrect to use $this in functions, but I don't know how I may replace that.

function get_dates($id) {
        $class = $this->db->query("SELECT * FROM schedule WHERE class_id='".$id."'");
        $class = $class->result_array();
        return $class;
    }

PS. I don't won't to work with mysql with standart php features and I strongly believe that it is possible to solve this problem with CI featues.

Why aren't you in an object context? Aren't you working with a model? If you use public function get_dates($id) { does it solve your error? The $this is actually correct but your context sounds wrong. By the way, your code is vulnerable to SQL injection. — sjagr
– sjagr, Commented Dec 9, 2014 at 16:46

Rooneyl · Accepted Answer · 2014-12-09 16:47:05Z

20

If you want to access the CI framework from inside a plain function, you need to get an instance of CI.

function get_dates($id) {
    $ci =& get_instance();

    $class = $ci->db->query("SELECT * FROM schedule WHERE class_id='".$id."'");
    $class = $class->result_array();
    return $class;
}

answered Dec 9, 2014 at 16:47

Rooneyl

7,9226 gold badges58 silver badges83 bronze badges

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

2 Comments

ltdev Over a year ago

@4clover: Note that it is consider best practice to keep the business logic inside models. Although the code Rooneyl wrote is valid, it's better to put queries inside models and methods.

Amani Over a year ago

what about CI 4 ?

saqibahmad · Accepted Answer · 2014-12-09 16:53:01Z

2

If You are in Helpers or library and want to access the controller object you have to user:

$ci =& get_instance();

Once you will get the controller object you can access any public method or member through $ci reference, you can get the model object as well by setting up a getter method to get the object and then simply reference that object for querying the db.

answered Dec 9, 2014 at 16:53

saqibahmad

9721 gold badge9 silver badges19 bronze badges

Collectives™ on Stack Overflow

CodeIgniter "Using $this when not in object context" in function

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related