search

carlescliment / calendar-bundle

A bundle for managing calendars in Symfony 2
40 stars 16 forks source link

Update EventRepository.php #17

Closed alexdrupal closed 9 years ago

alexdrupal commented 9 years ago

Fixed issue with date range. ->setTime(23,59) actually set time to 23:59:00 thus events with time 23:59:01 won't be selected.

carlescliment commented 9 years ago

Hey @alexdrupal, thanks a lot for this PR. It would be nice if you add a test covering the issue. Do you mind?

alexdrupal commented 9 years ago

Maybe you can guide me. Where I should insert this test?

carlescliment commented 9 years ago

Sorry for the delay in my response. I'll accept the PR and will let you know when I implement the test, just in case you want to have a look at it.

Thank you for contributing!