Log message #4202561

# At Username Text
# 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?