Adding italics in CSL introduces a random comma?

megvmegwork

I'm editing the "report" type for AMA style. This is what I have so far:

<else-if type="report" match="any">
<group prefix=" " delimiter=" ">
<text macro="editor"/>
<text variable="container-title" text-case="title" font-style="italic" suffix=","/>
<text variable="collection-title" font-style="italic" suffix=","/>
<group>
<text macro="publisher"/>
<group prefix=", ">
<date variable="issued">
<date-part name="year" suffix=","/>
</date>
<text variable="genre" font-style="italic" prefix=", "/>
<text variable="number" prefix=", No. "/>

It works fine, and gives me this:

Bongaarts J, Completing the fertility transition in the developing world: the role of educational differences and fertility preferences, New York: Population Council, 2003, Policy Research Division Working Papers, No. 177.

However, I would like "Policy Research Division Working Papers" (in the genre field) to be italicized, so I do this:

<else-if type="report" match="any">
<group prefix=" " delimiter=" ">
<text macro="editor"/>
<text variable="container-title" text-case="title" font-style="italic" suffix=","/>
<text variable="collection-title" font-style="italic" suffix=","/>
<group>
<text macro="publisher"/>
<group prefix=", ">
<date variable="issued">
<date-part name="year" suffix=","/>
</date>
<text variable="genre" font-style="italic" prefix=", "/>
<text variable="number" prefix=", No. "/>

Now the genre is italicized BUT there is a weird new comma in front of it, so it looks like this:

Bongaarts J, Completing the fertility transition in the developing world: the role of educational differences and fertility preferences, New York: Population Council, 2003,, Policy Research Division Working Papers, No. 177.

What am I doing wrong? Thanks in advance for any suggestions!

megvmegwork

Dangit, how do I include CSL here?

bwiernik

Put it in < code > tags

But even better would be to paste your code to a secret gist at gist.github.com and post a link here.

megvmegwork

Thanks! Here's the full code: https://gist.github.com/anonymous/2e3cfbf8fb62049258f3bcbe36347f3c
(Ignore the title macro weirdness, I just took that out)

damnation

This should work: https://gist.github.com/damnation333/cd20c3c42cc59ff21dcc8a1a40ec4ab8

I got rid of the various prefix commas and just added it as a delimiter in the group that sits on top of it.

adamsmith

FWIW, the reason this is different is that Zotero does try to remove duplicate punctuation, but that only goes so far -- at some point its ability to do so breaks, and adding new attributes like italics can do that. The solution is to do what damnation shows you (or something similar along those lines)

megvmegwork

Thank you both so much!

megvmegwork

Did something change in CSL between Feb 27 and now? None of the versions of my edited style (based on AMA, as described here) work anymore, nor does the one in this thread that damnation fixed. I haven't changed anything since then, though? I'm using the Style Editor in Zotero preferences, and it's saying "Error generating citations and bibliography: TypeError: termtxt is undefined".

adamsmith

It shouldn't, no. What version of Zotero are you using? Do you have the Propachi plugin installed (if you don't know, the answer is no)?

megvmegwork

I've tried version 4.0.29.16 (on Mac, if it matters) and version 5.0 beta (on Windows).

I do not have the Propachi plugin installed.

adamsmith

So if you select the text here https://gist.githubusercontent.com/damnation333/cd20c3c42cc59ff21dcc8a1a40ec4ab8/raw/cb9a7912f407fe1480b6dfef7e026a559322b878/rando.csl
(that's damnation's style above), and copy&paste it to the style editor in Zotero, you get that error? Does it depend on which references you have selected?

If so, I'd suggest making sure you have saved copies of your custom styles and then reset styles from the advanced tab of the Zotero preferences.

megvmegwork

Aha! You are correct. Thanks for your help.