From 8fe14adf3434cb42d66012127e7905766ce4d046 Mon Sep 17 00:00:00 2001
From: Maxime Leroy <maxime.leroy.4@ulaval.ca>
Date: Wed, 1 May 2019 21:48:19 -0400
Subject: [PATCH] fix(null obj)

---
 .../java/ca/ulaval/ima/mp/activity/MainActivity.java   |  2 +-
 .../java/ca/ulaval/ima/mp/adapter/MessageAdapter.java  | 10 ++++++----
 .../ca/ulaval/ima/mp/fragment/MessageFragment.java     |  5 +++--
 .../java/ca/ulaval/ima/mp/fragment/SoundFragment.java  |  2 +-
 4 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/app/src/main/java/ca/ulaval/ima/mp/activity/MainActivity.java b/app/src/main/java/ca/ulaval/ima/mp/activity/MainActivity.java
index 57a50f7..dc5981f 100644
--- a/app/src/main/java/ca/ulaval/ima/mp/activity/MainActivity.java
+++ b/app/src/main/java/ca/ulaval/ima/mp/activity/MainActivity.java
@@ -284,7 +284,7 @@ public void onVoiceChannelClicked(Channel channel) {
     public void onSearch(String search) {
         if (FileManager.requirePermissions(this)) {
             Intent intent = new Intent(this, FileManager.class);
-            intent.putExtra(FilePickerActivity.ARG_FILTER, Pattern.compile(search + ".*\\.(opus)"));
+            intent.putExtra(FilePickerActivity.ARG_FILTER, Pattern.compile(search + "*.*\\.(opus)"));
             intent.putExtra(FilePickerActivity.ARG_CLOSEABLE, true);
             intent.putExtra(FilePickerActivity.ARG_TITLE, getString(R.string.choose_file));
             intent.putExtra(FilePickerActivity.ARG_START_PATH, FileManager.importedDir.getAbsolutePath());
diff --git a/app/src/main/java/ca/ulaval/ima/mp/adapter/MessageAdapter.java b/app/src/main/java/ca/ulaval/ima/mp/adapter/MessageAdapter.java
index cfe363f..eead918 100644
--- a/app/src/main/java/ca/ulaval/ima/mp/adapter/MessageAdapter.java
+++ b/app/src/main/java/ca/ulaval/ima/mp/adapter/MessageAdapter.java
@@ -32,10 +32,12 @@ public ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
 
     @Override
     public void onBindViewHolder(final ViewHolder holder, int position) {
-        holder.mItem = mValues.get(position);
-        holder.mUserView.setText(mValues.get(position).author.username);
-        holder.mContentView.setText(mValues.get(position).content);
-        Picasso.get().load(mValues.get(position).author.getAvatarURI()).into(holder.mAvatarView);
+        if (position < mValues.size()) {
+            holder.mItem = mValues.get(position);
+            holder.mUserView.setText(mValues.get(position).author.username);
+            holder.mContentView.setText(mValues.get(position).content);
+            Picasso.get().load(mValues.get(position).author.getAvatarURI()).into(holder.mAvatarView);
+        }
     }
 
     @Override
diff --git a/app/src/main/java/ca/ulaval/ima/mp/fragment/MessageFragment.java b/app/src/main/java/ca/ulaval/ima/mp/fragment/MessageFragment.java
index a931192..d241317 100644
--- a/app/src/main/java/ca/ulaval/ima/mp/fragment/MessageFragment.java
+++ b/app/src/main/java/ca/ulaval/ima/mp/fragment/MessageFragment.java
@@ -125,7 +125,7 @@ public void run() {
 
     public void setActiveChannel(Channel channel) {
         this.loadMessages(channel);
-        if (getActivity() != null && channel.name != null) {
+        if (getActivity() != null && channel != null && channel.name != null) {
             getActivity().setTitle("#" + channel.name);
         }
         activeChannel = channel;
@@ -149,7 +149,8 @@ public void onClick(View v) {
             recyclerView.setAdapter(mAdapter);
         }
         this.setActiveChannel(this.activeChannel);
-        Gateway.server.setMessageHandler(this);
+        if (Gateway.server != null)
+            Gateway.server.setMessageHandler(this);
         return rootView;
     }
 
diff --git a/app/src/main/java/ca/ulaval/ima/mp/fragment/SoundFragment.java b/app/src/main/java/ca/ulaval/ima/mp/fragment/SoundFragment.java
index 77d3273..152d7d5 100644
--- a/app/src/main/java/ca/ulaval/ima/mp/fragment/SoundFragment.java
+++ b/app/src/main/java/ca/ulaval/ima/mp/fragment/SoundFragment.java
@@ -134,7 +134,7 @@ public void onFailure(String message) {
 
     public void setActiveChannel(Channel channel) {
         voiceChannel = channel;
-        if (getActivity() != null && channel.name != null) {
+        if (getActivity() != null && channel != null && channel.name != null) {
             getActivity().setTitle("@" + channel.name);
         }
         if (voiceChannel != null) {