Log message #4053987

# At Username Text
# Jul 19th 2017, 10:42 neon1024 Actually I could go look that up, rather than be lazy.
# Jul 19th 2017, 10:42 neon1024 Is the `beforeFind` event only actually fired when the query is executed?
# Jul 19th 2017, 10:36 dangnhutranluong hope to learn from you
# Jul 19th 2017, 10:35 dangnhutranluong i'm a newbie!
# Jul 19th 2017, 10:35 dangnhutranluong hi!
# Jul 19th 2017, 10:31 neon1024 I have a strange problem. I’m trying to assert on the contains array in a query instance. When I var_dump it, the `__debugInfo` create a public property of `$eagerLoader->contain()` which shows the array I’m expecting. However when I `var_dump($query->getEagerLoader()->contain()` it’s empty, what am I missing?
# Jul 19th 2017, 10:30 neon1024 Blimey that’s a beast @dereuromark will have to find some time to read it later :slightly_smiling_face:
# Jul 19th 2017, 10:20 adithya Authentication for mobile and web, couldn't get cakes users to work! Any other plugins that provide the same features ? Just authentication with social is also fine :grinning: will do the authorisation
# Jul 19th 2017, 10:19 dereuromark Some good stuff :slightly_smiling_face: https://www.sitepoint.com/poka-yoke-saving-projects-with-hyper-defensive-programming/
# Jul 19th 2017, 10:15 graziel how do you spoof user input for shell tests? ive ended up using `$input = new ConsoleInput("data://text/plain,shell_test@test.com\ntest\ntest\ny");` but it feels wrong
# Jul 19th 2017, 10:09 neon1024 `@return ¯\_(ツ)_/¯` :P
# Jul 19th 2017, 10:09 neon1024 No more `@return mixed` WOOT!
# Jul 19th 2017, 10:09 neon1024 Plus when the transition comes to PHP 7 the method return typehints should be easier :slightly_smiling_face:
# Jul 19th 2017, 10:08 dereuromark also better (strict/less) return types
# Jul 19th 2017, 10:08 neon1024 Thanks guys, that’s saved me a pull request submission :slightly_smiling_face:
# Jul 19th 2017, 10:08 dereuromark solving the root issue of having a better interface with clear responsibility separation is a win in many other directions as well.
# Jul 19th 2017, 10:07 dereuromark hasnt been fixed for years, wont happen anytime soon, but v4 will solve all this by never ever having any $this|... :slightly_smiling_face:
# Jul 19th 2017, 10:07 neon1024 I agree @dereuromark
# Jul 19th 2017, 10:07 neon1024 No help here either, https://github.com/phpDocumentor/fig-standards/blob/master/proposed/phpdoc.md#715-return
# Jul 19th 2017, 10:07 dereuromark as $this would be correct :slightly_smiling_face:
# Jul 19th 2017, 10:07 dereuromark its a phpstorm bug
# Jul 19th 2017, 10:06 neon1024 `array|self` is fine, as is `array|\Cake\ORM\Query`
# Jul 19th 2017, 10:06 dereuromark phpstorm still this day has the problem that once you have $this and sth else it completely gets confused, one of the main reasons the setter getter split will 400% improve IDE compatability :slightly_smiling_face:
# Jul 19th 2017, 10:06 neon1024 Yes
# Jul 19th 2017, 10:06 Art_ i have a problem with flash messages. I used the same code on two different controllers, but the flash is not displaying on the other one.
# Jul 19th 2017, 10:05 dereuromark array|$this is the issue
# Jul 19th 2017, 10:05 lorenzo lol
# Jul 19th 2017, 10:05 lorenzo we keep going back and forth with`$this` and `self` in the doc blocks
# Jul 19th 2017, 10:01 neon1024 Perhaps I will PR that
# Jul 19th 2017, 10:01 neon1024 Ah yes, self works as well
# Jul 19th 2017, 10:00 neon1024 Perhaps because the IDE assumes that if the return is `$this` then it’s a fluent interface, and thus any other return value is invalid?
# Jul 19th 2017, 10:00 neon1024 Although I’m not sure that’s worth a PR
# Jul 19th 2017, 09:59 neon1024 Works fine with , * @return array|\Cake\ORM\Query
# Jul 19th 2017, 09:59 neon1024 Ah, it doesn’t understand $this
# Jul 19th 2017, 09:58 neon1024 I’m getting an ‘expected Query’ in my IDE
# Jul 19th 2017, 09:58 neon1024 Anyone have an idea why PHP Storm doesn’t understand this docblock? https://github.com/cakephp/cakephp/blob/master/src/ORM/Query.php#L376
# Jul 19th 2017, 09:44 hmic you still might want to open an issue on your finding as this should work in some way or another
# Jul 19th 2017, 09:43 mirec I will take a look at it. thanks
# Jul 19th 2017, 09:43 ldsign Figured it out: $options['association']->getSource()->alias()
# Jul 19th 2017, 09:42 hmic use assetcompress already!
# Jul 19th 2017, 09:42 hmic which is more than just ugly!