isbn.equals("") || isbn.equals(null) というコードは isbn == null || isbn.equals("") を意図しているように見えたので、同等の意味になる TextUtils.isEmpty(isbn) に置き換えました。

isbn.equals(null) が true になることは無いし、isbn == null の意味だとしても、 isbn.equals("") よりも先に判定しないと isbn.equals("") で NullPointerException が発生していまいます。
Merge pull request #1 from zaki50/master

意図通りでないように見えるコードがあったので修正しました。