Filter package finders to return displayable by default
Context
As we start to use the package status
column for things like error states, we need to be sure we do not include error
package, or any non-displayable package in API responses or in package finder results when downloading a package.
Proposal
There is some opportunity to refactor the various <package_type>::PackageFinder
s to either use ::Packages::FinderHelper
, or inherit from one of the base package finder classes in order to share this common filtering. In the Helper or parent class, we should use the .displayable
scope when filtering results unless a different status is specifically specified.
Edited by Tim Rizzi