Fix a few warnings in ConfirmAvatarDialogFragment

This commit is contained in:
Sebastian Kürten
2020-12-03 12:11:45 +01:00
committed by Torsten Grote
parent 15021bffef
commit 64c0e9e9e4

View File

@@ -9,11 +9,13 @@ import android.view.View;
import android.widget.ImageView; import android.widget.ImageView;
import android.widget.TextView; import android.widget.TextView;
import org.briarproject.bramble.api.nullsafety.MethodsNotNullByDefault;
import org.briarproject.briar.R; import org.briarproject.briar.R;
import org.briarproject.briar.android.activity.BaseActivity; import org.briarproject.briar.android.activity.BaseActivity;
import javax.inject.Inject; import javax.inject.Inject;
import androidx.annotation.NonNull;
import androidx.appcompat.app.AlertDialog; import androidx.appcompat.app.AlertDialog;
import androidx.fragment.app.DialogFragment; import androidx.fragment.app.DialogFragment;
import androidx.fragment.app.FragmentActivity; import androidx.fragment.app.FragmentActivity;
@@ -21,6 +23,7 @@ import androidx.lifecycle.ViewModelProvider;
import static java.util.Objects.requireNonNull; import static java.util.Objects.requireNonNull;
@MethodsNotNullByDefault
public class ConfirmAvatarDialogFragment extends DialogFragment { public class ConfirmAvatarDialogFragment extends DialogFragment {
final static String TAG = ConfirmAvatarDialogFragment.class.getName(); final static String TAG = ConfirmAvatarDialogFragment.class.getName();
@@ -43,11 +46,12 @@ public class ConfirmAvatarDialogFragment extends DialogFragment {
} }
@Override @Override
public void onAttach(Context ctx) { public void onAttach(@NonNull Context ctx) {
super.onAttach(ctx); super.onAttach(ctx);
((BaseActivity) requireActivity()).getActivityComponent().inject(this); ((BaseActivity) requireActivity()).getActivityComponent().inject(this);
} }
@Override
public Dialog onCreateDialog(Bundle savedInstanceState) { public Dialog onCreateDialog(Bundle savedInstanceState) {
Bundle args = requireArguments(); Bundle args = requireArguments();
String argUri = requireNonNull(args.getString(ARG_URI)); String argUri = requireNonNull(args.getString(ARG_URI));
@@ -71,9 +75,7 @@ public class ConfirmAvatarDialogFragment extends DialogFragment {
.setMessage(R.string.dialog_confirm_profile_picture_question); .setMessage(R.string.dialog_confirm_profile_picture_question);
builder.setNegativeButton(R.string.cancel, null); builder.setNegativeButton(R.string.cancel, null);
builder.setPositiveButton(R.string.dialog_confirm_profile_picture_set, builder.setPositiveButton(R.string.dialog_confirm_profile_picture_set,
(dialog, id) -> { (dialog, id) -> settingsViewModel.setAvatar(uri));
settingsViewModel.setAvatar(uri);
});
ImageView imageView = view.findViewById(R.id.image); ImageView imageView = view.findViewById(R.id.image);
imageView.setImageResource(R.drawable.contact_connected); imageView.setImageResource(R.drawable.contact_connected);