| # |
Jul 23rd 2009, 08:54 |
alkemann |
ok |
| # |
Jul 23rd 2009, 08:53 |
jperras |
try and remember to keep the schema.php file up to date. it's not very essential now, but it will become more important as the codebase gets closer to stable |
| # |
Jul 23rd 2009, 08:53 |
alkemann |
ok good |
| # |
Jul 23rd 2009, 08:52 |
jperras |
remotes/origin/2.0 |
| # |
Jul 23rd 2009, 08:50 |
alkemann |
are u on the 2.0.x.x branch? |
| # |
Jul 23rd 2009, 08:49 |
jperras |
yeah, the schema was very old |
| # |
Jul 23rd 2009, 08:49 |
jperras |
yowza |
| # |
Jul 23rd 2009, 08:48 |
jperras |
cool |
| # |
Jul 23rd 2009, 08:47 |
alkemann |
ok. pushed fixed |
| # |
Jul 23rd 2009, 08:44 |
jperras |
and from the 2.0 branch, the schema.php file is incorrect |
| # |
Jul 23rd 2009, 08:44 |
jperras |
no, but I was setting up the application |
| # |
Jul 23rd 2009, 08:42 |
alkemann |
but u dont need schema to run the tests anyway :) |
| # |
Jul 23rd 2009, 08:42 |
alkemann |
but yea they need to be fixed after baking because of unconventional pk |
| # |
Jul 23rd 2009, 08:41 |
alkemann |
they arent even in the latest schema i have |
| # |
Jul 23rd 2009, 08:40 |
jperras |
article_pages_drafts and article_pages_revs |
| # |
Jul 23rd 2009, 08:37 |
alkemann |
oh im looking at fixtures |
| # |
Jul 23rd 2009, 08:37 |
alkemann |
jperras: which table? |
| # |
Jul 23rd 2009, 08:36 |
alkemann |
hmm.. ill fix that |
| # |
Jul 23rd 2009, 08:24 |
jperras |
there are primary keys defined twice in some tables |
| # |
Jul 23rd 2009, 08:24 |
jperras |
alkemann: also, there's an error in the schema |
| # |
Jul 23rd 2009, 08:24 |
jperras |
alkemann: just getting it setup now; got distracted for a bit |
| # |
Jul 23rd 2009, 08:18 |
alkemann |
jperras: u see it? |
| # |
Jul 23rd 2009, 07:17 |
jperras |
pulling repo now |
| # |
Jul 23rd 2009, 07:14 |
alkemann |
trying with and without containable. set recursive -1 or not. cant get it to do the same in the counter test |
| # |
Jul 23rd 2009, 07:12 |
jperras |
k |
| # |
Jul 23rd 2009, 07:12 |
alkemann |
typo. refresh bin |
| # |
Jul 23rd 2009, 07:12 |
jperras |
alkemann: public $alies = 'Article'; ? |
| # |
Jul 23rd 2009, 07:09 |
jperras |
yeah, gimme a minute |
| # |
Jul 23rd 2009, 07:08 |
alkemann |
jperras: can u pull the bakery 2.0 branch and run the test urself? |
| # |
Jul 23rd 2009, 07:07 |
alkemann |
jperras: http://bin.cakephp.org/view/1334665431 - this one doesnt do it |
| # |
Jul 23rd 2009, 07:07 |
jperras |
alkemann: oh? |
| # |
Jul 23rd 2009, 07:06 |
jperras |
we might be able to rewrite some portions of Model and DboSource to get the query a bit more optimized |
| # |
Jul 23rd 2009, 07:06 |
alkemann |
jperras: im not able to recreate it in a lesser test |
| # |
Jul 23rd 2009, 07:06 |
jperras |
alkemann: post an enhancement ticket |
| # |
Jul 23rd 2009, 07:05 |
Phally |
alkemann: if it is query #85 of article.test.php, then i have |
| # |
Jul 23rd 2009, 07:04 |
alkemann |
update is on the parent. scope is on the counted model |
| # |
Jul 23rd 2009, 07:04 |
jperras |
that's a side-effect of updateAll, iirc |
| # |
Jul 23rd 2009, 07:03 |
alkemann |
jperras: but the scope would be on the find('count' it still does not make sense on the update |
| # |
Jul 23rd 2009, 07:03 |
jperras |
if your counterscope is not set, then there shouldn't be any joins produced |
| # |
Jul 23rd 2009, 07:03 |
alkemann |
i am using scope yes |
| # |
Jul 23rd 2009, 07:03 |
jperras |
since you can scope on the associated model |