| # |
Oct 22nd 2009, 08:09 |
Ceeram |
which i knind of suggested with saying: Don't understand why PHP functions are tested even |
| # |
Oct 22nd 2009, 08:08 |
Ceeram |
so testUsingMbStrrpos() could be removed comletely as well |
| # |
Oct 22nd 2009, 08:07 |
Ceeram |
so replacing all the mc_strrpos(0 by Multibyte::strrpos() would result in duplicate tests |
| # |
Oct 22nd 2009, 08:06 |
Ceeram |
they are the same, one just uses php functions other one is using MB class |
| # |
Oct 22nd 2009, 08:06 |
Ceeram |
markstory: look for function testUsingMbStrrpos() and function testMultibyteStrrpos() |
| # |
Oct 22nd 2009, 08:04 |
Gothfunc |
seems a little circular |
| # |
Oct 22nd 2009, 08:04 |
Gothfunc |
if you have users hasMany articles, but each user also has a favourite article, what's the correct way to implement that? article_id in the users table with belongsTo an article? |
| # |
Oct 22nd 2009, 08:03 |
Ceeram |
^^ thats why i think it was copy/paste |
| # |
Oct 22nd 2009, 08:03 |
Ceeram |
thats why i stated why are we testing php functions anyway? |
| # |
Oct 22nd 2009, 08:02 |
Ceeram |
no just look at the tests and the ones above that, they are using MB class and running same test |
| # |
Oct 22nd 2009, 08:01 |
markstory |
t have 5.1 though. |
| # |
Oct 22nd 2009, 08:01 |
markstory |
I don |
| # |
Oct 22nd 2009, 08:01 |
markstory |
yeah :) |
| # |
Oct 22nd 2009, 08:00 |
Ceeram |
wake up a bit more and then take another look at it |
| # |
Oct 22nd 2009, 07:56 |
markstory |
*possibly php's |
| # |
Oct 22nd 2009, 07:56 |
Ceeram |
those are also in there already |
| # |
Oct 22nd 2009, 07:56 |
Ceeram |
the other test without the $offset param in mb_strrpos(0 just pass |
| # |
Oct 22nd 2009, 07:56 |
markstory |
probably, so the test needs to call Multibyte's functions instead of PHP's |
| # |
Oct 22nd 2009, 07:55 |
Ceeram |
not tests when the mb_strrpos is called directly woth $offset param |
| # |
Oct 22nd 2009, 07:55 |
Ceeram |
the Multibyte tests all pass |
| # |
Oct 22nd 2009, 07:55 |
Ceeram |
i think the test was first written with php function and then copy pasted and changed to cake MultiByte class but forgot to take out tests with just mb_strrpos |
| # |
Oct 22nd 2009, 07:54 |
Ceeram |
look at the file, you'll find the cake class test methods abiove that |
| # |
Oct 22nd 2009, 07:54 |
markstory |
mb_strrpos() gets defined by multibyte if it doesn't exist. |
| # |
Oct 22nd 2009, 07:53 |
markstory |
so why remove the tests, why not change them to use Multibyte::strrpos? |
| # |
Oct 22nd 2009, 07:53 |
Ceeram |
$offset parameter was introduced in 5.2.0 |
| # |
Oct 22nd 2009, 07:53 |
Ceeram |
http://php.net/manual/en/function.mb-strrpos.php |
| # |
Oct 22nd 2009, 07:53 |
Ceeram |
not Multibyte::strrpos() |
| # |
Oct 22nd 2009, 07:53 |
Ceeram |
is mb_strrpos |
| # |
Oct 22nd 2009, 07:52 |
Ceeram |
no look at the code it tests the php function not cakes |
| # |
Oct 22nd 2009, 07:52 |
markstory |
is there for me. |
| # |
Oct 22nd 2009, 07:52 |
markstory |
Multibyte::strrpos() ? |
| # |
Oct 22nd 2009, 07:51 |
Ceeram |
look more carefully |
| # |
Oct 22nd 2009, 07:51 |
Ceeram |
the mehotds arent within Multibyte class |
| # |
Oct 22nd 2009, 07:51 |
Ceeram |
markstory: about this one: http://code.cakephp.org/tickets/view/194#c483 |
| # |
Oct 22nd 2009, 07:51 |
markstory |
I think that's because the paths get changed around. |
| # |
Oct 22nd 2009, 07:49 |
Ceeram |
also there is a warning on that this both before and after the fix: Warning (2): array_filter() [function.array-filter]: The first argument should be an array [CORE/cake/libs/configure.php, line 708] |
| # |
Oct 22nd 2009, 07:49 |
Ceeram |
well its explains why they fail with me and not when you run them if this is caused by OS |
| # |
Oct 22nd 2009, 07:48 |
Ceeram |
http://code.cakephp.org/source/cake/basics.php#673 |
| # |
Oct 22nd 2009, 07:47 |
Ceeram |
np |
| # |
Oct 22nd 2009, 07:47 |
markstory |
oh, sorry its still early. |
| # |
Oct 22nd 2009, 07:47 |
Ceeram |
it gave back 4 instead of 3 |