Dbrant has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/172860

Change subject: Fix possible crash in full-text search.
......................................................................

Fix possible crash in full-text search.

Change-Id: I1c86be495aa7ecfb6640c269016040ca06286364
---
M wikipedia/src/main/java/org/wikipedia/search/FullSearchFragment.java
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/apps/android/wikipedia 
refs/changes/60/172860/1

diff --git 
a/wikipedia/src/main/java/org/wikipedia/search/FullSearchFragment.java 
b/wikipedia/src/main/java/org/wikipedia/search/FullSearchFragment.java
index f08042a..fb0c0a9 100644
--- a/wikipedia/src/main/java/org/wikipedia/search/FullSearchFragment.java
+++ b/wikipedia/src/main/java/org/wikipedia/search/FullSearchFragment.java
@@ -243,7 +243,8 @@
     private void getWikidataDescriptions(List<FullSearchResult> results) {
         List<String> idList = new ArrayList<String>();
         for (FullSearchResult r : results) {
-            if (app.getWikidataCache().get(r.getWikiBaseId()) == null) {
+            if (!TextUtils.isEmpty(r.getWikiBaseId())
+                    && app.getWikidataCache().get(r.getWikiBaseId()) == null) {
                 // not in our cache yet
                 idList.add(r.getWikiBaseId());
             }

-- 
To view, visit https://gerrit.wikimedia.org/r/172860
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1c86be495aa7ecfb6640c269016040ca06286364
Gerrit-PatchSet: 1
Gerrit-Project: apps/android/wikipedia
Gerrit-Branch: master
Gerrit-Owner: Dbrant <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to