Mark Story

Results 653 comments of Mark Story

@vlakoff Agreed. PHP provides `DateInterval` which can be used to define duration/interval lengths. Its API is not very elegant for dealing with media duration though.

Looks like while we have `beforeFind` in the API docs we never actually implemented it.

Makes sense to me. I don't think the refresh option existed when saving was first implemented, but now that it exists we should use it.

Looks like we're missing behavior like `Database\Query::_dirty()` provides ORM queries.

@ADmad Does anything else need to change on this? It looks good to me.

Since you tracked down the session construction, what is the stacktrace for the session creation? Generally it is supposed to happen during ServerRequestFactory::fromGlobals() during Http server setup, so the configuration...

Having a full stacktrace from where that 500 is coming from would help. You can see the source in that error message. AuthenticationComponent line 256. Adding `stackTrace()` at that line...

What entity are you putting into the the AuthorizationComponent `authorize()` method? The ORM resolver uses the class name of the entity to generate the name of the policy to use.

The `RequestAuthorizationMiddleware` doesn't support redirect options at this time. It could be added in a future release. Would you like to help add that option?

> my opinion : moving all Would you also be moving all the logic related to these options to the service as well?