Log message #4151165

# At Username Text
# Jun 13th 2018, 15:57 lorenzo I think a type architecture like that for the ORM is too much
# Jun 13th 2018, 15:57 josbeir ES doesn't assume stuff and takes various kind of date formats
# Jun 13th 2018, 15:56 josbeir yeah, dates are a bit lame too
# Jun 13th 2018, 15:56 lorenzo I guess ints is the only annoying one haha
# Jun 13th 2018, 15:56 lorenzo like ints
# Jun 13th 2018, 15:56 lorenzo I would start simple, and make the marshaller type cast the obvious stuff
# Jun 13th 2018, 15:55 josbeir yeah for instance
# Jun 13th 2018, 15:55 lorenzo in the marshaller you mean?
# Jun 13th 2018, 15:55 lorenzo where’s that?
# Jun 13th 2018, 15:55 josbeir yeah there's that but what if we had a way to configure those types explicitly ?
# Jun 13th 2018, 15:55 lorenzo as in, I guess you would expect the thing to convert very deeply nested properties to the right type, no?
# Jun 13th 2018, 15:54 lorenzo would you expect those types to understand your nested schema?
# Jun 13th 2018, 15:53 josbeir i'm finding myself doing too much basic stuff using entities/beforemarshal
# Jun 13th 2018, 15:53 josbeir @lorenzo here i am again, toughts on implementing something like Types for the ES plugin with a few standard types (like Date) ?
# Jun 13th 2018, 15:51 lorenzo the key is to check if afterSaveCommit is doing the work
# Jun 13th 2018, 15:51 lorenzo alright
# Jun 13th 2018, 15:51 itmpls gonna continue debugging after some yoga :slightly_smiling_face:
# Jun 13th 2018, 15:50 itmpls @lorenzo well it definitely does the stuff in _afterSave
# Jun 13th 2018, 15:50 neon1024 Thanks for the help :thumbsup:
# Jun 13th 2018, 15:49 neon1024 I’ll make a note in my readme about this :slightly_smiling_face:
# Jun 13th 2018, 15:49 dereuromark I tried to look into it, not too trivial change to get it done right
# Jun 13th 2018, 15:47 neon1024 I think I agree with your ticket here @dereuromark
# Jun 13th 2018, 15:47 dereuromark which it displays here :)
# Jun 13th 2018, 15:47 neon1024 Hmm, that doesn’t feel very verbose
# Jun 13th 2018, 15:47 neon1024 Hm, so that `- api` is my plugin command then?
# Jun 13th 2018, 15:47 dereuromark the rest is underlying php stuff not relevant IMO
# Jun 13th 2018, 15:46 dereuromark just bin/cake api
# Jun 13th 2018, 15:46 neon1024 Just a namespace in the plugin
# Jun 13th 2018, 15:46 neon1024 File uploaded https://cakesf.slack.com/files/U1BT622HW/FB67ATD5E/screen_shot_2018-06-13_at_16.46.19.png / https://slack-files.com/T053DPNCM-FB67ATD5E-1f44d7f178
# Jun 13th 2018, 15:46 dereuromark why the prefix?
# Jun 13th 2018, 15:46 neon1024 My command is `$ bin/cake gaia/supply.api sync Packages`
# Jun 13th 2018, 15:46 dereuromark if everyone (especially me) would bitch about everything first, the world would end :P
# Jun 13th 2018, 15:45 dereuromark but yours seem to be actual bugs of sorts, maybe you didnt sync the skeleton files?
# Jun 13th 2018, 15:45 neon1024 We all have different ways of working
# Jun 13th 2018, 15:45 dereuromark ;) I dont seem to encourage you hard enough
# Jun 13th 2018, 15:45 neon1024 I’ll go read the ticket :slightly_smiling_face:
# Jun 13th 2018, 15:45 neon1024 He forgets I’m lazy.
# Jun 13th 2018, 15:44 neon1024 He sure has high expectations of everyone else ;)
# Jun 13th 2018, 15:43 lorenzo it’s always about dereuromark :)
# Jun 13th 2018, 15:43 neon1024 This one? https://github.com/cakephp/cakephp/issues/12123
# Jun 13th 2018, 15:43 neon1024 Oh, you’re telling me to check the tickets because it’s your ticket