Mohammad Emran Hasan - Latest Comments in Extended Model for CodeIgniter

Re: Extended Model for CodeIgniter

Steve N. — Thu, 28 Oct 2010 18:50:12 -0000

Hi there,

Great library- thank you sharing. I'm having a bit of trouble though. I'm getting duplicate entry errors when I try to save a model object. It appears that it's trying to insert when it should update. Here's the code in one of my custom libraries that uses an extended model:
$this->CI->load->model('membership_model'); $membership = $this->CI->membership_model->find("merchantId = '$merchantId' AND customerId = '$customerId'"); if($membership) { // this membership already exists $membership['points']++; } else { // this is a new membership $this->CI->membership_model->create(); $membership['merchantId'] = $merchantId; $membership['customerId'] = $customerId; $membership['points'] = 1; $membership['created'] = date("Y-m-d H:i:s"); } $this->CI->membership_model->save($membership); ...

And this is the constructor for "membership_model.php":
function __construct() { // Call the Model constructor parent::__construct(); $this->loadTable('memberships'); $this->primaryKey = 'membershipId'; }

What am I doing wrong here?

Re: Extended Model for CodeIgniter

reneegate — Fri, 09 Jul 2010 01:25:09 -0000

Hello,
very nice work,has anybody developed multi tables model using mysql views?

reneegate

Re: Extended Model for CodeIgniter

u3nu3n — Tue, 10 Feb 2009 14:04:55 -0000

Thanks a lot~~
your extension save me lot of time! :)

Re: Extended Model for CodeIgniter

Abbas Mandal — Sun, 08 Feb 2009 02:04:02 -0000

Thanks Emran for this blog.Though Iam not a guru like some of you here,but i like the framework.
Iam a newbie to CI.Thanks again.

Abbas

Re: Extended Model for CodeIgniter

$1914876 — Fri, 30 Jan 2009 07:46:12 -0000

The only way I could get this to work in Code Igniter 1.7.0 was to replace the Model.php in the system/libraries directory as the above methods didn't work for me.

This could be because I'm using Matchbox 0.9.4 which hasn't been updated to include a lot of the new code in Loader.php yet...

But apart from that, this is such a time saver and a brilliant piece of kit to add to your Code Igniter arsenal.

Thanks heaps for creating it and looking forward to seeing the additional features you've got planned.

Re: Extended Model for CodeIgniter

Software_Miami — Sun, 18 Jan 2009 12:52:09 -0000

Emran and others,
Thanks for the innovative work Emran. The comments also proved extremely helpful.

@Michael Wales: Extremely helpful and encouraging as always (esp. in the CI forums).

@Jacob & @stensi: Thanks for the additions.

@ryan & @joshdavey & @abe: Good call all around. To come on someone's forum and sound so arrogant shows a lack of taste, respect and manners.

Re: Extended Model for CodeIgniter

Carlos — Sat, 10 Jan 2009 15:32:25 -0000

Why not using HMVC & Ignited Record & write a page module (like haughin' did) instead of hacking the core?
Saves even more time ^^

Re: Extended Model for CodeIgniter

Abe — Sat, 27 Dec 2008 04:26:27 -0000

Forgive the personality analysis, but your seemingly irate attitude towards the author necessitated commentary.

Do bear in mind that you're a guest on this website, not an unpaid asshole.

Your message is true, but your delivery is definitely flawed. I'd recommend trying to be a little more polite, or you'll keep getting comments from people like Ryan. While I agree with what you said, the way you came across was pretty overwhelmingly full of douchebaggery. I have not personally visited the CakePHP forums but if they're full of people such as yourself who display a genuine "inability" to be polite, I don't plan on visiting anytime soon.

It is rather apparent that you're a CakePHP fan, but don't turn it into some form of irrational zeal. Step one would be to resist all caps. Step two would be to assume that other people aren't as brilliant as you, and don't have the MIT license memorised. Step three would be to perhaps quietly suggest that the author credit the source that shows definite similarity to his code snippets, instead of full-force attacking him.

Thanks for the info, Emran. I was trying to extend Model.php, and I eventually just popped a customised copy of "Model.php" in my APPPATH/libraries folder and overrode the default Model file altogether.

Re: Extended Model for CodeIgniter

Abe — Sat, 27 Dec 2008 04:26:27 -0000

Forgive the personality analysis, but your seemingly irate attitude towards the author necessitated commentary.

Do bear in mind that you're a guest on this website, not an unpaid asshole.

Your message is true, but your delivery is definitely flawed. I'd recommend trying to be a little more polite, or you'll keep getting comments from people like Ryan. While I agree with what you said, the way you came across was pretty overwhelmingly full of douchebaggery. I have not personally visited the CakePHP forums but if they're full of people such as yourself who display a genuine "inability" to be polite, I don't plan on visiting anytime soon.

It is rather apparent that you're a CakePHP fan, but don't turn it into some form of irrational zeal. Step one would be to resist all caps. Step two would be to assume that other people aren't as brilliant as you, and don't have the MIT license memorised. Step three would be to perhaps quietly suggest that the author credit the source that shows definite similarity to his code snippets, instead of full-force attacking him.

Thanks for the info, Emran. I was trying to extend Model.php, and I eventually just popped a customised copy of "Model.php" in my APPPATH/libraries folder and overrode the default Model file altogether.

Re: Extended Model for CodeIgniter

Mohammad Emran Hasan — Mon, 27 Oct 2008 15:49:33 -0000

Thanks Jacob for pointing this out !!! I've been waiting for long to have CI support this. I will write a separate post with the update and a few additional features I planned long ago (probably post date: first week of Nov).

Everybody, please follow Jacob's two comments if you're using CodeIgniter 1.7.0.

Cheers!

Re: Extended Model for CodeIgniter

Mohammad Emran Hasan — Mon, 27 Oct 2008 14:49:33 -0000

Everybody, please follow Jacob's two comments if you're using CodeIgniter 1.7.0.

Cheers!

Re: Extended Model for CodeIgniter

Jacob Ball — Mon, 27 Oct 2008 06:11:18 -0000

Thanks for the reply, Emran! I hadn't seen your reply when I posted my fix below.

What I've done (in addition to the fix below) is rename it to MY_Model.php, placed it in system/application/libraries, updated the constructor to class MY_Model extends Model, and that's literally it. It's all working so far - I'll post back if I hit another snag :)

It may well be something they've fixed in these latest updates to CI, so you may wish to try it again, and if it works for, update your instructions accordingly.

Thanks for your contribution - it looks like it will be very useful!

Re: Extended Model for CodeIgniter

Jacob Ball — Mon, 27 Oct 2008 05:11:18 -0000

Thanks for the reply, Emran! I hadn't seen your reply when I posted my fix below.

What I've done (in addition to the fix below) is rename it to MY_Model.php, placed it in system/application/libraries, updated the constructor to class MY_Model extends Model, and that's literally it. It's all working so far - I'll post back if I hit another snag :)

It may well be something they've fixed in these latest updates to CI, so you may wish to try it again, and if it works for, update your instructions accordingly.

Thanks for your contribution - it looks like it will be very useful!

Re: Extended Model for CodeIgniter

Jacob Ball — Mon, 27 Oct 2008 00:30:36 -0000

Well, as always after posting a 'help me!' question, I've worked it out.

Replace Line 196 of MY_Model.php:

$this->fields = $this->db->field_names($table);

with

$this->fields = $this->db->list_fields($table);

as field_names() is deprecated, and clearly in 1.7.0 does not work!!

Hope this helps someone else :)

Re: Extended Model for CodeIgniter

Mohammad Emran Hasan — Mon, 27 Oct 2008 00:23:27 -0000

Michael's suggestion actually doesn't work, as I mentioned in my reply to him. You'll have to replace the actual Model.php in system/libraries to make it work. Hope this helps.

Re: Extended Model for CodeIgniter

Jacob Ball — Mon, 27 Oct 2008 00:02:19 -0000

I'd like to try using this, and I've tried it using Michael's suggestion, eg renaming to MY_Model etc, but I keep getting this error:

Fatal error: Call to undefined method CI_DB_mysql_driver::field_names() in C:homecodeignitersystemapplicationlibrariesMY_Model.php on line 196

and I just can't work out how to get around it... any help would be great!

Re: Extended Model for CodeIgniter

Jacob Ball — Sun, 26 Oct 2008 23:30:36 -0000

Well, as always after posting a 'help me!' question, I've worked it out.

Replace Line 196 of MY_Model.php:

$this->fields = $this->db->field_names($table);

with

$this->fields = $this->db->list_fields($table);

as field_names() is deprecated, and clearly in 1.7.0 does not work!!

Hope this helps someone else :)

Re: Extended Model for CodeIgniter

Mohammad Emran Hasan — Sun, 26 Oct 2008 23:23:27 -0000

Michael's suggestion actually doesn't work, as I mentioned in my reply to him. You'll have to replace the actual Model.php in system/libraries to make it work. Hope this helps.

Re: Extended Model for CodeIgniter

Jacob Ball — Sun, 26 Oct 2008 23:02:19 -0000

I'd like to try using this, and I've tried it using Michael's suggestion, eg renaming to MY_Model etc, but I keep getting this error:

Fatal error: Call to undefined method CI_DB_mysql_driver::field_names() in C:\home\codeigniter\system\application\libraries\MY_Model.php on line 196

and I just can't work out how to get around it... any help would be great!

Re: Extended Model for CodeIgniter

haran — Wed, 20 Aug 2008 15:20:10 -0000

Great work. It saves us a lot of time.

Re: Extended Model for CodeIgniter

haran — Wed, 20 Aug 2008 11:20:10 -0000

Great work. It saves us a lot of time.

Re: Extended Model for CodeIgniter

haran — Wed, 20 Aug 2008 11:20:10 -0000

Great work. It saves us a lot of time.

Re: Extended Model for CodeIgniter

Mariano Iglesias — Mon, 18 Aug 2008 19:40:02 -0000

@joshdavey: why do a lot of people INSIST ON NOT READING before posting? Check the previous messages, this was SOLVED long ago.

Re: Extended Model for CodeIgniter

joshdavey — Mon, 18 Aug 2008 19:00:29 -0000

@Mariano Iglesias

You really want him to re-name "query(), "getNumRows()", "$this->data" and other method/property names? These are standard method names used by pretty much every database abstraction layer in the history of programming.

Saying phpfour isn't respecting the MIT license is a bit harsh. He's just looking at other libraries for inspiration and building his own. Nothing wrong with that.

Re: Extended Model for CodeIgniter

Mariano Iglesias — Mon, 18 Aug 2008 15:40:02 -0000

@joshdavey: why do a lot of people INSIST ON NOT READING before posting? Check the previous messages, this was SOLVED long ago.