Comments on: The irony of Model::deleteAll() http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/ baking cakes with CakePHP Tue, 31 Jan 2012 15:12:14 +0000 hourly 1 http://wordpress.org/?v=3.3.1 By: cakebaker http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/comment-page-2/#comment-102678 cakebaker Thu, 14 Aug 2008 16:34:27 +0000 http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/#comment-102678 @Fordnox: Yes, that's a good alternative (at least if it is ok that auto_increment fields get reset). @Fordnox: Yes, that’s a good alternative (at least if it is ok that auto_increment fields get reset).

]]> By: Fordnox http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/comment-page-2/#comment-102593 Fordnox Wed, 13 Aug 2008 18:25:13 +0000 http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/#comment-102593 Have not read all the comment and i do not know what is being discussed in there, but for me is is easier to write such function in app_model.php function truncate() { return $this->query("TRUNCATE ".$this->table); } Have not read all the comment and i do not know what is being discussed in there, but for me is is easier to write such function in app_model.php

function truncate() {
return $this->query(“TRUNCATE “.$this->table);
}

]]> By: cakebaker http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/comment-page-2/#comment-59608 cakebaker Thu, 24 Jan 2008 18:08:59 +0000 http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/#comment-59608 @Cameron: At least to me it is redundant if you have to write something like: <pre> $this->MyModel->deleteAll(DELETE_ALL_RECORDS); </pre> But as you see in the other comments, half of the people think it is good to have such a flag, and the other half think it is not necessary ;-) @Cameron: At least to me it is redundant if you have to write something like:

$this->MyModel->deleteAll(DELETE_ALL_RECORDS);

But as you see in the other comments, half of the people think it is good to have such a flag, and the other half think it is not necessary ;-)

]]> By: Cameron Junge http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/comment-page-2/#comment-58871 Cameron Junge Mon, 21 Jan 2008 23:19:16 +0000 http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/#comment-58871 Ok, coming in a little late... how about a constant? define('DELETE_ALL_RECORDS', true); // ideally a class constant, maybe Model::$DELETE_ALL_RECORDS? Just for use as a flag really. I like the idea of using "true" rather than just assuming blank conditions mean you want to do it (could be a bug for instance). A little conscious thought isn't always a bad thing, and TRUE is a lot more intuitive than "1 = 1". Ok, coming in a little late…

how about a constant?

define(‘DELETE_ALL_RECORDS’, true);
// ideally a class constant, maybe Model::$DELETE_ALL_RECORDS?

Just for use as a flag really. I like the idea of using “true” rather than just assuming blank conditions mean you want to do it (could be a bug for instance). A little conscious thought isn’t always a bad thing, and TRUE is a lot more intuitive than “1 = 1″.

]]> By: cakebaker http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/comment-page-1/#comment-49012 cakebaker Thu, 27 Dec 2007 16:02:58 +0000 http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/#comment-49012 @Tim: Yes, that would be better :) @Tim: Yes, that would be better :)

]]> By: Tim Daldini http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/comment-page-1/#comment-48307 Tim Daldini Tue, 25 Dec 2007 15:35:19 +0000 http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/#comment-48307 what about the slightly less redundant $this->MyModel->deleteAll($confirmdelete = true); Or something. I haven't tried it but I think it should pass a true boolean to the function. what about the slightly less redundant $this->MyModel->deleteAll($confirmdelete = true);

Or something. I haven’t tried it but I think it should pass a true boolean to the function.

]]> By: cakebaker http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/comment-page-1/#comment-47701 cakebaker Mon, 24 Dec 2007 06:20:45 +0000 http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/#comment-47701 @ifunk: Thanks for your comment! Yes, if I read your snippet it is obvious what it does. On the other hand it introduces some redundancies... @ifunk: Thanks for your comment!

Yes, if I read your snippet it is obvious what it does. On the other hand it introduces some redundancies…

]]> By: ifunk http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/comment-page-1/#comment-46999 ifunk Sat, 22 Dec 2007 17:09:11 +0000 http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/#comment-46999 The solution to this problem.... $delete_all_records = 1; $this->MyModel->deleteAll($delete_all_records); It's obvious when you read over the code what it does and I doubt you could do it inadvertently :) The solution to this problem….

$delete_all_records = 1;
$this->MyModel->deleteAll($delete_all_records);

It’s obvious when you read over the code what it does and I doubt you could do it inadvertently :)

]]> By: cakebaker http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/comment-page-1/#comment-44824 cakebaker Mon, 17 Dec 2007 06:47:58 +0000 http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/#comment-44824 @purepear, Proleter: Thanks for your comments! @Proleter: Yes, it is like the messages the OS shows you when you want to delete some files. They don't prevent that you delete the wrong files. @purepear, Proleter: Thanks for your comments!

@Proleter: Yes, it is like the messages the OS shows you when you want to delete some files. They don’t prevent that you delete the wrong files.

]]> By: Proleter http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/comment-page-1/#comment-44065 Proleter Sat, 15 Dec 2007 07:31:20 +0000 http://cakebaker.42dh.com/2007/12/06/the-irony-of-modeldeleteall/#comment-44065 dho, I think that such small glitches helps me a lot to understand the framework and go deeper into it. Thanks for that. The "Are you sure?" parameters are more like a personal preference than necessity in the core code. It doesn't hurts putting three letters and no it doesn't gives extra protection. You can force the programmer to pass a parameter: "array('1=Yes, I wan't to delete all the records. I'm sure I want to do that. Yes I know what I'm doing!' " and still to accedently delete all the records. Mariano Iglesias, will you stop being bitch and start giving some productive (or at least technical) comments? You sound like a priest who gets offended about his religion. dho, I think that such small glitches helps me a lot to understand the framework and go deeper into it. Thanks for that.

The “Are you sure?” parameters are more like a personal preference than necessity in the core code. It doesn’t hurts putting three letters and no it doesn’t gives extra protection. You can force the programmer to pass a parameter: “array(’1=Yes, I wan’t to delete all the records. I’m sure I want to do that. Yes I know what I’m doing!’ ” and still to accedently delete all the records.

Mariano Iglesias, will you stop being bitch and start giving some productive (or at least technical) comments? You sound like a priest who gets offended about his religion.

]]>