Add direct book providers

[cdrini]

Closes #9419

Test with
https://openlibrary.org/books/OL38222427M.json

What we want is:

• UI for /addbook to propose web_book url Include web book URL in add book form #7077
• a /borrow endpoint the resolves an edition to web book url (see: https://github.com/internetarchive/openlibrary/blob/master/openlibrary/plugins/upstream/borrow.py#L108-L233)
• stats.incr metric for when web books type are fulfilled by /borrow
• Exploratory: Prototype: Enter a URL -> Book Import (Preview) #9405

On https://github.com/internetarchive/openlibrary/blob/master/openlibrary/plugins/upstream/borrow.py#L128C9-L128C40

edition = web.ctx.site.get(key)
provider = get_book_provider(edition)
if provider and provider.ebook_access(edition) == "public":
    raise web.seeother(provider.read_url)

Technical

Testing

Screenshot

Stakeholders

[cclauss]

Suggested change

	if edition.providers:
	return [EbookProvider.from_json(dict(p)) for p in edition.providers]
	else:
	return []
	return [EbookProvider.from_json(dict(p)) for p in edition.providers or []]

[cdrini]

Notes from rebase:

This code was in AffiliateLinks.html, to add the OPDS buy options to our buy options, but that code has since been updated to no longer take in page as a parameter. Should be an easy fix in some future PR where we want to re-enable this feature.

    $for provider in page.get('providers', []):
      $if provider.get('rel') == 'http://opds-spec.org/acquisition/buy' and provider.get('href'):
        $ name = provider.get('name', _('Official Vendor'))
        $ href = provider.get('href')
        $ price = provider.get('properties', {}).get('price', {}).get('value')
        $if price:
          $ price = '$%.2f' % price
        $:affiliate_link('direct', 'direct', name, href, price)

[mekarpeles]

This is for action="borrow" which is typically for borrowable books, but given most trusted book providers are for open access materials (and we tend to use "read" for these) we should make sure a case of /borrow?action=read also succeeds

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 16.21%. Comparing base (ce16a79) to head (84bdb7d).
Report is 55 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #6585      +/-   ##
==========================================
+ Coverage   16.06%   16.21%   +0.15%     
==========================================
  Files          90       90              
  Lines        4769     4786      +17     
  Branches      832      829       -3     
==========================================
+ Hits          766      776      +10     
- Misses       3480     3486       +6     
- Partials      523      524       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[cdrini]

Woooohooooo!!! This is so close!! I think this last set of changes, and then merge it up to the github in the sky!