Subject: [PATCH] Re: bug#12718: 24.2.50; ODT export (old) fails with wrong-type-argument

From: Jambunathan K <[email-masked]>
Date: Wed, 24 Oct 2012 12:55:53 +0530

Jambunathan K <[email-masked]> writes:

> When I try to export to ODT (without invoking the new exporter or
> anything), I get:
>
> OpenDocument export failed: Wrong type argument: stringp,
> (org-export-odt-preprocess-latex-fragments)

This is an `unforced error' from previous merge from Org.  I am
attaching a patch.

Here is a patch.  This is a user-facing error and gets in the way of
export.  So please clear this patch, asap.

=== modified file 'lisp/org/ChangeLog'
--- lisp/org/ChangeLog	2012-10-05 05:57:24 +0000
+++ lisp/org/ChangeLog	2012-10-24 07:20:00 +0000
@@ -1,3 +1,8 @@
+2012-10-24  Jambunathan K  <[email-masked]>
+
+	* org-lparse.el: Move some hooks from `org-lparse' to
+	`org-do-lparse (Bug#12718).
+
 2012-09-30  Abdó Roig-Maranges  <[email-masked]>
 
 	* org-html.el (org-export-html-preprocess)

=== modified file 'lisp/org/org-lparse.el'
--- lisp/org/org-lparse.el	2012-09-30 15:14:59 +0000
+++ lisp/org/org-lparse.el	2012-10-24 07:02:52 +0000
@@ -435,10 +435,6 @@ PUB-DIR specifies the publishing directo
   (let* ((org-lparse-backend (intern native-backend))
 	 (org-lparse-other-backend (and target-backend
 					(intern target-backend))))
-    (add-hook 'org-export-preprocess-hook
-	      'org-lparse-strip-experimental-blocks-maybe)
-    (add-hook 'org-export-preprocess-after-blockquote-hook
-	      'org-lparse-preprocess-after-blockquote)
     (unless (org-lparse-backend-is-native-p native-backend)
       (error "Don't know how to export natively to backend %s" native-backend))
 
@@ -446,12 +442,7 @@ PUB-DIR specifies the publishing directo
 		(org-lparse-reachable-p native-backend target-backend))
       (error "Don't know how to export to backend %s %s" target-backend
 	     (format "via %s" native-backend)))
-    (run-hooks 'org-export-first-hook)
-    (org-do-lparse arg hidden ext-plist to-buffer body-only pub-dir)
-    (remove-hook 'org-export-preprocess-hook
-		 'org-lparse-strip-experimental-blocks-maybe)
-    (remove-hook 'org-export-preprocess-after-blockquote-hook
-		 'org-lparse-preprocess-after-blockquote)))
+    (org-do-lparse arg hidden ext-plist to-buffer body-only pub-dir)))
 
 (defcustom org-lparse-use-flashy-warning nil
   "Control flashing of messages logged with `org-lparse-warn'.
@@ -594,6 +585,16 @@ version."
   (setq-default org-deadline-line-regexp org-deadline-line-regexp)
   (setq-default org-done-keywords org-done-keywords)
   (setq-default org-maybe-keyword-time-regexp org-maybe-keyword-time-regexp)
+
+  ;; Run first hook.
+  (run-hooks 'org-export-first-hook)
+
+  ;; Add pre-process hooks.
+  (add-hook 'org-export-preprocess-hook
+	    'org-lparse-strip-experimental-blocks-maybe)
+  (add-hook 'org-export-preprocess-after-blockquote-hook
+	    'org-lparse-preprocess-after-blockquote)
+
   (let* (hfy-user-sheet-assoc		; let `htmlfontify' know that
 					; we are interested in
 					; collecting styles
@@ -785,6 +786,11 @@ version."
 	 org-lparse-toc
 	 href
 	 )
+    ;; Remove pre-process hooks.
+    (remove-hook 'org-export-preprocess-hook
+		 'org-lparse-strip-experimental-blocks-maybe)
+    (remove-hook 'org-export-preprocess-after-blockquote-hook
+		 'org-lparse-preprocess-after-blockquote)
 
     (let ((inhibit-read-only t))
       (org-unmodified

> In GNU Emacs 24.2.50.2 (i686-pc-linux-gnu, X toolkit, Xaw3d scroll bars)
>  of 2012-10-24 on debian-6.05
> Bzr revision: 110628 [email-masked]
> Windowing system distributor `The X.Org Foundation', version 11.0.10707000
> Important settings:
>   value of $LANG: en_IN
>   locale-coding-system: iso-latin-1-unix
>   default enable-multibyte-characters: t