Re: Bug in cake beta 1.2 model
The problem is in /cake/libs/model/model.php line 364 and on. In the code it says: Line 1193 in Revision: 6398, Date: 2008-01-20 20:21:28 -0600 (Sun, 20 Jan 2008) --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups Cake PHP group. To post to this group, send email to cake-php@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/cake-php?hl=en -~--~~~~--~~--~--~---
Re: Bug in cake beta 1.2 model
I found (and fixed) a bug in the model.php file, and I can't report it in the trac system, so I thought I'd report it here in case someone sees it and can fix it. Whaaao!! Thanks a lot! I am a newbie to cake and since 2 days I was running around this strange behaviour when saving a HABTM relation. I thought it was necessary to write more code to handle this case, and I began to doubt of my cake understanding! So, thanks. --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups Cake PHP group. To post to this group, send email to cake-php@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/cake-php?hl=en -~--~~~~--~~--~--~---
Re: Bug in cake beta 1.2 model
@Chris I tried reporting to trac, but I got a 403 forbidden error when I tried to create a new ticket (at https://trac.cakephp.org/newticket). I realize now that I need to be registered (I just scanned the page and looked for the report a bug button and clicked), and I will report bugs through the proper channels in the future, because I totally get where you are coming from. :-) Nina On Jan 17, 10:01 pm, Chris Hartjes [EMAIL PROTECTED] wrote: On Jan 16, 2008 10:56 AM, Nina [EMAIL PROTECTED] wrote: Hi All I found (and fixed) a bug in the model.php file, and I can't report it in the trac system, so I thought I'd report it here in case someone sees it and can fix it. Nina, Is that can't or won't report the bug? It takes no more than 5 minutes to sign up for an account on trac in order to report a bug. I tell you this because I can say with great certainty that patches posted on the mailing list are extremely unlikely to be noticed. It's great that you might have found a bug, but PLEASE take the time to report it properly through trac. -- Chris Hartjes Internet Loudmouth Motto for 2008: Moving from herding elephants to handling snakes... @TheKeyBoard:http://www.littlehart.net/atthekeyboard --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups Cake PHP group. To post to this group, send email to cake-php@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/cake-php?hl=en -~--~~~~--~~--~--~---
Re: Bug in cake beta 1.2 model
On Jan 16, 2008 10:56 AM, Nina [EMAIL PROTECTED] wrote: Hi All I found (and fixed) a bug in the model.php file, and I can't report it in the trac system, so I thought I'd report it here in case someone sees it and can fix it. Nina, Is that can't or won't report the bug? It takes no more than 5 minutes to sign up for an account on trac in order to report a bug. I tell you this because I can say with great certainty that patches posted on the mailing list are extremely unlikely to be noticed. It's great that you might have found a bug, but PLEASE take the time to report it properly through trac. -- Chris Hartjes Internet Loudmouth Motto for 2008: Moving from herding elephants to handling snakes... @TheKeyBoard: http://www.littlehart.net/atthekeyboard --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups Cake PHP group. To post to this group, send email to cake-php@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/cake-php?hl=en -~--~~~~--~~--~--~---
Re: Bug in cake beta 1.2 model
Wow, I'm really glad you found this. I read this yesterday and thought well, I guess I'll leave it the way it is until I see problems Then today I was testing my code and found major problems with saving HABTM data. I thought it was because I was storing extra data in the join tables but then I remembered this post. Thanks for your work, I would have spent way to much time figuring this out (In fact I was just about to change me code to use $this-model-query() to save the data when I remembered this). Thanks, Dave On Jan 16, 8:56 am, Nina [EMAIL PROTECTED] wrote: Hi All I found (and fixed) a bug in the model.php file, and I can't report it in the trac system, so I thought I'd report it here in case someone sees it and can fix it. The problem is in /cake/libs/model/model.php line 364 and on. In the code it says: if (isset($v[$n])) { $v = $v[$n]; } $joined[$n] = $v; But it should say: if (isset($v[$n])) { $v = $v[$n]; $joined[$n] = $v; } This error is in the save function and it causes a save to save habtm associated fields wrong, putting empty values into the database. I found the bug by comparing to the pre_beta code. Sincerely Nina Jansen --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups Cake PHP group. To post to this group, send email to cake-php@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/cake-php?hl=en -~--~~~~--~~--~--~---
Bug in cake beta 1.2 model
Hi All I found (and fixed) a bug in the model.php file, and I can't report it in the trac system, so I thought I'd report it here in case someone sees it and can fix it. The problem is in /cake/libs/model/model.php line 364 and on. In the code it says: if (isset($v[$n])) { $v = $v[$n]; } $joined[$n] = $v; But it should say: if (isset($v[$n])) { $v = $v[$n]; $joined[$n] = $v; } This error is in the save function and it causes a save to save habtm associated fields wrong, putting empty values into the database. I found the bug by comparing to the pre_beta code. Sincerely Nina Jansen --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups Cake PHP group. To post to this group, send email to cake-php@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/cake-php?hl=en -~--~~~~--~~--~--~---