{"action":"created","issue":{"url":"https://api.github.com/repos/KWARC/LaTeXML-Plugin-sTeX/issues/53","repository_url":"https://api.github.com/repos/KWARC/LaTeXML-Plugin-sTeX","labels_url":"https://api.github.com/repos/KWARC/LaTeXML-Plugin-sTeX/issues/53/labels{/name}","comments_url":"https://api.github.com/repos/KWARC/LaTeXML-Plugin-sTeX/issues/53/comments","events_url":"https://api.github.com/repos/KWARC/LaTeXML-Plugin-sTeX/issues/53/events","html_url":"https://github.com/KWARC/LaTeXML-Plugin-sTeX/issues/53","id":131313712,"number":53,"title":"for attribute for n-th (n>1) definition wrong","user":{"login":"kohlhase","id":4037538,"avatar_url":"https://avatars.githubusercontent.com/u/4037538?v=3","gravatar_id":"","url":"https://api.github.com/users/kohlhase","html_url":"https://github.com/kohlhase","followers_url":"https://api.github.com/users/kohlhase/followers","following_url":"https://api.github.com/users/kohlhase/following{/other_user}","gists_url":"https://api.github.com/users/kohlhase/gists{/gist_id}","starred_url":"https://api.github.com/users/kohlhase/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kohlhase/subscriptions","organizations_url":"https://api.github.com/users/kohlhase/orgs","repos_url":"https://api.github.com/users/kohlhase/repos","events_url":"https://api.github.com/users/kohlhase/events{/privacy}","received_events_url":"https://api.github.com/users/kohlhase/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/KWARC/LaTeXML-Plugin-sTeX/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/KWARC/LaTeXML-Plugin-sTeX/labels/Priority","name":"Priority","color":"fbca04"}],"state":"open","locked":false,"assignee":{"login":"angerhang","id":7655454,"avatar_url":"https://avatars.githubusercontent.com/u/7655454?v=3","gravatar_id":"","url":"https://api.github.com/users/angerhang","html_url":"https://github.com/angerhang","followers_url":"https://api.github.com/users/angerhang/followers","following_url":"https://api.github.com/users/angerhang/following{/other_user}","gists_url":"https://api.github.com/users/angerhang/gists{/gist_id}","starred_url":"https://api.github.com/users/angerhang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/angerhang/subscriptions","organizations_url":"https://api.github.com/users/angerhang/orgs","repos_url":"https://api.github.com/users/angerhang/repos","events_url":"https://api.github.com/users/angerhang/events{/privacy}","received_events_url":"https://api.github.com/users/angerhang/received_events","type":"User","site_admin":false},"milestone":null,"comments":12,"created_at":"2016-02-04T11:04:05Z","updated_at":"2016-02-05T15:07:28Z","closed_at":null,"body":"The ```for``` attribute of a ```definition``` element should reference all the symbols defined in this definition (and only these). They should be generated by the ```\\defi*``` (and friends) in the definition body. \r\n\r\nUnfortunately, the sTeX plugin generates these wrong, for second and later definitions it also includes the symbols from the previous definitions. For instance in ```graphs/source/graphregular.en.tex``` we see \r\n```LaTeX\r\n\\begin{definition}\r\n  If in a graph $G$ all the vertices have the same degree, say $k$, then $G$ is called\r\n  \\defi[regular]{$k$-regular}, or simply \\defi{regular}. \r\n\\end{definition}\r\n\\begin{definition}\r\n  A \\atrefi{3-regular}{regular} graph is called \\defii{cubic}{graph}.\r\n\\end{definition}\r\n```\r\nAnd this generates\r\n```\r\n    <omdoc:definition for=\"graphregular?regular\" xml:id=\"graphregular.en.definition3\"\r\n...\r\n   <omdoc:definition for=\"graphregular?cubic-graph graphregular?regular\"\r\n```\r\nThe reference to  ```graphregular?regular``` is spurious and should not be there. I am sure that this is relatively easy to fix. "},"comment":{"url":"https://api.github.com/repos/KWARC/LaTeXML-Plugin-sTeX/issues/comments/180393644","html_url":"https://github.com/KWARC/LaTeXML-Plugin-sTeX/issues/53#issuecomment-180393644","issue_url":"https://api.github.com/repos/KWARC/LaTeXML-Plugin-sTeX/issues/53","id":180393644,"user":{"login":"dginev","id":348975,"avatar_url":"https://avatars.githubusercontent.com/u/348975?v=3","gravatar_id":"","url":"https://api.github.com/users/dginev","html_url":"https://github.com/dginev","followers_url":"https://api.github.com/users/dginev/followers","following_url":"https://api.github.com/users/dginev/following{/other_user}","gists_url":"https://api.github.com/users/dginev/gists{/gist_id}","starred_url":"https://api.github.com/users/dginev/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dginev/subscriptions","organizations_url":"https://api.github.com/users/dginev/orgs","repos_url":"https://api.github.com/users/dginev/repos","events_url":"https://api.github.com/users/dginev/events{/privacy}","received_events_url":"https://api.github.com/users/dginev/received_events","type":"User","site_admin":false},"created_at":"2016-02-05T15:07:28Z","updated_at":"2016-02-05T15:07:28Z","body":"The reason for that code block has been to record all defined symbols in a definition, as there can be multiple.\r\n\r\nWhat I wonder from Michael's example is whether the \\atrefi doesn't add to the for attribute by accident."}}