Re: Bug in cake beta 1.2 model

2008-01-30 Thread jakecake (newbie)

 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

2008-01-30 Thread jakecake (newbie)

 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

2008-01-20 Thread Nina

@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

2008-01-17 Thread Chris Hartjes

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

2008-01-17 Thread [EMAIL PROTECTED]

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

2008-01-16 Thread Nina

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
-~--~~~~--~~--~--~---