Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ReverseAdoc creates duplicated anchors #102

Open
ronaldtse opened this issue Jun 7, 2024 · 2 comments
Open

ReverseAdoc creates duplicated anchors #102

ronaldtse opened this issue Jun 7, 2024 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@ronaldtse
Copy link
Contributor

ronaldtse commented Jun 7, 2024

For document 002:

$ bundle exec reverse_adoc -rcoradoc/reverse_adoc/plugins/plateau \
  --split-sections 2 --external-images \
  -o sources/002-v4/document.adoc \
  reference-docs/plateau-document-02-v4.html
$ bundle exec metanorma site generate
...
Anchor toc_02 has already been used at line 36
Anchor tocF_02 has already been used at line 9707
Anchor tocT_02 has already been used at line 19690
<aborted>

Relevant snippets:

sources/002-v4/sections/appendix-F/section-02.adoc:

[[tocF_02]]
=== 徒歩道の記述とLOD

[[]]
==== 交通(徒歩道)モデル

徒歩道とは、公共測量標準図式における徒歩道及び庭園路(ただし、庭園路のうち、自動車ターミナル内の道路は、広場として取得するため、徒歩道には含まない)をいう。
...

sources/002-v4/sections/appendix-F/section-04.adoc:

[[tocF_02]]
=== 作業上の留意事項

 道路と徒歩道の接合部の表現

道路と徒歩道(tran:Track)の接合する場所を区切らない。
...
@ronaldtse ronaldtse added the bug Something isn't working label Jun 7, 2024
ronaldtse added a commit to metanorma/mn-samples-plateau that referenced this issue Jun 7, 2024
@hmdne
Copy link
Contributor

hmdne commented Jun 7, 2024

This originates from the incoming document. What would be the best behavior in your opinion? I think the current behavior is fine, because a writer can later debug duplicated IDs and fix them by hand.

ronaldtse added a commit to metanorma/mn-samples-plateau that referenced this issue Jun 10, 2024
ronaldtse added a commit to metanorma/mn-samples-plateau that referenced this issue Jun 10, 2024
@ronaldtse
Copy link
Contributor Author

This originates from the incoming document. What would be the best behavior in your opinion? I think the current behavior is fine, because a writer can later debug duplicated IDs and fix them by hand.

I guess this shows the original document is problematic. However, Coradoc needs to have verification methods of its own. It needs to know the document is invalid.

At least it should provide warnings when there are duplicated anchors detected at a (future) verification step.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: 🏔 High priority
Development

No branches or pull requests

2 participants