diff --git a/bookwyrm/importer.py b/bookwyrm/importer.py index b6e959cb3..a12884007 100644 --- a/bookwyrm/importer.py +++ b/bookwyrm/importer.py @@ -12,7 +12,7 @@ class Importer: service = 'Unknown' delimiter = ',' encoding = 'UTF-8' - mandatory_fields = ['ISBN13', 'Title', 'Author'] + mandatory_fields = ['Title', 'Author'] def create_job(self, user, csv_file, include_reviews, privacy): ''' check over a csv and creates a database entry for the job''' @@ -23,7 +23,7 @@ class Importer: ) for index, entry in enumerate(list(csv.DictReader(csv_file, delimiter=self.delimiter ))): if not all(x in entry for x in self.mandatory_fields): - raise ValueError('Author, title, and isbn must be in data.') + raise ValueError('Author and title must be in data.') entry = self.parse_fields(entry) self.save_item(job, index, entry) return job diff --git a/bookwyrm/librarything_import.py b/bookwyrm/librarything_import.py index 019573acb..0584daad9 100644 --- a/bookwyrm/librarything_import.py +++ b/bookwyrm/librarything_import.py @@ -12,8 +12,8 @@ class LibrarythingImporter(Importer): service = 'LibraryThing' delimiter = '\t' encoding = 'ISO-8859-1' - # mandatory_fields : fields matching the book ISBN13, title and author - mandatory_fields = ['ISBN', 'Title', 'Primary Author'] + # mandatory_fields : fields matching the book title and author + mandatory_fields = ['Title', 'Primary Author'] def parse_fields(self, initial): data = {} @@ -23,7 +23,10 @@ class LibrarythingImporter(Importer): data['Author'] = initial['Primary Author'] data['ISBN13'] = initial['ISBN'] data['My Review'] = initial['Review'] - data['My Rating'] = math.ceil(float(initial['Rating'])) + if initial['Rating']: + data['My Rating'] = math.ceil(float(initial['Rating'])) + else: + data['My Rating'] = '' data['Date Added'] = re.sub('\[|\]', '', initial['Entry Date']) data['Date Started'] = re.sub('\[|\]', '', initial['Date Started']) data['Date Read'] = re.sub('\[|\]', '', initial['Date Read']) diff --git a/bookwyrm/templates/import.html b/bookwyrm/templates/import.html index dd60ebff6..a7fde76df 100644 --- a/bookwyrm/templates/import.html +++ b/bookwyrm/templates/import.html @@ -10,7 +10,7 @@ <p>Data source</p> <div class="select {{ class }}"> <select name="source" id="source"> - <option value="GoodReads" {% if current == 'LibraryThing' %}selected{% endif %}> + <option value="GoodReads" {% if current == 'GoodReads' %}selected{% endif %}> GoodReads </option> <option value="LibraryThing" {% if current == 'LibraryThing' %}selected{% endif %}> diff --git a/bookwyrm/tests/data/librarything.tsv b/bookwyrm/tests/data/librarything.tsv index f0dca77fb..a707f2a9b 100644 --- a/bookwyrm/tests/data/librarything.tsv +++ b/bookwyrm/tests/data/librarything.tsv @@ -1,4 +1,4 @@ Book Id Title Sort Character Primary Author Primary Author Role Secondary Author Secondary Author Roles Publication Date Review Rating Comment Private Comment Summary Media Physical Description Weight Height Thickness Length Dimensions Page Count LCCN Acquired Date Started Date Read Barcode BCID Tags Collections Languages Original Languages LC Classification ISBN ISBNs Subjects Dewey Decimal Dewey Wording Other Call Number Copies Source Entry Date From Where OCLC Work id Lending Patron Lending Status Lending Start Lending End 5498194 Marelle 1 Cortázar, Julio Gallimard (1979), Poche 1979 chef d'oeuvre 4.5 Marelle by Julio Cortázar (1979) Broché 590 p.; 7.24 inches 1.28 pounds 7.24 inches 1.26 inches 4.96 inches 7.24 x 4.96 x 1.26 inches 590 [2007-04-16] [2007-05-08] roman, espagnol, expérimental, bohème, philosophie Your library French Spanish PQ7797 .C7145 [2070291340] 2070291340, 9782070291342 Cort�azar, Julio. Rayuela 863 Literature > Spanish And Portuguese > Spanish fiction 1 Amazon.fr [2006-08-09] 57814 5015319 Le grand incendie de Londres: Récit, avec incises et bifurcations, 1985-1987 (Fiction & Cie) 1 Roubaud, Jacques Seuil (1989), Unknown Binding 1989 5 Le grand incendie de Londres: Récit, avec incises et bifurcations, 1985-1987 (Fiction & Cie) by Jacques Roubaud (1989) Broché 411 p.; 7.72 inches 0.88 pounds 7.72 inches 1.02 inches 5.43 inches 7.72 x 5.43 x 1.02 inches 411 Your library English PQ2678 .O77 [2020104725] 2020104725, 9782020104722 Autobiographical fiction|Roubaud, Jacques > Fiction 813 American And Canadian > Fiction > Literature 1 Amazon.com [2006-07-25] 478910 -5015399 Le Maître et Marguerite 1 Boulgakov, Mikhaïl Pocket (1994), Poche 1994 5 Le Maître et Marguerite by Mikhaïl Boulgakov (1994) Broché 579 p.; 7.09 inches 0.66 pounds 7.09 inches 1.18 inches 4.33 inches 7.09 x 4.33 x 1.18 inches 579 Your library French PG3476 .B78 [2266062328] 2266062328, 9782266062329 Allegories|Bulgakov|Good and evil > Fiction|Humanities|Jerusalem > Fiction|Jesus Christ > Fiction|Literature|Mental illness > Fiction|Moscow (Russia) > Fiction|Novel|Pilate, Pontius, 1st cent. > Fiction|Political fiction|Russia > Fiction|Russian fiction|Russian publications (Form Entry)|Soviet Union > History > 1925-1953 > Fiction|literature 891.7342 1917-1945 > 1917-1991 (USSR) > Literature > Literature of other Indo-European languages > Other Languages > Russian > Russian Fiction 1 Amazon.fr [2006-07-25] 10151 +5015399 Le Maître et Marguerite 1 Boulgakov, Mikhaïl Pocket (1994), Poche 1994 Le Maître et Marguerite by Mikhaïl Boulgakov (1994) Broché 579 p.; 7.09 inches 0.66 pounds 7.09 inches 1.18 inches 4.33 inches 7.09 x 4.33 x 1.18 inches 579 Your library French PG3476 .B78 [2266062328] 2266062328, 9782266062329 Allegories|Bulgakov|Good and evil > Fiction|Humanities|Jerusalem > Fiction|Jesus Christ > Fiction|Literature|Mental illness > Fiction|Moscow (Russia) > Fiction|Novel|Pilate, Pontius, 1st cent. > Fiction|Political fiction|Russia > Fiction|Russian fiction|Russian publications (Form Entry)|Soviet Union > History > 1925-1953 > Fiction|literature 891.7342 1917-1945 > 1917-1991 (USSR) > Literature > Literature of other Indo-European languages > Other Languages > Russian > Russian Fiction 1 Amazon.fr [2006-07-25] 10151