From 315d7db56a831a85adfec6e4ecf30943a680b8cb Mon Sep 17 00:00:00 2001
From: Arthur Lefebvre <artlef@protonmail.com>
Date: Wed, 25 Mar 2026 15:45:50 +0100
Subject: [PATCH] Add a new tab to browse all books on the instance

---
 front/src/AppNavBar.vue                |  3 ++
 front/src/BooksBrowser.vue             |  4 +-
 front/src/InstanceBrowser.vue          | 61 ++++++++++++++++++++++++++
 front/src/api.js                       |  8 ++++
 front/src/locales/en.json              |  1 +
 front/src/locales/fr.json              |  1 +
 front/src/router.js                    |  2 +
 internal/apitest/fetchallbooks_test.go | 58 ++++++++++++++++++++++++
 internal/query/query.go                | 16 +++++++
 internal/routes/booksget.go            | 40 +++++++++++++++++
 internal/setup/setup.go                |  4 +-
 11 files changed, 195 insertions(+), 3 deletions(-)
 create mode 100644 front/src/InstanceBrowser.vue
 create mode 100644 internal/apitest/fetchallbooks_test.go
 create mode 100644 internal/routes/booksget.go

diff --git a/front/src/AppNavBar.vue b/front/src/AppNavBar.vue
index 96acfb3..df95366 100644
--- a/front/src/AppNavBar.vue
+++ b/front/src/AppNavBar.vue
@@ -83,6 +83,9 @@ onMounted(() => {
     <div id="navmenu" class="navbar-menu" :class="isMenuActive ? 'is-active' : ''">
       <div class="navbar-start">
         <NavBarSearch size-class="" class="is-hidden-touch" />
+        <RouterLink v-if="authStore.user" to="/browse" class="navbar-item" activeClass="is-active">
+          {{ $t('navbar.explore') }}
+        </RouterLink>
         <RouterLink v-if="authStore.user" to="/books" class="navbar-item" activeClass="is-active">
           {{ $t('navbar.mybooks') }}
         </RouterLink>
diff --git a/front/src/BooksBrowser.vue b/front/src/BooksBrowser.vue
index 1a0f8b1..787ca2a 100644
--- a/front/src/BooksBrowser.vue
+++ b/front/src/BooksBrowser.vue
@@ -17,8 +17,8 @@ const offset = computed(() => (pageNumber.value - 1) * limit)
 
 let currentFilterState = ref(FilterStates.READ)
 
-let data = ref(null)
-let error = ref(null)
+const data = ref(null)
+const error = ref(null)
 
 let totalBooksNumber = computed(() =>
   typeof data != 'undefined' && data.value != null ? data.value['count'] : 0,
diff --git a/front/src/InstanceBrowser.vue b/front/src/InstanceBrowser.vue
new file mode 100644
index 0000000..149837f
--- /dev/null
+++ b/front/src/InstanceBrowser.vue
@@ -0,0 +1,61 @@
+<script setup>
+import { ref, computed } from 'vue'
+import { getAllBooks } from './api.js'
+import { onBeforeRouteUpdate } from 'vue-router'
+import BookListElement from './BookListElement.vue'
+import Pagination from './Pagination.vue'
+
+const limit = 5
+const pageNumber = ref(1)
+
+const offset = computed(() => (pageNumber.value - 1) * limit)
+
+const data = ref(null)
+const error = ref(null)
+
+const pageTotal = computed(() => {
+  const countValue = data.value !== null ? data.value['count'] : 0
+  return Math.ceil(countValue / limit)
+})
+
+function fetchData() {
+  data.value = null
+  error.value = null
+  getAllBooks(data, error, limit, offset.value)
+}
+
+fetchData()
+
+onBeforeRouteUpdate(async (to, from) => {
+  pageNumber.value = 1
+  fetchData()
+})
+
+function pageChange(newPageNumber) {
+  pageNumber.value = newPageNumber
+  fetchData()
+}
+</script>
+
+<template>
+  <div>
+  <div>
+    <div v-if="error"> {{ error }}</div>
+    <div v-else-if="data && data.books && data.books.length > 0">
+      <div class="booksearchlist" v-for="book in data.books" :key="book.id">
+        <BookListElement v-bind="book" />
+      </div>
+    </div>
+    <div v-else-if="data === null">{{ $t('searchbook.loading') }}</div>
+    <div v-else>{{ $t('searchbook.noresult') }}</div>
+  </div>
+  <Pagination
+      :pageNumber="pageNumber"
+      :pageTotal="pageTotal"
+      maxItemDisplayed="11"
+      @pageChange="pageChange"
+      />
+  </div>
+</template>
+
+<style scoped></style>
diff --git a/front/src/api.js b/front/src/api.js
index 28f4cc9..913d97d 100644
--- a/front/src/api.js
+++ b/front/src/api.js
@@ -54,6 +54,14 @@ export function getMyBooks(data, error, arg, limit, offset) {
   return useFetch(data, error, '/ws/mybooks/' + arg + '?' + queryParams.toString())
 }
 
+export function getAllBooks(data, error, limit, offset) {
+  const queryParams = new URLSearchParams({
+    limit: limit,
+    offset: offset,
+  })
+  return useFetch(data, error, '/ws/books' + '?' + queryParams.toString())
+}
+
 export function getSearchBooks(data, error, searchterm, lang, searchInventaire, limit, offset) {
   const queryParams = new URLSearchParams({
     lang: lang,
diff --git a/front/src/locales/en.json b/front/src/locales/en.json
index 5f8853c..0b47c9c 100644
--- a/front/src/locales/en.json
+++ b/front/src/locales/en.json
@@ -6,6 +6,7 @@
   "navbar": {
     "mybooks": "My Books",
     "addbook": "Add Book",
+    "explore": "Explore",
     "logout": "Log out",
     "signup": "Sign up",
     "search": "Search",
diff --git a/front/src/locales/fr.json b/front/src/locales/fr.json
index 6272885..9b69422 100644
--- a/front/src/locales/fr.json
+++ b/front/src/locales/fr.json
@@ -5,6 +5,7 @@
   },
   "navbar": {
     "mybooks": "Mes Livres",
+    "explore": "Explorer",
     "addbook": "Ajouter Un Livre",
     "logout": "Se déconnecter",
     "signup": "S'inscrire",
diff --git a/front/src/router.js b/front/src/router.js
index ecbe0c3..ab01ee7 100644
--- a/front/src/router.js
+++ b/front/src/router.js
@@ -10,11 +10,13 @@ import Home from './Home.vue'
 import ScanBook from './ScanBook.vue'
 import SearchBook from './SearchBook.vue'
 import ImportInventaire from './ImportInventaire.vue'
+import InstanceBrowser from './InstanceBrowser.vue'
 import { useAuthStore } from './auth.store'
 
 const routes = [
   { path: '/', component: Home },
   { path: '/scan', component: ScanBook },
+  { path: '/browse', component: InstanceBrowser },
   { path: '/books', component: BooksBrowser },
   { path: '/book/:id', component: BookForm, props: true },
   { path: '/author/:id', component: AuthorForm, props: true },
diff --git a/internal/apitest/fetchallbooks_test.go b/internal/apitest/fetchallbooks_test.go
new file mode 100644
index 0000000..d0f35b8
--- /dev/null
+++ b/internal/apitest/fetchallbooks_test.go
@@ -0,0 +1,58 @@
+package apitest
+
+import (
+	"encoding/json"
+	"fmt"
+	"net/http"
+	"net/http/httptest"
+	"net/url"
+	"testing"
+
+	"git.artlef.fr/bibliomane/internal/dto"
+	"git.artlef.fr/bibliomane/internal/testutils"
+	"github.com/stretchr/testify/assert"
+)
+
+func TestFetchAllBooks(t *testing.T) {
+	result := testFetchBooks(t, "15", "0")
+	assert.Equal(t, int64(31), result.Count)
+	assert.Equal(t, 15, len(result.Books))
+}
+
+func testFetchBooks(t *testing.T, limit string, offset string) dto.BookSearchGet {
+	router := testutils.TestSetup()
+
+	u, err := url.Parse("/ws/books")
+	if err != nil {
+		t.Error(err)
+	}
+	if limit != "" {
+		q := u.Query()
+		q.Set("limit", limit)
+		u.RawQuery = q.Encode()
+	}
+	if offset != "" {
+		q := u.Query()
+		q.Set("offset", offset)
+		u.RawQuery = q.Encode()
+	}
+
+	q := u.Query()
+	q.Set("lang", "fr")
+	u.RawQuery = q.Encode()
+
+	token := testutils.ConnectDemoUser(router)
+	req, _ := http.NewRequest("GET", u.String(), nil)
+	req.Header.Add("Authorization", fmt.Sprintf("Bearer %s", token))
+	w := httptest.NewRecorder()
+	router.ServeHTTP(w, req)
+
+	var result dto.BookSearchGet
+	s := w.Body.String()
+	err = json.Unmarshal([]byte(s), &result)
+	if err != nil {
+		t.Error(err)
+	}
+	assert.Equal(t, 200, w.Code)
+	return result
+}
diff --git a/internal/query/query.go b/internal/query/query.go
index c936596..e4ba56a 100644
--- a/internal/query/query.go
+++ b/internal/query/query.go
@@ -103,6 +103,22 @@ func fetchUserBookGet(db *gorm.DB, userId uint) *gorm.DB {
 	return query
 }
 
+func FetchAllBooks(db *gorm.DB, userId uint, limit int, offset int) ([]dto.BookSearchGetBook, error) {
+	var books []dto.BookSearchGetBook
+	query := fetchBookQueryBuilder(db, userId)
+	query = query.Limit(limit)
+	query = query.Offset(offset)
+	res := query.Find(&books)
+	return books, res.Error
+}
+
+func FetchAllBooksCount(db *gorm.DB, userId uint) (int64, error) {
+	var count int64
+	query := fetchBookQueryBuilder(db, userId)
+	res := query.Count(&count)
+	return count, res.Error
+}
+
 func FetchBookSearchByAuthorGet(db *gorm.DB, userId uint, authorId uint64, limit int, offset int) ([]dto.BookSearchGetBook, error) {
 	var books []dto.BookSearchGetBook
 	query := fetchBookSearchByAuthorQuery(db, userId, authorId)
diff --git a/internal/routes/booksget.go b/internal/routes/booksget.go
new file mode 100644
index 0000000..30b6ed0
--- /dev/null
+++ b/internal/routes/booksget.go
@@ -0,0 +1,40 @@
+package routes
+
+import (
+	"net/http"
+
+	"git.artlef.fr/bibliomane/internal/appcontext"
+	"git.artlef.fr/bibliomane/internal/dto"
+	"git.artlef.fr/bibliomane/internal/myvalidator"
+	"git.artlef.fr/bibliomane/internal/query"
+)
+
+func GetBooksHandler(ac appcontext.AppContext) {
+	user, err := ac.GetAuthenticatedUser()
+	if err != nil {
+		myvalidator.ReturnErrorsAsJsonResponse(&ac, err)
+		return
+	}
+
+	limit, err := ac.GetQueryLimit()
+	if err != nil {
+		myvalidator.ReturnErrorsAsJsonResponse(&ac, err)
+		return
+	}
+	offset, err := ac.GetQueryOffset()
+	if err != nil {
+		myvalidator.ReturnErrorsAsJsonResponse(&ac, err)
+		return
+	}
+	books, err := query.FetchAllBooks(ac.Db, user.ID, limit, offset)
+	if err != nil {
+		myvalidator.ReturnErrorsAsJsonResponse(&ac, err)
+		return
+	}
+	count, err := query.FetchAllBooksCount(ac.Db, user.ID)
+	if err != nil {
+		myvalidator.ReturnErrorsAsJsonResponse(&ac, err)
+		return
+	}
+	ac.C.JSON(http.StatusOK, dto.BookSearchGet{Count: count, Inventaire: false, Books: books})
+}
diff --git a/internal/setup/setup.go b/internal/setup/setup.go
index 361ce3f..da3c1e4 100644
--- a/internal/setup/setup.go
+++ b/internal/setup/setup.go
@@ -39,7 +39,9 @@ func Setup(config *config.Config) *gin.Engine {
 	ws.GET("/appinfo", func(c *gin.Context) {
 		routes.GetAppInfo(appcontext.AppContext{C: c, Db: db, I18n: bundle, Config: config})
 	})
-
+	ws.GET("/books", func(c *gin.Context) {
+		routes.GetBooksHandler(appcontext.AppContext{C: c, Db: db, I18n: bundle, Config: config})
+	})
 	ws.GET("/mybooks/read", func(c *gin.Context) {
 		routes.GetMyBooksReadHandler(appcontext.AppContext{C: c, Db: db, I18n: bundle, Config: config})
 	})