| # |
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 |
| # |
Sep 10th 2019, 11:22 |
neon1024 |
Why don’t Table classes define associations as class properties? |
| # |
Sep 10th 2019, 11:21 |
neon1024 |
Wales, Scotland and NI will leave the union |
| # |
Sep 10th 2019, 11:20 |
neon1024 |
Ronald Dump will use his Greenland fund to buy England |
| # |
Sep 10th 2019, 11:20 |
neon1024 |
We’ll become a subsiduary of the USA soon |
| # |
Sep 10th 2019, 11:20 |
neon1024 |
From running an Empire which spanned most of the globe to a bickering childish embarassment |
| # |
Sep 10th 2019, 11:19 |
neon1024 |
They wheel out the “Will of the people” too, when it’s actually the “Will of 52% of the people” |
| # |
Sep 10th 2019, 11:19 |
neon1024 |
Pretty much spot on |
| # |
Sep 10th 2019, 11:19 |
neon1024 |
Lol |
| # |
Sep 10th 2019, 11:17 |
admad |
@neon1024 https://i.redd.it/i30s7h2hnpl31.jpg |
| # |
Sep 10th 2019, 11:03 |
dereuromark |
Wow, that bug was opened 4 years ago? :slightly_smiling_face: I wonder if they ever fix it. Should we have either a code sniffer or IdeHelper auto fix it? |
