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

Missing Delimiters if Organization Field Contains Multiple Entries for @misc #218

Open
lukeflo opened this issue Aug 28, 2023 · 3 comments
Assignees
Labels
fixed in dev Issue fixed in dev
Milestone

Comments

@lukeflo
Copy link

lukeflo commented Aug 28, 2023

As mentioned on TeX.SE, the delimiters are missing if an @misc contains multiple entries in the organization field. This may be due to line 548 in apa.bbx where the list format for misc is redefined with the following line: \DeclareListFormat[misc]{organization}{\mkbibemph{#1}}.

Thus, the usual listcount test is overwritten respectively deleted. You shall insert something like:

\DeclareListFormat[misc]{organization}{%
\mkbibemph{#1%
\ifthenelse{\value{listcount}<\value{liststop}}
{\multilistdelimiter}
{}}}

Or even some more sophisticated code which decides, if a multilistdelimiter or a finallistdelimiter should be inserted.

plk added a commit that referenced this issue Aug 29, 2023
@plk
Copy link
Owner

plk commented Aug 29, 2023

Should be fixed in the next release.

@lukeflo
Copy link
Author

lukeflo commented Aug 30, 2023

Should be fixed in the next release.

Just looked at your commit. In line 555, you use DeclareDelimFormat with the entryfield organization as context argument. AFAIK the optional context argument for this command can't be an entryfield, but has to be something like bib, biblist or a cite-command (Manual p. 129 ff.).

But since you're also one of the biblatex authors, I may be wrong. If thats the case, just forget my comment here... ;)

plk added a commit that referenced this issue Aug 30, 2023
@plk
Copy link
Owner

plk commented Aug 30, 2023

The context is basically arbitrary and is determined by \printdelim but you were right to note that that context wasn't being established anyway ... should be fixed now.

@plk plk added the fixed in dev Issue fixed in dev label Sep 2, 2023
@plk plk self-assigned this Sep 2, 2023
@plk plk added this to the 9.17 milestone Sep 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed in dev Issue fixed in dev
Projects
None yet
Development

No branches or pull requests

2 participants