internetarchive / internetarchive/openlibrary

Facets are not user sensitive (for print disabled books)

Open
#9,575 3 comments 0 reactions 0 assignees View on GitHub
Lead: @cdrini Module: Solr Priority: 3 Theme: Search Type: Bug
Dominant language
Python
Stars
6.7k
Forks
2k
Avg merge
2d 19h
Merged PRs (30d)
138

Description

### Problem

Why does this say “4 ebooks”
But then there are only 2 results (and each has 2 works but only 1 ebook for each)
This is because I don't see the books that are for folks with print disabled permissions but the facet doesn't reflect that.
https://openlibrary.org/search?subject=Apache+Solr

#### Evidence / Screenshot
![image](https://github.com/user-attachments/assets/85084b0a-2b5b-485a-be90-986aad0d5153)
![image](https://github.com/user-attachments/assets/223ca90a-1012-4b18-b55f-17faef33251a)

#### Relevant URL(s)
Slack thread: https://internetarchive.slack.com/archives/C0ETZV72L/p1721157389711519?thread_ts=1721115930.014879&cid=C0ETZV72L

### Reproducing the bug

1. Go to ...
2. Do ...

* Expected behavior:
* Actual behavior:

### Context

n/a

### Notes from this Issue's Lead

#### Proposal & constraints

#### Related files

#### Stakeholders

@cdrini


#### Instructions for Contributors

- Please [run these commands](https://github.com/internetarchive/openlibrary/wiki/Git-Cheat-Sheet#working-on-your-branch) to ensure your repository is up to date **before** [creating a new branch](https://github.com/internetarchive/openlibrary/wiki/Git-Cheat-Sheet#making-changes-and-creating-a-pull-request) to work on this issue and **each time after** pushing code to Github, because the pre-commit bot may add commits to your PRs upstream.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.