Skip to content

Commit 28250e0

Browse files
committed
Don't maintain Context.
1 parent 3379c59 commit 28250e0

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

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

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@
22

33
import android.content.Context;
44
import android.database.Cursor;
5+
6+
import androidx.annotation.NonNull;
57
import androidx.cursoradapter.widget.CursorAdapter;
68
import android.view.LayoutInflater;
79
import android.view.View;
@@ -20,14 +22,12 @@ class ContributionsListAdapter extends CursorAdapter {
2022

2123
private final ContributionDao contributionDao;
2224
private UploadService uploadService;
23-
private Context context;
2425

2526
public ContributionsListAdapter(Context context,
2627
Cursor c,
2728
int flags,
2829
ContributionDao contributionDao) {
2930
super(context, c, flags);
30-
this.context = context;
3131
this.contributionDao = contributionDao;
3232
}
3333

@@ -53,12 +53,12 @@ public void bindView(View view, Context context, Cursor cursor) {
5353
new DisplayableContribution.ContributionActions() {
5454
@Override
5555
public void retryUpload() {
56-
ContributionsListAdapter.this.retryUpload(contribution);
56+
ContributionsListAdapter.this.retryUpload(view.getContext(), contribution);
5757
}
5858

5959
@Override
6060
public void deleteUpload() {
61-
ContributionsListAdapter.this.deleteUpload(contribution);
61+
ContributionsListAdapter.this.deleteUpload(view.getContext(), contribution);
6262
}
6363
});
6464
views.bindModel(context, displayableContribution);
@@ -68,7 +68,7 @@ public void deleteUpload() {
6868
* Retry upload when it is failed
6969
* @param contribution contribution to be retried
7070
*/
71-
private void retryUpload(Contribution contribution) {
71+
private void retryUpload(@NonNull Context context, Contribution contribution) {
7272
if (NetworkUtils.isInternetConnectionEstablished(context)) {
7373
if (contribution.getState() == STATE_FAILED
7474
&& uploadService!= null) {
@@ -87,7 +87,7 @@ private void retryUpload(Contribution contribution) {
8787
* Delete a failed upload attempt
8888
* @param contribution contribution to be deleted
8989
*/
90-
private void deleteUpload(Contribution contribution) {
90+
private void deleteUpload(@NonNull Context context, Contribution contribution) {
9191
if (NetworkUtils.isInternetConnectionEstablished(context)) {
9292
if (contribution.getState() == STATE_FAILED) {
9393
Timber.d("Deleting failed contrib %s", contribution.toString());

0 commit comments

Comments
 (0)