Chore

1 files changed, 29 insertions, 29 deletions

diff --git a/app/src/main/java/it/niedermann/nextcloud/deck/ui/attachments/AttachmentAdapter.java b/app/src/main/java/it/niedermann/nextcloud/deck/ui/attachments/AttachmentAdapter.java
index cfdeb0880..f256db584 100644
--- a/app/src/main/java/it/niedermann/nextcloud/deck/ui/attachments/AttachmentAdapter.java
+++ b/app/src/main/java/it/niedermann/nextcloud/deck/ui/attachments/AttachmentAdapter.java
@@ -27,12 +27,14 @@ import it.niedermann.nextcloud.deck.util.AttachmentUtil;
 
 public class AttachmentAdapter extends RecyclerView.Adapter<AttachmentViewHolder> {
 
+    private static final String MIMETYPE_IMAGE_PREFIX = "image";
     private final Account account;
     private final long cardRemoteId;
     @NonNull
     private List<Attachment> attachments;
     private Context context;
 
+    @SuppressWarnings("WeakerAccess")
     public AttachmentAdapter(@NonNull Account account, long cardRemoteId, @NonNull List<Attachment> attachments) {
         super();
         this.attachments = attachments;
@@ -49,38 +51,36 @@ public class AttachmentAdapter extends RecyclerView.Adapter<AttachmentViewHolder
 
     @Override
     public void onBindViewHolder(@NonNull AttachmentViewHolder holder, int position) {
-        Attachment attachment = attachments.get(position);
-        String uri = AttachmentUtil.getUrl(account.getUrl(), cardRemoteId, attachment.getId());
-        if (attachment.getMimetype() != null) {
-            if (attachment.getMimetype().startsWith("image")) {
-                if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
-                    holder.binding.preview.setTransitionName(context.getString(R.string.transition_attachment_preview, String.valueOf(attachment.getLocalId())));
-                }
-                holder.binding.preview.setImageResource(R.drawable.ic_image_grey600_24dp);
-                Glide.with(context)
-                        .load(uri)
-                        .listener(new RequestListener<Drawable>() {
-                            @Override
-                            public boolean onLoadFailed(@Nullable GlideException e, Object model,
-                                                        Target<Drawable> target, boolean isFirstResource) {
-                                if (context instanceof FragmentActivity) {
-                                    ((FragmentActivity) context).supportStartPostponedEnterTransition();
-                                }
-                                return false;
+        final Attachment attachment = attachments.get(position);
+        final String uri = AttachmentUtil.getUrl(account.getUrl(), cardRemoteId, attachment.getId());
+        if (attachment.getMimetype() != null && attachment.getMimetype().startsWith(MIMETYPE_IMAGE_PREFIX)) {
+            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
+                holder.binding.preview.setTransitionName(context.getString(R.string.transition_attachment_preview, String.valueOf(attachment.getLocalId())));
+            }
+            holder.binding.preview.setImageResource(R.drawable.ic_image_grey600_24dp);
+            Glide.with(context)
+                    .load(uri)
+                    .listener(new RequestListener<Drawable>() {
+                        @Override
+                        public boolean onLoadFailed(@Nullable GlideException e, Object model,
+                                                    Target<Drawable> target, boolean isFirstResource) {
+                            if (context instanceof FragmentActivity) {
+                                ((FragmentActivity) context).supportStartPostponedEnterTransition();
                             }
+                            return false;
+                        }
 
-                            @Override
-                            public boolean onResourceReady(Drawable resource, Object model,
-                                                           Target<Drawable> target, DataSource dataSource, boolean isFirstResource) {
-                                if (context instanceof FragmentActivity) {
-                                    ((FragmentActivity) context).supportStartPostponedEnterTransition();
-                                }
-                                return false;
+                        @Override
+                        public boolean onResourceReady(Drawable resource, Object model,
+                                                       Target<Drawable> target, DataSource dataSource, boolean isFirstResource) {
+                            if (context instanceof FragmentActivity) {
+                                ((FragmentActivity) context).supportStartPostponedEnterTransition();
                             }
-                        })
-                        .error(R.drawable.ic_image_grey600_24dp)
-                        .into(holder.binding.preview);
-            }
+                            return false;
+                        }
+                    })
+                    .error(R.drawable.ic_image_grey600_24dp)
+                    .into(holder.binding.preview);
         }
     }