Make vCategory iterable

[niccokunzmann]

vCategory is used to list different categories of events. It would be nice to have an easy use of that.

from icalendar.prop import vCategory
c = vCategory.from_ical("APPOINTMENT,EDUCATION")
cats = list(c) # new feature
assert cats == ["APPOINTMENT", "EDUCATION"]

[devdanzin]

This seems like a good feature, but is the code example correct? Because from:

icalendar/src/icalendar/tests/prop/test_unit.py

Lines 331 to 339 in 39fef78

	def test_prop_vCategory(self):
	from icalendar.prop import vCategory
	catz = ['cat 1', 'cat 2', 'cat 3']
	v_cat = vCategory(catz)
	self.assertEqual(v_cat.to_ical(), b'cat 1,cat 2,cat 3')
	self.assertEqual(vCategory.from_ical(v_cat.to_ical()), catz)

It seems vCategory.from_ical doesn't return an instance of vCategory, but a list.

So should we work from:

from icalendar.prop import vCategory
c = vCategory(vCategory.from_ical("APPOINTMENT,EDUCATION"))
cats = list(c) # new feature
assert cats == ["APPOINTMENT", "EDUCATION"]

instead?

[niccokunzmann]

Hi, a list sounds good to me. I think that the documentation should reflect it so that it is clear how to use it.

[niccokunzmann]

Fixed in #670.