Fix crash when layout manager is set via XML

dpadrecyclerview/api/dpadrecyclerview.api

@@ -421,6 +421,7 @@ public final class com/rubensousa/dpadrecyclerview/layoutmanager/DpadLayoutParam
 }
 
 public final class com/rubensousa/dpadrecyclerview/layoutmanager/PivotLayoutManager : androidx/recyclerview/widget/RecyclerView$LayoutManager, androidx/recyclerview/widget/ItemTouchHelper$ViewDropHandler, androidx/recyclerview/widget/RecyclerView$SmoothScroller$ScrollVectorProvider {
+	public fun <init> (Landroid/content/Context;Landroid/util/AttributeSet;II)V
 	public fun <init> (Landroidx/recyclerview/widget/RecyclerView$LayoutManager$Properties;)V
 	public final fun addOnLayoutCompletedListener (Lcom/rubensousa/dpadrecyclerview/DpadRecyclerView$OnLayoutCompletedListener;)V
 	public final fun addOnViewFocusedListener (Lcom/rubensousa/dpadrecyclerview/OnViewFocusedListener;)V

dpadrecyclerview/src/androidTest/kotlin/com/rubensousa/dpadrecyclerview/test/tests/PivotLayoutManagerTest.kt

@@ -16,9 +16,13 @@
 
 package com.rubensousa.dpadrecyclerview.test.tests
 
+import androidx.fragment.app.Fragment
+import androidx.fragment.app.testing.launchFragment
 import androidx.recyclerview.widget.RecyclerView.LayoutManager.Properties
 import com.google.common.truth.Truth.assertThat
+import com.rubensousa.dpadrecyclerview.DpadRecyclerView
 import com.rubensousa.dpadrecyclerview.layoutmanager.PivotLayoutManager
+import com.rubensousa.dpadrecyclerview.test.R
 import org.junit.Test
 
 class PivotLayoutManagerTest {
@@ -41,4 +45,21 @@ class PivotLayoutManagerTest {
         assertThat(config.focusOutSideBack).isTrue()
     }
 
+    @Test
+    fun testPivotLayoutManagerFromXML() {
+        // given
+        val fragment = launchFragment<XMLFragment>()
+        var recyclerView: DpadRecyclerView? = null
+
+        // when
+        fragment.onFragment {
+            recyclerView = it.requireView() as DpadRecyclerView
+        }
+
+        // then
+        assertThat(recyclerView).isNotNull()
+    }
+
+    class XMLFragment : Fragment(R.layout.dpadrecyclerview_layout_name)
+
 }

dpadrecyclerview/src/androidTest/res/layout/dpadrecyclerview_layout_name.xml

@@ -0,0 +1,7 @@
+<?xml version="1.0" encoding="utf-8"?>
+<com.rubensousa.dpadrecyclerview.DpadRecyclerView xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    android:id="@+id/recyclerView"
+    android:layout_width="match_parent"
+    android:layout_height="match_parent"
+    app:layoutManager="com.rubensousa.dpadrecyclerview.layoutmanager.PivotLayoutManager" />

dpadrecyclerview/src/main/java/com/rubensousa/dpadrecyclerview/DpadRecyclerView.kt

@@ -64,7 +64,8 @@ open class DpadRecyclerView @JvmOverloads constructor(
         internal val DEBUG = BuildConfig.DEBUG
     }
 
-    private val viewHolderTaskExecutor = ViewHolderTaskExecutor()
+    // Nullable because setLayoutManager can be called by the parent class before this instantiated
+    private var viewHolderTaskExecutor: ViewHolderTaskExecutor? = ViewHolderTaskExecutor()
     private val focusableChildDrawingCallback = FocusableChildDrawingCallback()
     private val fadingEdge = FadingEdge()
 
@@ -212,7 +213,9 @@ open class DpadRecyclerView @JvmOverloads constructor(
 
     final override fun setLayoutManager(layout: LayoutManager?) {
         super.setLayoutManager(layout)
-        pivotLayoutManager?.removeOnViewHolderSelectedListener(viewHolderTaskExecutor)
+        viewHolderTaskExecutor?.let {
+            pivotLayoutManager?.removeOnViewHolderSelectedListener(it)
+        }
         pivotLayoutManager?.updateRecyclerView(null)
         if (pivotLayoutManager !== layout) {
             pivotLayoutManager?.layoutCompletedListener = null
@@ -223,7 +226,8 @@ open class DpadRecyclerView @JvmOverloads constructor(
 
         if (layout != null && layout !is PivotLayoutManager) {
             throw IllegalArgumentException(
-                "Only PivotLayoutManager is supported, but got $layout"
+                "Only com.rubensousa.dpadrecyclerview.layoutmanager.PivotLayoutManager" +
+                        ".PivotLayoutManager is supported, but got $layout"
             )
         }
         if (layout is PivotLayoutManager) {
@@ -233,7 +237,7 @@ open class DpadRecyclerView @JvmOverloads constructor(
                     hasPendingLayout = false
                 }
             }
-            layout.addOnViewHolderSelectedListener(viewHolderTaskExecutor)
+            viewHolderTaskExecutor?.let { layout.addOnViewHolderSelectedListener(it) }
             pivotLayoutManager = layout
         }
     }
@@ -1022,7 +1026,7 @@ open class DpadRecyclerView @JvmOverloads constructor(
      * @param task     Task to executed on the ViewHolder at the given position
      */
     fun setSelectedPosition(position: Int, task: ViewHolderTask) {
-        viewHolderTaskExecutor.schedule(position, task)
+        viewHolderTaskExecutor?.schedule(position, task)
         requireLayout().selectPosition(position, subPosition = 0, smooth = false)
     }
 
@@ -1041,7 +1045,7 @@ open class DpadRecyclerView @JvmOverloads constructor(
      * @param task     Task to executed on the ViewHolder at the given position
      */
     fun setSelectedPositionSmooth(position: Int, task: ViewHolderTask) {
-        viewHolderTaskExecutor.schedule(position, task)
+        viewHolderTaskExecutor?.schedule(position, task)
         requireLayout().selectPosition(position, subPosition = 0, smooth = true)
     }
 
@@ -1062,7 +1066,7 @@ open class DpadRecyclerView @JvmOverloads constructor(
      * @param task     Task to executed on the ViewHolder at the given position
      */
     fun setSelectedSubPosition(position: Int, subPosition: Int, task: ViewHolderTask) {
-        viewHolderTaskExecutor.schedule(position, subPosition, task)
+        viewHolderTaskExecutor?.schedule(position, subPosition, task)
         requireLayout().selectPosition(position, subPosition, smooth = false)
     }
 
@@ -1099,7 +1103,7 @@ open class DpadRecyclerView @JvmOverloads constructor(
      * @param task     Task to executed on the ViewHolder at the given position
      */
     fun setSelectedSubPositionSmooth(position: Int, subPosition: Int, task: ViewHolderTask) {
-        viewHolderTaskExecutor.schedule(position, subPosition, task)
+        viewHolderTaskExecutor?.schedule(position, subPosition, task)
         requireLayout().selectPosition(position, subPosition, smooth = true)
     }
 

dpadrecyclerview/src/main/java/com/rubensousa/dpadrecyclerview/layoutmanager/PivotLayoutManager.kt

@@ -79,6 +79,12 @@ class PivotLayoutManager(properties: Properties) : RecyclerView.LayoutManager(),
     private var isScrollingFromTouchEvent = false
     internal var layoutCompletedListener: DpadRecyclerView.OnLayoutCompletedListener? = null
 
+    // This is required for XML usage
+    @Suppress("unused", "UNUSED_PARAMETER")
+    constructor(context: Context, attrs: AttributeSet?, defStyleAttr: Int, defStyleRes: Int) : this(
+        Properties()
+    )
+
     override fun checkLayoutParams(layoutParams: RecyclerView.LayoutParams?): Boolean {
         return layoutParams is DpadLayoutParams
     }