Add import edition from inventaire

front/src/ImportListElement.vue

@@ -1,16 +1,17 @@
 <script setup>
-  import { computed } from 'vue'
+  import { ref, computed } from 'vue'
-  import { getInventaireImagePathOrDefault } from './api.js'
+  import { getInventaireImagePathOrDefault, postImportBook } from './api.js'
   import { useRouter } from 'vue-router'
 
   const router = useRouter();
   const props = defineProps({
-    id: String,
+    uri: String,
     title: String,
     image: String,
     publisher: String,
     date: String,
     isbn: String,
+    lang: String
   });
 
   function displayDate(date) {
@@ -22,26 +23,44 @@
       return date;
     }
   }
+  const error = ref(null);
+  const data = ref(null);
+  const importing = ref(false);
+
+  async function importInventaireEdition() {
+    importing.value = true;
+    const res = await postImportBook(props.uri, navigator.language.substring(0,2));
+    const json = await res.json();
+    if (res.ok) {
+      router.push(`/book/${json.id}`);
+    } else {
+      error.value = json;
+    }
+  }
   const imagePathOrDefault = computed(() => getInventaireImagePathOrDefault(props.image));
 </script>
 
 <template>
-  <div v-if="error" class="notification is-danger">
-    <p>{{error}}</p>
-  </div>
   <div class="columns no-padding box container has-background-dark">
-    <div class="media column no-margin clickable" @click="openBook">
+    <div v-if="importing && !data && !error">{{$t('importlistelement.importing')}}</div>
+    <div v-else class="media column no-margin clickable" @click="importInventaireEdition">
       <div class="media-left">
         <figure class="image mb-3">
           <img v-bind:src="imagePathOrDefault" v-bind:alt="title">
         </figure>
       </div>
       <div class="media-content">
-        <div class="is-size-4">{{title}}</div>
+
-        <div v-if="props.date" class="is-size-5">{{$t('importlistelement.releasedate') + "" +
+        <div v-if="error" class="has-text-danger">
-          displayDate(date)}}</div>
+          <p>{{error}}</p>
-        <div v-if="props.publisher" class="is-size-5">{{$t('importlistelement.publisher') + "" + publisher}}</div>
+        </div>
-        <div v-if="props.isbn" class="is-size-5">{{"ISBN: " + isbn}}</div>
+        <div v-else>
+          <div class="is-size-4">{{title}}</div>
+          <div v-if="props.date" class="is-size-5">{{$t('importlistelement.releasedate') + "" +
+            displayDate(date)}}</div>
+          <div v-if="props.publisher" class="is-size-5">{{$t('importlistelement.publisher') + "" + publisher}}</div>
+          <div v-if="props.isbn" class="is-size-5">{{"ISBN: " + isbn}}</div>
+        </div>
       </div>
     </div>
   </div>

front/src/InventaireImport.vue

@@ -1,32 +0,0 @@
-<script setup>
-  import { ref } from 'vue'
-  import { postImportBook } from './api.js'
-  import { useRouter } from 'vue-router'
-
-  const router = useRouter();
-
-  const props = defineProps({
-    inventaireid: String
-  });
-  const error = ref(null);
-  const data = ref(null);
-
-  async function importInventaireId() {
-    const res = await postImportBook(props.inventaireid, navigator.language.substring(0,2));
-    const json = await res.json();
-    if (res.ok) {
-      router.push(`/book/${json.id}`);
-    } else {
-      error.value = json;
-    }
-  }
-
-  importInventaireId();
-</script>
-
-<template>
-  <div v-if="error">{{error}}</div>
-  <div v-else-if="!data">Importing {{props.inventaireid}}...</div>
-</template>
-<style scoped></style>
-

front/src/locales/en.json

@@ -70,7 +70,8 @@
   },
   "importlistelement": {
     "releasedate":"Release date:",
-    "publisher":"Publisher:"
+    "publisher":"Publisher:",
+    "importing":"Importing..."
   }
 
 }

front/src/locales/fr.json

@@ -70,6 +70,7 @@
   },
   "importlistelement": {
     "releasedate":"Date de publication : ",
-    "publisher":"Maison d'édition : "
+    "publisher":"Maison d'édition : ",
+    "importing":"Import en cours..."
   }
 }

front/src/router.js

@@ -9,14 +9,12 @@ import LogIn from './LogIn.vue'
 import Home from './Home.vue'
 import SearchBook from './SearchBook.vue'
 import ImportInventaire from './ImportInventaire.vue'
-import InventaireImport from './InventaireImport.vue'
 import { useAuthStore } from './auth.store'
 
 const routes = [
   { path: '/', component: Home },
   { path: '/books', component: BooksBrowser },
   { path: '/book/:id', component: BookForm, props: true },
-  { path: '/importinventaire/:inventaireid', component: InventaireImport, props: true },
   { path: '/author/:id', component: AuthorForm, props: true },
   { path: '/search/:searchterm', component: SearchBook, props: true },
   { path: '/import/inventaire/:inventaireid', component: ImportInventaire, props: true },

internal/apitest/post_importbook_test.go

@@ -18,19 +18,19 @@ type hasId struct {
 }
 
 func TestPostImportBookHandler_Ok(t *testing.T) {
-	id := testPostImportBookHandler(t, "Q202975", http.StatusOK)
+	id := testPostImportBookHandler(t, "isbn:9782253004752", http.StatusOK)
 	book := testGetBook(t, strconv.FormatUint(uint64(id), 10), 200)
-	assert.Equal(t, "Les Hauts de Hurlevent", book.Title)
+	assert.Equal(t, "les Hauts de Hurle-Vent", book.Title)
 	assert.Equal(t, "Emily Brontë", book.Author)
-	assert.Equal(t, "Q202975", book.InventaireID)
+	assert.Equal(t, "isbn:9782253004752", book.InventaireID)
 }
 
 func TestPostImportBookHandler_OkAuthorKey(t *testing.T) {
-	id := testPostImportBookHandler(t, "Q1317915", http.StatusOK)
+	id := testPostImportBookHandler(t, "isbn:9782290033630", http.StatusOK)
 	book := testGetBook(t, strconv.FormatUint(uint64(id), 10), 200)
 	assert.Equal(t, "Dr Bloodmoney", book.Title)
 	assert.Equal(t, "Philip K. Dick", book.Author)
-	assert.Equal(t, "Q1317915", book.InventaireID)
+	assert.Equal(t, "isbn:9782290033630", book.InventaireID)
 }
 
 func TestPostImportBookHandler_NoOLID(t *testing.T) {
@@ -61,7 +61,10 @@ func testPostImportBookHandler(t *testing.T, openlibraryid string, expectedCode
 			t.Error(err)
 		}
 		return parsedId.ID
-	} else {
+	} else if expectedCode == 200 {
+		var stringerr = w.Body.String()
+		t.Error(stringerr)
 		return 0
 	}
+	return 0
 }

internal/inventaire/inventaire_test.go

@@ -43,7 +43,7 @@ func TestCallInventaireSearch_Offset(t *testing.T) {
 }
 
 func TestCallInventaireBook_BraveNewWorld(t *testing.T) {
-	result, err := CallInventaireBook("Q191949", "fr")
+	result, err := callInventaireBook("wd:Q191949", "fr")
 	if err != nil {
 		t.Error(err)
 	}
@@ -54,8 +54,8 @@ func TestCallInventaireBook_BraveNewWorld(t *testing.T) {
 	assert.Equal(t, "écrivain, romancier et philosophe britannique (1894–1963)", result.Author.Description)
 }
 
-func TestCallInventaireEdition_TestLimit(t *testing.T) {
+func TestCallInventaireEditionFromWork_TestLimit(t *testing.T) {
-	result, err := CallInventaireEdition("wd:Q339761", "fr", 10, 0)
+	result, err := CallInventaireEditionFromWork("wd:Q339761", "fr", 10, 0)
 	if err != nil {
 		t.Error(err)
 	}
@@ -63,8 +63,8 @@ func TestCallInventaireEdition_TestLimit(t *testing.T) {
 	assert.Equal(t, 10, len(result.Results))
 }
 
-func TestCallInventaireEdition_TestOffset(t *testing.T) {
+func TestCallInventaireEditionFromWork_TestOffset(t *testing.T) {
-	result, err := CallInventaireEdition("wd:Q3213142", "fr", 0, 0)
+	result, err := CallInventaireEditionFromWork("wd:Q3213142", "fr", 0, 0)
 	if err != nil {
 		t.Error(err)
 	}
@@ -101,3 +101,27 @@ func TestCallInventaireEdition_TestOffset(t *testing.T) {
 		},
 		result.Results[2])
 }
+
+func TestCallInventaireEdition(t *testing.T) {
+	result, err := CallInventaireEdition("isbn:9782266003698", "fr")
+	if err != nil {
+		t.Error(err)
+	}
+	assert.Equal(t,
+		InventaireEditionDetailedSingleResult{
+			Id:    "isbn:9782266003698",
+			Title: "Bel-Ami",
+			Author: &InventaireAuthorResult{
+				ID:          "Q9327",
+				Name:        "Guy de Maupassant",
+				Description: "écrivain et journaliste littéraire français (1850-1893)",
+			},
+			Description: "roman de Guy De Maupassant",
+			ISBN:        "978-2-266-00369-8",
+			Publisher:   "Pocket",
+			ReleaseDate: "1977",
+			Image:       "https://inventaire.io/img/entities/b237c0b5de5f6c765928d9eee26b55804a33557a",
+			Lang:        "fr",
+		},
+		result)
+}

internal/inventaire/inventairebook.go

@@ -43,7 +43,7 @@ func (i *InventaireBookResult) UnmarshalJSON(b []byte) error {
 		if err != nil {
 			return err
 		}
-		if inventaireEntity.WdId == i.ID {
+		if ("wd:" + inventaireEntity.WdId) == i.ID {
 			title, err := findLangageField(inventaireEntity.Labels, i.Lang)
 			if err != nil {
 				return err
@@ -73,20 +73,28 @@ func (i *InventaireBookResult) UnmarshalJSON(b []byte) error {
 }
 
 func findLangageField(multipleMessageFields map[string]json.RawMessage, lang string) (string, error) {
-	fieldToParse, ok := multipleMessageFields[lang]
+	if len(multipleMessageFields) == 0 {
-	if ok {
+		return "", errors.New("empty multilang field")
-		var parsedField string
-		err := json.Unmarshal(fieldToParse, &parsedField)
-		if err != nil {
-			return "", err
-		}
-		return parsedField, err
-	} else {
-		return "", errors.New("multi lang field could not be parsed")
 	}
+
+	var fieldToParse json.RawMessage
+	var ok bool
+	fieldToParse, ok = multipleMessageFields[lang]
+	if !ok {
+		for _, field := range multipleMessageFields {
+			fieldToParse = field
+			break
+		}
+	}
+	var parsedField string
+	err := json.Unmarshal(fieldToParse, &parsedField)
+	if err != nil {
+		return "", err
+	}
+	return parsedField, err
 }
 
-func CallInventaireBook(inventaireId string, lang string) (InventaireBookResult, error) {
+func callInventaireBook(inventaireId string, lang string) (InventaireBookResult, error) {
 	queryResult := InventaireBookResult{ID: inventaireId, Lang: lang}
 	u, err := computeInventaireApiUrl("entities")
 	if err != nil {
@@ -95,7 +103,7 @@ func CallInventaireBook(inventaireId string, lang string) (InventaireBookResult,
 	callapiutils.AddQueryParam(u, "action", "by-uris")
 	callapiutils.AddQueryParam(u, "relatives", "wdt:P50")
 	callapiutils.AddQueryParam(u, "lang", lang)
-	callapiutils.AddQueryParam(u, "uris", "wd:"+inventaireId)
+	callapiutils.AddQueryParam(u, "uris", inventaireId)
 
 	err = callapiutils.FetchAndParseResult(u, &queryResult)
 	return queryResult, err

internal/inventaire/inventaireedition.go

@@ -2,38 +2,19 @@ package inventaire
 
 import (
 	"encoding/json"
-	"math"
+	"net/url"
-	"sort"
 	"strings"
 
 	"git.artlef.fr/PersonalLibraryManager/internal/callapiutils"
 )
 
-type InventaireEditionResult struct {
-	Results []InventaireEditionResultBook `json:"results"`
-	Count   int64                         `json:"count"`
-}
-
-type InventaireEditionResultBook struct {
-	Id          string `json:"uri"`
-	Title       string `json:"title"`
-	ISBN        string `json:"isbn"`
-	Publisher   string `json:"publisher"`
-	ReleaseDate string `json:"date"`
-	Image       string `json:"image"`
-	Lang        string `json:"lang"`
-}
-
-type inventaireReverseClaimsResult struct {
-	Uris []string `json:"uris"`
-}
-
 type inventaireEditionQueryResult struct {
 	Entities []inventaireEditionQueryEntity
 }
 
 type inventaireEditionQueryEntity struct {
 	WdId        string
+	WorkId      string
 	EditionId   string
 	Title       string
 	ISBN        string
@@ -66,6 +47,10 @@ func (i *inventaireEditionQueryResult) UnmarshalJSON(b []byte) error {
 			return err
 		}
 		if parsedEntity.Type == "edition" {
+			workId, err := parseStringArrayFieldInJsonRaw(parsedEntity.Claims, "wdt:P629")
+			if err != nil {
+				return err
+			}
 			editionId, err := parseStringArrayFieldInJsonRaw(parsedEntity.Claims, "wdt:P123")
 			if err != nil {
 				return err
@@ -95,6 +80,7 @@ func (i *inventaireEditionQueryResult) UnmarshalJSON(b []byte) error {
 			}
 			i.Entities = append(i.Entities, inventaireEditionQueryEntity{
 				WdId:        parsedEntity.WdId,
+				WorkId:      workId,
 				EditionId:   editionId,
 				Title:       label,
 				ISBN:        isbn,
@@ -125,80 +111,26 @@ func parseStringArrayFieldInJsonRaw(jsonRawMap map[string]json.RawMessage, key s
 	return s, err
 }
 
-func CallInventaireEdition(inventaireId string, lang string, limit int, offset int) (InventaireEditionResult, error) {
+func callInventaireEditionEntities(uris []string) (inventaireEditionQueryResult, error) {
-	var queryResult InventaireEditionResult
+	var queryResult inventaireEditionQueryResult
-	uris, err := callInventaireUris(inventaireId)
+	u, err := getInventaireEditionEntitiesUri(strings.Join(uris, "|"))
 	if err != nil {
 		return queryResult, err
 	}
-	queryResult.Count = int64(len(uris.Uris))
-	sort.Strings(uris.Uris)
-	limitedUris := uris.Uris
-	if limit != 0 {
-		l := len(uris.Uris)
-		startIndex := int(math.Min(float64(offset), float64(l)))
-		endIndex := int(math.Min(float64(limit+offset), float64(l)))
-		limitedUris = uris.Uris[startIndex:endIndex]
-	}
-	editionEntities, err := callInventaireEditionEntities(limitedUris)
-
-	if err != nil {
-		return queryResult, err
-	}
-
-	sortedEntities := editionEntities.Entities
-	sort.Slice(sortedEntities, func(i, j int) bool {
-		return sortedEntities[i].Uri < sortedEntities[j].Uri
-	})
-
-	for _, entity := range sortedEntities {
-		publisher := ""
-		if entity.EditionId != "" {
-			publisher, err = callInventairePublisherGetName(entity.EditionId, lang)
-			if err != nil {
-				return queryResult, err
-			}
-		}
-		queryResult.Results = append(queryResult.Results, InventaireEditionResultBook{
-			Id:          entity.Uri,
-			ISBN:        entity.ISBN,
-			Title:       entity.Title,
-			ReleaseDate: entity.ReleaseDate,
-			Image:       entity.Image,
-			Publisher:   publisher,
-			Lang:        entity.Lang,
-		})
-	}
-	return queryResult, err
-}
-
-func callInventaireUris(inventaireId string) (inventaireReverseClaimsResult, error) {
-	var queryResult inventaireReverseClaimsResult
-	u, err := computeInventaireApiUrl("entities")
-	if err != nil {
-		return queryResult, err
-	}
-	callapiutils.AddQueryParam(u, "action", "reverse-claims")
-	callapiutils.AddQueryParam(u, "property", "wdt:P629")
-	callapiutils.AddQueryParam(u, "value", inventaireId)
-
 	err = callapiutils.FetchAndParseResult(u, &queryResult)
 	return queryResult, err
 }
 
-func callInventaireEditionEntities(uris []string) (inventaireEditionQueryResult, error) {
+func getInventaireEditionEntitiesUri(uris string) (*url.URL, error) {
-	var queryResult inventaireEditionQueryResult
 	u, err := computeInventaireApiUrl("entities")
 	if err != nil {
-		return queryResult, err
+		return u, err
 	}
 
 	callapiutils.AddQueryParam(u, "action", "by-uris")
-	callapiutils.AddQueryParam(u, "uris", strings.Join(uris, "|"))
+	callapiutils.AddQueryParam(u, "uris", uris)
 
-	err = callapiutils.FetchAndParseResult(u, &queryResult)
+	return u, err
-
-	return queryResult, err
 }
 
 type inventaireEditionPublisherResult struct {

internal/inventaire/inventaireeditionfrombook.go

@@ -0,0 +1,88 @@
+package inventaire
+
+import (
+	"math"
+	"sort"
+
+	"git.artlef.fr/PersonalLibraryManager/internal/callapiutils"
+)
+
+type InventaireEditionResult struct {
+	Results []InventaireEditionResultBook `json:"results"`
+	Count   int64                         `json:"count"`
+}
+
+type InventaireEditionResultBook struct {
+	Id          string `json:"uri"`
+	Title       string `json:"title"`
+	ISBN        string `json:"isbn"`
+	Publisher   string `json:"publisher"`
+	ReleaseDate string `json:"date"`
+	Image       string `json:"image"`
+	Lang        string `json:"lang"`
+}
+
+type inventaireReverseClaimsResult struct {
+	Uris []string `json:"uris"`
+}
+
+func CallInventaireEditionFromWork(workId string, lang string, limit int, offset int) (InventaireEditionResult, error) {
+	var queryResult InventaireEditionResult
+	uris, err := callInventaireUris(workId)
+	if err != nil {
+		return queryResult, err
+	}
+	queryResult.Count = int64(len(uris.Uris))
+	sort.Strings(uris.Uris)
+	limitedUris := uris.Uris
+	if limit != 0 {
+		l := len(uris.Uris)
+		startIndex := int(math.Min(float64(offset), float64(l)))
+		endIndex := int(math.Min(float64(limit+offset), float64(l)))
+		limitedUris = uris.Uris[startIndex:endIndex]
+	}
+	editionEntities, err := callInventaireEditionEntities(limitedUris)
+
+	if err != nil {
+		return queryResult, err
+	}
+
+	sortedEntities := editionEntities.Entities
+	sort.Slice(sortedEntities, func(i, j int) bool {
+		return sortedEntities[i].Uri < sortedEntities[j].Uri
+	})
+
+	for _, entity := range sortedEntities {
+		publisher := ""
+		if entity.EditionId != "" {
+			publisher, err = callInventairePublisherGetName(entity.EditionId, lang)
+			if err != nil {
+				return queryResult, err
+			}
+		}
+		queryResult.Results = append(queryResult.Results, InventaireEditionResultBook{
+			Id:          entity.Uri,
+			ISBN:        entity.ISBN,
+			Title:       entity.Title,
+			ReleaseDate: entity.ReleaseDate,
+			Image:       entity.Image,
+			Publisher:   publisher,
+			Lang:        entity.Lang,
+		})
+	}
+	return queryResult, err
+}
+
+func callInventaireUris(workId string) (inventaireReverseClaimsResult, error) {
+	var queryResult inventaireReverseClaimsResult
+	u, err := computeInventaireApiUrl("entities")
+	if err != nil {
+		return queryResult, err
+	}
+	callapiutils.AddQueryParam(u, "action", "reverse-claims")
+	callapiutils.AddQueryParam(u, "property", "wdt:P629")
+	callapiutils.AddQueryParam(u, "value", workId)
+
+	err = callapiutils.FetchAndParseResult(u, &queryResult)
+	return queryResult, err
+}

internal/inventaire/inventaireeditionfromid.go

@@ -0,0 +1,62 @@
+package inventaire
+
+import (
+	"fmt"
+)
+
+type InventaireEditionDetailedSingleResult struct {
+	Id          string
+	Title       string
+	Description string
+	Author      *InventaireAuthorResult
+	ISBN        string
+	Publisher   string
+	ReleaseDate string
+	Image       string
+	Lang        string
+}
+
+func CallInventaireEdition(inventaireId string, lang string) (InventaireEditionDetailedSingleResult, error) {
+	var result InventaireEditionDetailedSingleResult
+	editionQueryResults, err := callInventaireEditionEntities([]string{inventaireId})
+	if err != nil {
+		return result, err
+	}
+	var editionQueryResult inventaireEditionQueryEntity
+	if len(editionQueryResults.Entities) < 1 {
+		return result, fmt.Errorf("No edition found on inventaire for id %s", inventaireId)
+	}
+
+	editionQueryResult = editionQueryResults.Entities[0]
+
+	var publisher string
+	if editionQueryResult.EditionId != "" {
+		publisher, err = callInventairePublisherGetName(editionQueryResult.EditionId, lang)
+		if err != nil {
+			return result, err
+		}
+	}
+
+	var author *InventaireAuthorResult
+	var description string
+	if editionQueryResult.WorkId != "" {
+		workQueryResult, err := callInventaireBook(editionQueryResult.WorkId, lang)
+		if err != nil {
+			return result, err
+		}
+		author = workQueryResult.Author
+		description = workQueryResult.Description
+	}
+	result = InventaireEditionDetailedSingleResult{
+		Id:          editionQueryResult.Uri,
+		Title:       editionQueryResult.Title,
+		Author:      author,
+		Description: description,
+		ISBN:        editionQueryResult.ISBN,
+		Publisher:   publisher,
+		ReleaseDate: editionQueryResult.ReleaseDate,
+		Image:       editionQueryResult.Image,
+		Lang:        editionQueryResult.Lang,
+	}
+	return result, err
+}

internal/routes/bookpostimport.go

@@ -26,12 +26,12 @@ func PostImportBookHandler(ac appcontext.AppContext) {
 		myvalidator.ReturnErrorsAsJsonResponse(&ac, err)
 		return
 	}
-	inventaireBook, err := inventaire.CallInventaireBook(request.InventaireID, request.Lang)
+	inventaireEdition, err := inventaire.CallInventaireEdition(request.InventaireID, request.Lang)
 	if err != nil {
 		myvalidator.ReturnErrorsAsJsonResponse(&ac, err)
 		return
 	}
-	book, err := saveInventaireBookToDb(ac, inventaireBook, &user)
+	book, err := saveInventaireBookToDb(ac, inventaireEdition, &user)
 	if err != nil {
 		myvalidator.ReturnErrorsAsJsonResponse(&ac, err)
 		return
@@ -39,15 +39,15 @@ func PostImportBookHandler(ac appcontext.AppContext) {
 	ac.C.JSON(200, gin.H{"id": book.ID})
 }
 
-func saveInventaireBookToDb(ac appcontext.AppContext, inventaireBook inventaire.InventaireBookResult, user *model.User) (*model.Book, error) {
+func saveInventaireBookToDb(ac appcontext.AppContext, inventaireEdition inventaire.InventaireEditionDetailedSingleResult, user *model.User) (*model.Book, error) {
-	author, err := fetchOrCreateInventaireAuthor(ac, inventaireBook.Author)
+	author, err := fetchOrCreateInventaireAuthor(ac, inventaireEdition.Author)
 	if err != nil {
 		return nil, err
 	}
 	book := model.Book{
-		Title:            inventaireBook.Title,
+		Title:            inventaireEdition.Title,
-		SmallDescription: inventaireBook.Description,
+		SmallDescription: inventaireEdition.Description,
-		InventaireID:     inventaireBook.ID,
+		InventaireID:     inventaireEdition.Id,
 		Author:           *author,
 		AddedBy:          *user,
 	}

internal/routes/booksinventaireget.go

@@ -29,7 +29,7 @@ func GetInventaireBooks(ac appcontext.AppContext) {
 		return
 	}
 
-	inventaireEditionResult, err := inventaire.CallInventaireEdition(workId, params.Lang, limit, offset)
+	inventaireEditionResult, err := inventaire.CallInventaireEditionFromWork(workId, params.Lang, limit, offset)
 	if err != nil {
 		myvalidator.ReturnErrorsAsJsonResponse(&ac, err)
 		return