Skip to content
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.

Sign up for GitHub

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

Jump to bottom

bug: Categories are output into multiple instances of CATEGORIES lines. #156

Closed
zijianhuang opened this issue Oct 5, 2016 · 0 comments
Closed

bug: Categories are output into multiple instances of CATEGORIES lines. #156

zijianhuang opened this issue Oct 5, 2016 · 0 comments

Comments

@zijianhuang
Copy link

https://tools.ietf.org/html/rfc5545#section-3.8.1.2

The standard does not say "This property can be specified multiple times" in Conformance, though it does not forbid multiple instances either. MS Outlook is able to read an event with multiple CATEGORIES instances, but the export will output only 1 line.

It will be good the iCal component could conform to the standard when outputting ics though it could tolerate multiple instances of CATEGORIES.

I understand this is the legacy behavior of DDay.iCal component.
@zijianhuang zijianhuang changed the title Categories are output into multiple instances of CATEGORIES lines. bug: Categories are output into multiple instances of CATEGORIES lines. Oct 5, 2016
rianjs added a commit that referenced this issue Oct 19, 2016
@rianjs
Fix multiple categories serialization problem #156
24e8ffc
rianjs added a commit that referenced this issue Oct 19, 2016
@rianjs
Unit test for CATEOGORIES being included multiple times #156
fc60a47 
The issue is that the Values properties are being iterated one at a time,
which means if a property has multiple values, it's treated as a top-level
component/subcomponent, instead of being folded into a comma-separated
list. I wonder if ATTENDEES has the same problem?
rianjs added a commit that referenced this issue Oct 19, 2016
@rianjs
Merge pull request #173 from rianjs/MultipleCategories
127e993 
Unit test for CATEOGORIES being included multiple times #156
@axunonb axunonb closed this as completed Oct 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@zijianhuang @axunonb