emacs-orgmode
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [O] [PATCH] Prevent org-rss-headline dropping lower level headlines


From: Arun Isaac
Subject: Re: [O] [PATCH] Prevent org-rss-headline dropping lower level headlines
Date: Thu, 12 May 2016 09:36:09 +0530
User-agent: mu4e 0.9.17; emacs 24.5.1

> I mean that the HTML back-end adds data that may not be useful for the
> RSS back-end.

Yes, things like the <div id=...> tags are not useful. But, technically,
they aren't wrong either. And, the alternative is to modify ox-rss so it
depends less on ox-html. That is a bigger modification, not within the
scope of this patch, I think.

> My remark about `org-export-data-with-backend' is orthogonal to the
> output of the function. It is usually better because it doesn't depend
> on the name of the translators in HTML back-end.

I have made this modification.

> Could you send an updated patch? Thank you.

Please find attached an updated patch.

Thank you.

>From a8fd0b0f4eca51a38a432ad942b695b9e565d4fe Mon Sep 17 00:00:00 2001
From: Arun Isaac <address@hidden>
Date: Thu, 12 May 2016 09:21:18 +0530
Subject: [PATCH] ox-rss: Prevent dropping lower level headlines

* contrib/lisp/ox-rss.el (org-rss-headline): Transcode lower level
  headlines (headlines with level > 1) using the html backend.  Do not
  drop them.

Previously, lower level headlines were dropped and did not appear in the
exported RSS output.
---
 contrib/lisp/ox-rss.el | 106 ++++++++++++++++++++++++-------------------------
 1 file changed, 53 insertions(+), 53 deletions(-)

diff --git a/contrib/lisp/ox-rss.el b/contrib/lisp/ox-rss.el
index 39fce30..0c4a2f2 100644
--- a/contrib/lisp/ox-rss.el
+++ b/contrib/lisp/ox-rss.el
@@ -228,59 +228,59 @@ Return output file name."
   "Transcode HEADLINE element into RSS format.
 CONTENTS is the headline contents.  INFO is a plist used as a
 communication channel."
-  (unless (or (org-element-property :footnote-section-p headline)
-             ;; Only consider first-level headlines
-             (> (org-export-get-relative-level headline info) 1))
-    (let* ((author (and (plist-get info :with-author)
-                       (let ((auth (plist-get info :author)))
-                         (and auth (org-export-data auth info)))))
-          (htmlext (plist-get info :html-extension))
-          (hl-number (org-export-get-headline-number headline info))
-          (hl-home (file-name-as-directory (plist-get info :html-link-home)))
-          (hl-pdir (plist-get info :publishing-directory))
-          (hl-perm (org-element-property :RSS_PERMALINK headline))
-          (anchor (org-export-get-reference headline info))
-          (category (org-rss-plain-text
-                     (or (org-element-property :CATEGORY headline) "") info))
-          (pubdate0 (org-element-property :PUBDATE headline))
-          (pubdate (let ((system-time-locale "C"))
-                     (if pubdate0
-                         (format-time-string
-                          "%a, %d %b %Y %H:%M:%S %z"
-                          (org-time-string-to-time pubdate0)))))
-          (title (or (org-element-property :RSS_TITLE headline)
-                     (replace-regexp-in-string
-                      org-bracket-link-regexp
-                      (lambda (m) (or (match-string 3 m)
-                                 (match-string 1 m)))
-                      (org-element-property :raw-value headline))))
-          (publink
-           (or (and hl-perm (concat (or hl-home hl-pdir) hl-perm))
-               (concat
-                (or hl-home hl-pdir)
-                (file-name-nondirectory
-                 (file-name-sans-extension
-                  (plist-get info :input-file))) "." htmlext "#" anchor)))
-          (guid (if org-rss-use-entry-url-as-guid
-                    publink
-                  (org-rss-plain-text
-                   (or (org-element-property :ID headline)
-                       (org-element-property :CUSTOM_ID headline)
-                       publink)
-                   info))))
-      (if (not pubdate0) "" ;; Skip entries with no PUBDATE prop
-       (format
-        (concat
-         "<item>\n"
-         "<title>%s</title>\n"
-         "<link>%s</link>\n"
-         "<author>%s</author>\n"
-         "<guid isPermaLink=\"false\">%s</guid>\n"
-         "<pubDate>%s</pubDate>\n"
-         (org-rss-build-categories headline info) "\n"
-         "<description><![CDATA[%s]]></description>\n"
-         "</item>\n")
-        title publink author guid pubdate contents)))))
+  (if (> (org-export-get-relative-level headline info) 1)
+      (org-export-data-with-backend headline 'html info)
+    (unless (org-element-property :footnote-section-p headline)
+      (let* ((author (and (plist-get info :with-author)
+                         (let ((auth (plist-get info :author)))
+                           (and auth (org-export-data auth info)))))
+            (htmlext (plist-get info :html-extension))
+            (hl-number (org-export-get-headline-number headline info))
+            (hl-home (file-name-as-directory (plist-get info :html-link-home)))
+            (hl-pdir (plist-get info :publishing-directory))
+            (hl-perm (org-element-property :RSS_PERMALINK headline))
+            (anchor (org-export-get-reference headline info))
+            (category (org-rss-plain-text
+                       (or (org-element-property :CATEGORY headline) "") info))
+            (pubdate0 (org-element-property :PUBDATE headline))
+            (pubdate (let ((system-time-locale "C"))
+                       (if pubdate0
+                           (format-time-string
+                            "%a, %d %b %Y %H:%M:%S %z"
+                            (org-time-string-to-time pubdate0)))))
+            (title (or (org-element-property :RSS_TITLE headline)
+                       (replace-regexp-in-string
+                        org-bracket-link-regexp
+                        (lambda (m) (or (match-string 3 m)
+                                        (match-string 1 m)))
+                        (org-element-property :raw-value headline))))
+            (publink
+             (or (and hl-perm (concat (or hl-home hl-pdir) hl-perm))
+                 (concat
+                  (or hl-home hl-pdir)
+                  (file-name-nondirectory
+                   (file-name-sans-extension
+                    (plist-get info :input-file))) "." htmlext "#" anchor)))
+            (guid (if org-rss-use-entry-url-as-guid
+                      publink
+                    (org-rss-plain-text
+                     (or (org-element-property :ID headline)
+                         (org-element-property :CUSTOM_ID headline)
+                         publink)
+                     info))))
+       (if (not pubdate0) "" ;; Skip entries with no PUBDATE prop
+         (format
+          (concat
+           "<item>\n"
+           "<title>%s</title>\n"
+           "<link>%s</link>\n"
+           "<author>%s</author>\n"
+           "<guid isPermaLink=\"false\">%s</guid>\n"
+           "<pubDate>%s</pubDate>\n"
+           (org-rss-build-categories headline info) "\n"
+           "<description><![CDATA[%s]]></description>\n"
+           "</item>\n")
+          title publink author guid pubdate contents))))))
 
 (defun org-rss-build-categories (headline info)
   "Build categories for the RSS item."
-- 
2.8.2


reply via email to

[Prev in Thread] Current Thread [Next in Thread]