kumi/andOTP
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #634 from jsoberg/#631-AdapterPositionSafetyForNPE

Browse source 
#631 - Fix NPE from invalid adapter position
This commit is contained in:
Jakob Nixdorf 2020-09-25 06:12:21 +02:00 committed by GitHub
commit f358dabcb1
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 42 additions and 34 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

72
app/src/main/java/org/shadowice/flocke/andotp/View/EntryViewHolder.java
View file

@ -25,6 +25,8 @@ package org.shadowice.flocke.andotp.View;
import android.animation.ObjectAnimator;
import android.content.Context;
import android.graphics.ColorFilter;
import androidx.annotation.NonNull;
import androidx.cardview.widget.CardView;
import androidx.recyclerview.widget.RecyclerView;
import android.text.TextUtils;
@ -105,56 +107,62 @@ public class EntryViewHolder extends RecyclerView.ViewHolder
visibleImg.getDrawable().setColorFilter(colorFilter);
invisibleImg.getDrawable().setColorFilter(colorFilter);
// Setup onClickListeners
menuButton.setOnClickListener(new View.OnClickListener() {
@Override
public void onClick(View view) {
if (callback != null)
callback.onMenuButtonClicked(view, getAdapterPosition());
setupOnClickListeners(menuButton, copyButton);
setTapToReveal(tapToReveal);
}
});
copyButton.setOnClickListener(new View.OnClickListener() {
@Override
public void onClick(View view) {
if (callback != null)
callback.onCopyButtonClicked(value.getTag().toString(), getAdapterPosition());
}
});
private void setupOnClickListeners(ImageButton menuButton, ImageButton copyButton) {
menuButton.setOnClickListener(view ->
adapterPositionSafeCallback((callback, adapterPosition) ->
callback.onMenuButtonClicked(view, adapterPosition)
)
);
counterLayout.setOnClickListener(new View.OnClickListener() {
@Override
public void onClick(View view) {
if (callback != null)
callback.onCounterClicked(getAdapterPosition());
}
});
copyButton.setOnClickListener(view ->
adapterPositionSafeCallback((callback, adapterPosition) ->
callback.onCopyButtonClicked(value.getTag().toString(), adapterPosition)
)
);
counterLayout.setOnLongClickListener(new View.OnLongClickListener() {
@Override
public boolean onLongClick(View view) {
if (callback != null)
callback.onCounterLongPressed(getAdapterPosition());
counterLayout.setOnClickListener(view ->
adapterPositionSafeCallback(Callback::onCounterClicked)
);
counterLayout.setOnLongClickListener(view -> {
adapterPositionSafeCallback(Callback::onCounterLongPressed);
return false;
}
});
card.setOnClickListener(new SimpleDoubleClickListener() {
@Override
public void onSingleClick(View v) {
if (callback != null)
callback.onCardSingleClicked(getAdapterPosition(), value.getTag().toString());
adapterPositionSafeCallback((callback, adapterPosition) ->
callback.onCardSingleClicked(adapterPosition, value.getTag().toString())
);
}
@Override
public void onDoubleClick(View v) {
if (callback != null)
callback.onCardDoubleClicked(getAdapterPosition(), value.getTag().toString());
adapterPositionSafeCallback((callback, adapterPosition) ->
callback.onCardDoubleClicked(adapterPosition, value.getTag().toString())
);
}
});
}
setTapToReveal(tapToReveal);
@FunctionalInterface
private interface AdapterPositionSafeCallbackConsumer {
/** The specified {@link Callback} is guaranteed to be non-null, and adapterPosition is
* guaranteed to be a valid position. */
void accept(@NonNull Callback callback, int adapterPosition);
}
private void adapterPositionSafeCallback(AdapterPositionSafeCallbackConsumer safeCallback) {
int clickedPosition = getAdapterPosition();
if (callback != null && clickedPosition != RecyclerView.NO_POSITION) {
safeCallback.accept(callback, clickedPosition);
}
}
public void updateValues(Entry entry) {