Selaa lähdekoodia

resolve codacy warnings

Signed-off-by: Marcel Hibbe <dev@mhibbe.de>
Marcel Hibbe 2 vuotta sitten
vanhempi
commit
8ded97ee03

+ 1 - 0
app/src/main/java/com/nextcloud/talk/adapters/messages/PreviewMessageViewHolder.kt

@@ -97,6 +97,7 @@ abstract class PreviewMessageViewHolder(itemView: View?, payload: Any?) :
     }
 
     @SuppressLint("SetTextI18n")
+    @Suppress("NestedBlockDepth", "ComplexMethod", "LongMethod")
     override fun onBind(message: ChatMessage) {
         super.onBind(message)
         if (userAvatar != null) {

+ 5 - 0
app/src/main/java/com/nextcloud/talk/ui/dialog/MessageActionsDialog.kt

@@ -27,6 +27,7 @@ import android.content.Context
 import android.os.Bundle
 import android.os.Handler
 import android.os.Looper
+import android.util.Log
 import android.view.MotionEvent
 import android.view.View
 import android.view.ViewGroup
@@ -321,6 +322,7 @@ class MessageActionsDialog(
 
     inner class ReactionAddedObserver : Observer<ReactionAddedModel> {
         override fun onSubscribe(d: Disposable) {
+            // unused atm
         }
 
         override fun onNext(reactionAddedModel: ReactionAddedModel) {
@@ -333,6 +335,7 @@ class MessageActionsDialog(
         }
 
         override fun onError(e: Throwable) {
+            Log.e(TAG, "failure in ReactionAddedObserver", e)
         }
 
         override fun onComplete() {
@@ -342,6 +345,7 @@ class MessageActionsDialog(
 
     inner class ReactionDeletedObserver : Observer<ReactionDeletedModel> {
         override fun onSubscribe(d: Disposable) {
+            // unused atm
         }
 
         override fun onNext(reactionDeletedModel: ReactionDeletedModel) {
@@ -354,6 +358,7 @@ class MessageActionsDialog(
         }
 
         override fun onError(e: Throwable) {
+            Log.e(TAG, "failure in ReactionDeletedObserver", e)
         }
 
         override fun onComplete() {

+ 1 - 1
app/src/main/res/layout/reactions_inside_message.xml

@@ -33,4 +33,4 @@
         android:gravity="center_vertical"
         android:orientation="horizontal" >
     </LinearLayout>
-</HorizontalScrollView>
+</HorizontalScrollView>