Null fix for notifications

Author: maskara

app/src/main/java/fr/free/nrw/commons/mwapi/ApacheHttpClientMediaWikiApi.java

@@ -23,7 +23,6 @@
 import org.apache.http.util.EntityUtils;
 import org.mediawiki.api.ApiResult;
 import org.mediawiki.api.MWApi;
-import org.w3c.dom.Node;
 import org.w3c.dom.NodeList;
 
 import java.io.IOException;
@@ -47,14 +46,6 @@
 import io.reactivex.Single;
 import timber.log.Timber;
 
-import static fr.free.nrw.commons.notification.NotificationType.THANK_YOU_EDIT;
-import static fr.free.nrw.commons.notification.NotificationType.UNKNOWN;
-import static fr.free.nrw.commons.notification.NotificationUtils.getNotificationFromApiResult;
-import static fr.free.nrw.commons.notification.NotificationUtils.getNotificationType;
-import static fr.free.nrw.commons.notification.NotificationUtils.getNotificationsFromBundle;
-import static fr.free.nrw.commons.notification.NotificationUtils.isBundledNotification;
-import static fr.free.nrw.commons.notification.NotificationUtils.isCommonsNotification;
-
 /**
  * @author Addshore
  */
@@ -439,14 +430,16 @@ public List<Notification> getNotifications() {
                     .param("format", "xml")
                     .param("meta", "notifications")
                     .param("notformat", "model")
-                    //.param("notfilter", "!read")
                     .get()
                     .getNode("/api/query/notifications/list");
         } catch (IOException e) {
             Timber.e("Failed to obtain searchCategories", e);
         }
 
-        if (notificationNode == null) {
+        if (notificationNode == null
+                || notificationNode.getDocument() == null
+                || notificationNode.getDocument().getChildNodes() == null
+                || notificationNode.getDocument().getChildNodes().getLength() == 0) {
             return new ArrayList<>();
         }
 

app/src/main/java/fr/free/nrw/commons/notification/NotificationActivity.java

@@ -24,6 +24,7 @@
 import fr.free.nrw.commons.R;
 import fr.free.nrw.commons.Utils;
 import fr.free.nrw.commons.theme.NavigationBaseActivity;
+import fr.free.nrw.commons.utils.ViewUtil;
 import io.reactivex.Observable;
 import io.reactivex.android.schedulers.AndroidSchedulers;
 import io.reactivex.schedulers.Schedulers;
@@ -81,6 +82,7 @@ private void addNotifications() {
                         progressBar.setVisibility(View.GONE);
                     }, throwable -> {
                         Timber.e(throwable, "Error occurred while loading notifications");
+                        ViewUtil.showSnackbar(this, R.string.error_notifications);
                         progressBar.setVisibility(View.GONE);
                     });
         } else {

app/src/main/java/fr/free/nrw/commons/notification/NotificationRenderer.java

@@ -20,7 +20,6 @@
 
 public class NotificationRenderer extends Renderer<Notification> {
     @BindView(R.id.title) ReadMoreTextView title;
-    @BindView(R.id.description) ReadMoreTextView description;
     @BindView(R.id.time) TextView time;
     @BindView(R.id.icon) ImageView icon;
     private NotificationClicked listener;
@@ -49,12 +48,9 @@ protected View inflate(LayoutInflater layoutInflater, ViewGroup viewGroup) {
     public void render() {
         Notification notification = getContent();
         StringBuilder str = new StringBuilder(notification.notificationText);
-        str.append(" " );
+        str.append(" ");
         title.setText(str);
         time.setText(notification.date);
-        StringBuilder desc = new StringBuilder(notification.description);
-        desc.append(" ");
-        description.setText(desc);
         switch (notification.notificationType) {
             case THANK_YOU_EDIT:
                 icon.setImageResource(R.drawable.ic_edit_black_24dp);

app/src/main/res/values/strings.xml

@@ -247,4 +247,6 @@
   <string name="about_rate_us"><u>Rate Us</u></string>
   <string name="about_faq">Frequently Asked Questions</string>
   <string name="welcome_skip_button">Skip Tutorial</string>
+
+  <string name="error_notifications">Error fetching notifications</string>
 </resources>