Skip to content

Commit cebe1c2

Browse files
authored
chore : lint fix (commons-app#6099)
- Removed unused constants which were there in `Pref` - Removed unused parameter requireActivity from `Setting Fragment` Revert "chore : lint fix" This reverts commit 5992033.
1 parent 1d8d1d6 commit cebe1c2

File tree

3 files changed

+2
-10
lines changed

3 files changed

+2
-10
lines changed

app/src/main/java/fr/free/nrw/commons/settings/Prefs.kt

-4
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,7 @@
11
package fr.free.nrw.commons.settings
22

33
object Prefs {
4-
const val GLOBAL_PREFS = "fr.free.nrw.commons.preferences"
5-
6-
const val TRACKING_ENABLED = "eventLogging"
74
const val DEFAULT_LICENSE = "defaultLicense"
8-
const val UPLOADS_SHOWING = "uploadsShowing"
95
const val MANAGED_EXIF_TAGS = "managed_exif_tags"
106
const val DESCRIPTION_LANGUAGE = "languageDescription"
117
const val APP_UI_LANGUAGE = "appUiLanguage"

app/src/main/java/fr/free/nrw/commons/settings/SettingsActivity.kt

-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import fr.free.nrw.commons.theme.BaseActivity
1212
class SettingsActivity : BaseActivity() {
1313

1414
private lateinit var binding: ActivitySettingsBinding
15-
// private var settingsDelegate: AppCompatDelegate? = null
1615

1716
/**
1817
* to be called when the activity starts

app/src/main/java/fr/free/nrw/commons/settings/SettingsFragment.kt

+2-5
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
package fr.free.nrw.commons.settings
22

33
import android.Manifest.permission
4-
import android.annotation.SuppressLint
54
import android.app.Activity
65
import android.app.Dialog
76
import android.content.Context.MODE_PRIVATE
@@ -11,7 +10,6 @@ import android.net.Uri
1110
import android.os.Bundle
1211
import android.text.Editable
1312
import android.text.TextWatcher
14-
import android.view.KeyEvent
1513
import android.view.View
1614
import android.widget.AdapterView
1715
import android.widget.Button
@@ -131,7 +129,7 @@ class SettingsFragment : PreferenceFragmentCompat() {
131129
inAppCameraLocationPref?.setOnPreferenceChangeListener { _, newValue ->
132130
val isInAppCameraLocationTurnedOn = newValue as Boolean
133131
if (isInAppCameraLocationTurnedOn) {
134-
createDialogsAndHandleLocationPermissions(requireActivity())
132+
createDialogsAndHandleLocationPermissions()
135133
}
136134
true
137135
}
@@ -256,7 +254,7 @@ class SettingsFragment : PreferenceFragmentCompat() {
256254
*
257255
* @param activity
258256
*/
259-
private fun createDialogsAndHandleLocationPermissions(activity: Activity) {
257+
private fun createDialogsAndHandleLocationPermissions() {
260258
inAppCameraLocationPermissionLauncher.launch(arrayOf(permission.ACCESS_FINE_LOCATION))
261259
}
262260

@@ -284,7 +282,6 @@ class SettingsFragment : PreferenceFragmentCompat() {
284282
return object : PreferenceGroupAdapter(preferenceScreen) {
285283
override fun onBindViewHolder(holder: PreferenceViewHolder, position: Int) {
286284
super.onBindViewHolder(holder, position)
287-
val preference = getItem(position)
288285
val iconFrame: View? = holder.itemView.findViewById(R.id.icon_frame)
289286
iconFrame?.visibility = View.GONE
290287
}

0 commit comments

Comments
 (0)