burhanrashid52 · burhanrashid52 · Jul 23, 2024 · Jul 11, 2024 · Jul 22, 2024 · Jul 22, 2024
diff --git a/README.md b/README.md
@@ -130,7 +130,7 @@ We can draw shapes from [v.1.5.0](https://github.com/burhanrashid52/PhotoEditor/
 val shapeBuilder = ShapeBuilder()
     .withShapeOpacity(100)
     .withShapeType(ShapeType.Oval)
-    .withShapeSize(50f);
+    .withShapeSize(50f)
 
 photoEditor.setShape(mShapeBuilder)
 ```

diff --git a/app/src/androidTest/java/com/burhanrashid52/photoediting/EditImageActivityTest.kt b/app/src/androidTest/java/com/burhanrashid52/photoediting/EditImageActivityTest.kt
@@ -1,23 +1,20 @@
 package com.burhanrashid52.photoediting
 
 import android.content.Context
-import org.junit.runner.RunWith
-import androidx.test.rule.ActivityTestRule
-import junit.framework.TestCase
-import androidx.test.espresso.contrib.RecyclerViewActions
-import androidx.recyclerview.widget.RecyclerView
-import kotlin.Throws
 import android.content.Intent
 import android.net.Uri
 import android.view.View
 import android.view.ViewGroup
 import android.widget.FrameLayout
+import androidx.recyclerview.widget.RecyclerView
 import androidx.test.espresso.Espresso
 import androidx.test.espresso.NoMatchingViewException
 import androidx.test.espresso.action.ViewActions
 import androidx.test.espresso.assertion.ViewAssertions
+import androidx.test.espresso.contrib.RecyclerViewActions
 import androidx.test.espresso.matcher.ViewMatchers
 import androidx.test.filters.LargeTest
+import androidx.test.rule.ActivityTestRule
 import androidx.test.runner.AndroidJUnit4
 import junit.framework.TestCase.*
 import org.hamcrest.Description
@@ -27,6 +24,7 @@ import org.junit.Assert.assertNotEquals
 import org.junit.Ignore
 import org.junit.Rule
 import org.junit.Test
+import org.junit.runner.RunWith
 
 @RunWith(AndroidJUnit4::class)
 @LargeTest
@@ -49,9 +47,9 @@ class EditImageActivityTest {
     @Test
     fun checkIfBrushIsEnabledWhenClickedOnBrushTool() {
         val editImageActivity = mActivityRule.launchActivity(null)
-        assertEquals(editImageActivity.mPhotoEditor?.brushDrawableMode, false)
+        assertEquals(editImageActivity.mPhotoEditor.brushDrawableMode, false)
         Espresso.onView(ViewMatchers.withText(R.string.label_shape)).perform(ViewActions.click())
-        assertEquals(editImageActivity.mPhotoEditor?.brushDrawableMode ,true)
+        assertEquals(editImageActivity.mPhotoEditor.brushDrawableMode,true)
     }
 
     @Test
@@ -67,7 +65,7 @@ class EditImageActivityTest {
 
     @Test
     fun checkIfShapeIsEnabledWhenClickedOnBrushTool() {
-        val editImageActivity = mActivityRule.launchActivity(null)
+        mActivityRule.launchActivity(null)
         Espresso.onView(ViewMatchers.withText(R.string.label_shape)).perform(ViewActions.click())
         Espresso.onView(ViewMatchers.withText(R.string.label_shape)).check(
             ViewAssertions.matches(
@@ -108,15 +106,15 @@ class EditImageActivityTest {
     @Ignore("Flacky test. Need to optimize")
     fun checkIfDiscardDialogIsNotDisplayedWhenCacheIsEmpty() {
         val editImageActivity = mActivityRule.launchActivity(null)
-        TestCase.assertTrue(editImageActivity.mPhotoEditor!!.isCacheEmpty)
+        assertTrue(editImageActivity.mPhotoEditor.isCacheEmpty)
         Espresso.onView(ViewMatchers.withId(R.id.imgClose)).perform(ViewActions.click())
-        TestCase.assertTrue(editImageActivity.isDestroyed)
+        assertTrue(editImageActivity.isDestroyed)
     }
 
     @Test
     fun checkIfDiscardDialogIsDisplayedWhenCacheIsNotEmpty() {
         val editImageActivity = mActivityRule.launchActivity(null)
-        TestCase.assertTrue(editImageActivity.mPhotoEditor!!.isCacheEmpty)
+        assertTrue(editImageActivity.mPhotoEditor.isCacheEmpty)
         Espresso.onView(ViewMatchers.withId(R.id.rvConstraintTools))
             .perform(
                 RecyclerViewActions.scrollTo<RecyclerView.ViewHolder>(
@@ -388,7 +386,7 @@ class EditImageActivityTest {
         // Assert that the second emoji is selected (frame background is not null)
         Espresso.onView(withIndex(ViewMatchers.withId(ja.burhanrashid52.photoeditor.R.id.frmBorder), 1))
             .check { view: View, noViewFoundException: NoMatchingViewException? ->
-                TestCase.assertNotNull(
+                assertNotNull(
                     null,
                     view.background
                 )

diff --git a/app/src/main/java/com/burhanrashid52/photoediting/PhotoApp.kt b/app/src/main/java/com/burhanrashid52/photoediting/PhotoApp.kt
@@ -1,7 +1,6 @@
 package com.burhanrashid52.photoediting
 
 import android.app.Application
-import android.content.Context
 
 /**
  * Created by Burhanuddin Rashid on 1/23/2018.

diff --git a/app/src/main/java/com/burhanrashid52/photoediting/PropertiesBSFragment.kt b/app/src/main/java/com/burhanrashid52/photoediting/PropertiesBSFragment.kt
@@ -64,6 +64,7 @@ class PropertiesBSFragment : BottomSheetDialogFragment(), SeekBar.OnSeekBarChang
         }
     }
 
-    override fun onStartTrackingTouch(seekBar: SeekBar) {}
-    override fun onStopTrackingTouch(seekBar: SeekBar) {}
+    override fun onStartTrackingTouch(seekBar: SeekBar) = Unit
+
+    override fun onStopTrackingTouch(seekBar: SeekBar) = Unit
 }
diff --git a/app/src/main/java/com/burhanrashid52/photoediting/ShapeBSFragment.kt b/app/src/main/java/com/burhanrashid52/photoediting/ShapeBSFragment.kt
@@ -93,6 +93,7 @@ class ShapeBSFragment : BottomSheetDialogFragment(), SeekBar.OnSeekBarChangeList
         }
     }
 
-    override fun onStartTrackingTouch(seekBar: SeekBar) {}
-    override fun onStopTrackingTouch(seekBar: SeekBar) {}
+    override fun onStartTrackingTouch(seekBar: SeekBar) = Unit
+
+    override fun onStopTrackingTouch(seekBar: SeekBar) = Unit
 }
diff --git a/app/src/main/java/com/burhanrashid52/photoediting/StickerBSFragment.kt b/app/src/main/java/com/burhanrashid52/photoediting/StickerBSFragment.kt
@@ -3,22 +3,20 @@ package com.burhanrashid52.photoediting
 import android.annotation.SuppressLint
 import android.app.Dialog
 import android.graphics.Bitmap
-import android.graphics.BitmapFactory
 import android.graphics.drawable.Drawable
-import android.os.Bundle
 import android.view.LayoutInflater
 import android.view.View
 import android.view.ViewGroup
 import android.widget.ImageView
 import androidx.coordinatorlayout.widget.CoordinatorLayout
-import com.google.android.material.bottomsheet.BottomSheetDialogFragment
-import com.google.android.material.bottomsheet.BottomSheetBehavior.BottomSheetCallback
-import com.google.android.material.bottomsheet.BottomSheetBehavior
-import androidx.recyclerview.widget.RecyclerView
 import androidx.recyclerview.widget.GridLayoutManager
+import androidx.recyclerview.widget.RecyclerView
 import com.bumptech.glide.Glide
 import com.bumptech.glide.request.target.CustomTarget
 import com.bumptech.glide.request.transition.Transition
+import com.google.android.material.bottomsheet.BottomSheetBehavior
+import com.google.android.material.bottomsheet.BottomSheetBehavior.BottomSheetCallback
+import com.google.android.material.bottomsheet.BottomSheetDialogFragment
 
 class StickerBSFragment : BottomSheetDialogFragment() {
     private var mStickerListener: StickerListener? = null

diff --git a/app/src/main/java/com/burhanrashid52/photoediting/TextEditorDialogFragment.kt b/app/src/main/java/com/burhanrashid52/photoediting/TextEditorDialogFragment.kt
@@ -4,7 +4,6 @@ import android.content.Context
 import android.graphics.Color
 import android.graphics.drawable.ColorDrawable
 import android.os.Bundle
-import android.text.TextUtils
 import android.view.LayoutInflater
 import android.view.View
 import android.view.ViewGroup

diff --git a/app/src/main/java/com/burhanrashid52/photoediting/base/BaseActivity.kt b/app/src/main/java/com/burhanrashid52/photoediting/base/BaseActivity.kt
@@ -9,7 +9,6 @@ import android.view.WindowManager
 import android.widget.Toast
 import androidx.activity.result.contract.ActivityResultContracts
 import androidx.appcompat.app.AppCompatActivity
-import androidx.core.app.ActivityCompat
 import androidx.core.content.ContextCompat
 import com.google.android.material.snackbar.Snackbar
 

diff --git a/app/src/main/java/com/burhanrashid52/photoediting/base/BaseFragment.kt b/app/src/main/java/com/burhanrashid52/photoediting/base/BaseFragment.kt
@@ -1,9 +1,5 @@
 package com.burhanrashid52.photoediting.base
 
-import android.os.Bundle
-import android.view.LayoutInflater
-import android.view.View
-import android.view.ViewGroup
 import androidx.fragment.app.Fragment
 
 /**

diff --git a/photoeditor/src/main/java/ja/burhanrashid52/photoeditor/DrawingView.kt b/photoeditor/src/main/java/ja/burhanrashid52/photoeditor/DrawingView.kt
@@ -52,7 +52,7 @@ class DrawingView @JvmOverloads constructor(
         paint.xfermode = PorterDuffXfermode(PorterDuff.Mode.SRC_OVER)
 
         // apply shape builder parameters
-        currentShapeBuilder?.apply {
+        currentShapeBuilder.apply {
             paint.strokeWidth = this.shapeSize
             // 'paint.color' must be called before 'paint.alpha',
             // otherwise 'paint.alpha' value will be overwritten.

diff --git a/photoeditor/src/main/java/ja/burhanrashid52/photoeditor/MultiTouchListener.kt b/photoeditor/src/main/java/ja/burhanrashid52/photoeditor/MultiTouchListener.kt
@@ -126,10 +126,6 @@ internal class MultiTouchListener(
         } ?: false
     }
 
-    fun setOnMultiTouchListener(onMultiTouchListener: OnMultiTouchListener?) {
-        this.onMultiTouchListener = onMultiTouchListener
-    }
-
     private inner class ScaleGestureListener : ScaleGestureDetector.SimpleOnScaleGestureListener() {
         private var mPivotX = 0f
         private var mPivotY = 0f
@@ -172,7 +168,6 @@ internal class MultiTouchListener(
     }
 
     internal interface OnMultiTouchListener {
-        fun onEditTextClickListener(text: String, colorCode: Int)
         fun onRemoveViewListener(removedView: View)
     }
 

diff --git a/photoeditor/src/main/java/ja/burhanrashid52/photoeditor/PhotoEditorImpl.kt b/photoeditor/src/main/java/ja/burhanrashid52/photoeditor/PhotoEditorImpl.kt
@@ -69,7 +69,7 @@ internal class PhotoEditorImpl @SuppressLint("ClickableViewAccessibility") const
     }
 
     override fun addText(text: String, styleBuilder: TextStyleBuilder?) {
-        drawingView?.enableDrawing(false)
+        drawingView.enableDrawing(false)
         val multiTouchListener = getMultiTouchListener(isTextPinchScalable)
         val textGraphic = Text(
             photoEditorView,
@@ -112,7 +112,7 @@ internal class PhotoEditorImpl @SuppressLint("ClickableViewAccessibility") const
     }
 
     override fun addEmoji(emojiTypeface: Typeface?, emojiName: String) {
-        drawingView?.enableDrawing(false)
+        drawingView.enableDrawing(false)
         val multiTouchListener = getMultiTouchListener(true)
         val emoji = Emoji(
             photoEditorView,
@@ -150,7 +150,7 @@ internal class PhotoEditorImpl @SuppressLint("ClickableViewAccessibility") const
     }
 
     override fun setBrushDrawingMode(brushDrawingMode: Boolean) {
-        drawingView?.enableDrawing(brushDrawingMode)
+        drawingView.enableDrawing(brushDrawingMode)
     }
 
     override val brushDrawableMode: Boolean
@@ -159,29 +159,29 @@ internal class PhotoEditorImpl @SuppressLint("ClickableViewAccessibility") const
     override fun setOpacity(@IntRange(from = 0, to = 100) opacity: Int) {
         var opacityValue = opacity
         opacityValue = (opacityValue / 100.0 * 255.0).toInt()
-        drawingView?.currentShapeBuilder?.withShapeOpacity(opacityValue)
+        drawingView.currentShapeBuilder.withShapeOpacity(opacityValue)
     }
 
     override var brushSize: Float
-        get() = drawingView?.currentShapeBuilder?.shapeSize ?: 0f
+        get() = drawingView.currentShapeBuilder.shapeSize
         set(size) {
-            drawingView?.currentShapeBuilder?.withShapeSize(size)
+            drawingView.currentShapeBuilder.withShapeSize(size)
         }
     override var brushColor: Int
-        get() = drawingView?.currentShapeBuilder?.shapeColor ?: 0
+        get() = drawingView.currentShapeBuilder.shapeColor
         set(color) {
-            drawingView?.currentShapeBuilder?.withShapeColor(color)
+            drawingView.currentShapeBuilder.withShapeColor(color)
         }
 
     override fun setBrushEraserSize(brushEraserSize: Float) {
-        drawingView?.eraserSize = brushEraserSize
+        drawingView.eraserSize = brushEraserSize
     }
 
     override val eraserSize: Float
-        get() = drawingView?.eraserSize ?: 0f
+        get() = drawingView.eraserSize
 
     override fun brushEraser() {
-        drawingView?.brushEraser()
+        drawingView.brushEraser()
     }
 
     override fun undo(): Boolean {
@@ -267,15 +267,11 @@ internal class PhotoEditorImpl @SuppressLint("ClickableViewAccessibility") const
 
     // region Shape
     override fun setShape(shapeBuilder: ShapeBuilder) {
-        drawingView?.currentShapeBuilder = shapeBuilder
+        drawingView.currentShapeBuilder = shapeBuilder
     } // endregion
 
-    companion object {
-        private const val TAG = "PhotoEditor"
-    }
-
     init {
-        drawingView?.setBrushViewChangeListener(mBrushDrawingStateListener)
+        drawingView.setBrushViewChangeListener(mBrushDrawingStateListener)
         val mDetector = GestureDetector(
             context,
             PhotoEditorImageViewListener(
@@ -287,7 +283,7 @@ internal class PhotoEditorImpl @SuppressLint("ClickableViewAccessibility") const
                 }
             )
         )
-        imageView?.setOnTouchListener { _, event ->
+        imageView.setOnTouchListener { _, event ->
             mOnPhotoEditorListener?.onTouchSourceImage(event)
             mDetector.onTouchEvent(event)
         }

diff --git a/photoeditor/src/main/java/ja/burhanrashid52/photoeditor/PhotoSaverTask.kt b/photoeditor/src/main/java/ja/burhanrashid52/photoeditor/PhotoSaverTask.kt
@@ -84,8 +84,4 @@ internal class PhotoSaverTask(
         return bitmap
     }
 
-    companion object {
-        const val TAG = "PhotoSaverTask"
-    }
-
 }
diff --git a/photoeditor/src/main/java/ja/burhanrashid52/photoeditor/Vector2D.kt b/photoeditor/src/main/java/ja/burhanrashid52/photoeditor/Vector2D.kt
@@ -5,8 +5,8 @@ import kotlin.math.atan2
 import kotlin.math.sqrt
 
 internal class Vector2D : PointF {
-    constructor() : super() {}
-    constructor(x: Float, y: Float) : super(x, y) {}
+    constructor() : super()
+    constructor(x: Float, y: Float) : super(x, y)
 
     private fun normalize() {
         val length = sqrt((x * x + y * y).toDouble()).toFloat()