[dev] Re: [cvs] commit: nag tasklists.php

John Morrissey jwm at horde.net
Mon Aug 18 11:40:43 PDT 2003


On Mon, Aug 18, 2003 at 04:31:25PM +0200, Jan Schneider wrote:
> That didn't change your last patch. The current logic is: "if setting the
> tasklist description fails, show the error message: the tasklist couldn't
> be saved." Huh?

I figure you're referring to tasklist.php:85? I'm not sure what you mean -
what should the code be doing? The existing success message was 'The task
list has been saved.' To create a corresponding failure message, I used 'The
tasklist couldn't be saved.' The changes are batched up in the share and
comitted by the setDescription() call, so either all of it succeeds, or all
of it fails.

john
-- 
John Morrissey          _o            /\         ----  __o
jwm at horde.net        _-< \_          /  \       ----  <  \,
www.horde.net/    __(_)/_(_)________/    \_______(_) /_(_)__


More information about the dev mailing list