Skip to content

Commit 40c1ea3

Browse files
authored
Merge pull request commons-app#681 from dbrant/crash2
Fix crash when uploading multiple photos.
2 parents be62646 + 5923e9a commit 40c1ea3

File tree

2 files changed

+7
-10
lines changed

2 files changed

+7
-10
lines changed

app/src/main/java/fr/free/nrw/commons/contributions/ContributionsListAdapter.java

-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ public View newView(Context context, Cursor cursor, ViewGroup viewGroup) {
2424
return parent;
2525
}
2626

27-
//FIXME: Potential cause of wrong image display bug
2827
@Override
2928
public void bindView(View view, Context context, Cursor cursor) {
3029
final ContributionViewHolder views = (ContributionViewHolder)view.getTag();

app/src/main/java/fr/free/nrw/commons/upload/MultipleUploadListFragment.java

+7-9
Original file line numberDiff line numberDiff line change
@@ -24,12 +24,12 @@
2424
import android.widget.RelativeLayout;
2525
import android.widget.TextView;
2626

27-
import fr.free.nrw.commons.MediaWikiImageView;
27+
import com.facebook.drawee.view.SimpleDraweeView;
28+
2829
import fr.free.nrw.commons.R;
2930
import fr.free.nrw.commons.contributions.Contribution;
3031
import fr.free.nrw.commons.media.MediaDetailPagerFragment;
3132

32-
3333
public class MultipleUploadListFragment extends Fragment {
3434

3535
public interface OnMultipleUploadInitiatedHandler {
@@ -47,12 +47,10 @@ public interface OnMultipleUploadInitiatedHandler {
4747
private boolean imageOnlyMode;
4848

4949
private static class UploadHolderView {
50-
Uri imageUri;
51-
52-
MediaWikiImageView image;
53-
TextView title;
54-
55-
RelativeLayout overlay;
50+
private Uri imageUri;
51+
private SimpleDraweeView image;
52+
private TextView title;
53+
private RelativeLayout overlay;
5654
}
5755

5856
private class PhotoDisplayAdapter extends BaseAdapter {
@@ -79,7 +77,7 @@ public View getView(int i, View view, ViewGroup viewGroup) {
7977
if(view == null) {
8078
view = getLayoutInflater(null).inflate(R.layout.layout_upload_item, null);
8179
holder = new UploadHolderView();
82-
holder.image = (MediaWikiImageView) view.findViewById(R.id.uploadImage);
80+
holder.image = (SimpleDraweeView) view.findViewById(R.id.uploadImage);
8381
holder.title = (TextView) view.findViewById(R.id.uploadTitle);
8482
holder.overlay = (RelativeLayout) view.findViewById(R.id.uploadOverlay);
8583

0 commit comments

Comments
 (0)