Merge pull request #381 from openstax/fix_search_dupes

Return only the latest version of each book in the books endpoint
openstax · Sep 7, 2022 · f6c2d76 · f6c2d76
2 parents d5d5048 + ab00e88
commit f6c2d76
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/app/controllers/api/v1/books_controller.rb b/app/controllers/api/v1/books_controller.rb
@@ -18,10 +18,17 @@ class BooksController < OpenStax::Api::V1::ApiController
     def index
       OSU::AccessPolicy.require_action_allowed! :index, current_api_user, OpenStax::Content::Book
 
-      books = abl.approved_books(archive: archive).map do |book|
+      books = abl.approved_books(archive: archive).group_by(&:uuid).map do |uuid, books|
+        latest_book = books.sort_by(&:committed_at).last
+
         # We use book.slug.titleize here to avoid fetching all book ToCs,
         # because the actual book title is not in the ABL
-        { uuid: book.uuid, version: book.version, title: book.slug.titleize, slug: book.slug }
+        {
+          uuid: latest_book.uuid,
+          version: latest_book.version,
+          title: latest_book.slug.titleize,
+          slug: latest_book.slug
+        }
       end
 
       render json: books