What is wrong with this CodeIgniter code?

Question

I am trying to port an application's few functions from a CodeIgniter application to another existing CodeIgniter application. Both applications on themselves are working very well but when I added this thing it gives the following error:

Fatal error: Call to a member function order_by() on null in …\application\core\MY_Model.php on line 7

In this question I have removed parts unrelated to the error to simplify code.

//MY_Model.php model file

<?php
class MY_Model extends CI_Model {
    protected $_order_by = '';

    public function get(){
        $this->db->order_by($this->_order_by);
    }
}

//article_m.php model file

<?php
class Article_m extends MY_Model
{
    protected $_order_by = 'pubdate desc, id desc';
}

//frontend.php controller file

<?php
class Frontend extends MY_Controller
{
    function __construct()
    {
    $this->load->model('article_m');  
    }
    function index()
    {
    $this->article_m->get();
    }
}

Please help. Thank you!

CodeGodie · Accepted Answer · 2015-03-07 13:45:33Z

2

whenever calling any $this->db ... you have to make sure to load your database library. Check in application\config\autoload.php for the following:

$autoload['libraries'] = array('database');

answered Mar 7, 2015 at 13:45

CodeGodie

12.2k6 gold badges40 silver badges66 bronze badges

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

3 Comments

Rod Over a year ago

that worked thanks! is there anyway to load database from controller itself?

CodeGodie Over a year ago

you should not load any database information in the controller as this is not good MVC practice. All database interaction should be done in your Model. If you do not want to autoload the the database library, you can run $this->load->library('database') in your model method prior creating your queries.

Rod Over a year ago

Sorry to be not clear what I meant, I actually wanted to ask this:- Is there anyway to load the database library from a controller? I did not mean database info from controller but load/activate database library. I seem to have got the answer by the way. I got this line from an app using this to load database lib from controller :- $this->load->database();

Collectives™ on Stack Overflow

What is wrong with this CodeIgniter code?

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related