From 39dc1a1776cf364b8c9f5ab8d4a0449dd18e7651 Mon Sep 17 00:00:00 2001
From: George Kankava <george.kankava@devfactory.com>
Date: Tue, 8 Mar 2016 12:07:47 +0400
Subject: [PATCH] squid:S1066 - Collapsible "if" statements should be merged

---
 .../jigdraw/draw/views/JigsawGridView.java    | 35 +++++++------------
 1 file changed, 13 insertions(+), 22 deletions(-)

diff --git a/app/src/main/java/com/jigdraw/draw/views/JigsawGridView.java b/app/src/main/java/com/jigdraw/draw/views/JigsawGridView.java
index 81549e6..7dbc7f1 100644
--- a/app/src/main/java/com/jigdraw/draw/views/JigsawGridView.java
+++ b/app/src/main/java/com/jigdraw/draw/views/JigsawGridView.java
@@ -147,11 +147,9 @@ public void onScroll(AbsListView view, int firstVisibleItem,
         }
 
         public void checkAndHandleFirstVisibleCellChange() {
-            if (mCurrentFirstVisibleItem != mPreviousFirstVisibleItem) {
-                if (mCellIsMobile && mMobileItemId != INVALID_ID) {
-                    updateNeighborViewsForId(mMobileItemId);
-                    handleCellSwitch();
-                }
+            if (mCurrentFirstVisibleItem != mPreviousFirstVisibleItem && (mCellIsMobile && mMobileItemId != INVALID_ID)) {
+                updateNeighborViewsForId(mMobileItemId);
+                handleCellSwitch();
             }
         }
 
@@ -160,11 +158,9 @@ public void checkAndHandleLastVisibleCellChange() {
                     + mCurrentVisibleItemCount;
             int previousLastVisibleItem = mPreviousFirstVisibleItem
                     + mPreviousVisibleItemCount;
-            if (currentLastVisibleItem != previousLastVisibleItem) {
-                if (mCellIsMobile && mMobileItemId != INVALID_ID) {
-                    updateNeighborViewsForId(mMobileItemId);
-                    handleCellSwitch();
-                }
+            if (currentLastVisibleItem != previousLastVisibleItem && (mCellIsMobile && mMobileItemId != INVALID_ID)) {
+                updateNeighborViewsForId(mMobileItemId);
+                handleCellSwitch();
             }
         }
 
@@ -268,20 +264,16 @@ public boolean onTouchEvent(@NonNull MotionEvent event) {
             case MotionEvent.ACTION_UP:
                 touchEventsEnded();
 
-                if (mHoverCell != null) {
-                    if (mDropListener != null) {
-                        mDropListener.onActionDrop();
-                    }
+                if (mHoverCell != null && mDropListener != null) {
+                    mDropListener.onActionDrop();
                 }
                 break;
 
             case MotionEvent.ACTION_CANCEL:
                 touchEventsCancelled();
 
-                if (mHoverCell != null) {
-                    if (mDropListener != null) {
-                        mDropListener.onActionDrop();
-                    }
+                if (mHoverCell != null && mDropListener != null) {
+                    mDropListener.onActionDrop();
                 }
                 break;
 
@@ -312,10 +304,9 @@ public void setOnDragListener(OnDragListener dragListener) {
 
     public void startEditMode(int position) {
         requestDisallowInterceptTouchEvent(true);
-        if (isPostHoneycomb())
-            if (position != -1) {
-                startDragAtPosition(position);
-            }
+        if (isPostHoneycomb() && position != -1) {
+            startDragAtPosition(position);
+        }
         mIsEditMode = true;
     }