[Dev] question about attribute editors

[Ted Leung]

On Jan 5, 2006, at 2:40 PM, Bryan Stearns wrote:

> Today's lesson on Theory vs Practice:
>
> Theory: The normal picking process ought to do the right thing: if  
> the attribute is of type String with cardinality 'list', you ought  
> to get the regular StringAttributeEditor, which is supposed to  
> format and parse with ',' as the separator.
>
> Practice: Having typed that, I went to look at the source code - it  
> looks like StringAttributeEditor.GetAttributeValue will do the  
> right thing with 'list' cardinality, but  
> StringAttributeEditor.SetAttributeValue only handles 'single'. (The  
> one case where we currently use something like this is the email  
> 'toAddress' attribute, where EmailAddressAttributeEditor gets used  
> - it's a subclass of StringAttributeEditor that implements its own  
> SetAttributeValue.
>
> I'll make StringAttributeEditor.SetAttributeValue handle the 'list'  
> case shortly. (Note, though, that our general strategy for dealing  
> with 'list' things in textctrls assumes that the strings being  
> edited will never contain ',' - I think this is a bad assumption,  
> but I don't have a fix for it.)

I'd like to see if I could get this to work using the 0.6 builds that  
were distributed.  Can I make this field read only or something so  
that it doesn't rely on a change to  
StringAttributeEditor.SetAttributeValue?

Ted