Dbrant has uploaded a new change for review.

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

Change subject: Fix NPE crash in SearchFragment.
......................................................................

Fix NPE crash in SearchFragment.

In the case when the user taps the "Go" button on the soft keyboard, the
closeSearch() method was being called twice! (it's called automatically
from navigateToTitle())

Bug: T146142
Change-Id: Ia835403f4728bef64370f65df41bd430ea597564
---
M app/src/main/java/org/wikipedia/search/SearchFragment.java
1 file changed, 0 insertions(+), 1 deletion(-)


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

diff --git a/app/src/main/java/org/wikipedia/search/SearchFragment.java 
b/app/src/main/java/org/wikipedia/search/SearchFragment.java
index 383fc59..5da34c1 100644
--- a/app/src/main/java/org/wikipedia/search/SearchFragment.java
+++ b/app/src/main/java/org/wikipedia/search/SearchFragment.java
@@ -106,7 +106,6 @@
             }
             if (firstResult != null) {
                 navigateToTitle(firstResult, false, 0);
-                closeSearch();
             }
             return true;
         }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia835403f4728bef64370f65df41bd430ea597564
Gerrit-PatchSet: 1
Gerrit-Project: apps/android/wikipedia
Gerrit-Branch: master
Gerrit-Owner: Dbrant <dbr...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to