New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error when set a null title on event #84

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@bit-dragon

bit-dragon commented Mar 13, 2013

When you create an event with a null value you get an error, cause can't found the method replace. So if I get data from Google Calendar with null value on title, the calendar crash.

I modified a test file with null value for an event to test this issue.

@elyou

This comment has been minimized.

Show comment
Hide comment
@elyou

elyou Mar 13, 2013

thank you for the fix but i think there should be no events without titles, because it will show as an event slot without any important data (beside the start date if it isn't an : allDay event)

elyou commented on 9f02d09 Mar 13, 2013

thank you for the fix but i think there should be no events without titles, because it will show as an event slot without any important data (beside the start date if it isn't an : allDay event)

This comment has been minimized.

Show comment
Hide comment
@bit-dragon

bit-dragon Mar 13, 2013

Owner

Yeah! Maybe you right and I have to check firsts if the event doesn't have title. I used this way on my application and I thought that maybe could be nessesary.

Owner

bit-dragon replied Mar 13, 2013

Yeah! Maybe you right and I have to check firsts if the event doesn't have title. I used this way on my application and I thought that maybe could be nessesary.

@arshaw

This comment has been minimized.

Show comment
Hide comment
@arshaw
Member

arshaw commented Jul 30, 2013

@arshaw arshaw closed this Jul 30, 2013

@arshaw

This comment has been minimized.

Show comment
Hide comment
@arshaw

arshaw Aug 11, 2013

Member

just released in v1.6.3

Member

arshaw commented Aug 11, 2013

just released in v1.6.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment