ADmad

Results 347 comments of ADmad

@LordSimal For the solution suggested by markstory you also have to explicitly specify that the `enumstatus` type should be used for the column through the [Table::_initializeSchema()](https://book.cakephp.org/4/en/orm/saving-data.html#saving-complex-types) hook.

I just used the example provided in the manual to avoid typing it out here :). We should probably update the manual example too to use `Table::initialize()`.

CakePHP already includes a `CorsBuilder` class whose instance is accessible through `Response::cors()`. It provides various convenience methods for setting required CORS headers. The reason there's no `CorsMiddleware` is because what...

You can open a pull request with a proposed implementation.

Both `ORM\Query::all()` and `ORM\Query::count()` cache the results. The docblock of `Query::count()` explicitly specifies this: > If the query has not been modified, and the count has already been performed the...

> The query has been modified by calling bind() after count(). In ndm2's words: "Not sure if anyone wants to take up the task of introducing more state into the...

Anymore feedback on this?

A noteworthy difference is the hook interface methods seem to receive only the testcase classname as string argument as opposed to testcase instance in `TestListener` methods.

Yes that's a concern. I am not sure how easy/difficult the change would be. Personally I am not too keen digging into this right away. When I noticed the deprecation...

Use [Table::subquery()](https://book.cakephp.org/4/en/orm/query-builder.html#subqueries) as stated in the manual.