adesigns / calendar-bundle

This bundle allows you to integrate the jQuery FullCalendar plugin into your Symfony2 application.
MIT License
96 stars 60 forks source link

Correcting some bug & Logic Optmization #15

Closed ScreamZ closed 7 years ago

ScreamZ commented 10 years ago

For IDE better \t than \T :)

why u set : $endDatetime = null, $allDay = false Should be better if you make a allday = true or put a default endDateTime

If the user doesnt specify these parameters he gonna get an exception directly because he doesnt define a endDate for a non-allDay event So why allow these parameters optionnal if they make an instant exception if they're not specify.

Make them necessary or change to a default endDate, or set allday true by default

Thanks :)

Hope u enjoy :)