ADmad

Results 347 comments of ADmad

Given how easy the implementation of your proposed fragment() method it, you can just add it to your `AppView`.

Also check my comment here https://github.com/cakephp/cakephp/pull/17954#discussion_r1793021654. We need to consider the rendering of templates for emails and cells too, not just controllers, in which cause the use of `View::$name` as...

My issue isn't really with the dot notation. It's with array structure like this where the aliases and config keys can be mixed at the same level: ```php $usersQuery->contain([ 'Articles'...

> Would it suffice to deprecate only this call style in 5.x? I would be happy with just that. > It looks like this call style can be detected by...

> Copilot is right that calling prefix('admin', []) without a callback does nothing useful. None of these methods make sense without a callback, so callback is a must. So to...

Some type of grouping / breathing points is a must. A continuous wall of text is very difficult to read.

> Thought about this a bit more, what happens with plugins, and the core command line tools for moving webroot files around? Do those start to support both public and...

Our standard doesn't use tabs, so frankly I don't really care if a sniffs creates errors from those who do. One can simply disable that sniff in their app config.

I wouldn't make this change in a minor either, it would create a lot of confusion and documentation inconsistency.

> For some reason 4.x is now protected already, even though just created. The protection rule is set for `*.x`, that's way. I have temporarily changed it to `3.x`, so...