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

Cannot 'open in editor' (DCLP) #217

Closed
samosafuz opened this issue Jul 10, 2020 · 5 comments
Closed

Cannot 'open in editor' (DCLP) #217

samosafuz opened this issue Jul 10, 2020 · 5 comments
Assignees

Comments

@samosafuz
Copy link
Member

This morning I attempted to edit a pair of related files in DCLP (TM 65628 and TM 65823) and was unable to open either one. I suspect that the problem is related to the fact that both texts are aggregated with a pair of Michigan APIS entries (michigan.apis.1990 and michigan.apis.3413).

My suspicion is based on the web address on the error screen I received, which suggests that the system is trying to open an APIS entry first (as opposed to the DCLP record?):

image

The fact that there are two APIS entries to choose from may be part of the problem (there is no APIS identifier at all in the address on the error screen). But when I tested this hypothesis by opening another item in DCLP with multiple APIS entries aggregated (i.e., berkeley.apis.376), the publication was successfully created.

image

Maybe the fact that the Berkeley papyrus also exists in DDbDP has prevented the same error. But I'm not sure what has gone wrong.

Vestiges of the error appear now in my user dashboard.

image

I find this odd because I've attempted to open both TM 65628 and 65823 a few times for the sake of confirming the error and writing the ticket, yet there's only this one item on my dashboard. FWIW, clicking on the item under publication navigates to http://papyri.info/editor/publications/89369, with the following error screen.

image

Since I'm preparing a substantial pull request which will update the idnos in APIS records to assist in DCLP aggregation, the potential for some sort of glitch involving APIS and DCLP is of particular concern.

@samosafuz
Copy link
Member Author

The same issue happened to Paul van der Laan when attempting to open TM 60866 in editor. That file has a Columbia APIS entry, too (columbia.apis.p367).

@samosafuz
Copy link
Member Author

Paul van der Laan reports the same issue when attempting to open TM 60610 (yale.apis.0010620000) and TM 60862 (= columbia.apis.p363) in editor. There does seem to be an issue for DCLP items with APIS entries.

@ryanfb ryanfb self-assigned this Nov 9, 2020
@jcowey
Copy link
Member

jcowey commented Dec 11, 2020

Out of curiosity I navigated to http://papyri.info/dclp/60866 (mentioned in the comment above). I then hit open in editor. I got the "We are sorry, but something went wrong notice." What I found interesting is that my news feed reported:
You started editing papyri.info/apis (28 minutes ago)
That would seem to indicate to me that neither the identifier of the APIS record nor that of the DCLP record is not being picked up at all, when "open in editor" is clicked.
Normally in DCLP the news feed will say:
C. Michael Sampson started editing 64796 (about 7 hours ago)
Where 64796 indicates the DCLP identifier.
Is the number server somehow not properly jigged to take DCLP identifier in conjunction with an APIS identifier?

@ryanfb
Copy link
Member

ryanfb commented Jan 13, 2021

Yes, this looks to be an issue with how the papyrological navigator is linking to the editor. I'll continue looking into it, but in the meantime a workaround is to use the editor's "Advanced Create" and paste the identifier e.g. papyri.info/apis/yale.apis.0010620000 into the PN ID field in the "From PN ID" section, which works for me with the examples above. In cases where there's DDbDP/HGV records in addition to DCLP/APIS ones, it looks like the "Open in Editor" link may still work correctly.

@ryanfb
Copy link
Member

ryanfb commented Jan 13, 2021

It looks like the issue was that for DCLP+APIS cases, the link was made from a DCLP context trying to use the apisid idno, which isn't in the DCLP record. I've pushed a fix which should use the DCLP identifiers instead, which should resolve this issue.

@hcayless: If you can pull the latest navigator with this change, I think re-indexing (or whichever process generates the HTML output) should resolve this issue on production.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants