| # |
Sep 10th 2019, 16:01 |
 dereuromark |
:slightly_smiling_face: |
| # |
Sep 10th 2019, 16:01 |
dereuromark |
Doing the anti composer way sure is not proper. |
| # |
Sep 10th 2019, 15:59 |
neon1024 |
I guess that’s developers for you! |
| # |
Sep 10th 2019, 15:59 |
neon1024 |
I bet most people think most other people are ‘doing it wrong’ |
| # |
Sep 10th 2019, 15:58 |
neon1024 |
“Proper” is there such a thing |
| # |
Sep 10th 2019, 15:32 |
challgren |
Sweet PDU failure! |
| # |
Sep 10th 2019, 15:25 |
dereuromark |
No, proper setup of PHP apps :slightly_smiling_face: |
| # |
Sep 10th 2019, 15:24 |
neon1024 |
> we have been doing this for a while now |
| # |
Sep 10th 2019, 15:24 |
neon1024 |
@dereuromark So you’re an expert as multi-core setups? |
| # |
Sep 10th 2019, 15:14 |
dereuromark |
You are only creating issues, not solving a single real one :slightly_smiling_face: Trust us, we have been doing this for a while now. |
| # |
Sep 10th 2019, 15:14 |
dereuromark |
Yeah, avoid side effects. Really dont dont dont go down this road. |
| # |
Sep 10th 2019, 15:00 |
challgren |
Unless you are using a floppy disk or zip disk it wont really save you anything |
| # |
Sep 10th 2019, 14:59 |
challgren |
Your only saving 128mbs of space. Which in this day is nothing. Plus composer will cache everything locally so you wont even have duplicate network calls going out |
| # |
Sep 10th 2019, 14:57 |
admad |
Spend your time and effort on things that actually matter, not pointless optimisations |
| # |
Sep 10th 2019, 14:57 |
rsteenge |
:,) |
| # |
Sep 10th 2019, 14:56 |
admad |
@rsteenge are your 20 applications run from a floppy disk? |
| # |
Sep 10th 2019, 14:55 |
rsteenge |
I understand but what if you like all the applications to have the same core? for example 20 different applications |
| # |
Sep 10th 2019, 14:38 |
ricksaccous |
@rsteenge this is not recommended, especially since one project might be easier to update and another might take longer, etc |
| # |
Sep 10th 2019, 14:37 |
rsteenge |
is it possible in CakePHP 3 that multiple applications share the same CakePHP core? |
| # |
Sep 10th 2019, 13:50 |
admad |
indeed |
| # |
Sep 10th 2019, 13:50 |
dereuromark |
immutable object operation without reassignment sure is sth hard to spot. |
| # |
Sep 10th 2019, 13:50 |
dereuromark |
which is quite cool indeed. |
| # |
Sep 10th 2019, 13:49 |
admad |
@dereuromark i skimmed through it. If i understand it correctly with proper use of those annotations psalm could for e.g. warn user if they use `$this->response->withFoo()` in controller and fail to return / assign the method's return value. |
| # |
Sep 10th 2019, 13:42 |
admad |
psalm should become the defacto static analyzer for PHP. Matt Brown's development pace is awesome. |
| # |
Sep 10th 2019, 13:41 |
admad |
i got the release notification, haven't read the article yet :slightly_smiling_face: |
| # |
Sep 10th 2019, 13:39 |
dereuromark |
@admad If you're bored^^ https://psalm.dev/articles/immutability-and-beyond |
| # |
Sep 10th 2019, 13:34 |
neon1024 |
I fixed the title :slightly_smiling_face: |
| # |
Sep 10th 2019, 13:33 |
neon1024 |
Here we are, feel free to edit if you need to https://github.com/cakephp/cakephp/issues/13638 |
| # |
Sep 10th 2019, 13:28 |
admad |
above i said `addBehavior()` by mistake, the association methods `hasOne()` etc. :slightly_smiling_face: |
| # |
Sep 10th 2019, 13:24 |
neon1024 |
Righto |
| # |
Sep 10th 2019, 13:24 |
neon1024 |
Or er, both perhaps? |
| # |
Sep 10th 2019, 13:24 |
admad |
core, first the properties need to be assinged |
| # |
Sep 10th 2019, 13:23 |
neon1024 |
Open it against the core or bake? |
| # |
Sep 10th 2019, 13:23 |
neon1024 |
Of course. We were discussing why PHP Storm doesn’t add the `$` and discussions turned instead, to why Cake doesn’t just declare class properties right in the Table class |
| # |
Sep 10th 2019, 13:21 |
admad |
@neon1024 mind opening an issue for this for 4.x? |
| # |
Sep 10th 2019, 13:18 |
admad |
maybe `addBehavior()` should set the property like `loadModel()` does |
| # |
Sep 10th 2019, 13:18 |
admad |
actually, my bad. It doesn't set the associations as properties, they are fetched using magic `__get()` |
| # |
Sep 10th 2019, 13:14 |
admad |
it should, have you declared them? |
| # |
Sep 10th 2019, 13:14 |
neon1024 |
Rather than having to use `@property` |
| # |
Sep 10th 2019, 13:14 |
neon1024 |
Y |
| # |
Sep 10th 2019, 13:13 |
admad |
@neon1024 it does, but since the properties are added dynamically your IDE doesn't "see" them |