Checks for null picture and pictureBorder views (for gtasks, we should update those views to work with pictures)

pull/14/head
Sam Bosley 13 years ago
parent 1e30f4cfdf
commit 28d910a9e8

@ -390,18 +390,22 @@ public class TaskAdapter extends CursorAdapter implements Filterable {
// image view // image view
final AsyncImageView pictureView = viewHolder.picture; { final AsyncImageView pictureView = viewHolder.picture; {
if(task.getValue(Task.USER_ID) == 0) { if (pictureView != null) {
pictureView.setVisibility(View.GONE); if(task.getValue(Task.USER_ID) == 0) {
viewHolder.pictureBorder.setVisibility(View.GONE); pictureView.setVisibility(View.GONE);
} else { if (viewHolder.pictureBorder != null)
pictureView.setVisibility(View.VISIBLE); viewHolder.pictureBorder.setVisibility(View.GONE);
viewHolder.pictureBorder.setVisibility(View.VISIBLE); } else {
pictureView.setUrl(null); pictureView.setVisibility(View.VISIBLE);
try { if (viewHolder.pictureBorder != null)
JSONObject user = new JSONObject(task.getValue(Task.USER)); viewHolder.pictureBorder.setVisibility(View.VISIBLE);
pictureView.setUrl(user.optString("picture")); //$NON-NLS-1$ pictureView.setUrl(null);
} catch (JSONException e) { try {
Log.w("astrid", "task-adapter-image", e); //$NON-NLS-1$ //$NON-NLS-2$ JSONObject user = new JSONObject(task.getValue(Task.USER));
pictureView.setUrl(user.optString("picture")); //$NON-NLS-1$
} catch (JSONException e) {
Log.w("astrid", "task-adapter-image", e); //$NON-NLS-1$ //$NON-NLS-2$
}
} }
} }
} }
@ -416,9 +420,10 @@ public class TaskAdapter extends CursorAdapter implements Filterable {
} else { } else {
checkBoxView.setButtonDrawable(IMPORTANCE_RESOURCES[value]); checkBoxView.setButtonDrawable(IMPORTANCE_RESOURCES[value]);
} }
if (pictureView.getVisibility() == View.VISIBLE) { if (pictureView != null && pictureView.getVisibility() == View.VISIBLE) {
checkBoxView.setVisibility(View.INVISIBLE); checkBoxView.setVisibility(View.INVISIBLE);
viewHolder.pictureBorder.setBackgroundResource(IMPORTANCE_RESOURCES_LARGE[value]); if (viewHolder.pictureBorder != null)
viewHolder.pictureBorder.setBackgroundResource(IMPORTANCE_RESOURCES_LARGE[value]);
} else { } else {
checkBoxView.setVisibility(View.VISIBLE); checkBoxView.setVisibility(View.VISIBLE);
} }

Loading…
Cancel
Save