Rollback Ktlint update (#6865)

Task/Issue URL:
https://app.asana.com/1/137249556945/project/488551667048375/task/1211506961789626?focus=true

### Description

- Rolls back to ktlint 0.50.0, the last version I saw working without
issue prior to Ktlint 1.0 and addition of new rules. We were using
`0.47.1` with Kotlinter prior to the move to Spotless
- Apply a one off formatting to the codebase to prevent others needing
to make the changes incrementally.
- We still ratchetFrom develop to make doubly sure that when this goes
in that if anything, people still only format files they touch. But that
should be minimal.

### Steps to test this PR

Apply patch

```
Index: code-formatting.gradle
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/code-formatting.gradle b/code-formatting.gradle
--- a/code-formatting.gradle	(revision b355a47)
+++ b/code-formatting.gradle	(date 1759319873227)
@@ -1,7 +1,6 @@
 apply plugin: 'com.diffplug.spotless'
 
 spotless {
-    ratchetFrom 'origin/develop'
     kotlin {
         target 'src/*/kotlin/**/*.kt', 'src/*/java/**/*.kt'
         ktlint("0.50.0")
```

- [x] Checkout the branch
- [x] Run `.gradlew spotlessCheck`. You _might_ need to add --no-daemon
so that Spotless picks up the .editorConfig changes. Let me know if you
do.
- [x] Make some code edits (re-order imports, move java packages up for
instance)
- [x] Run `.gradlew spotlessApply`
- [x] Formatting should be applied
 

### UI changes
N/A

Author: mikescamell

.editorconfig

@@ -45,7 +45,7 @@ ij_kotlin_field_annotation_wrap = normal
 ij_kotlin_finally_on_new_line = false
 ij_kotlin_if_rparen_on_new_line = true
 ij_kotlin_import_nested_classes = true
-ij_kotlin_imports_layout = *, java.**, javax.**, kotlin.**, ^
+ij_kotlin_imports_layout=*,java.**,javax.**,kotlin.**,^
 ij_kotlin_insert_whitespaces_in_simple_one_line_method = true
 ij_kotlin_keep_blank_lines_before_right_brace = 0
 ij_kotlin_keep_blank_lines_in_code = 1
@@ -95,3 +95,4 @@ ktlint_standard_no-wildcard-imports = disabled
 ktlint_standard_filename = disabled
 ktlint_standard_package-name = disabled
 ktlint_standard_annotation = disabled
+ktlint_standard_kdoc-wrapping = disabled

ad-click/ad-click-impl/src/main/java/com/duckduckgo/adclick/impl/DataRemovalAdClickWorker.kt

@@ -30,11 +30,11 @@ import com.duckduckgo.app.lifecycle.MainProcessLifecycleObserver
 import com.duckduckgo.common.utils.DispatcherProvider
 import com.duckduckgo.di.scopes.AppScope
 import com.squareup.anvil.annotations.ContributesMultibinding
-import java.util.concurrent.TimeUnit
-import javax.inject.Inject
 import kotlinx.coroutines.withContext
 import logcat.LogPriority.VERBOSE
 import logcat.logcat
+import java.util.concurrent.TimeUnit
+import javax.inject.Inject
 
 @ContributesWorker(AppScope::class)
 class DataRemovalAdClickWorker(context: Context, workerParameters: WorkerParameters) :

ad-click/ad-click-impl/src/main/java/com/duckduckgo/adclick/impl/DuckDuckGoAdClickData.kt

@@ -21,10 +21,10 @@ import com.duckduckgo.adclick.impl.store.exemptions.AdClickExemptionsDao
 import com.duckduckgo.adclick.impl.store.exemptions.AdClickExemptionsDatabase
 import com.duckduckgo.adclick.impl.store.exemptions.AdClickTabExemptionEntity
 import com.duckduckgo.common.utils.DispatcherProvider
-import java.util.concurrent.ConcurrentHashMap
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.launch
 import logcat.logcat
+import java.util.concurrent.ConcurrentHashMap
 
 interface AdClickData {
 

ad-click/ad-click-impl/src/main/java/com/duckduckgo/adclick/impl/DuckDuckGoAdClickManager.kt

@@ -23,9 +23,9 @@ import com.duckduckgo.app.browser.UriString
 import com.duckduckgo.di.scopes.AppScope
 import com.squareup.anvil.annotations.ContributesBinding
 import dagger.SingleInstanceIn
-import javax.inject.Inject
 import logcat.logcat
 import okhttp3.internal.publicsuffix.PublicSuffixDatabase
+import javax.inject.Inject
 
 @SingleInstanceIn(AppScope::class)
 @ContributesBinding(AppScope::class)

ad-click/ad-click-impl/src/main/java/com/duckduckgo/adclick/impl/pixels/AdClickDailyReportingWorker.kt

@@ -29,11 +29,11 @@ import com.duckduckgo.app.lifecycle.MainProcessLifecycleObserver
 import com.duckduckgo.common.utils.DispatcherProvider
 import com.duckduckgo.di.scopes.AppScope
 import com.squareup.anvil.annotations.ContributesMultibinding
-import java.util.concurrent.TimeUnit
-import javax.inject.Inject
 import kotlinx.coroutines.withContext
 import logcat.LogPriority.VERBOSE
 import logcat.logcat
+import java.util.concurrent.TimeUnit
+import javax.inject.Inject
 
 @ContributesWorker(AppScope::class)
 class AdClickDailyReportingWorker(context: Context, workerParameters: WorkerParameters) :

ad-click/ad-click-impl/src/main/java/com/duckduckgo/adclick/impl/remoteconfig/AdClickAttributionFeature.kt

@@ -21,14 +21,14 @@ import com.duckduckgo.di.scopes.AppScope
 import com.duckduckgo.feature.toggles.api.Toggle
 import com.duckduckgo.feature.toggles.api.Toggle.DefaultFeatureValue
 
+/**
+ * This is the class that represents the adClickAttribution feature flags.
+ */
 @ContributesRemoteFeature(
     scope = AppScope::class,
     featureName = "adClickAttribution",
     settingsStore = AdClickAttributionFeatureSettingsStore::class,
 )
-/**
- * This is the class that represents the adClickAttribution feature flags.
- */
 interface AdClickAttributionFeature {
     /**
      * @return `true` when the remote config has the global "adClickAttribution" feature flag enabled

ad-click/ad-click-impl/src/main/java/com/duckduckgo/adclick/impl/remoteconfig/AdClickAttributionRepository.kt

@@ -23,9 +23,9 @@ import com.duckduckgo.adclick.impl.store.AdClickAttributionLinkFormatEntity
 import com.duckduckgo.adclick.impl.store.AdClickDao
 import com.duckduckgo.adclick.impl.store.AdClickDatabase
 import com.duckduckgo.common.utils.DispatcherProvider
-import java.util.concurrent.CopyOnWriteArrayList
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.launch
+import java.util.concurrent.CopyOnWriteArrayList
 
 interface AdClickAttributionRepository {
     fun updateAll(

ad-click/ad-click-impl/src/test/java/com/duckduckgo/adclick/impl/DuckDuckGoAdClickDataTest.kt

@@ -21,7 +21,6 @@ import com.duckduckgo.adclick.impl.store.exemptions.AdClickExemptionsDao
 import com.duckduckgo.adclick.impl.store.exemptions.AdClickExemptionsDatabase
 import com.duckduckgo.common.test.CoroutineTestRule
 import com.duckduckgo.feature.toggles.api.Toggle
-import java.util.concurrent.TimeUnit
 import kotlinx.coroutines.test.TestScope
 import org.junit.Assert.assertEquals
 import org.junit.Assert.assertFalse
@@ -35,6 +34,7 @@ import org.mockito.kotlin.never
 import org.mockito.kotlin.reset
 import org.mockito.kotlin.verify
 import org.mockito.kotlin.whenever
+import java.util.concurrent.TimeUnit
 
 class DuckDuckGoAdClickDataTest {
 

ad-click/ad-click-impl/src/test/java/com/duckduckgo/adclick/impl/DuckDuckGoAdClickManagerTest.kt

@@ -20,7 +20,6 @@ import androidx.test.ext.junit.runners.AndroidJUnit4
 import com.duckduckgo.adclick.api.AdClickManager
 import com.duckduckgo.adclick.impl.pixels.AdClickPixelName
 import com.duckduckgo.adclick.impl.pixels.AdClickPixels
-import java.util.concurrent.TimeUnit
 import org.junit.Assert.assertFalse
 import org.junit.Assert.assertTrue
 import org.junit.Before
@@ -33,6 +32,7 @@ import org.mockito.kotlin.times
 import org.mockito.kotlin.verify
 import org.mockito.kotlin.verifyNoInteractions
 import org.mockito.kotlin.whenever
+import java.util.concurrent.TimeUnit
 
 @RunWith(AndroidJUnit4::class)
 class DuckDuckGoAdClickManagerTest {

ad-click/ad-click-impl/src/test/java/com/duckduckgo/adclick/impl/pixels/RealAdClickPixelsTest.kt

@@ -24,8 +24,6 @@ import com.duckduckgo.adclick.impl.Exemption
 import com.duckduckgo.app.statistics.pixels.Pixel
 import com.duckduckgo.app.statistics.pixels.Pixel.PixelType.Count
 import com.duckduckgo.common.test.api.InMemorySharedPreferences
-import java.time.Instant
-import java.util.concurrent.TimeUnit
 import org.junit.Assert.assertEquals
 import org.junit.Assert.assertFalse
 import org.junit.Assert.assertTrue
@@ -38,6 +36,8 @@ import org.mockito.kotlin.mock
 import org.mockito.kotlin.never
 import org.mockito.kotlin.verify
 import org.mockito.kotlin.whenever
+import java.time.Instant
+import java.util.concurrent.TimeUnit
 
 @RunWith(AndroidJUnit4::class)
 class RealAdClickPixelsTest {