Fixed weird behaviour if scroll was too fast (description text was me…

…ssed) Signed-off-by: Thomas Gaudin <[email protected]>
WikiJourney · Dec 6, 2015 · 4ed3678 · 4ed3678 · nymous · Dec 7, 2015
1 parent ee3067b
commit 4ed3678
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/app/src/main/java/com/wikijourney/wikijourney/functions/PoiListAdapter.java b/app/src/main/java/com/wikijourney/wikijourney/functions/PoiListAdapter.java
@@ -102,7 +102,7 @@ public void onBindViewHolder(ViewHolder holder, int position) {
         String mPoiDescription = mPoiList.get(position).getDescription();
 
         // We add a Listener, so that a tap on the card opens a WebView to the WP page
-        if (mPoiSitelink != null) {
+        if (mPoiSitelink != null && !"".equals(mPoiSitelink)) {
             holder.mReadMoreButton.setOnClickListener(new View.OnClickListener() {
                 @Override
                 public void onClick(View view) {
@@ -124,15 +124,17 @@ public void onClick(View view) {
 
             if (mPoiDescription == null) {
                 downloadWikipediaExtract(holder, poiName, position);
-            } else if(!"".equals(mPoiDescription)){
+            } else if ("".equals(mPoiDescription)) {
+                holder.mPoiDescription.setVisibility(View.GONE);
+            } else {
                 holder.mPoiDescription.setVisibility(View.VISIBLE);
                 holder.mPoiDescription.setText(mPoiDescription);
             }
         }
 
         holder.mPoiPicture.setImageDrawable(ContextCompat.getDrawable(context, R.drawable.logo_cut));
         // We use Picasso to download the Wikipedia article image
-        if (mPoiImageUrl != null && !mPoiImageUrl.equals("")) {
+        if (mPoiImageUrl != null && !"".equals(mPoiImageUrl)) {
             displayArticleImage(holder, mPoiImageUrl);
         }