Skip to content

Commit 7f9a744

Browse files
committed
Remove Manas Tungare's isbn-to-bibtex fetcher in the queue
1 parent 8f7188d commit 7f9a744

File tree

7 files changed

+6
-200
lines changed

7 files changed

+6
-200
lines changed

CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ We refer to [GitHub issues](https://github.com/JabRef/jabref/issues) by using `#
6666
### Removed
6767

6868
- We removed some obsolete notifications. [#5555](https://github.com/JabRef/jabref/issues/5555)
69-
69+
- We removed an internal step in the [ISBN-to-BibTeX fetcher](https://docs.jabref.org/import-using-publication-identifiers/isbntobibtex): The [ISBN to BibTeX Converter](https://manas.tungare.name/software/isbn-to-bibtex) by [@manastungare](https://github.com/manastungare) is not used anymore, because it is offline: "people using this tool have not been generating enough sales for Amazon."
7070

7171

7272

src/main/java/org/jabref/logic/importer/fetcher/IsbnFetcher.java

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -47,17 +47,10 @@ public Optional<BibEntry> performSearchById(String identifier) throws FetcherExc
4747

4848
IsbnViaEbookDeFetcher isbnViaEbookDeFetcher = new IsbnViaEbookDeFetcher(importFormatPreferences);
4949
Optional<BibEntry> bibEntry = isbnViaEbookDeFetcher.performSearchById(identifier);
50-
51-
// nothing found at ebook.de, try chimbori.com
52-
if (!bibEntry.isPresent()) {
53-
LOGGER.debug("No entry found at ebook.de try chimbori.com");
54-
IsbnViaChimboriFetcher isbnViaChimboriFetcher = new IsbnViaChimboriFetcher(importFormatPreferences);
55-
bibEntry = isbnViaChimboriFetcher.performSearchById(identifier);
56-
}
5750

58-
//nothing found at ebook.de and chimbori.com, try ottobib
51+
// nothing found at ebook.de: try ottobib
5952
if (!bibEntry.isPresent()) {
60-
LOGGER.debug("No entry found at ebook.de and chimbori.com try ottobib");
53+
LOGGER.debug("No entry found at ebook.de; trying ottobib");
6154
IsbnViaOttoBibFetcher isbnViaOttoBibFetcher = new IsbnViaOttoBibFetcher(importFormatPreferences);
6255
bibEntry = isbnViaOttoBibFetcher.performSearchById(identifier);
6356
}

src/main/java/org/jabref/logic/importer/fetcher/IsbnViaChimboriFetcher.java

Lines changed: 0 additions & 93 deletions
This file was deleted.

src/test/java/org/jabref/logic/importer/WebFetchersTest.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
import java.util.stream.Collectors;
66

77
import org.jabref.logic.importer.fetcher.AbstractIsbnFetcher;
8-
import org.jabref.logic.importer.fetcher.IsbnViaChimboriFetcher;
98
import org.jabref.logic.importer.fetcher.IsbnViaEbookDeFetcher;
109
import org.jabref.logic.importer.fetcher.IsbnViaOttoBibFetcher;
1110
import org.jabref.logic.importer.fetcher.MrDLibFetcher;
@@ -41,7 +40,6 @@ void getIdBasedFetchersReturnsAllFetcherDerivingFromIdBasedFetcher() throws Exce
4140
expected.remove(AbstractIsbnFetcher.class);
4241
expected.remove(IdBasedParserFetcher.class);
4342
// Remove special ISBN fetcher since we don't want to expose them to the user
44-
expected.remove(IsbnViaChimboriFetcher.class);
4543
expected.remove(IsbnViaEbookDeFetcher.class);
4644
expected.remove(IsbnViaOttoBibFetcher.class);
4745
assertEquals(expected, getClasses(idFetchers));

src/test/java/org/jabref/logic/importer/fetcher/IsbnFetcherTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -97,10 +97,10 @@ void searchByEntryWithISBNSuccessful() throws FetcherException {
9797

9898
/**
9999
* This test searches for a valid ISBN. See https://www.amazon.de/dp/3728128155/?tag=jabref-21 However, this ISBN is
100-
* not available on ebook.de. The fetcher should something as it falls back to Chimbori
100+
* not available on ebook.de. The fetcher should something as it falls back to OttoBib
101101
*/
102102
@Test
103-
void searchForIsbnAvailableAtChimboriButNonOnEbookDe() throws FetcherException {
103+
void searchForIsbnAvailableAtOttoBibButNonOnEbookDe() throws FetcherException {
104104
Optional<BibEntry> fetchedEntry = fetcher.performSearchById("3728128155");
105105
assertNotEquals(Optional.empty(), fetchedEntry);
106106
}

src/test/java/org/jabref/logic/importer/fetcher/IsbnViaChimboriFetcherTest.java

Lines changed: 0 additions & 92 deletions
This file was deleted.

src/test/java/org/jabref/logic/importer/fetcher/IsbnViaOttoBibFetcherTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ public void searchByIdSuccessfulWithLongISBN() throws FetcherException {
6565
@Test
6666
@Override
6767
public void authorsAreCorrectlyFormatted() throws Exception {
68-
BibEntry bibEntry = new BibEntry();
68+
bibEntry = new BibEntry();
6969
bibEntry.setType(StandardEntryType.Book);
7070
bibEntry.setCiteKey("dumas2018fundamentals");
7171
bibEntry.setField(StandardField.TITLE, "Fundamentals of business process management");

0 commit comments

Comments
 (0)