open-oni / open-oni

Open ONI (Open Online Newspaper Initiative) Django web app
https://open-oni.github.io/
Other
48 stars 23 forks source link

Titles with no place associations behavior #277

Open jduss4 opened 7 years ago

jduss4 commented 7 years ago

Recently, we discovered several of our titles are missing the 752 marc field. This meant that though they could display data indicating that they were from "Lincoln, Neb," they were not actually associated with that core_place in the database.

It led to some funky behavior:

I suspect that there may have been more effects which I didn't test, like not being reflected by city and state filters / counts on search result pages, not displaying in the search select forms, etc.

We are filling out a form to get their marc catalog records changed, but I am wondering if we want to have a contingency plan for this in ONI. Specifically:

jechols commented 7 years ago

The work I'm doing to pull newspapers by city will skip anything lacking a city just like the current system skips anything lacking a state. We could hack it up to include those, I think, but it may result in odd errors since so much of the logic expects some kind of location value.

Given that there are likely to be more papers lacking a city than lacking a state, I'm thinking my work will need more testing than I was hoping.

jechols commented 7 years ago

Actually, looking closer at this, there may be a way to just make the newspaper list a lot more agnostic than what I've done so far. I'll push something for #278 in the next few days hopefully.

jduss4 commented 6 years ago

@jechols do you know if this is done?

jechols commented 6 years ago

Ugh, I clearly dropped the ball somewhere here. I think this will require a re-test on a vanilla instance of ONI :(

raybrarian commented 2 years ago

Related: #28