Trevoke / org-gtd.el

A package for using GTD with org-mode

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

defcustom group argument value seems incorrect in org-gtd-refile.el

doolio opened this issue · comments

The following defcustom :group argument value seems incorrect in org-gtd-refile.el:

:group 'org-gtd-organize

Should it not have a value of 'org-gtd-refile instead? In addition, is this file missing a defgroup form to ensure its customization settings appear under the org-gtd parent group?

Well, "refiling" is not a completely meaningful concept to surface from an org-gtd perspective. As you may have noticed between 2.0 and 3.0, I'm trying to move away from org-mode language as much where possible.

This may be a tactical error. Maybe I should let archive, refile, etc. just exist on their own, but ... "refile" has a very, VERY specific purpose in org-gtd, as opposed to the completely open and flexible tool that org-refile truly is, and it really is only meaningful as part of organize.

This being said, like for everything else, if you have an argument, I'm listening.

Well, "refiling" is not a completely meaningful concept to surface from an org-gtd perspective.

I do too. In fact, I was going to suggest at some point that the org-gtd-refile.el file be merged into the org-gtd-organize.el file.

Sure, but look at #160. The logic around sorting/storing is not "understood" yet, and until it is, I'd rather keep it separate. It's easier in code to bring things together than to separate them.

If we want to mess with org-gtd-refile.el then we should consider renaming it org-gtd-store.el or something like that, but I don't think the time is right for it. I think that, first, we should move the refiling logic in "add to project" to org-gtd-refile.el (something I'll work on in because of #160) and then we should see what we find.