Re: [Orgmode] Minor bug: org-agenda-holidays

[Noorul Islam K M]

Paul Sexton <address@hidden> writes:

> Yes.
> "Wrong type argument: commandp, list-calendar-holidays"
>
>

I think the following patch will fix the issue for you.

Log
[[[

Make org-agenda-holidays compatible with older versions of emacs.

* lisp/org-agenda.el (org-agenda-holidays): Make this function compatible
  with older versions of emacs.

]]]

Thanks and Regards
Noorul

diff --git a/lisp/org-agenda.el b/lisp/org-agenda.el
index 19535b4..217c701 100644
--- a/lisp/org-agenda.el
+++ b/lisp/org-agenda.el
@@ -7685,7 +7685,9 @@ the cursor position."
 (defun org-agenda-holidays ()
   "Display the holidays for the 3 months around the cursor date."
   (interactive)
-  (org-agenda-execute-calendar-command 'list-calendar-holidays))
+  (if (fboundp 'list-calendar-holidays)
+      (org-agenda-execute-calendar-command 'list-calendar-holidays)
+    (org-agenda-execute-calendar-command 'calendar-list-holidays)))
 
 (defvar calendar-longitude)
 (defvar calendar-latitude)

> On 22/12/2010 8:11 p.m., Noorul Islam K M wrote:
>> Paul Sexton<address@hidden>  writes:
>>
>>> In org-agenda.el, the function org-agenda-holidays is coded as:
>>>
>>> (defun org-agenda-holidays ()
>>>    "Display the holidays for the 3 months around the cursor date."
>>>    (interactive)
>>>    (org-agenda-execute-calendar-command 'list-calendar-holidays))
>>>
>>> 'list-calendar-holidays' does not exist. It should be
>>> 'calendar-list-holidays'.
>>>
>> Did you  get any error when you executed this function?
>>
>> Thanks and Regards
>> Noorul