Re: [Roundup-devel] timelog deployment issues

[John P. R. <ro...@cs...>]

In message <200...@ek...>,
Richard Jones writes:

>On Sun, 26 Jan 2003 5:36 am, John P. Rouillard wrote:
>> How do I implement timelogging via email. Having something available
>> only via the web interface isn't very useful. Ideas?
>
>Not sure. We probably need some command interface in the email gateway (for 
>handling "help" initially) and I suppose it could be extensible through the 
>interfaces file.

The other thing I was thinking of is adding the field at the end of the
subject line, athough I have wanted inline commands as well. Then I could

  Subject: [issue99] jdfjkdfsklj lkjdsfalkdsfa [timelog=0:20]

and tell the email somehow that timelog belongs to the message and
call a similar handler to what exists in the web interface.

>> So how do I handle this. The changes will also have to be done to my
>> patches in 0.6.0. I currently plan on trying:
>>
>> 	self.form[':msg_fields'].getlist
>>
>> to always get a list, then concatenate the list and pass it through
>> for processing. Is this the best way to handle these storage objects?
>> Here is my original quote.
>
>Probably - though I'd allow comma-separated lists too, so perhaps splitting 
>off the multilink list-handling code form parsePropsFromForm would be good 
>(ie. this:
>
>        if isinstance(proptype, hyperdb.Multilink):
>            # multiple values are OK
>            if isinstance(value, type([])):
>                # it's a list of MiniFieldStorages
>                value = [i.value.strip() for i in value]
>            else:
>                # it's a MiniFieldStorage, but may be a comma-separated list
>                # of values
>                value = [i.strip() for i in value.value.split(',')]
>
>            # filter out the empty bits
>            value = filter(None, value)
>

I did something much dumber.

        if self.form.has_key(':msg_fields') :
	    msgFields = join(self.form.getlist(':msg_fields'),',')
	    for i in split(msgFields,',') :

Just join the list with ,'s and the split everything. Seems to work
so far.

				-- rouilj
John Rouillard
===========================================================================
My employers don't acknowledge my existence much less my opinions.