[Ml-yokadi] Note support

[Sébastien Renard]

Le Mardi 9 Mars 2010 23:44:57, Aurélien Gâteau a écrit :
> It's true taskcmd.py is getting quite large, but why do you think those
> new modules should be subclasses of TaskCmd?

a bug is a task.
a not is a task.
Si it makes sense to subclass TaskCmd no ?
 
> I am not sure because I feel it is very specific to [tn]_list, I can't
> see any way to reuse this for other commands.

Right.
 
> > - idem for _renderList (which is easier to move) but I don't know
> > where...
> 
> Actually I am wondering whether it would make sense to create a module
> dedicated to task/note listing...

Why not ? It could be the right way.
 
> > - Did you only split the code into small functions or do you also change
> > t_list behaviour ? (except small things like changing tui.error with
> > Y.Except.)
> 
> The only behavior change is that t_list now skips tasks which have the
> @note keyword. Any other behavior change is a bug :)

Ok ;-)
 
> > - I agree with change note to _note to be consistent with bug keywords.
> 
> Will do, but I want to implement k_edit before, so that it's easy for me
> to migrate... The only tricky part here is the case where one tries to
> rename a keyword using an existing keyword name. I think Yokadi should
> ask whether the user wants to merge keywords in this case. What do you
> think?

Right. User should decide if he wants to merge or use another keyword name.
-- 
Sébastien