Hi Roland,

What you're suggesting is a pretty major change to data returned by
Cake - major in that there would be a lot of user-end code that needs
changing if this were to happen to the core. Though I think it's
awesome that you'd be willing to write a patch and submit it, maybe
you should first submit a ticket to Lighthouse ( 
http://cakephp.lighthouseapp.com
) for some feedback from the core development team. To be honest, I
found the structure of returned data a bit odd when I was starting out
with Cake as well, so you're not the only one.

- Jamie

On May 7, 5:46 am, Roland Bock <feedback2...@eudoxos.de> wrote:
> Yes, and validation, too, I guess...
>
> I hope to find the time :-)
>
> Thanks for your help!
>
> Regards,
>
> Roland
>
>
>
>
>
>
>
> > You could probably write a Behaviour, but that will take additional
> > processing, which slows down the application.
>
> > On May 7, 2:48 pm, Roland Bock <feedback2...@eudoxos.de> wrote:
> >> John Andersen wrote:
> >>> I agree that the structure could be better, but as it is, is the way
> >>> we have to work with it, and I consider it sufficient for my usage.
> >>> Enjoy,
> >>>    John
> >> OK, two questions:
>
> >> a) Do you think I could tweak current Models with a Behavior in order to
> >> get the structure I prefer?
>
> >> b) Do you think it would make sense to send a patch that would change
> >> the data structure (maybe for cake-1.4)?
>
> >> Thanks and regards,
>
> >> Roland
>
> >>> On May 7, 12:30 pm, Roland Bock <feedback2...@eudoxos.de> wrote:
> >>>> Hi,
> >>>> John Andersen wrote:
> >>>>> I do think that if you don't like something in CakePHP, please write
> >>>>> your own solution to it, then offer it to the CakePHP developers.
> >>>> Sure. I'd like to understand first, though :-)
> >>>> In CakePHP, Parent -> Node -> Child leads to different ways to handle a
> >>>> single Node:
> >>>> $node['id'];    // in case Node has been retrieved by Parent
> >>>> $node[Child'];
> >>>> but
> >>>> $currentRecord['Node']['id']; // Node has been retrieved by Node
> >>>> $currentRecord['Child'];
> >>>> This seems to add complexity.
> >>>> Now, here is what I would aim for (including the Ghost you added)
> >>>> Array
> >>>> (
> >>>>     [Parent] => Array
> >>>>         (
> >>>>             [0] => Array
> >>>>                 (
> >>>>                     [id] => 1
> >>>>                     [Node] => Array
> >>>>                         (
> >>>>                             [0] => Array
> >>>>                                 (
> >>>>                                     [id] => 17
> >>>>                                     [Child] => Array
> >>>>                                         (
> >>>>                                         )
> >>>>                                 )
> >>>>                             [1] => Array
> >>>>                                 (
> >>>>                                     [id] => 19
> >>>>                                     [Child] => Array
> >>>>                                         (
> >>>>                                         )
> >>>>                                 )
> >>>>                         )
> >>>>                     [Ghost] => Array
> >>>>                         (
> >>>>                             [0] => Array
> >>>>                                 (
> >>>>                                     [id] => 2
> >>>>                                 )
> >>>>                             [1] => Array
> >>>>                                 (
> >>>>                                     [id] => 3
> >>>>                                 )
> >>>>                         )
> >>>>                 )
> >>>>         )
> >>>> )
> >>>> This way I can say
> >>>> $resultSet             <- This is an array of Parents
> >>>> $currentRecord = $resultSet['Parent']['0']
> >>>> $currentRecord         <- This is a Parent
> >>>> $currentRecord['id']
> >>>> $currentRecord['Node']
> >>>> $currentRecord['Node'][0]['id']
> >>>> $currentRecord['Node'][0]['Child']
> >>>> $currentRecord['Ghost]
> >>>> In the original CakePHP structure,
> >>>> $resultSet  <- This is an array of arrays(Parent,Nodes, Ghosts)
> >>>> $currentRecord = $resultSet['0']
> >>>> $currentRecord        <- This is a Parent/Nodes/Ghosts array
> >>>> $currentRecord['Parent']
> >>>> $currentRecord['Parent']['id']
> >>>> $currentRecord['Node']
> >>>> $currentRecord['Node'][0]['Child]
> >>>> $currentRecord['Ghost']
> >>>> So basically, my way, you need to write *less* to access members of
> >>>> 'Parent'. The rest is pretty similar. But, (huge benefit, I think), you
> >>>> can handle a Node always in the same way, regardless of whether you get
> >>>> it included in a Parent or use the Node Model directly:
> >>>> My way:
> >>>> $node['id'];  // no matter what
> >>>> $node['Child'];
> >>>> Remeber the CakePHP way:
> >>>> $node['id'];    // in case Node has been retrieved by Parent
> >>>> $node[Child'];
> >>>> but
> >>>> $currentRecord['Node']['id']; // Node has been retrieved by Node
> >>>> $currentRecord['Child'];
> >>>>> Does the above helps you to clarify why the original structure is
> >>>>> good?
> >>>> Not yet, I am afraid. Maybe I was able to intrigue you with my proposal? 
> >>>> :-)
> >>>> Regards,
> >>>> Roland
> >>> [snip]
> >>> Check out the new CakePHP Questions sitehttp://cakeqs.organdhelp others 
> >>> with their CakePHP related questions.
> >>> You received this message because you are subscribed to the Google Groups 
> >>> "CakePHP" group.
> >>> To post to this group, send email to cake-php@googlegroups.com
> >>> To unsubscribe from this group, send email to
> >>> cake-php+unsubscr...@googlegroups.com For more options, visit this group 
> >>> athttp://groups.google.com/group/cake-php?hl=en
> >> Check out the new CakePHP Questions sitehttp://cakeqs.organdhelp others 
> >> with their CakePHP related questions.
>
> >> You received this message because you are subscribed to the Google Groups 
> >> "CakePHP" group.
> >> To post to this group, send email to cake-php@googlegroups.com
> >> To unsubscribe from this group, send email to
> >> cake-php+unsubscr...@googlegroups.com For more options, visit this group 
> >> athttp://groups.google.com/group/cake-php?hl=en
>
> > Check out the new CakePHP Questions sitehttp://cakeqs.organd help others 
> > with their CakePHP related questions.
>
> > You received this message because you are subscribed to the Google Groups 
> > "CakePHP" group.
> > To post to this group, send email to cake-php@googlegroups.com
> > To unsubscribe from this group, send email to
> > cake-php+unsubscr...@googlegroups.com For more options, visit this group 
> > athttp://groups.google.com/group/cake-php?hl=en
>
> Check out the new CakePHP Questions sitehttp://cakeqs.organd help others with 
> their CakePHP related questions.
>
> You received this message because you are subscribed to the Google Groups 
> "CakePHP" group.
> To post to this group, send email to cake-php@googlegroups.com
> To unsubscribe from this group, send email to
> cake-php+unsubscr...@googlegroups.com For more options, visit this group 
> athttp://groups.google.com/group/cake-php?hl=en

Check out the new CakePHP Questions site http://cakeqs.org and help others with 
their CakePHP related questions.

You received this message because you are subscribed to the Google Groups 
"CakePHP" group.
To post to this group, send email to cake-php@googlegroups.com
To unsubscribe from this group, send email to
cake-php+unsubscr...@googlegroups.com For more options, visit this group at 
http://groups.google.com/group/cake-php?hl=en

Reply via email to