Clean code

app/src/main/java/net/pokeranalytics/android/ui/activity/SettingsActivity.kt

@@ -9,25 +9,25 @@ import net.pokeranalytics.android.ui.activity.components.PokerAnalyticsActivity
 
 class SettingsActivity : PokerAnalyticsActivity() {
 
-    companion object {
-        fun newInstance(context: Context) {
-            val intent = Intent(context, SettingsActivity::class.java)
-            context.startActivity(intent)
-        }
+	companion object {
+		fun newInstance(context: Context) {
+			val intent = Intent(context, SettingsActivity::class.java)
+			context.startActivity(intent)
+		}
 
-        /**
-         * Create a new instance for result
-         */
-        fun newInstanceForResult(fragment: Fragment, requestCode: Int) {
-            val intent = Intent(fragment.requireContext(), SettingsActivity::class.java)
-            fragment.startActivityForResult(intent, requestCode)
-        }
+		/**
+		 * Create a new instance for result
+		 */
+		fun newInstanceForResult(fragment: Fragment, requestCode: Int) {
+			val intent = Intent(fragment.requireContext(), SettingsActivity::class.java)
+			fragment.startActivityForResult(intent, requestCode)
+		}
 
-    }
+	}
 
-    override fun onCreate(savedInstanceState: Bundle?) {
-        super.onCreate(savedInstanceState)
-        setContentView(R.layout.activity_settings)
-    }
+	override fun onCreate(savedInstanceState: Bundle?) {
+		super.onCreate(savedInstanceState)
+		setContentView(R.layout.activity_settings)
+	}
 
 }

app/src/main/java/net/pokeranalytics/android/ui/fragment/ReportsFragment.kt

@@ -7,6 +7,7 @@ import android.view.ViewGroup
 import androidx.recyclerview.widget.LinearLayoutManager
 import kotlinx.android.synthetic.main.fragment_stats.*
 import net.pokeranalytics.android.R
+import net.pokeranalytics.android.ui.activity.ComparisonChartActivity
 import net.pokeranalytics.android.ui.adapter.RowRepresentableAdapter
 import net.pokeranalytics.android.ui.adapter.RowRepresentableDelegate
 import net.pokeranalytics.android.ui.adapter.StaticRowRepresentableDataSource
@@ -56,6 +57,11 @@ class ReportsFragment : PokerAnalyticsFragment(), StaticRowRepresentableDataSour
 		return rowRepresentation
 	}
 
+	override fun onRowSelected(position: Int, row: RowRepresentable, fromAction: Boolean) {
+		super.onRowSelected(position, row, fromAction)
+		ComparisonChartActivity.newInstance(requireContext())
+	}
+
 
 	// Business
 

app/src/main/java/net/pokeranalytics/android/ui/fragment/SettingsFragment.kt

@@ -30,7 +30,6 @@ import java.util.*
 
 class SettingsFragment : PokerAnalyticsFragment(), RowRepresentableDelegate, StaticRowRepresentableDataSource {
 
-	private lateinit var parentActivity: PokerAnalyticsActivity
 
 	companion object {
 
@@ -53,15 +52,8 @@ class SettingsFragment : PokerAnalyticsFragment(), RowRepresentableDelegate, Sta
 		const val REQUEST_CODE_CURRENCY: Int = 100
 	}
 
-	override fun stringForRow(row: RowRepresentable): String {
-		return when (row) {
-			SettingRow.VERSION -> BuildConfig.VERSION_NAME + if (BuildConfig.DEBUG) " (${BuildConfig.VERSION_CODE}) DEBUG" else ""
-			SettingRow.CURRENCY -> Currency.getInstance(Preferences.getCurrencyLocale(this.parentActivity)).symbol
-			else -> ""
-		}
-	}
-
 	private lateinit var settingsAdapterRow: RowRepresentableAdapter
+	private lateinit var parentActivity: PokerAnalyticsActivity
 
 
 	override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?, savedInstanceState: Bundle?): View? {
@@ -87,6 +79,14 @@ class SettingsFragment : PokerAnalyticsFragment(), RowRepresentableDelegate, Sta
 		return rowRepresentation
 	}
 
+	override fun stringForRow(row: RowRepresentable): String {
+		return when (row) {
+			SettingRow.VERSION -> BuildConfig.VERSION_NAME + if (BuildConfig.DEBUG) " (${BuildConfig.VERSION_CODE}) DEBUG" else ""
+			SettingRow.CURRENCY -> Currency.getInstance(Preferences.getCurrencyLocale(this.parentActivity)).symbol
+			else -> ""
+		}
+	}
+
 	override fun onRowSelected(position: Int, row: RowRepresentable, fromAction: Boolean) {
 
 		when (row) {