* Bug: Docstrings of =org-agenda-todo-ignore-with-date= and =org-agenda-todo-ignore-timestamp= unclear [9.2 (release_9.2-215-g5b39d8 @ /home/mbork/others-works/emacs/org-mode/lisp/)]
@ 2019-06-18 6:55 Marcin Borkowski
2019-06-19 19:19 ` Nicolas Goaziou
0 siblings, 1 reply; 12+ messages in thread
From: Marcin Borkowski @ 2019-06-18 6:55 UTC (permalink / raw)
To: emacs-orgmode
Remember to cover the basics, that is, what you expected to happen and
what in fact did happen. You don't know how to make a good report? See
https://orgmode.org/manual/Feedback.html#Feedback
Your bug report will be posted to the Org mailing list.
------------------------------------------------------------------------
The docstrings of =org-agenda-todo-ignore-with-date= and
=org-agenda-todo-ignore-timestamp= do not really tell what these
settings are doing. They should (imho) emphasize that the former just
omits from the global todo list all entries with at least one active
timestamp (in the hedaline or the body), and the latter checks the first
active timestamp which is not SCHEDULED or DEADLINE and compares it to
today's date.
It could also be mentioned in the manual.
I have GNU papers for Emacs signed. If this is enough, I'd be happy to
submit a patch. If so, what would be better: to patch only the
docstrings or the docstrings and the manual.
Best,
mb
Emacs : GNU Emacs 27.0.50 (build 2, x86_64-pc-linux-gnu, GTK+ Version 3.24.8)
of 2019-06-15
Package: Org mode version 9.2 (release_9.2-215-g5b39d8 @ /home/mbork/others-works/emacs/org-mode/lisp/)
--
Marcin Borkowski
http://mbork.pl
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: Bug: Docstrings of =org-agenda-todo-ignore-with-date= and =org-agenda-todo-ignore-timestamp= unclear [9.2 (release_9.2-215-g5b39d8 @ /home/mbork/others-works/emacs/org-mode/lisp/)]
2019-06-18 6:55 Bug: Docstrings of =org-agenda-todo-ignore-with-date= and =org-agenda-todo-ignore-timestamp= unclear [9.2 (release_9.2-215-g5b39d8 @ /home/mbork/others-works/emacs/org-mode/lisp/)] Marcin Borkowski
@ 2019-06-19 19:19 ` Nicolas Goaziou
2019-06-23 11:23 ` Marcin Borkowski
0 siblings, 1 reply; 12+ messages in thread
From: Nicolas Goaziou @ 2019-06-19 19:19 UTC (permalink / raw)
To: Marcin Borkowski; +Cc: emacs-orgmode
Hello,
Marcin Borkowski <mbork@mbork.pl> writes:
> The docstrings of =org-agenda-todo-ignore-with-date= and
> =org-agenda-todo-ignore-timestamp= do not really tell what these
> settings are doing. They should (imho) emphasize that the former just
> omits from the global todo list all entries with at least one active
> timestamp (in the hedaline or the body), and the latter checks the first
> active timestamp which is not SCHEDULED or DEADLINE and compares it to
> today's date.
>
> It could also be mentioned in the manual.
>
> I have GNU papers for Emacs signed. If this is enough, I'd be happy to
> submit a patch. If so, what would be better: to patch only the
> docstrings or the docstrings and the manual.
I would say the latter. Thank you.
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: Bug: Docstrings of =org-agenda-todo-ignore-with-date= and =org-agenda-todo-ignore-timestamp= unclear [9.2 (release_9.2-215-g5b39d8 @ /home/mbork/others-works/emacs/org-mode/lisp/)]
2019-06-19 19:19 ` Nicolas Goaziou
@ 2019-06-23 11:23 ` Marcin Borkowski
2019-06-23 19:29 ` Samuel Wales
2019-06-24 19:09 ` Marcin Borkowski
0 siblings, 2 replies; 12+ messages in thread
From: Marcin Borkowski @ 2019-06-23 11:23 UTC (permalink / raw)
To: Nicolas Goaziou; +Cc: emacs-orgmode
[-- Attachment #1: Type: text/plain, Size: 995 bytes --]
On 2019-06-19, at 21:19, Nicolas Goaziou <mail@nicolasgoaziou.fr> wrote:
> Hello,
>
> Marcin Borkowski <mbork@mbork.pl> writes:
>
>> The docstrings of =org-agenda-todo-ignore-with-date= and
>> =org-agenda-todo-ignore-timestamp= do not really tell what these
>> settings are doing. They should (imho) emphasize that the former just
>> omits from the global todo list all entries with at least one active
>> timestamp (in the hedaline or the body), and the latter checks the first
>> active timestamp which is not SCHEDULED or DEADLINE and compares it to
>> today's date.
>>
>> It could also be mentioned in the manual.
>>
>> I have GNU papers for Emacs signed. If this is enough, I'd be happy to
>> submit a patch. If so, what would be better: to patch only the
>> docstrings or the docstrings and the manual.
>
> I would say the latter. Thank you.
I only patched the doctsrings for now. If this is fine, I'll do
a similar thing in the manual.
WDYT?
--
Marcin Borkowski
http://mbork.pl
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-Make-two-docstrings-more-clear.patch --]
[-- Type: text/x-patch; size=1.79KiB, Size: 1833 bytes --]
From d5a1c1ff3a9a7ccbf2765332971973b2c4233e16 Mon Sep 17 00:00:00 2001
From: Marcin Borkowski <mbork@mbork.pl>
Date: Sun, 23 Jun 2019 13:22:04 +0200
Subject: [PATCH] Make two docstrings more clear
The docstrings of `org-agenda-todo-ignore-with-date` and
`org-agenda-todo-ignore-timestamp` were not very clear. Now they are
better.
---
lisp/org-agenda.el | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/lisp/org-agenda.el b/lisp/org-agenda.el
index fbf520c5e..daf51601d 100644
--- a/lisp/org-agenda.el
+++ b/lisp/org-agenda.el
@@ -690,9 +690,10 @@ potentially much shorter TODO lists."
"Non-nil means don't show entries with a date in the global todo list.
You can use this if you prefer to mark mere appointments with a TODO keyword,
but don't want them to show up in the TODO list.
-When this is set, it also covers deadlines and scheduled items, the settings
-of `org-agenda-todo-ignore-scheduled' and `org-agenda-todo-ignore-deadlines'
-will be ignored.
+When this is set, it also covers deadlines and scheduled items
+(more precisely, every entry with at least one active timestamp will be ignored),
+the settings of `org-agenda-todo-ignore-scheduled'
+and `org-agenda-todo-ignore-deadlines' will be ignored.
See also the variable `org-agenda-tags-todo-honor-ignore-options'."
:group 'org-agenda-skip
:group 'org-agenda-todo-list
@@ -700,6 +701,9 @@ See also the variable `org-agenda-tags-todo-honor-ignore-options'."
(defcustom org-agenda-todo-ignore-timestamp nil
"Non-nil means don't show entries with a timestamp.
+If this variable is non-nil, Org agenda looks at the first active
+timestamp in the entry which is neither a DEADLINE nor
+a SCHEDULED timestamp and compares it to \"today\".
This applies when creating the global todo list.
Valid values are:
--
2.22.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: Bug: Docstrings of =org-agenda-todo-ignore-with-date= and =org-agenda-todo-ignore-timestamp= unclear [9.2 (release_9.2-215-g5b39d8 @ /home/mbork/others-works/emacs/org-mode/lisp/)]
2019-06-23 11:23 ` Marcin Borkowski
@ 2019-06-23 19:29 ` Samuel Wales
2019-06-24 19:08 ` Marcin Borkowski
2019-06-24 19:09 ` Marcin Borkowski
1 sibling, 1 reply; 12+ messages in thread
From: Samuel Wales @ 2019-06-23 19:29 UTC (permalink / raw)
To: Marcin Borkowski; +Cc: emacs-orgmode, Nicolas Goaziou
this looks good, but i wonder if newcomers get confused about that todo thing.
i am not proposing any solution, just brainstorming.
the word todo is used in a few contexts in org:
- the todo keyword per se [TODO]
- todoish keywords as opposed to doneish keywords
- the global todo view as opposed to the general views that cover
time [called daily/weekly], structured search [called tags], and
free-form search [called search]
newcomers might wonder if the global todo view does anything that the
general views cannot do, otherwise why does it exist.
some newcomers might even wonder if todo means keyword as opposed to blank.
don't take this too seriously, but i wonder if that todo- confuses
newcomers even though the docstring is now clear.
On 6/23/19, Marcin Borkowski <mbork@mbork.pl> wrote:
>
> On 2019-06-19, at 21:19, Nicolas Goaziou <mail@nicolasgoaziou.fr> wrote:
>
>> Hello,
>>
>> Marcin Borkowski <mbork@mbork.pl> writes:
>>
>>> The docstrings of =org-agenda-todo-ignore-with-date= and
>>> =org-agenda-todo-ignore-timestamp= do not really tell what these
>>> settings are doing. They should (imho) emphasize that the former just
>>> omits from the global todo list all entries with at least one active
>>> timestamp (in the hedaline or the body), and the latter checks the first
>>> active timestamp which is not SCHEDULED or DEADLINE and compares it to
>>> today's date.
>>>
>>> It could also be mentioned in the manual.
>>>
>>> I have GNU papers for Emacs signed. If this is enough, I'd be happy to
>>> submit a patch. If so, what would be better: to patch only the
>>> docstrings or the docstrings and the manual.
>>
>> I would say the latter. Thank you.
>
> I only patched the doctsrings for now. If this is fine, I'll do
> a similar thing in the manual.
>
> WDYT?
>
> --
> Marcin Borkowski
> http://mbork.pl
>
--
The Kafka Pandemic
What is misopathy?
https://thekafkapandemic.blogspot.com/2013/10/why-some-diseases-are-wronged.html
The disease DOES progress. MANY people have died from it. And ANYBODY
can get it at any time.
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: Bug: Docstrings of =org-agenda-todo-ignore-with-date= and =org-agenda-todo-ignore-timestamp= unclear [9.2 (release_9.2-215-g5b39d8 @ /home/mbork/others-works/emacs/org-mode/lisp/)]
2019-06-23 19:29 ` Samuel Wales
@ 2019-06-24 19:08 ` Marcin Borkowski
0 siblings, 0 replies; 12+ messages in thread
From: Marcin Borkowski @ 2019-06-24 19:08 UTC (permalink / raw)
To: Samuel Wales; +Cc: emacs-orgmode, Nicolas Goaziou
On 2019-06-23, at 21:29, Samuel Wales <samologist@gmail.com> wrote:
> this looks good, but i wonder if newcomers get confused about that todo thing.
>
> i am not proposing any solution, just brainstorming.
>
> the word todo is used in a few contexts in org:
>
> - the todo keyword per se [TODO]
> - todoish keywords as opposed to doneish keywords
> - the global todo view as opposed to the general views that cover
> time [called daily/weekly], structured search [called tags], and
> free-form search [called search]
>
> newcomers might wonder if the global todo view does anything that the
> general views cannot do, otherwise why does it exist.
>
> some newcomers might even wonder if todo means keyword as opposed to blank.
>
> don't take this too seriously, but i wonder if that todo- confuses
> newcomers even though the docstring is now clear.
Good points, though (as you said) this is out of scope of my humble
patch:-).
Best,
--
Marcin Borkowski
http://mbork.pl
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: Bug: Docstrings of =org-agenda-todo-ignore-with-date= and =org-agenda-todo-ignore-timestamp= unclear [9.2 (release_9.2-215-g5b39d8 @ /home/mbork/others-works/emacs/org-mode/lisp/)]
2019-06-23 11:23 ` Marcin Borkowski
2019-06-23 19:29 ` Samuel Wales
@ 2019-06-24 19:09 ` Marcin Borkowski
2019-07-07 7:51 ` Marcin Borkowski
1 sibling, 1 reply; 12+ messages in thread
From: Marcin Borkowski @ 2019-06-24 19:09 UTC (permalink / raw)
To: Nicolas Goaziou; +Cc: emacs-orgmode
[-- Attachment #1: Type: text/plain, Size: 1234 bytes --]
On 2019-06-23, at 13:23, Marcin Borkowski <mbork@mbork.pl> wrote:
> On 2019-06-19, at 21:19, Nicolas Goaziou <mail@nicolasgoaziou.fr> wrote:
>
>> Hello,
>>
>> Marcin Borkowski <mbork@mbork.pl> writes:
>>
>>> The docstrings of =org-agenda-todo-ignore-with-date= and
>>> =org-agenda-todo-ignore-timestamp= do not really tell what these
>>> settings are doing. They should (imho) emphasize that the former just
>>> omits from the global todo list all entries with at least one active
>>> timestamp (in the hedaline or the body), and the latter checks the first
>>> active timestamp which is not SCHEDULED or DEADLINE and compares it to
>>> today's date.
>>>
>>> It could also be mentioned in the manual.
>>>
>>> I have GNU papers for Emacs signed. If this is enough, I'd be happy to
>>> submit a patch. If so, what would be better: to patch only the
>>> docstrings or the docstrings and the manual.
>>
>> I would say the latter. Thank you.
>
> I only patched the doctsrings for now. If this is fine, I'll do
> a similar thing in the manual.
I attach two more patches: one for the manual and one for the (apparent)
bug in `org-agenda-check-for-timestamp-as-reason-to-ignore-todo-item`.
WDYT?
--
Marcin Borkowski
http://mbork.pl
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0002-Explain-the-org-agenda-todo-ignore-.-variables-in-th.patch --]
[-- Type: text/x-patch; size=1.41KiB, Size: 1448 bytes --]
From 89e1254d04802b13f4bd8c753287bce4eb1ae044 Mon Sep 17 00:00:00 2001
From: Marcin Borkowski <mbork@mbork.pl>
Date: Mon, 24 Jun 2019 21:05:32 +0200
Subject: [PATCH] Explain the `org-agenda-todo-ignore-...` variables in the
manual
---
doc/org-manual.org | 12 +++++++-----
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/doc/org-manual.org b/doc/org-manual.org
index 440888b4e..573c59b99 100644
--- a/doc/org-manual.org
+++ b/doc/org-manual.org
@@ -8545,11 +8545,13 @@ it more compact:
#+vindex: org-agenda-todo-ignore-with-date
Some people view a TODO item that has been /scheduled/ for execution
or have a /deadline/ (see [[*Timestamps]]) as no longer /open/.
- Configure the variables ~org-agenda-todo-ignore-scheduled~,
- ~org-agenda-todo-ignore-deadlines~,
- ~org-agenda-todo-ignore-timestamp~ and/or
- ~org-agenda-todo-ignore-with-date~ to exclude such items from the
- global TODO list.
+ Configure the variables ~org-agenda-todo-ignore-scheduled~ to
+ exclude some or all scheduled items from the global TODO list,
+ ~org-agenda-todo-ignore-deadlines~ to exclude some or all items with
+ a deadline set, ~org-agenda-todo-ignore-timestamp~ to exclude some
+ or all items with an active timestamp other than a DEADLINE or
+ a SCHEDULED timestamp and/or ~org-agenda-todo-ignore-with-date~ to
+ exclude items with at least one active timestamp.
-
#+vindex: org-agenda-todo-list-sublevels
--
2.22.0
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #3: 0003-Fix-a-bug-when-org-agenda-todo-ignore-deadlines-is-s.patch --]
[-- Type: text/x-patch; size=914B, Size: 914 bytes --]
From 0bf1e6e6b25847ce4bee74487782b82c822138a3 Mon Sep 17 00:00:00 2001
From: Marcin Borkowski <mbork@mbork.pl>
Date: Mon, 24 Jun 2019 21:05:56 +0200
Subject: [PATCH] Fix a bug when `org-agenda-todo-ignore-deadlines` is set to t
---
lisp/org-agenda.el | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lisp/org-agenda.el b/lisp/org-agenda.el
index daf51601d..673c3c311 100644
--- a/lisp/org-agenda.el
+++ b/lisp/org-agenda.el
@@ -5444,7 +5444,7 @@ This function is invoked if `org-agenda-todo-ignore-deadlines',
(and org-agenda-todo-ignore-deadlines
(re-search-forward org-deadline-time-regexp end t)
(cond
- ((memq org-agenda-todo-ignore-deadlines '(t all)) t)
+ ((eq org-agenda-todo-ignore-deadlines 'all) t)
((eq org-agenda-todo-ignore-deadlines 'far)
(not (org-deadline-close-p (match-string 1))))
((eq org-agenda-todo-ignore-deadlines 'future)
--
2.22.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: Bug: Docstrings of =org-agenda-todo-ignore-with-date= and =org-agenda-todo-ignore-timestamp= unclear [9.2 (release_9.2-215-g5b39d8 @ /home/mbork/others-works/emacs/org-mode/lisp/)]
2019-06-24 19:09 ` Marcin Borkowski
@ 2019-07-07 7:51 ` Marcin Borkowski
2019-07-09 10:22 ` Nicolas Goaziou
0 siblings, 1 reply; 12+ messages in thread
From: Marcin Borkowski @ 2019-07-07 7:51 UTC (permalink / raw)
To: Nicolas Goaziou; +Cc: emacs-orgmode
Hello there,
any comments?
Best,
mb
On 2019-06-24, at 21:09, Marcin Borkowski <mbork@mbork.pl> wrote:
> On 2019-06-23, at 13:23, Marcin Borkowski <mbork@mbork.pl> wrote:
>
>> On 2019-06-19, at 21:19, Nicolas Goaziou <mail@nicolasgoaziou.fr> wrote:
>>
>>> Hello,
>>>
>>> Marcin Borkowski <mbork@mbork.pl> writes:
>>>
>>>> The docstrings of =org-agenda-todo-ignore-with-date= and
>>>> =org-agenda-todo-ignore-timestamp= do not really tell what these
>>>> settings are doing. They should (imho) emphasize that the former just
>>>> omits from the global todo list all entries with at least one active
>>>> timestamp (in the hedaline or the body), and the latter checks the first
>>>> active timestamp which is not SCHEDULED or DEADLINE and compares it to
>>>> today's date.
>>>>
>>>> It could also be mentioned in the manual.
>>>>
>>>> I have GNU papers for Emacs signed. If this is enough, I'd be happy to
>>>> submit a patch. If so, what would be better: to patch only the
>>>> docstrings or the docstrings and the manual.
>>>
>>> I would say the latter. Thank you.
>>
>> I only patched the doctsrings for now. If this is fine, I'll do
>> a similar thing in the manual.
>
> I attach two more patches: one for the manual and one for the (apparent)
> bug in `org-agenda-check-for-timestamp-as-reason-to-ignore-todo-item`.
>
> WDYT?
>
> --
> Marcin Borkowski
> http://mbork.pl
> From 89e1254d04802b13f4bd8c753287bce4eb1ae044 Mon Sep 17 00:00:00 2001
> From: Marcin Borkowski <mbork@mbork.pl>
> Date: Mon, 24 Jun 2019 21:05:32 +0200
> Subject: [PATCH] Explain the `org-agenda-todo-ignore-...` variables in the
> manual
>
> ---
> doc/org-manual.org | 12 +++++++-----
> 1 file changed, 7 insertions(+), 5 deletions(-)
>
> diff --git a/doc/org-manual.org b/doc/org-manual.org
> index 440888b4e..573c59b99 100644
> --- a/doc/org-manual.org
> +++ b/doc/org-manual.org
> @@ -8545,11 +8545,13 @@ it more compact:
> #+vindex: org-agenda-todo-ignore-with-date
> Some people view a TODO item that has been /scheduled/ for execution
> or have a /deadline/ (see [[*Timestamps]]) as no longer /open/.
> - Configure the variables ~org-agenda-todo-ignore-scheduled~,
> - ~org-agenda-todo-ignore-deadlines~,
> - ~org-agenda-todo-ignore-timestamp~ and/or
> - ~org-agenda-todo-ignore-with-date~ to exclude such items from the
> - global TODO list.
> + Configure the variables ~org-agenda-todo-ignore-scheduled~ to
> + exclude some or all scheduled items from the global TODO list,
> + ~org-agenda-todo-ignore-deadlines~ to exclude some or all items with
> + a deadline set, ~org-agenda-todo-ignore-timestamp~ to exclude some
> + or all items with an active timestamp other than a DEADLINE or
> + a SCHEDULED timestamp and/or ~org-agenda-todo-ignore-with-date~ to
> + exclude items with at least one active timestamp.
>
> -
> #+vindex: org-agenda-todo-list-sublevels
> --
> 2.22.0
>
> From 0bf1e6e6b25847ce4bee74487782b82c822138a3 Mon Sep 17 00:00:00 2001
> From: Marcin Borkowski <mbork@mbork.pl>
> Date: Mon, 24 Jun 2019 21:05:56 +0200
> Subject: [PATCH] Fix a bug when `org-agenda-todo-ignore-deadlines` is set to t
>
> ---
> lisp/org-agenda.el | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lisp/org-agenda.el b/lisp/org-agenda.el
> index daf51601d..673c3c311 100644
> --- a/lisp/org-agenda.el
> +++ b/lisp/org-agenda.el
> @@ -5444,7 +5444,7 @@ This function is invoked if `org-agenda-todo-ignore-deadlines',
> (and org-agenda-todo-ignore-deadlines
> (re-search-forward org-deadline-time-regexp end t)
> (cond
> - ((memq org-agenda-todo-ignore-deadlines '(t all)) t)
> + ((eq org-agenda-todo-ignore-deadlines 'all) t)
> ((eq org-agenda-todo-ignore-deadlines 'far)
> (not (org-deadline-close-p (match-string 1))))
> ((eq org-agenda-todo-ignore-deadlines 'future)
--
Marcin Borkowski
http://mbork.pl
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: Bug: Docstrings of =org-agenda-todo-ignore-with-date= and =org-agenda-todo-ignore-timestamp= unclear [9.2 (release_9.2-215-g5b39d8 @ /home/mbork/others-works/emacs/org-mode/lisp/)]
2019-07-07 7:51 ` Marcin Borkowski
@ 2019-07-09 10:22 ` Nicolas Goaziou
2019-07-11 19:06 ` Marcin Borkowski
0 siblings, 1 reply; 12+ messages in thread
From: Nicolas Goaziou @ 2019-07-09 10:22 UTC (permalink / raw)
To: Marcin Borkowski; +Cc: emacs-orgmode
Hello,
Marcin Borkowski <mbork@mbork.pl> writes:
> Hello there,
>
> any comments?
Thank you for the patches.
I applied the first one. Could you expound the commit message in the
second one: what bug is it?
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: Bug: Docstrings of =org-agenda-todo-ignore-with-date= and =org-agenda-todo-ignore-timestamp= unclear [9.2 (release_9.2-215-g5b39d8 @ /home/mbork/others-works/emacs/org-mode/lisp/)]
2019-07-09 10:22 ` Nicolas Goaziou
@ 2019-07-11 19:06 ` Marcin Borkowski
2019-07-12 8:53 ` Nicolas Goaziou
0 siblings, 1 reply; 12+ messages in thread
From: Marcin Borkowski @ 2019-07-11 19:06 UTC (permalink / raw)
To: Nicolas Goaziou; +Cc: emacs-orgmode
On 2019-07-09, at 12:22, Nicolas Goaziou <mail@nicolasgoaziou.fr> wrote:
> Hello,
>
> Marcin Borkowski <mbork@mbork.pl> writes:
>
>> Hello there,
>>
>> any comments?
>
> Thank you for the patches.
>
> I applied the first one. Could you expound the commit message in the
> second one: what bug is it?
Thanks. I meant that - according to the manual - `t' should be
synonymous with `near' and not `all'. I can fix the commit message, but
could you confirm that I'm right first?
Best,
--
Marcin Borkowski
http://mbork.pl
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: Bug: Docstrings of =org-agenda-todo-ignore-with-date= and =org-agenda-todo-ignore-timestamp= unclear [9.2 (release_9.2-215-g5b39d8 @ /home/mbork/others-works/emacs/org-mode/lisp/)]
2019-07-11 19:06 ` Marcin Borkowski
@ 2019-07-12 8:53 ` Nicolas Goaziou
2019-07-14 20:30 ` Marcin Borkowski
0 siblings, 1 reply; 12+ messages in thread
From: Nicolas Goaziou @ 2019-07-12 8:53 UTC (permalink / raw)
To: Marcin Borkowski; +Cc: emacs-orgmode
Hello,
Marcin Borkowski <mbork@mbork.pl> writes:
> Thanks. I meant that - according to the manual - `t' should be
> synonymous with `near' and not `all'. I can fix the commit message, but
> could you confirm that I'm right first?
AFAICT, your guess sounds right.
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: Bug: Docstrings of =org-agenda-todo-ignore-with-date= and =org-agenda-todo-ignore-timestamp= unclear [9.2 (release_9.2-215-g5b39d8 @ /home/mbork/others-works/emacs/org-mode/lisp/)]
2019-07-12 8:53 ` Nicolas Goaziou
@ 2019-07-14 20:30 ` Marcin Borkowski
2019-07-14 22:00 ` Nicolas Goaziou
0 siblings, 1 reply; 12+ messages in thread
From: Marcin Borkowski @ 2019-07-14 20:30 UTC (permalink / raw)
To: Nicolas Goaziou; +Cc: emacs-orgmode
[-- Attachment #1: Type: text/plain, Size: 426 bytes --]
On 2019-07-12, at 10:53, Nicolas Goaziou <mail@nicolasgoaziou.fr> wrote:
> Hello,
>
> Marcin Borkowski <mbork@mbork.pl> writes:
>
>> Thanks. I meant that - according to the manual - `t' should be
>> synonymous with `near' and not `all'. I can fix the commit message, but
>> could you confirm that I'm right first?
>
> AFAICT, your guess sounds right.
Thanks. How about this?
Best,
--
Marcin Borkowski
http://mbork.pl
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-Fix-a-bug-when-org-agenda-todo-ignore-deadlines-is-s.patch --]
[-- Type: text/x-patch; size=996B, Size: 996 bytes --]
From 669b4347f35dd87815e9f232ab72533d6514d7d1 Mon Sep 17 00:00:00 2001
From: Marcin Borkowski <mbork@mbork.pl>
Date: Mon, 24 Jun 2019 21:05:56 +0200
Subject: [PATCH] Fix a bug when `org-agenda-todo-ignore-deadlines` is set to t
Make `t' synonymous with `near' instead of `all' (as is stated in the
docstring).
---
lisp/org-agenda.el | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lisp/org-agenda.el b/lisp/org-agenda.el
index daf51601d..673c3c311 100644
--- a/lisp/org-agenda.el
+++ b/lisp/org-agenda.el
@@ -5444,7 +5444,7 @@ This function is invoked if `org-agenda-todo-ignore-deadlines',
(and org-agenda-todo-ignore-deadlines
(re-search-forward org-deadline-time-regexp end t)
(cond
- ((memq org-agenda-todo-ignore-deadlines '(t all)) t)
+ ((eq org-agenda-todo-ignore-deadlines 'all) t)
((eq org-agenda-todo-ignore-deadlines 'far)
(not (org-deadline-close-p (match-string 1))))
((eq org-agenda-todo-ignore-deadlines 'future)
--
2.22.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: Bug: Docstrings of =org-agenda-todo-ignore-with-date= and =org-agenda-todo-ignore-timestamp= unclear [9.2 (release_9.2-215-g5b39d8 @ /home/mbork/others-works/emacs/org-mode/lisp/)]
2019-07-14 20:30 ` Marcin Borkowski
@ 2019-07-14 22:00 ` Nicolas Goaziou
0 siblings, 0 replies; 12+ messages in thread
From: Nicolas Goaziou @ 2019-07-14 22:00 UTC (permalink / raw)
To: Marcin Borkowski; +Cc: emacs-orgmode
Hello,
Marcin Borkowski <mbork@mbork.pl> writes:
> Thanks. How about this?
Applied. Thank you.
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2019-07-14 22:01 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-06-18 6:55 Bug: Docstrings of =org-agenda-todo-ignore-with-date= and =org-agenda-todo-ignore-timestamp= unclear [9.2 (release_9.2-215-g5b39d8 @ /home/mbork/others-works/emacs/org-mode/lisp/)] Marcin Borkowski
2019-06-19 19:19 ` Nicolas Goaziou
2019-06-23 11:23 ` Marcin Borkowski
2019-06-23 19:29 ` Samuel Wales
2019-06-24 19:08 ` Marcin Borkowski
2019-06-24 19:09 ` Marcin Borkowski
2019-07-07 7:51 ` Marcin Borkowski
2019-07-09 10:22 ` Nicolas Goaziou
2019-07-11 19:06 ` Marcin Borkowski
2019-07-12 8:53 ` Nicolas Goaziou
2019-07-14 20:30 ` Marcin Borkowski
2019-07-14 22:00 ` Nicolas Goaziou
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).