Issue Number | 817 |
---|---|
Summary | Error with a single citation's full history page |
Created | 2024-05-01 17:05:03 |
Issue Type | Bug |
Submitted By | Boggess, Cynthia (NIH/NCI) [C] |
Assigned To | Kline, Bob (NIH/NCI) [C] |
Status | Resolved |
Resolved | 2024-06-24 11:42:57 |
Resolution | Fixed |
Path | /home/bkline/backups/jira/oceebms/issue.435930 |
Bonnie has reported that Citation EBMS ID: 62333 will not display it's full history page. Upon clicking the citation's title from the Article Search retrieval list, an error message appears "The website encountered an unexpected error. Try again later." The same error also displays when the citation's pmid 15213888 is searched using the pmid quick search.
This bug seems to be isolated to just this citation.
Turns out this is a bug in the data. It appears that the old CiteMS system recorded a state for the article in connection with the Supportive and Palliative Care board's "Sexuality and Reproductive Issues" topic, but stored the Pediatric Treatment board (not Supportive and Palliative Care) for the state (it got the board right for the other states for that topic). It's surprising that this hasn't popped up earlier, because this isn't the only instance in which the legacy system made this kind of mistake in recording the wrong board for an article state. You'll get the same error for this article on all the tiers. I have implemented a workaround on my own server, so the full article page is viewable there. Do we have a name for the next release? 🙂
I rarely work with legacy citations, but Bonnie and the board manager are more likely to during something like a comprehensive review.
I think we are up to state parks starting with an "I"
Ticket has been added to the new Ironwood
sprint.
Working on migrating the workaround into a new branch and onto DEV.
The workaround has been committed to the new branch, pushed to GitHub, and installed on EBMS DEV. You should be able to pull up the full article page for EBMS 62333 on that tier now. Ready for user testing.
Bonnie and I have taken a look at this citation in dev and it is displaying the full history page but we are seeing a yellow banner at the top that says "collecting state information"
And this banner is displaying for all citations on the full history page not just 62333.
I am attaching a screen shot of the banner for 62333 as well as a recent 2024 citaiton.
Good catch. That was from code I had added while I was troubleshooting the problem and testing the solution. I had dropped it from what got checked into version control but I had neglected to install that final version on the DEV server. That message should no longer appear.
OK, looks good. I am no longer seeing the yellow banner.
Bonnie has identified another citation that is having the same problem in PROD. PMID 17242397
Full history page will not open when PMID quick search is used or when the article title link is clicked from an article search results list.
I tested the citation in DEV and it works fine and no yellow banner appears.
I am mentioning this as an additional example and also because this citation is a bit different. I am not seeing any states recorded for the wrong board. Unless I am missing something. Please take a look just in case.
I believe this is the same behavior (and cause) as originally described for this ticket. The reason you don't see any bogus boards in the article display where the workaround has been installed is that this workaround ignores the state record's board value and instead looks up the board actually associated with the state's topic. I'm pretty sure that if you go back and look at the article for which this ticket was originally opened, you'll see that all of the boards are correct for the article's topic state history.
Ok that makes sense and yes, the original article is correct in dev. So this citation was a nice test of your workaround...which is working. This will also give us another example to test in prod when we move this fix over. Thanks for checking!
File Name | Posted | User |
---|---|---|
ebms-dev-Jira-817_other-citation.PNG | 2024-06-14 10:01:03 | Boggess, Cynthia (NIH/NCI) [C] |
ebms-dev-Jira-817.PNG | 2024-06-14 10:01:03 | Boggess, Cynthia (NIH/NCI) [C] |
Elapsed: 0:00:00.000612