Closed alexskr closed 1 year ago
Is it even worth fixing this issue? virtual ontology ids are legacy and have not been in use for a number of years.
I know the reason for the breakage, and it shouldn't be a difficult fix. I'll take a look at this tomorrow.
I agree though, that having ontology ID resolver code in the Rails app feels a little messy. If there are better alternatives, I'm all ears.
we could look at the logs and determine which virtual ids are still in use and add redirects on the web server level. After this, we could remove the resolver from the rails. There is no need for it in the ontoportal appliance anyway. That or perhaps have a generic redirect list in the app where redirects can be added/removed on demand
@graybeal expressed a preference that we continue to accept the old numeric identifiers and resolve them to the corresponding acronyms in our current system. So, I fixed the code because it wasn't a significant effort. I agree that we should use a better solution for resolving the old IDs, but I think spending time on this should be a lower priority than a lot of the other things we have to work on. In the event we decide to tackle this, I think the work should be tracked under a new ticket that's characterized as an enhancement request.
virtual (legacy numeric based ontology id) to acronym resolver no longer works. UI throws 500 error when browsing bioportal.bioontology.org/ontologies/1032 which used to redirect to bioportal.bioontology.org/ontologies/NCIT