Setting seeAlso and relations in translators

[Alex T]

Hello everyone,

There's a long-standing issue record in the translators repo [1] and a PR in the main repo [2] for enabling the seeAlso functionality in translators. The PR didn't get much attention since it was submitted back in April and I wonder what's necessary to move it forward.

I happen to need this functionality for a translator change I'd like to propose, that's why I'm asking, and I'm also ready to take care of the PR even if the original submitter isn't interested any more (it certainly needs at least a proper rebase). I haven't personally analyzed the proposed approach, plus I'm not familiar with Zotero code [yet], so a comment or review from someone who does would be quite valuable, and I (or the original PR author) could take it from there.

It looks like the relevant part of Zotero code was simply commented out at some point, without any explanatory comment attached, and was never reenabled. There was an earlier PR that tried to simply uncomment the code, but AFAIU it's not as simple as that, and the PR I've referenced adds more logic, but I just don't know if it's correct, and simply needs maintainer attention to get merged, or a rework is required, so would be grateful for a comment.

[1] https://github.com/zotero/translators/issues/2338

[2] https://github.com/zotero/zotero/pull/3085

regards,

Alex

[Frank Bennett]

Hi Alex,

I have assumed that the feature is either just low on the team's priority list, or conflicts with future plans for relations. From my dive into it, it looked like the original relations were stored as arrays of target objects, but then that was changed to use arrays of IDs after that code block was commented out.

The fix that I worked out for Jurism is reliable, but I'm not familiar enough with the code and its development trajectory to know whether it's the best approach.

I agree that it would be great to get this back up and running in Zotero. Relations applied by a translator can save a lot of curation time for some tasks.

--
You received this message because you are subscribed to the Google Groups "zotero-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to zotero-dev+...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/zotero-dev/CADqm%2BCdPsVMZd1Ejecv4bFRVJA906FMwKNLPVXsZjtG%3DCWh3ig%40mail.gmail.com.

[Alex T]

Thanks Frank, this makes sense to me. Now, it'd be nice to get a comment from the Zotero team on this - is this just not a priority now, are there any more fundamental problems that need to be solved first, etc?

To view this discussion on the web visit https://groups.google.com/d/msgid/zotero-dev/CAJgpGgA1jMV1nViCdo7E%2BLReHBnpRWFRPWC6NrqB5fhqqbrjWg%40mail.gmail.com.