-
-
Notifications
You must be signed in to change notification settings - Fork 294
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
Export of creator-type fields from embedded CSL variables + change authors to directors for CSL-JSON film+tvBroadcast + export tvBroadcast to biblatex video #365
Comments
I currently see these as potential creator fields; what Bib(La)TeX fields to these correspond to? |
In what sense does Zotero not export the director properly? |
Overlap between CSL and biblatex is minimal here: ¹
It exports “Director” as CSL Styles that support CSL |
I can convert director to the appropriate field in the *TeX exports; if you have a testcase + expected output, that would be much appreciated. WRT "director" not being exported correctly, since Pandoc JSON uses CSL JSON internally, and since creator order is lost in the transformation from Zotero item to CSL-JSON item, I cannot patch this behavior (without recreating the Zotero to CSL-JSON formatter myself). This problem should be fixed when it is fixed in Zotero. I wish I could do more than defer the problem, but there's nothing else I can do for this one. |
The current (expected) behaviour in the 365 branch should be that if you specify a creator-type multiple times, it will replace the normal Zotero creator type, but will stack other than that; so multiple |
I have a version that passes all existing tests at http://tempsend.com/CAA34AAEF7 |
What about a bit of brute force then? Just convert all authors in motion_picture items to directors before writing out “Pandoc JSON” – motion_picture items do not have (genuine) authors this could conflict with. |
In those cases where the behavior is unequivocal, then yes. But I see that for a zotero "Film" reference, nothing but the director is exported; other creator types are simply silently dropped on the floor. Is that desirable? |
Thank you – but:
returns just
|
Well, it was untested :) Do you have a full reference for me? I can stick those straight in my test suite. |
Not at all. But there are no CSL variables for any of these. For bib*tex there are at least producer and scriptwriter (biblatex-apa / -fiwi) – but never authors. |
New version at http://tempsend.com/22F7908B24 for (I think) editor fix & director fix |
full references: PQPQFN3D
director: looks good, editor: not yet, still: |
http://tempsend.com/22F7908B24: It seems labels that do not match a valid CSL variable are no longer accepted when using the Result when trying to export to “Pandoc JSON”: Popup “An error occurred while trying to export the selected file.” I don’t observe this error with the ``X-author: Doe || John` format. I found this quite useful for “commenting out” variables when testing, so it’d be great to get the original behaviour back. |
One more thing: for the CSL |
It looks like http://tempsend.com/4CA88E0D7E addresses your needs, yeah? Source references here (I added the X-field to see what would happen), BibLaTeX output, CSL-JSON output |
@nickbart1980 I missed your last comment. You mean authors changed to directors, right? |
Ok, see #368. But number is in both :tv ( |
Update Export of creator-type fields from embedded CSL variables #365.json
I don't know what's going on then on my end; here it only appears once:
|
One comes from a proper Zotero field, the other is embedded in the Extra field. (Zotero is pretty inconsistent wrt fields it offers for video material.) |
Ah, hold on, it's in the "number" field! Let me see what's up with that. |
Wait, when you say that
do you mean that stuff will break for existing users unless they use "DeclareSourceMap"? In that case I'd prefer to stick this behind a preference. I don't know the prevalence of -apa/-fiwi, but unless they constitute the bulk of the use, I can't just silently break stuff. |
Fair enough. |
Does |
The biblatex manual lists biblatex-chicago and biblatex-apa, on the other hand, recognize a Finally, biblatex-fiwi, a style focusing on film studies, uses a My recommendation would be to use the |
OK, if I add the
|
This seems to be based purely on what the biblatex manual lists as required and optional fields, but I would argue this is too narrow: It’s perfectly possible for a report to have editors (and indeed biblatex-chicago formats this properly). The video entries, too, rather closely match the following from the current version of biblatex-chicago (
(The examples I general, I wouldn’t drop Zotero fields just because they are not listed in the biblatex manual, provided they are being used in at least one of the other more popular packages. One problem is of course that different biblatex packages sometimes require different fields, and IIRC in the past I have sometimes argued in favour of BBT using/generating simple and clear field names and content, e.g.:
rather than biblatex-chicago’s
or biblatex-apa’s
(the latter from As I have argued elsewhere repeatedly, I feel the least bad option in such cases is to have users sort this out themselves, e.g., via the |
I have no idea what popular packages are, and what fields they use though. |
@njbart do you know how |
Well, the package called This is where
Not sure where you got that from. (If |
I was just wrong on that, I hadn't looked through the whole manual. |
citeproc-js recognizes
{:label: lastName || firstName}
in Zotero’s Extra field as a two-field name value. (See zotero/zotero#846.)BBT could use the same syntax (not too pretty, but then again, we don’t have to invent one from scratch).
This would be great to add creator types for items that don’t currently support them (e.g., “Director”¹ for “Film”, “Editor” for “Report” and “Web page”, …).
¹ Zotero does have a “Director” field but does not export it properly.
The text was updated successfully, but these errors were encountered: