| # |
Feb 1st 2018, 17:19 |
joopm |
but cookbook writes it does not matter |
| # |
Feb 1st 2018, 17:13 |
joopm |
extension is intalled rewriting i havent edited |
| # |
Feb 1st 2018, 17:12 |
joopm |
but log file is empty |
| # |
Feb 1st 2018, 17:12 |
joopm |
well i gave 777 permission for tmp and log |
| # |
Feb 1st 2018, 17:09 |
joopm |
thank you i try what you suggest |
| # |
Feb 1st 2018, 17:09 |
joopm |
hello hmic |
| # |
Feb 1st 2018, 17:08 |
hmic |
like neccessary extensions, rewriting, and so on |
| # |
Feb 1st 2018, 17:07 |
hmic |
make sure your server meete the prerequisites too |
| # |
Feb 1st 2018, 17:07 |
hmic |
enable debug in the app, if you created a new app template from cakephp/app that should be the case already |
| # |
Feb 1st 2018, 17:07 |
hmic |
joopm: check the serverlog |
| # |
Feb 1st 2018, 16:44 |
joopm |
but it shows only blank page the php version 7.1 |
| # |
Feb 1st 2018, 16:44 |
joopm |
i created a linux server,and created a cake project |
| # |
Feb 1st 2018, 16:43 |
joopm |
i faced with a challange |
| # |
Feb 1st 2018, 16:43 |
joopm |
hello all |
| # |
Feb 1st 2018, 16:37 |
 steinkel |
well done! |
| # |
Feb 1st 2018, 16:37 |
neon1024 |
I’ll take 98% |
| # |
Feb 1st 2018, 16:37 |
neon1024 |
https://coveralls.io/github/davidyell/CakePHP-Seo :) |
| # |
Feb 1st 2018, 16:37 |
steinkel |
indeed it's a feature |
| # |
Feb 1st 2018, 16:37 |
neon1024 |
We’ll call it an integration test and noone will notice! |
| # |
Feb 1st 2018, 16:36 |
steinkel |
hehe |
| # |
Feb 1st 2018, 16:36 |
neon1024 |
A small crime ;) |
| # |
Feb 1st 2018, 16:36 |
neon1024 |
Yeah, I think on balance it’s not too bad |
| # |
Feb 1st 2018, 16:36 |
steinkel |
ok, but you are using the router with this approach |
| # |
Feb 1st 2018, 16:35 |
neon1024 |
I went with this in the end which seemed to work okay |
| # |
Feb 1st 2018, 16:35 |
neon1024 |
Ahh ok |
| # |
Feb 1st 2018, 16:35 |
neon1024 |
https://github.com/davidyell/CakePHP-Seo/blob/master/tests/TestCase/View/Helper/SeoHelperTest.php#L260 |
| # |
Feb 1st 2018, 16:35 |
steinkel |
you would mock the protected method, not the Router:url one |
| # |
Feb 1st 2018, 16:34 |
neon1024 |
I’d prefer to use a mock, as I don’t need to test the router |
| # |
Feb 1st 2018, 16:34 |
neon1024 |
It was the mock that caused that error :slightly_smiling_face: |
| # |
Feb 1st 2018, 16:34 |
steinkel |
@neon1024 use a protected _getUrl() method in your helper and mock it |
| # |
Feb 1st 2018, 16:23 |
neon1024 |
Well that’s nice and simple |
| # |
Feb 1st 2018, 16:23 |
neon1024 |
Ahh, I can just add routes in the test |
| # |
Feb 1st 2018, 16:17 |
neon1024 |
Not sure how else my helper can create a url without the router |
| # |
Feb 1st 2018, 16:17 |
neon1024 |
Well, https://stackoverflow.com/posts/5961194/revisions |
| # |
Feb 1st 2018, 16:15 |
neon1024 |
`1) Seo\Tests\View\Helper\SeoHelperTest::testPageLink with data set #0 (‘Examples’, 1, ‘next’, false, ‘/examples?page=2’) Static method “url” cannot be invoked on mock object` |
| # |
Feb 1st 2018, 16:15 |
neon1024 |
Anyone tried mocking the `Router::url()` method? |
| # |
Feb 1st 2018, 16:03 |
hmic |
but, it's a neccessity for the PSR interface IMHO and might very well be defined there too, have a look! |
| # |
Feb 1st 2018, 16:02 |
hmic |
implement good/thorow tests around it |
| # |
Feb 1st 2018, 16:02 |
hmic |
if you use it, mention it and to extra checks on upgrades |
| # |
Feb 1st 2018, 16:02 |
hmic |
exactly like that i suppose. |
| # |
Feb 1st 2018, 15:44 |
mikesmoniker |
I noticed the docs use `Cake\Http\CallbackStream` in some controller/response examples, but that class is marked `@internal`. I’m used to that annotation meaning something along the lines of _this is part of our internal API and we don’t promise we won’t mess with it_. Anyone know how that plays in Cake? |