Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Manas Tungare's isbn-to-bibtex fetcher in the queue #5613

Merged
merged 1 commit into from
Nov 16, 2019
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ We refer to [GitHub issues](https://github.com/JabRef/jabref/issues) by using `#
### Removed

- We removed some obsolete notifications. [#5555](https://github.com/JabRef/jabref/issues/5555)

- 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."



Expand Down
11 changes: 2 additions & 9 deletions src/main/java/org/jabref/logic/importer/fetcher/IsbnFetcher.java
Original file line number Diff line number Diff line change
Expand Up @@ -47,17 +47,10 @@ public Optional<BibEntry> performSearchById(String identifier) throws FetcherExc

IsbnViaEbookDeFetcher isbnViaEbookDeFetcher = new IsbnViaEbookDeFetcher(importFormatPreferences);
Optional<BibEntry> bibEntry = isbnViaEbookDeFetcher.performSearchById(identifier);

// nothing found at ebook.de, try chimbori.com
if (!bibEntry.isPresent()) {
LOGGER.debug("No entry found at ebook.de try chimbori.com");
IsbnViaChimboriFetcher isbnViaChimboriFetcher = new IsbnViaChimboriFetcher(importFormatPreferences);
bibEntry = isbnViaChimboriFetcher.performSearchById(identifier);
}

//nothing found at ebook.de and chimbori.com, try ottobib
// nothing found at ebook.de: try ottobib
if (!bibEntry.isPresent()) {
LOGGER.debug("No entry found at ebook.de and chimbori.com try ottobib");
LOGGER.debug("No entry found at ebook.de; trying ottobib");
IsbnViaOttoBibFetcher isbnViaOttoBibFetcher = new IsbnViaOttoBibFetcher(importFormatPreferences);
bibEntry = isbnViaOttoBibFetcher.performSearchById(identifier);
}
Expand Down

This file was deleted.

2 changes: 0 additions & 2 deletions src/test/java/org/jabref/logic/importer/WebFetchersTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
import java.util.stream.Collectors;

import org.jabref.logic.importer.fetcher.AbstractIsbnFetcher;
import org.jabref.logic.importer.fetcher.IsbnViaChimboriFetcher;
import org.jabref.logic.importer.fetcher.IsbnViaEbookDeFetcher;
import org.jabref.logic.importer.fetcher.IsbnViaOttoBibFetcher;
import org.jabref.logic.importer.fetcher.MrDLibFetcher;
Expand Down Expand Up @@ -41,7 +40,6 @@ void getIdBasedFetchersReturnsAllFetcherDerivingFromIdBasedFetcher() throws Exce
expected.remove(AbstractIsbnFetcher.class);
expected.remove(IdBasedParserFetcher.class);
// Remove special ISBN fetcher since we don't want to expose them to the user
expected.remove(IsbnViaChimboriFetcher.class);
expected.remove(IsbnViaEbookDeFetcher.class);
expected.remove(IsbnViaOttoBibFetcher.class);
assertEquals(expected, getClasses(idFetchers));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -97,10 +97,10 @@ void searchByEntryWithISBNSuccessful() throws FetcherException {

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

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ public void searchByIdSuccessfulWithLongISBN() throws FetcherException {
@Test
@Override
public void authorsAreCorrectlyFormatted() throws Exception {
BibEntry bibEntry = new BibEntry();
bibEntry = new BibEntry();
bibEntry.setType(StandardEntryType.Book);
bibEntry.setCiteKey("dumas2018fundamentals");
bibEntry.setField(StandardField.TITLE, "Fundamentals of business process management");
Expand Down