[emacs-wiki-discuss] Re: Extra brackets in Planner-Muse

[Marco Gidde]

Jim Ottaway <address@hidden> writes:

>>>>>> Marco Gidde <address@hidden> writes:
>
>> Yep, and since I don't use muse-wiki but could reproduce the described
>> effect, the missing brackets are the crucial point.
>
>> planner-multi-copy-or-move-task expects explicit links within the
>> parenthesis, probably to avoid treeting some random notes (like this
>> one) as links. When it finds something different, it falls back to
>> planner-copy-or-move-task-basic and the multiple links vanish.
>
>
> This patch seems to work.  I had to make a new, simpler date regexp:
> planner-date-regexp is not suitable since it only matches if the
> string is only a date.  I removed the groupings that appear to be
> unnecessary.  

I am not sure wether this "let's handle all possible special cases" is
really eligible. If planner was responsible for inserting the date
without the brackets in the first place, I would consider this
behaviour the bug. If it was written by the OP for convenience one
might argue, that it should be possible to leave out all the
brackets. In any case I would prefer a clean and consistent solution,
either every link with or every link without brackets. Just my 2ct.


Cheers

-- 
Marco Gidde