Re: [PATCH] org-contacts.el: add expire feature

[Karl Voit <devnull@Karl-Voit.at>]

* Daimrod <daimrod@gmail.com> wrote:
>
> Feng Shu <tumashu@gmail.com> writes:
>
>> * test
>>   :PROPERTIES:
>>   :EMAIL: test1@gmail.com  test2@gmail.com  test3@gmail.com
>>   :PHONE:  123456  123457 123458
>>   :EXPIRE:  test2@gmail.com 123457
>>   :END:
>>
>> when completing or exporting to vcard,  the emails and  phones in the
>> expire property (test2@gmail.com and 123457) will be ignore

This is a very good patch, fixing an issue I also do have currently.

> Since the purpose of this property is to ignore some values when
> exporting to vcard, don't you think it would be better to name it IGNORE
> or VCARD_IGNORE? (and of course to rename all functions accordingly)

I totally agree.

At first, I could not follow Feng Shu's explanation because I
thought that some (meta-) data gets expired after a certain period
of time. But then I realized that he meant that these things expired
in the past.

From the user point of view, I also do think that renaming the
property from :EXPIRE: to :IGNORE: would be better in terms of
understanding its purpose and how it works.

(If you do tend to keep the wording, I would at least rename it to
:EXPIRED: which emphasizes the fact that these things expired in the
past.)

-- 
mail|git|SVN|photos|postings|SMS|phonecalls|RSS|CSV|XML to Org-mode:
       > get Memacs from https://github.com/novoid/Memacs <

https://github.com/novoid/extract_pdf_annotations_to_orgmode + more on github