From 63b6942f6b2649c70c30cdced6c033ff2607724f Mon Sep 17 00:00:00 2001 From: Nick White Date: Mon, 21 Mar 2022 16:26:55 +0000 Subject: rescribe: move getBookIdFromUrl() to gbook.go, and add tests for it --- cmd/rescribe/gui.go | 47 ----------------------------------------------- 1 file changed, 47 deletions(-) (limited to 'cmd/rescribe/gui.go') diff --git a/cmd/rescribe/gui.go b/cmd/rescribe/gui.go index a75b816..a9d5fbb 100644 --- a/cmd/rescribe/gui.go +++ b/cmd/rescribe/gui.go @@ -13,7 +13,6 @@ import ( "log" "os" "path/filepath" - "regexp" "strings" "fyne.io/fyne/v2" @@ -43,52 +42,6 @@ var trainingNames = map[string]string{ "rescribev8_fast": "Latin (early printing)", } -// getBookIdFromUrl returns a 12 character Google Book ID from -// a Google URL, or an error if one can't be found. -// Example URLs: -// https://books.google.it/books?id=QjQepCuN8JYC -// https://www.google.it/books/edition/_/VJbr-Oe2au0C -func getBookIdFromUrl(url string) (string, error) { - lurl := strings.ToLower(url) - if len(url) == 12 && !strings.ContainsAny(url, "?/:") { - return url, nil - } - - matchUrl, err := regexp.MatchString("https://www.google.[^\\/]*/books/", url) - if err != nil { - return "", err - } - - if strings.HasPrefix(lurl, "https://books.google") { - start := strings.Index(lurl, "?id=") - if start == -1 { - start = strings.Index(lurl, "&id=") - } - - if start >= 0 { - start += 4 - if len(url[start:]) < 12 { - return "", fmt.Errorf("Could not find book ID in URL") - } - return url[start : start+12], nil - } - - return "", fmt.Errorf("Could not find book ID in URL") - } - if matchUrl == true { - start := strings.Index(lurl, "edition/_/") - - if start >= 0 { - start += 10 - if len(url[start:]) < 12 { - return "", fmt.Errorf("Could not find book ID in URL") - } - return url[start : start+12], nil - } - } - return "", fmt.Errorf("Could not find book ID in URL") -} - // copyStdoutToChan creates a pipe to copy anything written // to the file also to a rune channel. func copyStdoutToChan() (chan rune, error) { -- cgit v1.2.1-24-ge1ad