Docstring ? (here and below :))

What about None or [] then?

What about making this a JSON blob as we do for milestones/priorities and such?

No need to put the list in a dict, you can directly put the list in the json.dumps(), which will also make the getter much simpler

I think we should keep this part as dum as possible and do the split in the controller

Same here, imho, it should return the list, not a string.

Side note: You can concatenate a list using ', '.join(list) for example:

>>> ', '.join(['a', 'b', 'c'])
'a, b, c'

Which also simplifies the code quite a bit :)

Note that is field.data returns a list, you can then use {{ field.data | join(', ') }}

I don't want to return a list, this string is directly used by the html page. Its much harder to do this splitting in html/jinja than it is to do it here. However, I will use your concatenation though

What are you proposing exactly? This section does the normalization of removing white spaces around the commas. We need this.

Will this add an unwanted comma after the last item? I need to test it...

Yup, you're right. I'll revise this.

Actually I might be able to remove this function based off one of your other comments...

I won't add a trailing coma no:

>>> ', '.join(['a', 'b', 'c'])
'a, b, c'

Agreed, but I think we could move it above (going to add a comment to where I think we could place it)

I think this is the place where I would do the split on , and .strip()

Applied recommended changes and rebased...

I guess we should rename the argument then list_str, maybe data_obj?

I would prefer we do something like:

if key.name == 'bugzilla':
    self.assertNotNone(key.data)

Or, even better that we check precisely what key.data is :)

Same as above if possible :)

Few nitpicking the rest is looking good, I'm going to test it locally

I think there is typo here This is required ...

     I think there is typo here This is required ...

No typo here - this is correct. Perhaps it's not the best wording though. I'll change it...

You can drop the () ;-)

assertFalse item not in list? Could we avoid the double negation here? :)

Added a few commits and rebased :)

ack!

This is looking all good to me and local testing seem fine as well :)