Pittsburgh-NEH-Institute / pr-app

eXist-db app development
MIT License
3 stars 1 forks source link

[BUG] Partial decades should show all values #42

Closed djbpitt closed 2 years ago

djbpitt commented 2 years ago

Expected behavior

If we select a decade and a publisher the decade returns with only the matches checked. This means that if we add a publisher that follow-up query will be constrained by the prior month-dates, and we want it to be constrained by the prior decade, since that's what we selected.

Actual behavior

If we selected an entire decade, the entire decade should be returned, including month-years that don't match the previous query.

Reproduction steps

Check “Penny Satirist, The” and “Times, The” and “1830” (decade). The search returns showing only July 1830 and January 1838, since those are the only month-years in the 1830s when those publishers issued articles. But if we add a new publisher that published in a different month-year in that decade, we won't find the hit because although it looks as if the decade is checked, our XQuery looks only at the checked month-years.

djbpitt commented 2 years ago

Fixed