alx/mu4e
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

changed from before/setq to around/let to limit scope

Browse Source 
let is better than setq, as it doesn't permanently alter the value of
org-overriding-default-time. The setq has knock-on impacts in future
captures that use the value of this variable, if it is set.
This commit is contained in:
plantarum
2021-01-26 13:52:43 -05:00
parent 51f0a6b739
commit a440510cdd
1 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
mu4e/mu4e.texi
View File

@ -3453,12 +3453,13 @@ You can extract the event time directly, and have the @code{org-capture}
functions use that to set the @code{datetree} location: functions use that to set the @code{datetree} location:
@lisp @lisp
(defun my-catch-event-time (event reply-status) (defun my-catch-event-time (orig-fun &rest args)
"Set org-overriding-default-time to the start time of the capture event" "Set org-overriding-default-time to the start time of the capture event"
(setq org-overriding-default-time (let ((org-overriding-default-time (date-to-time
(date-to-time (gnus-icalendar-event:start event)))) (gnus-icalendar-event:start (car args)))))
(apply orig-fun args)))
(advice-add 'gnus-icalendar:org-event-save :before #'my-catch-event-time) (advice-add 'gnus-icalendar:org-event-save :around #'my-catch-event-time)
@end lisp @end lisp
If you do this, you'll want to omit the @code{:timeprompt t} setting If you do this, you'll want to omit the @code{:timeprompt t} setting