Just-in-case null pointer check in person list fragment (should never happen)

astrid/plugin-src/com/todoroo/astrid/people/PersonViewFragment.java

@@ -139,18 +139,20 @@ public class PersonViewFragment extends TaskListFragment {
     }
 
     public void handleStatusButtonClicked() {
-        String status = user.getValue(User.STATUS);
+        if (user != null) { // Just in case
-        if (TextUtils.isEmpty(status) || "null".equals(status)) { // Add friend case //$NON-NLS-1$
+            String status = user.getValue(User.STATUS);
-            user.setValue(User.PENDING_STATUS, User.PENDING_REQUEST);
+            if (TextUtils.isEmpty(status) || "null".equals(status)) { // Add friend case //$NON-NLS-1$
-        } else if (User.STATUS_OTHER_PENDING.equals(status)) { // Accept friend case
+                user.setValue(User.PENDING_STATUS, User.PENDING_REQUEST);
-            user.setValue(User.PENDING_STATUS, User.PENDING_APPROVE);
+            } else if (User.STATUS_OTHER_PENDING.equals(status)) { // Accept friend case
-        }
+                user.setValue(User.PENDING_STATUS, User.PENDING_APPROVE);
-
+            }
-        ContentValues setValues = user.getSetValues();
+
-        if (setValues != null && setValues.containsKey(User.PENDING_STATUS.name)) {
+            ContentValues setValues = user.getSetValues();
-            userDao.saveExisting(user);
+            if (setValues != null && setValues.containsKey(User.PENDING_STATUS.name)) {
-            userStatusButton.setVisibility(View.GONE);
+                userDao.saveExisting(user);
-            refreshData(false);
+                userStatusButton.setVisibility(View.GONE);
+                refreshData(false);
+            }
         }
     }