refactored AudioPlayer

4 files changed, 353 insertions, 67 deletions

diff --git a/src/main/java/de/pixart/messenger/services/AudioPlayer.java b/src/main/java/de/pixart/messenger/services/AudioPlayer.java
new file mode 100644
index 000000000..6c3b55344
--- /dev/null
+++ b/src/main/java/de/pixart/messenger/services/AudioPlayer.java
@@ -0,0 +1,280 @@
+package de.pixart.messenger.services;
+
+import android.content.res.ColorStateList;
+import android.media.MediaPlayer;
+import android.os.Build;
+import android.os.Handler;
+import android.support.v4.content.ContextCompat;
+import android.view.View;
+import android.widget.ImageButton;
+import android.widget.RelativeLayout;
+import android.widget.SeekBar;
+import android.widget.TextView;
+
+import java.lang.ref.WeakReference;
+import java.util.Locale;
+
+import de.pixart.messenger.R;
+import de.pixart.messenger.entities.Message;
+import de.pixart.messenger.ui.adapter.MessageAdapter;
+import de.pixart.messenger.utils.WeakReferenceSet;
+
+public class AudioPlayer implements View.OnClickListener, MediaPlayer.OnCompletionListener, SeekBar.OnSeekBarChangeListener, Runnable {
+
+    private static final int REFRESH_INTERVAL = 250;
+    private static final Object LOCK = new Object();
+    private static MediaPlayer player = null;
+    private static Message currentlyPlayingMessage = null;
+    private final MessageAdapter messageAdapter;
+    private final WeakReferenceSet<RelativeLayout> audioPlayerLayouts = new WeakReferenceSet<>();
+
+    private final Handler handler = new Handler();
+
+    public AudioPlayer(MessageAdapter adapter) {
+        this.messageAdapter = adapter;
+        synchronized (AudioPlayer.LOCK) {
+            if (AudioPlayer.player != null) {
+                AudioPlayer.player.setOnCompletionListener(this);
+            }
+        }
+    }
+
+    private static String formatTime(int ms) {
+        return String.format(Locale.ENGLISH, "%d:%02d", ms / 60000, Math.min(Math.round((ms % 60000) / 1000f), 59));
+    }
+
+    public void init(RelativeLayout audioPlayer, Message message) {
+        synchronized (AudioPlayer.LOCK) {
+            audioPlayer.setTag(message);
+            if (init(ViewHolder.get(audioPlayer), message)) {
+                this.audioPlayerLayouts.addWeakReferenceTo(audioPlayer);
+                this.stopRefresher(true);
+            } else {
+                this.audioPlayerLayouts.removeWeakReferenceTo(audioPlayer);
+            }
+        }
+    }
+
+    private boolean init(ViewHolder viewHolder, Message message) {
+        viewHolder.runtime.setTextColor(this.messageAdapter.getMessageTextColor(viewHolder.darkBackground, false));
+        viewHolder.progress.setOnSeekBarChangeListener(this);
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
+            ColorStateList color = ContextCompat.getColorStateList(messageAdapter.getContext(), viewHolder.darkBackground ? R.color.white70 : R.color.bubble);
+            viewHolder.progress.setThumbTintList(color);
+            viewHolder.progress.setProgressTintList(color);
+        }
+        viewHolder.playPause.setAlpha(viewHolder.darkBackground ? 0.7f : 0.57f);
+        viewHolder.playPause.setOnClickListener(this);
+        if (message == currentlyPlayingMessage) {
+            if (AudioPlayer.player != null && AudioPlayer.player.isPlaying()) {
+                viewHolder.playPause.setImageResource(viewHolder.darkBackground ? R.drawable.ic_pause_white_36dp : R.drawable.ic_pause_black_36dp);
+                viewHolder.progress.setEnabled(true);
+            } else {
+                viewHolder.playPause.setImageResource(viewHolder.darkBackground ? R.drawable.ic_play_arrow_white_36dp : R.drawable.ic_play_arrow_black_36dp);
+                viewHolder.progress.setEnabled(false);
+            }
+            return true;
+        } else {
+            viewHolder.playPause.setImageResource(viewHolder.darkBackground ? R.drawable.ic_play_arrow_white_36dp : R.drawable.ic_play_arrow_black_36dp);
+            viewHolder.runtime.setText(formatTime(message.getFileParams().runtime));
+            viewHolder.progress.setProgress(0);
+            viewHolder.progress.setEnabled(false);
+            return false;
+        }
+    }
+
+    @Override
+    public synchronized void onClick(View v) {
+        if (v.getId() == R.id.play_pause) {
+            synchronized (LOCK) {
+                startStop((ImageButton) v);
+            }
+        }
+    }
+
+    private void startStop(ImageButton playPause) {
+        final RelativeLayout audioPlayer = (RelativeLayout) playPause.getParent();
+        final ViewHolder viewHolder = ViewHolder.get(audioPlayer);
+        final Message message = (Message) audioPlayer.getTag();
+        if (startStop(viewHolder, message)) {
+            this.audioPlayerLayouts.clear();
+            this.audioPlayerLayouts.addWeakReferenceTo(audioPlayer);
+            stopRefresher(true);
+        }
+    }
+
+    private boolean playPauseCurrent(ViewHolder viewHolder) {
+        viewHolder.playPause.setAlpha(viewHolder.darkBackground ? 0.7f : 0.57f);
+        if (player.isPlaying()) {
+            viewHolder.progress.setEnabled(false);
+            player.pause();
+            viewHolder.playPause.setImageResource(viewHolder.darkBackground ? R.drawable.ic_play_arrow_white_36dp : R.drawable.ic_play_arrow_black_36dp);
+        } else {
+            viewHolder.progress.setEnabled(true);
+            player.start();
+            this.stopRefresher(true);
+            viewHolder.playPause.setImageResource(viewHolder.darkBackground ? R.drawable.ic_pause_white_36dp : R.drawable.ic_pause_black_36dp);
+        }
+        return false;
+    }
+
+    private boolean play(ViewHolder viewHolder, Message message) {
+        AudioPlayer.player = new MediaPlayer();
+        try {
+            AudioPlayer.currentlyPlayingMessage = message;
+            AudioPlayer.player.setDataSource(messageAdapter.getFileBackend().getFile(message).getAbsolutePath());
+            AudioPlayer.player.setOnCompletionListener(this);
+            AudioPlayer.player.prepare();
+            AudioPlayer.player.start();
+            viewHolder.progress.setEnabled(true);
+            viewHolder.playPause.setImageResource(viewHolder.darkBackground ? R.drawable.ic_pause_white_36dp : R.drawable.ic_pause_black_36dp);
+            return true;
+        } catch (Exception e) {
+            AudioPlayer.currentlyPlayingMessage = null;
+            return false;
+        }
+    }
+
+    private boolean startStop(ViewHolder viewHolder, Message message) {
+        if (message == currentlyPlayingMessage && player != null) {
+            return playPauseCurrent(viewHolder);
+        }
+        if (AudioPlayer.player != null) {
+            stopCurrent();
+        }
+        return play(viewHolder, message);
+    }
+
+    private void stopCurrent() {
+        if (AudioPlayer.player.isPlaying()) {
+            AudioPlayer.player.stop();
+        }
+        AudioPlayer.player.release();
+        AudioPlayer.player = null;
+        resetPlayerUi();
+    }
+
+    private void resetPlayerUi() {
+        for (WeakReference<RelativeLayout> audioPlayer : audioPlayerLayouts) {
+            resetPlayerUi(audioPlayer.get());
+        }
+    }
+
+    private void resetPlayerUi(RelativeLayout audioPlayer) {
+        if (audioPlayer == null) {
+            return;
+        }
+        final ViewHolder viewHolder = ViewHolder.get(audioPlayer);
+        final Message message = (Message) audioPlayer.getTag();
+        viewHolder.playPause.setImageResource(viewHolder.darkBackground ? R.drawable.ic_play_arrow_white_36dp : R.drawable.ic_play_arrow_black_36dp);
+        if (message != null) {
+            viewHolder.runtime.setText(formatTime(message.getFileParams().runtime));
+        }
+        viewHolder.progress.setProgress(0);
+        viewHolder.progress.setEnabled(false);
+    }
+
+    @Override
+    public void onCompletion(MediaPlayer mediaPlayer) {
+        synchronized (AudioPlayer.LOCK) {
+            this.stopRefresher(false);
+            if (AudioPlayer.player == mediaPlayer) {
+                AudioPlayer.currentlyPlayingMessage = null;
+                AudioPlayer.player = null;
+            }
+            mediaPlayer.release();
+            resetPlayerUi();
+        }
+    }
+
+    @Override
+    public void onProgressChanged(SeekBar seekBar, int progress, boolean fromUser) {
+        synchronized (AudioPlayer.LOCK) {
+            final RelativeLayout audioPlayer = (RelativeLayout) seekBar.getParent();
+            final Message message = (Message) audioPlayer.getTag();
+            if (fromUser && message == AudioPlayer.currentlyPlayingMessage) {
+                float percent = progress / 100f;
+                int duration = AudioPlayer.player.getDuration();
+                int seekTo = Math.round(duration * percent);
+                AudioPlayer.player.seekTo(seekTo);
+            }
+        }
+    }
+
+    @Override
+    public void onStartTrackingTouch(SeekBar seekBar) {
+
+    }
+
+    @Override
+    public void onStopTrackingTouch(SeekBar seekBar) {
+
+    }
+
+    public void stop() {
+        synchronized (AudioPlayer.LOCK) {
+            stopRefresher(false);
+            if (AudioPlayer.player != null) {
+                stopCurrent();
+            }
+            AudioPlayer.currentlyPlayingMessage = null;
+        }
+    }
+
+    private void stopRefresher(boolean runOnceMore) {
+        this.handler.removeCallbacks(this);
+        if (runOnceMore) {
+            this.handler.post(this);
+        }
+    }
+
+    @Override
+    public void run() {
+        synchronized (AudioPlayer.LOCK) {
+            if (AudioPlayer.player != null) {
+                boolean renew = false;
+                final int current = player.getCurrentPosition();
+                final int duration = player.getDuration();
+                for (WeakReference<RelativeLayout> audioPlayer : audioPlayerLayouts) {
+                    renew |= refreshAudioPlayer(audioPlayer.get(), current, duration);
+                }
+                if (renew && AudioPlayer.player.isPlaying()) {
+                    handler.postDelayed(this, REFRESH_INTERVAL);
+                }
+            }
+        }
+    }
+
+    private boolean refreshAudioPlayer(RelativeLayout audioPlayer, int current, int duration) {
+        if (audioPlayer == null || audioPlayer.getVisibility() != View.VISIBLE) {
+            return false;
+        }
+        final ViewHolder viewHolder = ViewHolder.get(audioPlayer);
+        viewHolder.progress.setProgress(current * 100 / duration);
+        viewHolder.runtime.setText(formatTime(current) + " / " + formatTime(duration));
+        return true;
+    }
+
+    public static class ViewHolder {
+        private TextView runtime;
+        private SeekBar progress;
+        private ImageButton playPause;
+        private boolean darkBackground = false;
+
+        public static ViewHolder get(RelativeLayout audioPlayer) {
+            ViewHolder viewHolder = (ViewHolder) audioPlayer.getTag(R.id.TAG_AUDIO_PLAYER_VIEW_HOLDER);
+            if (viewHolder == null) {
+                viewHolder = new ViewHolder();
+                viewHolder.runtime = (TextView) audioPlayer.findViewById(R.id.runtime);
+                viewHolder.progress = (SeekBar) audioPlayer.findViewById(R.id.progress);
+                viewHolder.playPause = (ImageButton) audioPlayer.findViewById(R.id.play_pause);
+                audioPlayer.setTag(R.id.TAG_AUDIO_PLAYER_VIEW_HOLDER, viewHolder);
+            }
+            return viewHolder;
+        }
+
+        public void setDarkBackground(boolean darkBackground) {
+            this.darkBackground = darkBackground;
+        }
+    }
+}
\ No newline at end of file
diff --git a/src/main/java/de/pixart/messenger/ui/ConversationFragment.java b/src/main/java/de/pixart/messenger/ui/ConversationFragment.java
index 29862861c..d6e3f49da 100644
--- a/src/main/java/de/pixart/messenger/ui/ConversationFragment.java
+++ b/src/main/java/de/pixart/messenger/ui/ConversationFragment.java
@@ -889,6 +889,9 @@ public class ConversationFragment extends Fragment implements EditMessage.Keyboa
     @Override
     public void onStop() {
         super.onStop();
+        if (activity == null || !activity.isChangingConfigurations()) {
+            messageListAdapter.stopAudioPlayer();
+        }
         if (this.conversation != null) {
             final String msg = mEditMessage.getText().toString();
             this.conversation.setNextMessage(msg);
@@ -915,6 +918,7 @@ public class ConversationFragment extends Fragment implements EditMessage.Keyboa
             this.conversation.setNextMessage(msg);
             if (this.conversation != conversation) {
                 updateChatState(this.conversation, msg);
+                messageListAdapter.stopAudioPlayer();
             }
             this.conversation.trim();
         }
diff --git a/src/main/java/de/pixart/messenger/ui/adapter/MessageAdapter.java b/src/main/java/de/pixart/messenger/ui/adapter/MessageAdapter.java
index d923172b5..0a3cb7008 100644
--- a/src/main/java/de/pixart/messenger/ui/adapter/MessageAdapter.java
+++ b/src/main/java/de/pixart/messenger/ui/adapter/MessageAdapter.java
@@ -1,7 +1,6 @@
 package de.pixart.messenger.ui.adapter;
 
 import android.content.ActivityNotFoundException;
-import android.content.Context;
 import android.content.Intent;
 import android.content.pm.PackageInfo;
 import android.content.pm.PackageManager;
@@ -26,7 +25,6 @@ import android.text.util.Linkify;
 import android.util.DisplayMetrics;
 import android.util.Log;
 import android.view.ActionMode;
-import android.view.LayoutInflater;
 import android.view.Menu;
 import android.view.MenuItem;
 import android.view.View;
@@ -49,7 +47,6 @@ import java.io.File;
 import java.io.IOException;
 import java.lang.ref.WeakReference;
 import java.net.URL;
-import java.util.HashMap;
 import java.util.List;
 import java.util.Locale;
 import java.util.concurrent.RejectedExecutionException;
@@ -66,6 +63,7 @@ import de.pixart.messenger.entities.Message;
 import de.pixart.messenger.entities.Message.FileParams;
 import de.pixart.messenger.entities.Transferable;
 import de.pixart.messenger.persistance.FileBackend;
+import de.pixart.messenger.services.AudioPlayer;
 import de.pixart.messenger.services.MessageArchiveService;
 import de.pixart.messenger.services.NotificationService;
 import de.pixart.messenger.ui.ConversationActivity;
@@ -82,7 +80,6 @@ import de.pixart.messenger.utils.UIHelper;
 import de.pixart.messenger.xmpp.mam.MamReference;
 import ezvcard.Ezvcard;
 import ezvcard.VCard;
-import nl.changer.audiowife.AudioWife;
 
 public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextView.CopyHandler {
 
@@ -131,12 +128,12 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
     private boolean mIndicateReceived = false;
     private OnQuoteListener onQuoteListener;
     private final ListSelectionManager listSelectionManager = new ListSelectionManager();
-    private AudioWife audioWife = new AudioWife();
-    private HashMap<Integer, AudioWife> audioPlayer;
+    private final AudioPlayer audioPlayer;
     private boolean mUseWhiteBackground = false;
 
     public MessageAdapter(ConversationActivity activity, List<Message> messages) {
         super(activity, 0, messages);
+        this.audioPlayer = new AudioPlayer(this);
         this.activity = activity;
         metrics = getContext().getResources().getDisplayMetrics();
         updatePreferences();
@@ -179,7 +176,7 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
         return getItemViewType(getItem(position));
     }
 
-    private int getMessageTextColor(boolean onDark, boolean primary) {
+    public int getMessageTextColor(boolean onDark, boolean primary) {
         if (onDark) {
             return ContextCompat.getColor(activity, primary ? R.color.dark : R.color.primary);
         } else {
@@ -336,10 +333,8 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
     }
 
     private void displayInfoMessage(ViewHolder viewHolder, String text, boolean darkBackground) {
-        viewHolder.aw_player.setVisibility(View.GONE);
-        if (viewHolder.download_button != null) {
-            viewHolder.download_button.setVisibility(View.GONE);
-        }
+        viewHolder.download_button.setVisibility(View.GONE);
+        viewHolder.audioPlayer.setVisibility(View.GONE);
         viewHolder.image.setVisibility(View.GONE);
         viewHolder.messageBody.setVisibility(View.VISIBLE);
         viewHolder.messageBody.setText(text);
@@ -349,11 +344,9 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
     }
 
     private void displayDecryptionFailed(ViewHolder viewHolder, boolean darkBackground) {
-        viewHolder.aw_player.setVisibility(View.GONE);
-        if (viewHolder.download_button != null) {
-            viewHolder.download_button.setVisibility(View.GONE);
-        }
+        viewHolder.download_button.setVisibility(View.GONE);
         viewHolder.image.setVisibility(View.GONE);
+        viewHolder.audioPlayer.setVisibility(View.GONE);
         viewHolder.messageBody.setVisibility(View.VISIBLE);
         viewHolder.messageBody.setText(getContext().getString(
                 R.string.decryption_failed));
@@ -363,10 +356,8 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
     }
 
     private void displayEmojiMessage(final ViewHolder viewHolder, final String body) {
-        viewHolder.aw_player.setVisibility(View.GONE);
-        if (viewHolder.download_button != null) {
-            viewHolder.download_button.setVisibility(View.GONE);
-        }
+        viewHolder.download_button.setVisibility(View.GONE);
+        viewHolder.audioPlayer.setVisibility(View.GONE);
         viewHolder.image.setVisibility(View.GONE);
         viewHolder.messageBody.setVisibility(View.VISIBLE);
         viewHolder.messageBody.setIncludeFontPadding(false);
@@ -381,7 +372,7 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
         contact = contact.split(":")[1];
         contact = contact.split("\\?")[0];
         String add_contact = activity.getString(R.string.add_to_contact_list) + " (" + contact + ")";
-        viewHolder.aw_player.setVisibility(View.GONE);
+        viewHolder.audioPlayer.setVisibility(View.GONE);
         viewHolder.download_button.setVisibility(View.VISIBLE);
         viewHolder.download_button.setText(add_contact);
         viewHolder.download_button.setOnClickListener(new OnClickListener() {
@@ -552,7 +543,7 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
     }
 
     private void displayDownloadableMessage(ViewHolder viewHolder, final Message message, String text) {
-        viewHolder.aw_player.setVisibility(View.GONE);
+        viewHolder.audioPlayer.setVisibility(View.GONE);
         viewHolder.image.setVisibility(View.GONE);
         viewHolder.messageBody.setVisibility(View.GONE);
         viewHolder.download_button.setVisibility(View.VISIBLE);
@@ -567,35 +558,8 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
         });
     }
 
-    private void displayAudioMessage(ViewHolder viewHolder, final Message message, int position) {
-        if (audioPlayer == null) {
-            audioPlayer = new HashMap<>();
-        }
-        viewHolder.image.setVisibility(View.GONE);
-        viewHolder.messageBody.setVisibility(View.GONE);
-        if (viewHolder.download_button != null) {
-            viewHolder.download_button.setVisibility(View.GONE);
-        }
-        viewHolder.aw_player.setVisibility(View.VISIBLE);
-        Uri audioFile = Uri.fromFile(activity.xmppConnectionService.getFileBackend().getFile(message));
-
-        audioWife = audioPlayer.get(position);
-        if (audioWife == null) {
-            audioWife = new AudioWife();
-            audioWife.init(getContext(), audioFile);
-            audioPlayer.put(position, audioWife);
-            RelativeLayout vg = new RelativeLayout(activity);
-            LayoutInflater layoutInflater = (LayoutInflater) getContext().getSystemService(Context.LAYOUT_INFLATER_SERVICE);
-            audioWife.useDefaultUi(vg, layoutInflater);
-            viewHolder.aw_player.addView(audioWife.getPlayerUi());
-        } else {
-            audioWife.cleanPlayerUi();
-            viewHolder.aw_player.addView(audioWife.getPlayerUi());
-        }
-    }
-
     private void displayOpenableMessage(ViewHolder viewHolder, final Message message) {
-        viewHolder.aw_player.setVisibility(View.GONE);
+        viewHolder.audioPlayer.setVisibility(View.GONE);
         viewHolder.image.setVisibility(View.GONE);
         viewHolder.messageBody.setVisibility(View.GONE);
         viewHolder.download_button.setVisibility(View.VISIBLE);
@@ -669,7 +633,7 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
     }
 
     private void displayLocationMessage(ViewHolder viewHolder, final Message message) {
-        viewHolder.aw_player.setVisibility(View.GONE);
+        viewHolder.audioPlayer.setVisibility(View.GONE);
         viewHolder.messageBody.setVisibility(View.GONE);
         String url = GeoHelper.MapPreviewUri(message);
         viewHolder.image.setVisibility(View.VISIBLE);
@@ -720,13 +684,20 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
 
     }
 
-    private void displayImageMessage(ViewHolder viewHolder,
-                                     final Message message) {
-        viewHolder.aw_player.setVisibility(View.GONE);
-        if (viewHolder.download_button != null) {
-            viewHolder.download_button.setVisibility(View.GONE);
-        }
+    private void displayAudioMessage(ViewHolder viewHolder, Message message, boolean darkBackground) {
+        viewHolder.image.setVisibility(View.GONE);
         viewHolder.messageBody.setVisibility(View.GONE);
+        viewHolder.download_button.setVisibility(View.GONE);
+        final RelativeLayout audioPlayer = viewHolder.audioPlayer;
+        audioPlayer.setVisibility(View.VISIBLE);
+        AudioPlayer.ViewHolder.get(audioPlayer).setDarkBackground(darkBackground);
+        this.audioPlayer.init(audioPlayer, message);
+    }
+
+    private void displayImageMessage(ViewHolder viewHolder, final Message message) {
+        viewHolder.download_button.setVisibility(View.GONE);
+        viewHolder.messageBody.setVisibility(View.GONE);
+        viewHolder.audioPlayer.setVisibility(View.GONE);
         viewHolder.image.setVisibility(View.VISIBLE);
         FileParams params = message.getFileParams();
         double target = metrics.density * 200;
@@ -799,7 +770,7 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
                         .findViewById(R.id.message_box);
                 viewHolder.contact_picture = (ImageView) view
                         .findViewById(R.id.message_photo);
-                viewHolder.aw_player = (ViewGroup) view.findViewById(R.id.aw_player);
+                viewHolder.audioPlayer = (RelativeLayout) view.findViewById(R.id.audio_player);
                 viewHolder.download_button = (Button) view
                         .findViewById(R.id.download_button);
                 viewHolder.resend_button = (Button) view
@@ -825,7 +796,7 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
                         .findViewById(R.id.message_box);
                 viewHolder.contact_picture = (ImageView) view
                         .findViewById(R.id.message_photo);
-                viewHolder.aw_player = (ViewGroup) view.findViewById(R.id.aw_player);
+                viewHolder.audioPlayer = (RelativeLayout) view.findViewById(R.id.audio_player);
                 viewHolder.download_button = (Button) view
                         .findViewById(R.id.download_button);
                 viewHolder.indicator = (ImageView) view
@@ -860,7 +831,7 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
         }
 
 
-        boolean darkBackground = (type == SENT && (!isInValidSession || !mUseWhiteBackground));
+        boolean darkBackground = false; // default: (type == SENT && (!isInValidSession || !mUseWhiteBackground));
 
         if (type == DATE_SEPARATOR) {
             if (UIHelper.today(message.getTimeSent())) {
@@ -917,7 +888,6 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
                 });
 
         final Transferable transferable = message.getTransferable();
-        String mimeType = message.getMimeType();
         if (transferable != null && transferable.getStatus() != Transferable.STATUS_UPLOADING) {
             if (transferable.getStatus() == Transferable.STATUS_OFFER) {
                 displayDownloadableMessage(viewHolder, message, activity.getString(R.string.download_x_file, UIHelper.getFileDescriptionString(activity, message)));
@@ -929,14 +899,12 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
         } else if (message.getType() == Message.TYPE_IMAGE && message.getEncryption() != Message.ENCRYPTION_PGP && message.getEncryption() != Message.ENCRYPTION_DECRYPTION_FAILED) {
             displayImageMessage(viewHolder, message);
         } else if (message.getType() == Message.TYPE_FILE && message.getEncryption() != Message.ENCRYPTION_PGP && message.getEncryption() != Message.ENCRYPTION_DECRYPTION_FAILED) {
-            if (message.getFileParams().width > 0) {
+            if (message.getFileParams().width > 0 && message.getFileParams().height > 0) {
                 displayImageMessage(viewHolder, message);
+            } else if (message.getFileParams().runtime > 0) {
+                displayAudioMessage(viewHolder, message, darkBackground);
             } else {
-                if (mimeType != null) {
-                    if (message.getMimeType().startsWith("audio/")) {
-                        displayAudioMessage(viewHolder, message, position);
-                    } else displayOpenableMessage(viewHolder, message);
-                } else displayOpenableMessage(viewHolder, message);
+                displayOpenableMessage(viewHolder, message);
             }
         } else if (message.getEncryption() == Message.ENCRYPTION_PGP) {
             if (account.isPgpDecryptionServiceConnected()) {
@@ -1052,6 +1020,14 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
         }
     }
 
+    public FileBackend getFileBackend() {
+        return activity.xmppConnectionService.getFileBackend();
+    }
+
+    public void stopAudioPlayer() {
+        audioPlayer.stop();
+    }
+
     public interface OnQuoteListener {
         public void onQuote(String text);
     }
@@ -1189,7 +1165,6 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
         protected LinearLayout message_box;
         protected Button download_button;
         protected Button resend_button;
-        protected ViewGroup aw_player;
         protected ImageView image;
         protected ImageView indicator;
         protected ImageView indicatorReceived;
@@ -1201,6 +1176,7 @@ public class MessageAdapter extends ArrayAdapter<Message> implements CopyTextVie
         protected TextView encryption;
         public Button load_more_messages;
         public ImageView edit_indicator;
+        public RelativeLayout audioPlayer;
     }
 
     class BitmapWorkerTask extends AsyncTask<Message, Void, Bitmap> {
diff --git a/src/main/java/de/pixart/messenger/utils/WeakReferenceSet.java b/src/main/java/de/pixart/messenger/utils/WeakReferenceSet.java
new file mode 100644
index 000000000..b6ccbd560
--- /dev/null
+++ b/src/main/java/de/pixart/messenger/utils/WeakReferenceSet.java
@@ -0,0 +1,26 @@
+package de.pixart.messenger.utils;
+
+import java.lang.ref.WeakReference;
+import java.util.HashSet;
+import java.util.Iterator;
+
+public class WeakReferenceSet<T> extends HashSet<WeakReference<T>> {
+
+    public void removeWeakReferenceTo(T reference) {
+        for (Iterator<WeakReference<T>> iterator = iterator(); iterator.hasNext(); ) {
+            if (reference == iterator.next().get()) {
+                iterator.remove();
+            }
+        }
+    }
+
+
+    public void addWeakReferenceTo(T reference) {
+        for (WeakReference<T> weakReference : this) {
+            if (reference == weakReference.get()) {
+                return;
+            }
+        }
+        this.add(new WeakReference<>(reference));
+    }
+}
\ No newline at end of file