| # |
Dec 19th 2017, 11:50 |
 neon1024 |
Let me finish this and I’ll look into the plugin to see if I can add a test case |
| # |
Dec 19th 2017, 11:50 |
joop |
working wit anonymus function is a bit harder than the previus task demands |
| # |
Dec 19th 2017, 11:50 |
neon1024 |
Perhaps I’ve got a bug in my code, but adding the annotation makes PHP Stan pass |
| # |
Dec 19th 2017, 11:49 |
neon1024 |
File uploaded https://cakesf.slack.com/files/U1BT622HW/F8GP4EPS6/php_stan.txt / https://slack-files.com/T053DPNCM-F8GP4EPS6-781ed3a9ef |
| # |
Dec 19th 2017, 11:48 |
neon1024 |
Sorry that’s not what I meant. I meant PHP Stan gives me an error |
| # |
Dec 19th 2017, 11:48 |
neon1024 |
For search manager at least |
| # |
Dec 19th 2017, 11:48 |
neon1024 |
Seems to be required according to my PHPStorm |
| # |
Dec 19th 2017, 11:48 |
dereuromark |
method would not be needed with mixin. |
| # |
Dec 19th 2017, 11:48 |
neon1024 |
Perhaps because it has a strange prefix namespace? |
| # |
Dec 19th 2017, 11:47 |
neon1024 |
Instead of `* @mixin \Muffin\Slug\Model\Behavior\SlugBehavior` |
| # |
Dec 19th 2017, 11:47 |
neon1024 |
`* @mixin \Muffin\Slug.Slug` |
| # |
Dec 19th 2017, 11:47 |
neon1024 |
Plus it seems to have incorrectly detected the UseMuffin/Slug plugin |
| # |
Dec 19th 2017, 11:45 |
neon1024 |
I’ve had to add `* @method \Search\Model\Behavior\SearchBehavior searchManager()` for my PHP Stan to pass |
| # |
Dec 19th 2017, 11:45 |
neon1024 |
@dereuromark Have I done something wrong? The ide helper isn’t generating a `@method` docblock for my call to `$this->searchManager()` in my table classes |
| # |
Dec 19th 2017, 10:51 |
neon1024 |
Cool, I’m over here though ;) https://github.com/dereuromark/cakephp-ide-helper/blob/master/docs/Annotations.md#view |
| # |
Dec 19th 2017, 10:44 |
dereuromark |
Thought this was clear, I added a note: https://github.com/dereuromark/cakephp-ide-helper/commit/41151c5ea1e635d785b1c3d27e07936a8a79ccdb |
| # |
Dec 19th 2017, 10:39 |
neon1024 |
:) |
| # |
Dec 19th 2017, 10:39 |
dereuromark |
yes they do |
| # |
Dec 19th 2017, 10:38 |
heke |
yes, but ... "pthreads v3 is restricted to operating in CLI only" , curl supposedly gives you option to query multiple APIs the same time , in webserver ... |
| # |
Dec 19th 2017, 10:38 |
neon1024 |
Hey @dereuromark do the annotations for Docblocks allow Cake code to pass PHP Stan? |
| # |
Dec 19th 2017, 10:35 |
neon1024 |
I didn’t really understand his talk as it was complicated, but is that multi-thread php what you mean? |
| # |
Dec 19th 2017, 10:35 |
neon1024 |
https://github.com/krakjoe/pthreads |
| # |
Dec 19th 2017, 10:34 |
neon1024 |
Isn’t Krakjoe doing that? Let me check |
| # |
Dec 19th 2017, 10:34 |
neon1024 |
I haven’t really used Guzzle though! *disclosure |
| # |
Dec 19th 2017, 10:34 |
heke |
lol, is that a wrapper around curl , it would be first time to hear php support multithread ... |
| # |
Dec 19th 2017, 10:34 |
neon1024 |
I suppose the thought process boils down to if Cake will be similar to Symfony in providing a way to wire up a number of components. However as Cake is opinionated and RAD I think that’s why it provides it’s own tooling within the framework |
| # |
Dec 19th 2017, 10:33 |
neon1024 |
Seems to, http://docs.guzzlephp.org/en/stable/quickstart.html#concurrent-requests |
| # |
Dec 19th 2017, 10:33 |
heke |
jeps query multple at the same time |
| # |
Dec 19th 2017, 10:32 |
neon1024 |
Pretty certain Guzzle allows for multi request, but let me check |
| # |
Dec 19th 2017, 10:32 |
neon1024 |
Multi-threading? Is that like multi-curl? |
| # |
Dec 19th 2017, 10:32 |
heke |
@neon1024 Idk, I see curl as the best, because it provides multithreading , ( in which php sucks ) |
| # |
Dec 19th 2017, 10:30 |
dereuromark |
you didnt before? ;) |
| # |
Dec 19th 2017, 10:29 |
neon1024 |
Going to try out your IdeHelper plugin @dereuromark :slightly_smiling_face: Looks good |
| # |
Dec 19th 2017, 10:25 |
rchavik |
i'm calling $this->Number->toReadableSize(100000) but the return value is the format string: "{0,number,#,###.##} KB". where did i screw up? |
| # |
Dec 19th 2017, 09:59 |
neon1024 |
There are some maintainability concerns, as there was with the phinx/migrations but some libraries are far more established and mature |
| # |
Dec 19th 2017, 09:58 |
neon1024 |
Why does the core have it’s own Http Client and not just depend on something like Guzzle? |
| # |
Dec 19th 2017, 09:51 |
dereuromark |
you are free to use whatever templating language you want, the core stays lean and simple however at this time. |
| # |
Dec 19th 2017, 09:50 |
dereuromark |
that was the scope of my ticket and rfc :slightly_smiling_face: nothing more than that, ever. |
| # |
Dec 19th 2017, 09:50 |
dereuromark |
for bake it was just common sense to not use php anymore to generate php, its a bad idea to use a non-standard solution for generating code of the same language |
| # |
Dec 19th 2017, 09:49 |
dereuromark |
no, there is no framework decision, it is a opt-in plugin, thats all |
| # |
Dec 19th 2017, 09:49 |
neon1024 |
I only put bake in require-dev though |