Removes the use of the custom OpenClass annotation and the associated kotlin-allopen plugin

app/build.gradle

@@ -19,7 +19,6 @@
 plugins {
     id "com.android.application"
     id "kotlin-android"
-    id "kotlin-allopen"
     id "jacoco"
     id "org.jlleitschuh.gradle.ktlint" version "14.0.1"
 }
@@ -131,10 +130,6 @@ android {
     }
 }
 
-allOpen {
-    annotation("com.vrem.annotation.OpenClass")
-}
-
 // keystore -------------------------------------------------
 signingConfig()
 

app/src/main/kotlin/com/vrem/wifianalyzer/Configuration.kt

@@ -17,12 +17,9 @@
  */
 package com.vrem.wifianalyzer
 
-import com.vrem.annotation.OpenClass
-
 const val SIZE_MIN = 1024
 const val SIZE_MAX = 4096
 
-@OpenClass
 class Configuration(
     val largeScreen: Boolean,
 ) {

app/src/main/kotlin/com/vrem/wifianalyzer/DrawerNavigation.kt

@@ -21,9 +21,7 @@ import android.content.res.Configuration
 import androidx.appcompat.app.ActionBarDrawerToggle
 import androidx.appcompat.widget.Toolbar
 import androidx.drawerlayout.widget.DrawerLayout
-import com.vrem.annotation.OpenClass
 
-@OpenClass
 class DrawerNavigation(
     private val mainActivity: MainActivity,
     private val toolbar: Toolbar,

app/src/main/kotlin/com/vrem/wifianalyzer/MainActivity.kt

@@ -30,7 +30,6 @@ import androidx.core.splashscreen.SplashScreen.Companion.installSplashScreen
 import androidx.core.view.GravityCompat
 import androidx.drawerlayout.widget.DrawerLayout
 import com.google.android.material.navigation.NavigationView
-import com.vrem.annotation.OpenClass
 import com.vrem.util.createContext
 import com.vrem.wifianalyzer.navigation.NavigationMenu
 import com.vrem.wifianalyzer.navigation.NavigationMenuControl
@@ -41,7 +40,6 @@ import com.vrem.wifianalyzer.settings.Settings
 import com.vrem.wifianalyzer.wifi.accesspoint.ConnectionView
 import com.vrem.wifianalyzer.wifi.scanner.ScannerService
 
-@OpenClass
 class MainActivity :
     AppCompatActivity(),
     NavigationMenuControl,

app/src/main/kotlin/com/vrem/wifianalyzer/export/ExportIntent.kt

@@ -19,9 +19,7 @@
 package com.vrem.wifianalyzer.export
 
 import android.content.Intent
-import com.vrem.annotation.OpenClass
 
-@OpenClass
 class ExportIntent {
     internal fun intent(
         title: String,

app/src/main/kotlin/com/vrem/wifianalyzer/navigation/NavigationMenuController.kt

@@ -23,10 +23,8 @@ import androidx.core.view.forEach
 import androidx.core.view.get
 import com.google.android.material.bottomnavigation.BottomNavigationView
 import com.google.android.material.navigation.NavigationView
-import com.vrem.annotation.OpenClass
 import com.vrem.wifianalyzer.R
 
-@OpenClass
 class NavigationMenuController(
     navigationMenuControl: NavigationMenuControl,
     val drawerNavigationView: NavigationView = navigationMenuControl.findViewById(R.id.nav_drawer),

app/src/main/kotlin/com/vrem/wifianalyzer/navigation/options/OptionMenu.kt

@@ -22,10 +22,8 @@ import android.app.Activity
 import android.view.Menu
 import android.view.MenuItem
 import androidx.appcompat.view.menu.MenuBuilder
-import com.vrem.annotation.OpenClass
 import com.vrem.wifianalyzer.R
 
-@OpenClass
 class OptionMenu {
     var menu: Menu? = null
 

app/src/main/kotlin/com/vrem/wifianalyzer/permission/ApplicationPermission.kt

@@ -20,9 +20,7 @@ package com.vrem.wifianalyzer.permission
 import android.Manifest
 import android.app.Activity
 import android.content.pm.PackageManager
-import com.vrem.annotation.OpenClass
 
-@OpenClass
 class ApplicationPermission(
     private val activity: Activity,
     private val permissionDialog: PermissionDialog = PermissionDialog(activity),

app/src/main/kotlin/com/vrem/wifianalyzer/permission/LocationPermission.kt

@@ -21,10 +21,8 @@ import android.app.Activity
 import android.location.LocationManager
 import android.os.Build
 import androidx.annotation.RequiresApi
-import com.vrem.annotation.OpenClass
 import com.vrem.util.buildMinVersionP
 
-@OpenClass
 class LocationPermission(
     private val activity: Activity,
 ) {

app/src/main/kotlin/com/vrem/wifianalyzer/permission/PermissionDialog.kt

@@ -21,11 +21,9 @@ import android.app.Activity
 import android.app.AlertDialog
 import android.content.DialogInterface
 import android.view.View
-import com.vrem.annotation.OpenClass
 import com.vrem.util.buildMinVersionP
 import com.vrem.wifianalyzer.R
 
-@OpenClass
 class PermissionDialog(
     private val activity: Activity,
 ) {

app/src/main/kotlin/com/vrem/wifianalyzer/permission/PermissionService.kt

@@ -18,9 +18,7 @@
 package com.vrem.wifianalyzer.permission
 
 import android.app.Activity
-import com.vrem.annotation.OpenClass
 
-@OpenClass
 class PermissionService(
     private val activity: Activity,
     private val locationPermission: LocationPermission = LocationPermission(activity),

app/src/main/kotlin/com/vrem/wifianalyzer/settings/Settings.kt

@@ -18,7 +18,6 @@
 package com.vrem.wifianalyzer.settings
 
 import android.content.SharedPreferences.OnSharedPreferenceChangeListener
-import com.vrem.annotation.OpenClass
 import com.vrem.util.buildMinVersionQ
 import com.vrem.util.defaultCountryCode
 import com.vrem.util.defaultLanguageTag
@@ -40,7 +39,6 @@ import com.vrem.wifianalyzer.wifi.model.Strength
 import java.util.Locale
 import kotlin.enums.EnumEntries
 
-@OpenClass
 class Settings(
     private val repository: Repository,
 ) {

app/src/main/kotlin/com/vrem/wifianalyzer/vendor/VendorAdapter.kt

@@ -22,13 +22,11 @@ import android.view.View
 import android.view.ViewGroup
 import android.widget.ArrayAdapter
 import android.widget.TextView
-import com.vrem.annotation.OpenClass
 import com.vrem.wifianalyzer.MainContext
 import com.vrem.wifianalyzer.R
 import com.vrem.wifianalyzer.databinding.VendorDetailsBinding
 import com.vrem.wifianalyzer.vendor.model.VendorService
 
-@OpenClass
 internal class VendorAdapter(
     context: Context,
     private val vendorService: VendorService,

app/src/main/kotlin/com/vrem/wifianalyzer/vendor/model/VendorService.kt

@@ -18,14 +18,12 @@
 package com.vrem.wifianalyzer.vendor.model
 
 import android.content.res.Resources
-import com.vrem.annotation.OpenClass
 import com.vrem.util.EMPTY
 import com.vrem.util.readFile
 import com.vrem.wifianalyzer.R
 import java.util.Locale
 import java.util.TreeMap
 
-@OpenClass
 class VendorService(
     private val resources: Resources,
 ) {

app/src/main/kotlin/com/vrem/wifianalyzer/wifi/accesspoint/AccessPointDetail.kt

@@ -23,15 +23,13 @@ import android.widget.ImageView
 import android.widget.TextView
 import androidx.annotation.LayoutRes
 import androidx.core.content.ContextCompat
-import com.vrem.annotation.OpenClass
 import com.vrem.wifianalyzer.MainContext
 import com.vrem.wifianalyzer.R
 import com.vrem.wifianalyzer.wifi.model.WiFiAdditional
 import com.vrem.wifianalyzer.wifi.model.WiFiDetail
 import com.vrem.wifianalyzer.wifi.model.WiFiSecurity
 import com.vrem.wifianalyzer.wifi.model.WiFiSignal
 
-@OpenClass
 class AccessPointDetail {
     fun makeView(
         convertView: View?,

app/src/main/kotlin/com/vrem/wifianalyzer/wifi/accesspoint/AccessPointPopup.kt

@@ -19,10 +19,8 @@ package com.vrem.wifianalyzer.wifi.accesspoint
 
 import android.app.AlertDialog
 import android.view.View
-import com.vrem.annotation.OpenClass
 import com.vrem.wifianalyzer.wifi.model.WiFiDetail
 
-@OpenClass
 class AccessPointPopup {
     fun show(view: View): AlertDialog {
         val alertDialog: AlertDialog =

app/src/main/kotlin/com/vrem/wifianalyzer/wifi/accesspoint/AccessPointsAdapter.kt

@@ -22,13 +22,11 @@ import android.view.ViewGroup
 import android.widget.BaseExpandableListAdapter
 import android.widget.ExpandableListView
 import android.widget.ImageView
-import com.vrem.annotation.OpenClass
 import com.vrem.wifianalyzer.R
 import com.vrem.wifianalyzer.wifi.model.WiFiData
 import com.vrem.wifianalyzer.wifi.model.WiFiDetail
 import com.vrem.wifianalyzer.wifi.scanner.UpdateNotifier
 
-@OpenClass
 class AccessPointsAdapter(
     private val accessPointsAdapterData: AccessPointsAdapterData = AccessPointsAdapterData(),
     private val accessPointDetail: AccessPointDetail = AccessPointDetail(),

app/src/main/kotlin/com/vrem/wifianalyzer/wifi/accesspoint/AccessPointsAdapterData.kt

@@ -18,7 +18,6 @@
 package com.vrem.wifianalyzer.wifi.accesspoint
 
 import android.widget.ExpandableListView
-import com.vrem.annotation.OpenClass
 import com.vrem.wifianalyzer.MainContext
 import com.vrem.wifianalyzer.SIZE_MAX
 import com.vrem.wifianalyzer.SIZE_MIN
@@ -30,7 +29,6 @@ import com.vrem.wifianalyzer.wifi.model.WiFiDetail
 import com.vrem.wifianalyzer.wifi.predicate.makeAccessPointsPredicate
 import java.security.MessageDigest
 
-@OpenClass
 class AccessPointsAdapterData(
     private val accessPointsAdapterGroup: AccessPointsAdapterGroup = AccessPointsAdapterGroup(),
     val wiFiDetails: MutableList<WiFiDetail> = mutableListOf(),

app/src/main/kotlin/com/vrem/wifianalyzer/wifi/accesspoint/AccessPointsAdapterGroup.kt

@@ -18,12 +18,10 @@
 package com.vrem.wifianalyzer.wifi.accesspoint
 
 import android.widget.ExpandableListView
-import com.vrem.annotation.OpenClass
 import com.vrem.wifianalyzer.MainContext
 import com.vrem.wifianalyzer.wifi.model.GroupBy
 import com.vrem.wifianalyzer.wifi.model.WiFiDetail
 
-@OpenClass
 class AccessPointsAdapterGroup(
     val expanded: MutableSet<String> = mutableSetOf(),
     var groupBy: GroupBy = GroupBy.NONE,

app/src/main/kotlin/com/vrem/wifianalyzer/wifi/channelgraph/ChannelGraphView.kt

@@ -20,7 +20,6 @@ package com.vrem.wifianalyzer.wifi.channelgraph
 import android.view.View
 import com.jjoe64.graphview.GraphView
 import com.jjoe64.graphview.series.TitleLineGraphSeries
-import com.vrem.annotation.OpenClass
 import com.vrem.wifianalyzer.MainContext
 import com.vrem.wifianalyzer.R
 import com.vrem.wifianalyzer.settings.Settings
@@ -83,7 +82,6 @@ internal fun makeGraphViewWrapper(wiFiBand: WiFiBand): GraphViewWrapper {
     return graphViewWrapper
 }
 
-@OpenClass
 internal class ChannelGraphView(
     private val wiFiBand: WiFiBand,
     private var dataManager: DataManager = DataManager(),

app/src/main/kotlin/com/vrem/wifianalyzer/wifi/channelgraph/DataManager.kt

@@ -18,13 +18,11 @@
 package com.vrem.wifianalyzer.wifi.channelgraph
 
 import com.jjoe64.graphview.series.TitleLineGraphSeries
-import com.vrem.annotation.OpenClass
 import com.vrem.wifianalyzer.wifi.graphutils.GraphDataPoint
 import com.vrem.wifianalyzer.wifi.graphutils.GraphViewWrapper
 import com.vrem.wifianalyzer.wifi.graphutils.MIN_Y
 import com.vrem.wifianalyzer.wifi.model.WiFiDetail
 
-@OpenClass
 internal class DataManager {
     fun newSeries(wiFiDetails: List<WiFiDetail>): Set<WiFiDetail> = wiFiDetails.toSet()
 

app/src/main/kotlin/com/vrem/wifianalyzer/wifi/filter/adapter/BasicFilterAdapter.kt

@@ -17,12 +17,10 @@
  */
 package com.vrem.wifianalyzer.wifi.filter.adapter
 
-import com.vrem.annotation.OpenClass
 import com.vrem.wifianalyzer.settings.Settings
 
-@OpenClass
 abstract class BasicFilterAdapter<T>(
-    var selections: Set<T>,
+    open var selections: Set<T>,
 ) {
     fun selections(selections: Set<T>) {
         this.selections = selections

app/src/main/kotlin/com/vrem/wifianalyzer/wifi/filter/adapter/EnumFilterAdapter.kt

@@ -42,7 +42,7 @@ abstract class EnumFilterAdapter<T : Enum<T>>(
         selections(defaults.toSet())
     }
 
-    fun color(selection: T): Int = if (selections.contains(selection)) R.color.selected else R.color.regular
+    open fun color(selection: T): Int = if (selections.contains(selection)) R.color.selected else R.color.regular
 
     fun contains(selection: T): Boolean = selections.contains(selection)
 }

app/src/main/kotlin/com/vrem/wifianalyzer/wifi/filter/adapter/FiltersAdapter.kt

@@ -17,13 +17,11 @@
  */
 package com.vrem.wifianalyzer.wifi.filter.adapter
 
-import com.vrem.annotation.OpenClass
 import com.vrem.wifianalyzer.MainContext
 import com.vrem.wifianalyzer.navigation.NavigationMenu
 import com.vrem.wifianalyzer.settings.Settings
 import java.io.Serializable
 
-@OpenClass
 class FiltersAdapter(
     private val settings: Settings,
 ) {

app/src/main/kotlin/com/vrem/wifianalyzer/wifi/graphutils/GraphColors.kt

@@ -17,7 +17,6 @@
  */
 package com.vrem.wifianalyzer.wifi.graphutils
 
-import com.vrem.annotation.OpenClass
 import com.vrem.wifianalyzer.MainContext
 import com.vrem.wifianalyzer.R
 
@@ -30,7 +29,6 @@ data class GraphColor(
 
 internal val transparent = GraphColor(0x009E9E9E, 0x009E9E9E)
 
-@OpenClass
 class GraphColors {
     private val availableGraphColors: MutableList<GraphColor> = mutableListOf()
     private val currentGraphColors: ArrayDeque<GraphColor> = ArrayDeque()

app/src/main/kotlin/com/vrem/wifianalyzer/wifi/graphutils/GraphViewWrapper.kt

@@ -22,7 +22,6 @@ import com.jjoe64.graphview.LegendRenderer
 import com.jjoe64.graphview.series.BaseSeries
 import com.jjoe64.graphview.series.DataPointInterface
 import com.jjoe64.graphview.series.Series
-import com.vrem.annotation.OpenClass
 import com.vrem.wifianalyzer.MainContext
 import com.vrem.wifianalyzer.SIZE_MAX
 import com.vrem.wifianalyzer.SIZE_MIN
@@ -32,7 +31,6 @@ import com.vrem.wifianalyzer.wifi.accesspoint.AccessPointPopup
 import com.vrem.wifianalyzer.wifi.model.WiFiDetail
 import java.security.MessageDigest
 
-@OpenClass
 class GraphViewWrapper(
     val graphView: GraphView,
     var graphLegend: GraphLegend,

app/src/main/kotlin/com/vrem/wifianalyzer/wifi/graphutils/SeriesCache.kt

@@ -19,10 +19,8 @@ package com.vrem.wifianalyzer.wifi.graphutils
 
 import com.jjoe64.graphview.series.BaseSeries
 import com.jjoe64.graphview.series.Series
-import com.vrem.annotation.OpenClass
 import com.vrem.wifianalyzer.wifi.model.WiFiDetail
 
-@OpenClass
 class SeriesCache {
     private val cache: MutableMap<WiFiDetail, BaseSeries<GraphDataPoint>> = mutableMapOf()
 

app/src/main/kotlin/com/vrem/wifianalyzer/wifi/graphutils/SeriesOptions.kt

@@ -20,7 +20,6 @@ package com.vrem.wifianalyzer.wifi.graphutils
 import com.jjoe64.graphview.series.BaseSeries
 import com.jjoe64.graphview.series.LineGraphSeries
 import com.jjoe64.graphview.series.TitleLineGraphSeries
-import com.vrem.annotation.OpenClass
 
 private fun BaseSeries<GraphDataPoint>.removeSeriesColor(graphColors: GraphColors) =
     graphColors.addColor(this.color.toLong())
@@ -55,7 +54,6 @@ private fun BaseSeries<GraphDataPoint>.drawBackground(drawBackground: Boolean) {
     }
 }
 
-@OpenClass
 class SeriesOptions(
     private val graphColors: GraphColors = GraphColors(),
 ) {