-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Phantom parentheses appear in non-parenthetical (text-style) citations when converting from org-mode (org-cite) #8509
Comments
This is probably related to the implementation in #7329, so paging @Carlos-M-D, @jgm, and @bdarcus, in case any of you have any ideas. |
I just ran the command again with [ Citation
{ citationId = "dominici2014overview"
, citationPrefix = []
, citationSuffix = []
, citationMode = AuthorInText
, citationNoteNum = 0
, citationHash = 0
}
, Citation
{ citationId = "krewinkel2017formatting"
, citationPrefix = []
, citationSuffix = []
, citationMode = NormalCitation
, citationNoteNum = 0
, citationHash = 0
}
, Citation
{ citationId = "macfarlanejgm"
, citationPrefix = [ Str "and" ]
, citationSuffix = []
, citationMode = NormalCitation
, citationNoteNum = 0
, citationHash = 0
}
] |
Looks like the problem is here, if I'm not mistaken. The |
(That is, not just the first author. Fixes jgm#8509.)
Pull request submitted! I hope I did that right. |
Explain the problem.
When using org-mode citations, with the org-cite syntax and the
cite/t
style, parentheses are correctly removed, but from the first author only. Subsequent authors, and the associated dates of publications, are still in parentheses.Normal
cite:@...
parenthetical styles still work. Single-author and no-date cite/t styles work. It's just multiple-authorcite/t
styles that are broken.Inputs:
refs.bib:
Command:
Output:
Dominici (2014; Krewinkel and Winkler 2017; and MacFarlane et al., n.d.)
Expected Output:
Dominici 2014; Krewinkel and Winkler 2017; and MacFarlane et al., n.d.
I.e., no parentheses.
Currently I'm working around this by just adding individual
cite/t
declarations around each one, like this:[cite/t:@dominici2014overview] [cite/t:@krewinkel2017formatting] and [cite/t:@macfarlanejgm]
but that's not ideal.Pandoc version?
What version of pandoc are you using, on what OS? (If it's not the latest release, please try with the latest release before reporting the issue.)
2.19.2 on NixOS. But I also tried this in Try Pandoc, with the same results.
The text was updated successfully, but these errors were encountered: