[PATCH] Fix CUSTOM_ID completion for PROPERTY drawers

[PATCH] Fix CUSTOM_ID completion for PROPERTY drawers

[Bernt Hansen]

The wrong name was added for CUSTOM_ID in the completion list for
properties in commit 931e13d (Custom IDs: Documentation and
improvements)
---

This patch is available at git://git.norang.ca/org-mode for-carsten

 lisp/org.el |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/lisp/org.el b/lisp/org.el
index 3c95baa..41e0e48 100644
--- a/lisp/org.el
+++ b/lisp/org.el
@@ -11148,7 +11148,7 @@ These are properties that are not defined in the property drawer,
 but in some other way.")
 
 (defconst org-default-properties
-  '("ARCHIVE" "CATEGORY" "SUMMARY" "DESCRIPTION" "CUSTOM_LINK"
+  '("ARCHIVE" "CATEGORY" "SUMMARY" "DESCRIPTION" "CUSTOM_ID"
     "LOCATION" "LOGGING" "COLUMNS" "VISIBILITY"
     "TABLE_EXPORT_FORMAT" "TABLE_EXPORT_FILE"
     "EXPORT_FILE_NAME" "EXPORT_TITLE" "EXPORT_AUTHOR" "EXPORT_DATE"
-- 
1.6.3.rc1

[PATCH] Fix section id on HTML export when no custom id is present

[Bernt Hansen]

This fixes a bug where section ids were all set to 'nil'.
suffix and href were identical so suffix is no more.

Before this change the table of contents had the correct id but the
heading levels all had id="nil"
---

This patch is available at git://git.norang.ca/org-mode for-carsten

 lisp/org-html.el |    9 ++++-----
 1 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/lisp/org-html.el b/lisp/org-html.el
index 0f3d113..0c416af 100644
--- a/lisp/org-html.el
+++ b/lisp/org-html.el
@@ -1817,7 +1817,7 @@ When TITLE is nil, just close all open levels."
 	 (preferred (cdr (assoc target org-export-preferred-target-alist)))
 	 (remove (or preferred target))
 	 (l org-level-max)
-	 snumber href suffix)
+	 snumber href)
     (setq extra-targets (remove remove extra-targets))
     (setq extra-targets
 	  (mapconcat (lambda (x)
@@ -1874,12 +1874,11 @@ When TITLE is nil, just close all open levels."
 				 level snumber)
 			 " " title)))
 	(unless (= head-count 1) (insert "\n</div>\n"))
-	(setq href (cdr (assoc (concat "sec-" snumber) org-export-preferred-target-alist)))
-	(setq suffix (or href (concat "sec-" snumber)))
+	(setq href (or (cdr (assoc (concat "sec-" snumber) org-export-preferred-target-alist)) (concat "sec-" snumber)))
 	(insert (format "\n<div id=\"outline-container-%s\" class=\"outline-%d\">\n<h%d id=\"%s\">%s%s</h%d>\n<div class=\"outline-text-%d\" id=\"text-%s\">\n"
-			suffix level level href
+			href level level href
 			extra-targets
-			title level level suffix))
+			title level level href))
 	(org-open-par)))))
 
 (defun org-export-html-get-tag-class-name (tag)
-- 
1.6.3.rc1

Re: [PATCH] Fix CUSTOM_ID completion for PROPERTY drawers

[Carsten Dominik]

Applied, thanks.

- Carsten

On Apr 19, 2009, at 6:29 PM, Bernt Hansen wrote:

> The wrong name was added for CUSTOM_ID in the completion list for
> properties in commit 931e13d (Custom IDs: Documentation and
> improvements)
> ---
>
> This patch is available at git://git.norang.ca/org-mode for-carsten
>
> lisp/org.el |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/lisp/org.el b/lisp/org.el
> index 3c95baa..41e0e48 100644
> --- a/lisp/org.el
> +++ b/lisp/org.el
> @@ -11148,7 +11148,7 @@ These are properties that are not defined in  
> the property drawer,
> but in some other way.")
>
> (defconst org-default-properties
> -  '("ARCHIVE" "CATEGORY" "SUMMARY" "DESCRIPTION" "CUSTOM_LINK"
> +  '("ARCHIVE" "CATEGORY" "SUMMARY" "DESCRIPTION" "CUSTOM_ID"
>     "LOCATION" "LOGGING" "COLUMNS" "VISIBILITY"
>     "TABLE_EXPORT_FORMAT" "TABLE_EXPORT_FILE"
>     "EXPORT_FILE_NAME" "EXPORT_TITLE" "EXPORT_AUTHOR" "EXPORT_DATE"
> -- 
> 1.6.3.rc1
>
>
>
> _______________________________________________
> Emacs-orgmode mailing list
> Remember: use `Reply All' to send replies to the list.
> Emacs-orgmode@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-orgmode

Re: [PATCH] Fix section id on HTML export when no custom id is present

[Carsten Dominik]

Yes, a bug, but your fix is not the right one, see the discussions  
with Sebastian about the naming of containers and IDs.

Fixed, in a different way.

Thanks.

- Carsten

On Apr 19, 2009, at 6:29 PM, Bernt Hansen wrote:

> This fixes a bug where section ids were all set to 'nil'.
> suffix and href were identical so suffix is no more.
>
> Before this change the table of contents had the correct id but the
> heading levels all had id="nil"
> ---
>
> This patch is available at git://git.norang.ca/org-mode for-carsten
>
> lisp/org-html.el |    9 ++++-----
> 1 files changed, 4 insertions(+), 5 deletions(-)
>
> diff --git a/lisp/org-html.el b/lisp/org-html.el
> index 0f3d113..0c416af 100644
> --- a/lisp/org-html.el
> +++ b/lisp/org-html.el
> @@ -1817,7 +1817,7 @@ When TITLE is nil, just close all open levels."
> 	 (preferred (cdr (assoc target org-export-preferred-target-alist)))
> 	 (remove (or preferred target))
> 	 (l org-level-max)
> -	 snumber href suffix)
> +	 snumber href)
>     (setq extra-targets (remove remove extra-targets))
>     (setq extra-targets
> 	  (mapconcat (lambda (x)
> @@ -1874,12 +1874,11 @@ When TITLE is nil, just close all open  
> levels."
> 				 level snumber)
> 			 " " title)))
> 	(unless (= head-count 1) (insert "\n</div>\n"))
> -	(setq href (cdr (assoc (concat "sec-" snumber) org-export- 
> preferred-target-alist)))
> -	(setq suffix (or href (concat "sec-" snumber)))
> +	(setq href (or (cdr (assoc (concat "sec-" snumber) org-export- 
> preferred-target-alist)) (concat "sec-" snumber)))
> 	(insert (format "\n<div id=\"outline-container-%s\" class=\"outline- 
> %d\">\n<h%d id=\"%s\">%s%s</h%d>\n<div class=\"outline-text-%d\" id= 
> \"text-%s\">\n"
> -			suffix level level href
> +			href level level href
> 			extra-targets
> -			title level level suffix))
> +			title level level href))
> 	(org-open-par)))))
>
> (defun org-export-html-get-tag-class-name (tag)
> -- 
> 1.6.3.rc1
>
>
>
> _______________________________________________
> Emacs-orgmode mailing list
> Remember: use `Reply All' to send replies to the list.
> Emacs-orgmode@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-orgmode

Re: [PATCH] Fix section id on HTML export when no custom id is present

[Bernt Hansen]

Carsten Dominik <carsten.dominik@gmail.com> writes:

> Yes, a bug, but your fix is not the right one, see the discussions
> with Sebastian about the naming of containers and IDs.
>
> Fixed, in a different way.
>
> Thanks.

Thanks for fixing it :)

-Bernt