search

DHTMLX / scheduler

GPL version of JavaScript Event Scheduler
https://dhtmlx.com/docs/products/dhtmlxScheduler/
GNU General Public License v2.0
315 stars 111 forks source link

Fixed error with modified occurrence #34

Closed candelom closed 6 years ago

candelom commented 7 years ago

When showing/hiding calendar in an angular2 component (where the component loads some calendar events), this error used to appear: "Cannot read property event_pid of undefined".

AlexKlimenkov commented 6 years ago

Added null check from scheduler.addEvent to the main version - the new event can be immediately deleted from onEventAdded, so probably non-existing record here is a valid scenario. changes will be included in the next version of scheduler, this repo will be updated at next release. Didn't add changes from scheduler._is_modified_occurence, I can't see an expected scenario for a null/undefined argument there (thus a js exception will be an expected behavior). Please reopen if you can share some use case when an empty value there is expected.