# |
Jul 19th 2017, 15:29 |
neon1024 |
Well if it’s a value object surely the constructor just checks the params? |
# |
Jul 19th 2017, 15:28 |
neon1024 |
The only down side I have with value objects is they make the api look really complex when calling methods |
# |
Jul 19th 2017, 15:28 |
inoas |
I can understand the desire of the IDE folks but the problem is that burzums config object to me doesn't do a good job on making sure the options "object" is in a well formed state that can be consumed |
# |
Jul 19th 2017, 15:28 |
neon1024 |
Perhaps I’ll spend some time properly reading the stuff :slightly_smiling_face: |
# |
Jul 19th 2017, 15:27 |
inoas |
well that's not where I would go... sadly |
# |
Jul 19th 2017, 15:27 |
neon1024 |
I like being able to type hint against them :slightly_smiling_face: |
# |
Jul 19th 2017, 15:27 |
neon1024 |
I’m all in favour of value objects |
# |
Jul 19th 2017, 15:27 |
inoas |
or forgetting one, or having some that used to do things but now do not do them anymore |
# |
Jul 19th 2017, 15:27 |
inoas |
you never have the problem of miss-typing a hashkey and not realizing again |
# |
Jul 19th 2017, 15:27 |
inoas |
After that it is easy to use $options as you can rely on them |
# |
Jul 19th 2017, 15:26 |
inoas |
neon1024 all I am saying is you push $options into a function that takes an option object (basically an array) and the method/function then runs it through a small schema validator. |
# |
Jul 19th 2017, 15:26 |
domgeek |
https://github.com/cakephp/cakephp/issues/6072 please check this out |
# |
Jul 19th 2017, 15:24 |
neon1024 |
Go well my brainy friend! ;) |
# |
Jul 19th 2017, 15:24 |
neon1024 |
Geez Ionas, you’re so far ahead of me on these things. I don’t even understand half of it! |
# |
Jul 19th 2017, 15:19 |
inoas |
domgeek I am not sure I can help you (or we can) but can you repost the gist with issue |
# |
Jul 19th 2017, 15:19 |
inoas |
and this is what I fear... and there hasn't been really a clear statement where this will go https://github.com/cakephp/cakephp/issues/10866#issuecomment-313391901 |
# |
Jul 19th 2017, 15:19 |
domgeek |
sorry to have tagged them but i posted this same post earlier on.. nobody replied me :sad: |
# |
Jul 19th 2017, 15:18 |
inoas |
jeremyharris about feedback, maybe this is not the best solution, but what's your take on validation $options before consumption against a pattern/schema like so https://gist.github.com/ionas/94e961bf414c78362442041081dad3ca |
# |
Jul 19th 2017, 15:18 |
inoas |
what issue anyway I didnt see it but "having problem with script" |
# |
Jul 19th 2017, 15:17 |
jarard |
app works fine otherwise, but when my console tries to query a model I get the error |
# |
Jul 19th 2017, 15:17 |
neon1024 |
Sorry @domgeek I don’t know a solution to your issue |
# |
Jul 19th 2017, 15:17 |
inoas |
domgeek try to work with the community not single peeps ;) |
# |
Jul 19th 2017, 15:17 |
jeremyharris |
I'm not at a computer at the moment sorry |
# |
Jul 19th 2017, 15:17 |
jarard |
guys anyone familiar with missing mysql connection when running Console? |
# |
Jul 19th 2017, 15:16 |
domgeek |
@neon1024 @jeremyharris plz i need help here |
# |
Jul 19th 2017, 15:16 |
jeremyharris |
Input is always welcome :slightly_smiling_face: the framework has always been a team effort |
# |
Jul 19th 2017, 15:16 |
neon1024 |
@jeremyharris Oh in Stream!? Right, thanks :slightly_smiling_face: |
# |
Jul 19th 2017, 15:16 |
inoas |
not necessarily but in 95% of the cases and that's enough for me to be happy and productive with cakephp ;) |
# |
Jul 19th 2017, 15:15 |
neon1024 |
We’ll see, I’m sure the core team will make the right choice in the end |
# |
Jul 19th 2017, 15:15 |
neon1024 |
I found the routing example very confusing. I think @steinkel has been working on routing scopes which, if I understand correctly, would address those concerns raised |
# |
Jul 19th 2017, 15:15 |
inoas |
i mean use strict, strict method signatures, the pattern matching for $options that I have suggested or something simlar strict |
# |
Jul 19th 2017, 15:14 |
neon1024 |
Well the errors are being overhauled soon |
# |
Jul 19th 2017, 15:14 |
inoas |
and maybe a warning/exception if you use the generic table/entity ;) |
# |
Jul 19th 2017, 15:14 |
neon1024 |
The discussion intrigues me as I never think about the actual design of the framework. I do find the posts by @OP contradictory though, complaining about statics in one message, but advocating more statics in another. It seems like someone who enjoys another framework more, but wants those features in CakePHP. |
# |
Jul 19th 2017, 15:14 |
inoas |
the only changes I am looking for is ultra strict modes + verbose exceptions AND nullinflector across the model layer so that plural/singular is only being used for view template rendering / option labels / flash messages |
# |
Jul 19th 2017, 15:13 |
inoas |
neon1024 tbh I am in the camp of 95% happy ;) |
# |
Jul 19th 2017, 15:12 |
dereuromark |
any quick fixes for https://github.com/cakephp/cakephp/issues/10723#issuecomment-316418716 ? I would have expected the ORM to format this to valid MySQL or throw an exception. |
# |
Jul 19th 2017, 15:12 |
neon1024 |
Hmm, interesting ticket Ionas :slightly_smiling_face: |
# |
Jul 19th 2017, 15:11 |
jeremyharris |
Inoas no we don't, I think he's in the camp of people that want one. Or at least docs on how to implement one |
# |
Jul 19th 2017, 15:10 |
jeremyharris |
@neon1024 there's a test in the adapter class here https://github.com/cakephp/cakephp/blob/master/tests/TestCase/Http/Client/Adapter/StreamTest.php |
# |
Jul 19th 2017, 15:09 |
inoas |
the only queue I know is middlewarequeue |