get_return_method bug in My_Model.php, version 0.93

SaltJunky · September 2011

I got the following error when using $this->examples_model->get_return_method();

Message: Missing argument 1 for MY_Model::get_return_method()...
Filename: core/MY_Model.php
Line Number: 1849

The code from application/core/MY_Model around line 1842:


/**
 * Set the default return type
 *
 * @access	public
 * @param	string	return type (object, array, query, auto)
 * @return	void
*/	
public function get_return_method($return_method)
{
	return $this->return_method;
}

It looks like there was a copy paste from set_return_method, including doc comments and an erroneous $return_method argument.

admin · September 2011

Thanks for the report. That's been fixed in the master branch:
https://github.com/daylightstudio/FUEL-CMS/commit/45fa4d4089e8df21fa0734e103ded6351f4defba

SaltJunky · September 2011

Wow - quick response. I'm 80% of the way into my first FUEL project - I really like the ability to quickly develop custom modules for my clients. Thanks for your great work!

Howdy, Stranger!

Categories

get_return_method bug in My_Model.php, version 0.93

Comments