Revert "Task/code cleanup"

app/src/main/java/com/sduduzog/slimlauncher/MainActivity.kt

@@ -27,12 +27,12 @@ class MainActivity : AppCompatActivity(), SharedPreferences.OnSharedPreferenceCh
     }
 
     private fun dispatchBack() {
-        subscribers.forEach { if (it.onBack()) return@forEach }
+        for (s in subscribers) if (s.onBack()) return
         completeBackAction()
     }
 
     private fun dispatchHome() {
-        subscribers.forEach { it.onHome() }
+        for (s in subscribers) s.onHome()
     }
 
     override fun onCreate(savedInstanceState: Bundle?) {

app/src/main/java/com/sduduzog/slimlauncher/adapters/AddAppAdapter.kt

@@ -11,7 +11,7 @@ import com.sduduzog.slimlauncher.utils.OnAppClickedListener
 
 class AddAppAdapter(private val listener: OnAppClickedListener) : RecyclerView.Adapter<AddAppAdapter.ViewHolder>() {
 
-    private val apps: MutableList<App> = mutableListOf()
+    private var apps: List<App> = listOf()
 
     override fun getItemCount(): Int = apps.size
 
@@ -28,9 +28,8 @@ class AddAppAdapter(private val listener: OnAppClickedListener) : RecyclerView.A
         return ViewHolder(view)
     }
 
-    fun setItems(apps: List<App>) {
-        this.apps.clear()
-        this.apps.addAll(apps)
+    fun setItems(apps: List<App>){
+        this.apps = apps
         notifyDataSetChanged()
     }
 
@@ -38,6 +37,8 @@ class AddAppAdapter(private val listener: OnAppClickedListener) : RecyclerView.A
 
         val appName: TextView = itemView.findViewById(R.id.aa_list_item_app_name)
 
-        override fun toString(): String = "${super.toString()} '${appName.text}'"
+        override fun toString(): String {
+            return super.toString() + " '${appName.text}'"
+        }
     }
 }

app/src/main/java/com/sduduzog/slimlauncher/adapters/CustomAppsAdapter.kt

@@ -16,7 +16,7 @@ import com.sduduzog.slimlauncher.utils.OnShitDoneToAppsListener
 
 class CustomAppsAdapter(private val listener: OnShitDoneToAppsListener) : RecyclerView.Adapter<CustomAppsAdapter.ViewHolder>(), OnItemActionListener {
 
-    private val apps: MutableList<HomeApp> = mutableListOf()
+    private var apps: MutableList<HomeApp> = mutableListOf()
     private lateinit var touchHelper: ItemTouchHelper
 
     override fun getItemCount(): Int = apps.size
@@ -42,16 +42,13 @@ class CustomAppsAdapter(private val listener: OnShitDoneToAppsListener) : Recycl
     }
 
     fun setItems(apps: List<HomeApp>) {
-        this.apps.apply {
-            clear()
-            addAll(sanitiseIndexes(apps))
-        }
+        this.apps = sanitiseIndexes(apps) as MutableList<HomeApp>
         notifyDataSetChanged()
     }
 
     private fun sanitiseIndexes(apps: List<HomeApp>): List<HomeApp> {
-        apps.indices.forEach {
-            apps[it].sortingIndex = it
+        for (i in apps.indices) {
+            apps[i].sortingIndex = i
         }
         return apps
     }
@@ -85,11 +82,12 @@ class CustomAppsAdapter(private val listener: OnShitDoneToAppsListener) : Recycl
     }
 
     inner class ViewHolder(itemView: View) : RecyclerView.ViewHolder(itemView) {
-
         val dragHandle: TextView = itemView.findViewById(R.id.ca_list_item_drag_handle)
         val appName: TextView = itemView.findViewById(R.id.ca_list_item_app_name)
         val menuIcon: ImageView = itemView.findViewById(R.id.ca_list_item_more_icon)
 
-        override fun toString(): String = "${super.toString()} '${appName.text}'"
+        override fun toString(): String {
+            return super.toString() + " '${appName.text}'"
+        }
     }
 }

app/src/main/java/com/sduduzog/slimlauncher/adapters/HomeAdapter.kt

@@ -13,7 +13,7 @@ import com.sduduzog.slimlauncher.utils.OnLaunchAppListener
 class HomeAdapter(private val listener: OnLaunchAppListener)
     : RecyclerView.Adapter<HomeAdapter.ViewHolder>() {
 
-    private val apps: MutableList<HomeApp> = mutableListOf()
+    private var apps: List<HomeApp> = listOf()
 
     override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): ViewHolder {
         val view = LayoutInflater.from(parent.context)
@@ -22,7 +22,7 @@ class HomeAdapter(private val listener: OnLaunchAppListener)
     }
 
     override fun onBindViewHolder(holder: ViewHolder, position: Int) {
-        val item = apps[position]
+        val item = apps.elementAt(position)
         holder.mLabelView.text = item.appName
         holder.mLabelView.setOnClickListener {
             listener.onLaunch(item, it)
@@ -32,15 +32,16 @@ class HomeAdapter(private val listener: OnLaunchAppListener)
     override fun getItemCount(): Int = apps.size
 
     fun setItems(list: List<HomeApp>) {
-        apps.addAll(list)
+        this.apps = list
         notifyDataSetChanged()
     }
 
 
     inner class ViewHolder(mView: View) : RecyclerView.ViewHolder(mView) {
-
         val mLabelView: TextView = mView.findViewById(R.id.home_fragment_list_item_app_name)
 
-        override fun toString(): String = "${super.toString()} '${mLabelView.text}'"
+        override fun toString(): String {
+            return super.toString() + " '" + mLabelView.text + "'"
+        }
     }
 }

app/src/main/java/com/sduduzog/slimlauncher/data/model/App.kt

@@ -1,7 +1,7 @@
 package com.sduduzog.slimlauncher.data.model
 
 data class App(
-        val appName: String,
-        val packageName: String,
-        val activityName: String
+        var appName: String,
+        var packageName: String,
+        var activityName: String
 )

app/src/main/java/com/sduduzog/slimlauncher/dialogs/ChangeThemeDialog.kt

@@ -9,13 +9,13 @@ import androidx.core.content.edit
 import androidx.fragment.app.DialogFragment
 import com.sduduzog.slimlauncher.R
 
-class ChangeThemeDialog private constructor() : DialogFragment() {
+class ChangeThemeDialog : DialogFragment(){
 
     private lateinit var settings: SharedPreferences
 
     override fun onCreateDialog(savedInstanceState: Bundle?): Dialog {
-        val builder = AlertDialog.Builder(context)
-        settings = context!!.getSharedPreferences(getString(R.string.prefs_settings), MODE_PRIVATE)
+        val builder = AlertDialog.Builder(context!!)
+        settings  = context!!.getSharedPreferences(getString(R.string.prefs_settings), MODE_PRIVATE)
 
         val active = settings.getInt(getString(R.string.prefs_settings_key_theme), 0)
         builder.setTitle(R.string.choose_theme_dialog_title)
@@ -29,6 +29,8 @@ class ChangeThemeDialog private constructor() : DialogFragment() {
     }
 
     companion object {
-        fun getThemeChooser(): ChangeThemeDialog = ChangeThemeDialog()
+        fun getThemeChooser(): ChangeThemeDialog{
+            return ChangeThemeDialog()
+        }
     }
 }

app/src/main/java/com/sduduzog/slimlauncher/dialogs/ChooseTimeFormatDialog.kt

@@ -9,21 +9,22 @@ import androidx.core.content.edit
 import androidx.fragment.app.DialogFragment
 import com.sduduzog.slimlauncher.R
 
-class ChooseTimeFormatDialog private constructor() : DialogFragment() {
+class ChooseTimeFormatDialog : DialogFragment(){
 
     private lateinit var settings: SharedPreferences
 
     override fun onCreateDialog(savedInstanceState: Bundle?): Dialog {
-        val builder = AlertDialog.Builder(context)
+        val builder = AlertDialog.Builder(context!!)
         settings = context!!.getSharedPreferences(getString(R.string.prefs_settings), MODE_PRIVATE)
 
         val is24Hour = settings.getBoolean(getString(R.string.prefs_settings_key_time_format), true)
         val index = if (is24Hour) 1 else 0
         builder.setTitle(R.string.choose_time_format_dialog_title)
-        builder.setSingleChoiceItems(R.array.time_format_array, index) { dialogInterface, i ->
+        builder.setSingleChoiceItems(R.array.time_format_array, index) {dialogInterface, i ->
             dialogInterface.dismiss()
             settings.edit {
-                putBoolean(getString(R.string.prefs_settings_key_time_format), i != 0)
+                val b = i != 0
+                putBoolean(getString(R.string.prefs_settings_key_time_format), b)
             }
 
         }
@@ -32,6 +33,8 @@ class ChooseTimeFormatDialog private constructor() : DialogFragment() {
 
 
     companion object {
-        fun getInstance(): ChooseTimeFormatDialog = ChooseTimeFormatDialog()
+        fun getInstance(): ChooseTimeFormatDialog{
+            return ChooseTimeFormatDialog()
+        }
     }
 }

app/src/main/java/com/sduduzog/slimlauncher/dialogs/RenameAppDialog.kt

@@ -12,22 +12,21 @@ import com.sduduzog.slimlauncher.data.MainViewModel
 import com.sduduzog.slimlauncher.data.model.HomeApp
 import kotlinx.android.synthetic.main.customise_apps_fragment.*
 
-class RenameAppDialog private constructor(
-        private val app: HomeApp,
-        private val model: MainViewModel
-) : DialogFragment() {
+class RenameAppDialog : DialogFragment() {
+
+    private lateinit var app: HomeApp
+    private lateinit var model: MainViewModel
 
     override fun onCreateDialog(savedInstanceState: Bundle?): Dialog {
         val view = LayoutInflater.from(context).inflate(R.layout.rename_dialog_edit_text, customise_apps_fragment, false)
         val editText: EditText = view.findViewById(R.id.rename_editText)
         editText.text.append(app.appName)
-        val builder = AlertDialog.Builder(context!!).apply {
-            setTitle("Rename ${app.appName}")
-            setView(view)
-            setPositiveButton("DONE") { _, _ ->
-                val name = editText.text.toString()
-                updateApp(name)
-            }
+        val builder = AlertDialog.Builder(context!!)
+        builder.setTitle("Rename ${app.appName}")
+        builder.setView(view)
+        builder.setPositiveButton("DONE") { _, _ ->
+            val name = editText.text.toString()
+            updateApp(name)
         }
         editText.setOnEditorActionListener { v, _, _ ->
             val name = v.text.toString()
@@ -48,6 +47,11 @@ class RenameAppDialog private constructor(
     }
 
     companion object {
-        fun getInstance(app: HomeApp, model: MainViewModel) = RenameAppDialog(app, model)
+        fun getInstance(app: HomeApp, model: MainViewModel): RenameAppDialog {
+            return RenameAppDialog().apply {
+                this.model = model
+                this.app = app
+            }
+        }
     }
 }

app/src/main/java/com/sduduzog/slimlauncher/ui/main/HomeFragment.kt

@@ -1,11 +1,6 @@
 package com.sduduzog.slimlauncher.ui.main
 
-import android.content.ActivityNotFoundException
-import android.content.BroadcastReceiver
-import android.content.ComponentName
-import android.content.Context
-import android.content.Intent
-import android.content.IntentFilter
+import android.content.*
 import android.os.Build
 import android.os.Bundle
 import android.provider.AlarmClock
@@ -24,8 +19,7 @@ import com.sduduzog.slimlauncher.utils.BaseFragment
 import com.sduduzog.slimlauncher.utils.OnLaunchAppListener
 import kotlinx.android.synthetic.main.home_fragment.*
 import java.text.SimpleDateFormat
-import java.util.Date
-import java.util.Locale
+import java.util.*
 
 
 class HomeFragment : BaseFragment(), OnLaunchAppListener {
@@ -70,7 +64,9 @@ class HomeFragment : BaseFragment(), OnLaunchAppListener {
         activity?.registerReceiver(receiver, IntentFilter(Intent.ACTION_TIME_TICK))
     }
 
-    override fun getFragmentView(): ViewGroup = home_fragment
+    override fun getFragmentView(): ViewGroup {
+        return home_fragment
+    }
 
     override fun onResume() {
         super.onResume()
@@ -104,11 +100,10 @@ class HomeFragment : BaseFragment(), OnLaunchAppListener {
 
         home_fragment_date.setOnClickListener {
             try {
-                Intent(Intent.ACTION_MAIN).apply {
-                    addCategory(Intent.CATEGORY_APP_CALENDAR)
-                    flags = Intent.FLAG_ACTIVITY_NEW_TASK
-                    launchActivity(it, this)
-                }
+                val intent = Intent(Intent.ACTION_MAIN)
+                intent.addCategory(Intent.CATEGORY_APP_CALENDAR)
+                intent.flags = Intent.FLAG_ACTIVITY_NEW_TASK
+                launchActivity(it, intent)
             } catch (e: ActivityNotFoundException) {
                 // Do nothing, we've failed :(
             }
@@ -142,15 +137,12 @@ class HomeFragment : BaseFragment(), OnLaunchAppListener {
         val twenty4Hour = context?.getSharedPreferences(getString(R.string.prefs_settings), Context.MODE_PRIVATE)
                 ?.getBoolean(getString(R.string.prefs_settings_key_time_format), true)
         val date = Date()
-        when (twenty4Hour) {
-            is Boolean -> {
-                val fWatchTime = SimpleDateFormat("h:mm aa", Locale.ROOT)
-                home_fragment_time.text = fWatchTime.format(date)
-            }
-            else -> {
-                val fWatchTime = SimpleDateFormat("H:mm", Locale.ROOT)
-                home_fragment_time.text = fWatchTime.format(date)
-            }
+        if (twenty4Hour as Boolean) {
+            val fWatchTime = SimpleDateFormat("h:mm aa", Locale.ROOT)
+            home_fragment_time.text = fWatchTime.format(date)
+        } else {
+            val fWatchTime = SimpleDateFormat("H:mm", Locale.ROOT)
+            home_fragment_time.text = fWatchTime.format(date)
         }
         val fWatchDate = SimpleDateFormat("EEE, MMM dd", Locale.ROOT)
         home_fragment_date.text = fWatchDate.format(date)
@@ -158,15 +150,14 @@ class HomeFragment : BaseFragment(), OnLaunchAppListener {
 
     override fun onLaunch(app: HomeApp, view: View) {
         try {
+            val intent = Intent()
             val name = ComponentName(app.packageName, app.activityName)
-            Intent().apply {
-                action = Intent.ACTION_MAIN
-                addCategory(Intent.CATEGORY_LAUNCHER)
-                flags = Intent.FLAG_ACTIVITY_NEW_TASK or Intent.FLAG_ACTIVITY_RESET_TASK_IF_NEEDED
-                component = name
-                resolveActivity(activity!!.packageManager)?.let {
-                    launchActivity(view, this)
-                }
+            intent.action = Intent.ACTION_MAIN
+            intent.addCategory(Intent.CATEGORY_LAUNCHER)
+            intent.flags = Intent.FLAG_ACTIVITY_NEW_TASK or Intent.FLAG_ACTIVITY_RESET_TASK_IF_NEEDED
+            intent.component = name
+            intent.resolveActivity(activity!!.packageManager)?.let {
+                launchActivity(view, intent)
             }
         } catch (e: Exception) {
             // Do no shit yet

app/src/main/java/com/sduduzog/slimlauncher/ui/options/AddAppFragment.kt

@@ -37,7 +37,6 @@ class AddAppFragment : BaseFragment(), OnAppClickedListener {
         activity?.let {
             viewModel = ViewModelProviders.of(it).get(MainViewModel::class.java)
         } ?: throw Error("How the fuck is this fragment alive while there's no activity?")
-
         viewModel.installedApps.observe(this, Observer {
             val homeApps = viewModel.apps.value.orEmpty()
             it?.let { apps ->
@@ -65,8 +64,10 @@ class AddAppFragment : BaseFragment(), OnAppClickedListener {
             // Do nothing
         }
 
-        override fun onTextChanged(s: CharSequence, start: Int, before: Int, count: Int) {
-            LoadInstalledApps(viewModel, s.toString()).execute(context!!.packageManager)
+        override fun onTextChanged(s: CharSequence?, start: Int, before: Int, count: Int) {
+            s?.let {
+                LoadInstalledApps(viewModel, s.toString()).execute(context!!.packageManager)
+            }
         }
     }
 

app/src/main/java/com/sduduzog/slimlauncher/ui/options/CustomiseAppsFragment.kt

@@ -109,10 +109,10 @@ class CustomiseAppsFragment : BaseFragment(), OnShitDoneToAppsListener {
     }
 
     private fun showPopupMenu(view: View): PopupMenu {
-        return PopupMenu(context!!, view).apply {
-            menuInflater.inflate(R.menu.customise_apps_popup_menu, menu)
-            show()
-        }
+        val popup = PopupMenu(context!!, view)
+        popup.menuInflater.inflate(R.menu.customise_apps_popup_menu, popup.menu)
+        popup.show()
+        return popup
     }
 
     override fun onAppsUpdated(list: List<HomeApp>) {
@@ -123,8 +123,7 @@ class CustomiseAppsFragment : BaseFragment(), OnShitDoneToAppsListener {
         showPopupMenu(view).setOnMenuItemClickListener {
             when (it.itemId) {
                 R.id.ca_menu_rename -> {
-                    RenameAppDialog.getInstance(app, viewModel)
-                            .show(childFragmentManager, "SettingsListAdapter")
+                    RenameAppDialog.getInstance(app, viewModel).show(childFragmentManager, "SettingsListAdapter")
                 }
                 R.id.ca_menu_remove -> {
                     viewModel.remove(app)

app/src/main/java/com/sduduzog/slimlauncher/utils/BaseFragment.kt

@@ -15,11 +15,13 @@ abstract class BaseFragment : Fragment() {
 
     abstract fun getFragmentView(): ViewGroup
 
+
     override fun onResume() {
         super.onResume()
         val settings = context!!.getSharedPreferences(getString(R.string.prefs_settings), AppCompatActivity.MODE_PRIVATE)
         val active = settings.getInt(getString(R.string.prefs_settings_key_theme), 0)
 
+
         if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.M) {
             when (active) {
                 0, 3, 5 -> {

app/src/main/java/com/sduduzog/slimlauncher/utils/LoadInstalledApps.kt

@@ -10,7 +10,7 @@ import android.provider.MediaStore
 import com.sduduzog.slimlauncher.BuildConfig
 import com.sduduzog.slimlauncher.data.MainViewModel
 import com.sduduzog.slimlauncher.data.model.App
-import java.util.Collections
+import java.util.*
 
 class LoadInstalledApps(private val viewModel: MainViewModel?, private val filterString: String = "") : AsyncTask<PackageManager, Unit, List<App>>() {
 
@@ -24,13 +24,10 @@ class LoadInstalledApps(private val viewModel: MainViewModel?, private val filte
         main.addCategory(Intent.CATEGORY_LAUNCHER)
         val activitiesList = pm.queryIntentActivities(main, 0)
         Collections.sort(activitiesList, ResolveInfo.DisplayNameComparator(pm))
-        activitiesList.indices.forEach {
-            val item = activitiesList[it]
+        for (i in activitiesList.indices) {
+            val item = activitiesList[i]
             val activity = item.activityInfo
-            val app = App(
-                    activitiesList[it].loadLabel(pm).toString(),
-                    activity.applicationInfo.packageName, activity.name
-            )
+            val app = App(activitiesList[i].loadLabel(pm).toString(), activity.applicationInfo.packageName, activity.name)
             list.add(app)
         }