Skip to content

Add Usenix.org translator - #3477

Merged
AbeJellinek merged 3 commits into
zotero:masterfrom
timlst:usenix-curly-brackets-fix
Jul 29, 2025
Merged

Add Usenix.org translator#3477
AbeJellinek merged 3 commits into
zotero:masterfrom
timlst:usenix-curly-brackets-fix

Conversation

@timlst

@timlst timlst commented Jul 23, 2025

Copy link
Copy Markdown
Contributor

Conference hosted on usenix.org include BibTex style syntax for capitalization in their page metadata.
E.g. this paper includes 5 brackets within the title

  • PR adds a simple translator which removes brace pairs from title field
  • tries to account for escaped braces intended in the title, but could not find an example

timlst added 2 commits July 23, 2025 17:58
Fix issue where BibTeX escapes were present in title metadata.
@alex-ter

Copy link
Copy Markdown
Contributor

I wonder if the whole new translator is meaningful here. It seems to me it should rather be an edit for the Embedded Metadata translator directly. Maybe even a general utility "cleanup" function (or an update to the existing one)?

There's a set of smaller things I'd note, but let's maybe first see what maintainers think of the general idea here.

@timlst

timlst commented Jul 28, 2025

Copy link
Copy Markdown
Contributor Author

Totally agree, this is a somewhat quick&dirty solution for myself, that I just happened to PR because it annoyed me - my JS was barely enough for this, I didn't want to touch anything bigger

@AbeJellinek

Copy link
Copy Markdown
Member

I actually think this should be its own translator. We don't tend to add fixes to EM for individual sites. If a specific site needs a metadata fix, we wrap EM with a new translator. It's a bit annoying, but it's probably better than crowding EM with tons of tiny edge-case fixes. (I don't know of any other sites that need this fix, at least off the top of my head.)

Comment thread usenix.org.js Outdated
Comment thread usenix.org.js
Comment thread usenix.org.js Outdated
Comment thread usenix.org.js Outdated
@AbeJellinek

Copy link
Copy Markdown
Member

Thanks! I guess USENIX doesn't like requests from our CI, but this looks good.

@AbeJellinek
AbeJellinek merged commit a12fe98 into zotero:master Jul 29, 2025
1 check failed
@alex-ter

Copy link
Copy Markdown
Contributor

Thanks for elaborating, Abe, good to know. I was looking at it as a more generic "cleanup" functionality that could be helpful for other sites, but indeed if that's the only one, having a dedicated translator makes sense to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants