Comments on: Extended Model for CodeIgniter

By: Steve N.

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?

By: reneegate

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

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

reneegate

By: Code Updates (HTTP class, Extended CodeIgniter Model, Cross-domain AJAX transport) | Md Emran Hasan (phpfour)

Wed, 18 Feb 2009 21:27:53 +0000

[...] original version of the Extended Model for CodeIgniter has been serving many people well. Although most users loved [...]

By: haran

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

Great work. It saves us a lot of time.

By: Mariano Iglesias

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.

By: joshdavey

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.

By: Emran Hasan

Emran Hasan — Fri, 08 Aug 2008 13:34:23 +0000

@Ryan: I fully agree with Mariano and would misjudge the CI community in the same way, seeing your response.

What Mariano did was just to remind me something that I forgot, although in a bit rough way, but he had no bad intention in mind and I thanked him for that.

We’re all community enthusiastic, let’s not create divisions and point fingers.

By: Mariano Iglesias

Mariano Iglesias — Fri, 08 Aug 2008 13:13:55 +0000

Having been (and currently beeing) on several open source projects I know this is not unheard of. But people like Ryan on his comment above (showing that his brain consists only of derogative terms) certainly do not help the projects.

For the sake of CI, I hope you have more of the “other kind” of people. You know, people that KNOW something about something, and HELP the project back, for a change.

By: Ryan

Ryan — Sun, 03 Aug 2008 01:39:03 +0000

@Mariano Douchebag Boy

Go away you miserable creature. Take your nastiness to a cake forum.

By: stensi

stensi — Wed, 23 Jul 2008 02:22:49 +0000

Although after testing, that doesn’t work o.O

Leaving out the CI_ and putting it in application/libraries allows it to run perfectly fine.

I wonder why CodeIgniter doesn’t allow the CI_ override for the Model?