CodeIgniter Error on library function call

Question

Trying to fill options in dropdown by fetching data from database into a view. However I get the error:

Message: Undefined property: CI_Loader::$AdminDataHelper

Filename: forms/user.php

Fatal error: Call to a member function get_colleges() on a non-object in /ci/app/views/includes/forms/user.php on line 54

    <select id="college" name="college-selector" >   
         <?php 
               foreach ($this->AdminDataHelper->get_colleges() as $colleges)
               {
                 echo "<option value='".$colleges['id']."'>".$colleges['description']."</option>";
               }
        ?>                   
     </select>

My Library class:-

class AdminDataHelper {

public function __construct()
{
    $this->CI =& get_instance();
    $this->CI->load->model('admin_m');      
}

public function get_colleges(){
    return $this->CI->admin_m->get_colleges();
}

My model:-

class Admin_M extends CI_Model {

function __construct()
{
    // Call the Model constructor
    parent::__construct();
    $this->global_db = $this->load->database('global', TRUE); 
}

public function get_colleges()
{
    $this->global_db->select('id, description');
    $this->global_db->from('College');
    $result = $this->global_db->get();
    $data=$result->result_array();
    return $data;
}

In my autoload.php file I added the library.

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

I think you need to carefully read the docs. You cannot use a name like that. Only the first letter of the file name is capitalized and then use all lower-case when referencing it. Admindatahelper.php and then $this->admindatahelper->. Follow the examples and see the "naming conventions" section here: codeigniter.com/user_guide/general/… — Sparky
– Sparky, Commented Jan 19, 2017 at 23:45
I also don't think you can call a Library from within a View. You need to call it from your Controller and only send the data into your View. — Sparky
– Sparky, Commented Jan 19, 2017 at 23:48
You are right. The issue was with the naming convention. Thanks — jstandshigh
– jstandshigh, Commented Jan 19, 2017 at 23:59

Sparky · Accepted Answer · 2017-01-20 00:01:07Z

2

Carefully read the docs. You cannot use a name like that. Only the first letter of the file name is capitalized and then use all lower-case when referencing it.

Follow the examples and see the "naming conventions" section:

File names must be capitalized. For example: Myclass.php

Class declarations must be capitalized. For example: class Myclass

Class names and file names must match.

Admindatahelper.php and $this->admindatahelper

answered Jan 20, 2017 at 0:01

Sparky

98.8k26 gold badges202 silver badges293 bronze badges

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

Comments

talkhabi · Accepted Answer · 2017-01-19 23:49:27Z

1

You should to use Model in this case instead of using Library, however this will work:

$this->adminDataHelper->get_colleges();

the first character must start with lowercase letter.

answered Jan 19, 2017 at 23:49

talkhabi

2,7693 gold badges22 silver badges29 bronze badges

Collectives™ on Stack Overflow

CodeIgniter Error on library function call

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