Skip to content

Commit 20ae836

Browse files
david-allisonlukstbit
authored andcommitted
refactor(destinations): replace 'bundleOf' with Bundle()
Deprecated in androidx:core 1.18.0 https://developer.android.com/jetpack/androidx/releases/core#core_and_core-ktx_version_118_2 Assisted-by: Claude Opus 4.8 - all
1 parent a65805f commit 20ae836

8 files changed

Lines changed: 20 additions & 19 deletions

File tree

AnkiDroid/src/main/java/com/ichi2/anki/pages/CardInfoDestination.kt

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ package com.ichi2.anki.pages
1717

1818
import android.content.Context
1919
import android.content.Intent
20-
import androidx.core.os.bundleOf
20+
import android.os.Bundle
2121
import com.ichi2.anki.R
2222
import com.ichi2.anki.SingleFragmentActivity
2323
import com.ichi2.anki.libanki.CardId
@@ -39,7 +39,11 @@ data class CardInfoDestination(
3939
simplifiedTitle.toSentenceCase(context, R.string.sentence_card_stats_previous_card_study),
4040
)
4141
val cardInfoTitle = sentenceStrings.firstOrNull { it != simplifiedTitle } ?: title
42-
val arguments = bundleOf(CardInfoFragment.KEY_TITLE to cardInfoTitle, CardInfoFragment.KEY_CARD_ID to cardId)
42+
val arguments =
43+
Bundle().apply {
44+
putString(CardInfoFragment.KEY_TITLE, cardInfoTitle)
45+
putLong(CardInfoFragment.KEY_CARD_ID, cardId)
46+
}
4347
return SingleFragmentActivity.getIntent(
4448
context,
4549
fragmentClass = CardInfoFragment::class,

AnkiDroid/src/main/java/com/ichi2/anki/pages/CsvImporter.kt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ import android.os.Bundle
2121
import android.view.View
2222
import android.webkit.WebView
2323
import androidx.activity.OnBackPressedCallback
24-
import androidx.core.os.bundleOf
2524
import com.google.android.material.appbar.MaterialToolbar
2625
import com.ichi2.anki.CollectionManager
2726
import com.ichi2.anki.R
@@ -112,7 +111,7 @@ class CsvImporter : PageFragment() {
112111
context: Context,
113112
filePath: String,
114113
): Intent {
115-
val arguments = bundleOf(KEY_FILE_PATH to filePath)
114+
val arguments = Bundle().apply { putString(KEY_FILE_PATH, filePath) }
116115
return SingleFragmentActivity.getIntent(context, fragmentClass = CsvImporter::class, arguments)
117116
}
118117
}

AnkiDroid/src/main/java/com/ichi2/anki/pages/DeckOptions.kt

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ import android.webkit.JavascriptInterface
2323
import android.webkit.WebResourceRequest
2424
import android.webkit.WebView
2525
import androidx.activity.OnBackPressedCallback
26-
import androidx.core.os.bundleOf
2726
import androidx.core.view.isVisible
2827
import androidx.fragment.app.FragmentActivity
2928
import anki.collection.OpChanges
@@ -257,7 +256,11 @@ class DeckOptions : PageFragment() {
257256
context: Context,
258257
deckId: DeckId,
259258
): Intent =
260-
SingleFragmentActivity.getIntent(context, fragmentClass = DeckOptions::class, arguments = bundleOf(KEY_DECK_ID to deckId))
259+
SingleFragmentActivity.getIntent(
260+
context,
261+
fragmentClass = DeckOptions::class,
262+
arguments = Bundle().apply { putLong(KEY_DECK_ID, deckId) },
263+
)
261264
}
262265
}
263266

AnkiDroid/src/main/java/com/ichi2/anki/pages/ImageOcclusion.kt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ import android.webkit.WebView
2424
import android.widget.TextView
2525
import androidx.activity.addCallback
2626
import androidx.core.os.BundleCompat
27-
import androidx.core.os.bundleOf
2827
import androidx.core.view.isVisible
2928
import androidx.fragment.app.viewModels
3029
import com.google.android.material.appbar.MaterialToolbar
@@ -152,7 +151,7 @@ class ImageOcclusion : PageFragment(R.layout.page_image_occlusion) {
152151
context: Context,
153152
args: ImageOcclusionArgs,
154153
): Intent {
155-
val arguments = bundleOf(IO_ARGS_KEY to args)
154+
val arguments = Bundle().apply { putParcelable(IO_ARGS_KEY, args) }
156155
return SingleFragmentActivity.getIntent(context, ImageOcclusion::class, arguments)
157156
}
158157
}

AnkiDroid/src/main/java/com/ichi2/anki/preferences/Preferences.kt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ import android.os.Bundle
2525
import android.view.View
2626
import androidx.activity.OnBackPressedCallback
2727
import androidx.annotation.XmlRes
28-
import androidx.core.os.bundleOf
2928
import androidx.core.view.updateLayoutParams
3029
import androidx.fragment.app.Fragment
3130
import androidx.fragment.app.FragmentFactory
@@ -254,7 +253,7 @@ class PreferencesActivity :
254253
context: Context,
255254
initialFragment: KClass<out Fragment>? = null,
256255
): Intent {
257-
val arguments = bundleOf(INITIAL_FRAGMENT_EXTRA to initialFragment?.jvmName)
256+
val arguments = Bundle().apply { putString(INITIAL_FRAGMENT_EXTRA, initialFragment?.jvmName) }
258257
return Intent(context, PreferencesActivity::class.java).apply {
259258
putExtra(FRAGMENT_NAME_EXTRA, PreferencesFragment::class.jvmName)
260259
putExtra(FRAGMENT_ARGS_EXTRA, arguments)

AnkiDroid/src/main/java/com/ichi2/anki/previewer/PreviewerFragment.kt

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ import android.view.Menu
2424
import android.view.MenuItem
2525
import android.view.View
2626
import androidx.appcompat.widget.Toolbar
27-
import androidx.core.os.bundleOf
2827
import androidx.core.view.ViewCompat
2928
import androidx.core.view.doOnLayout
3029
import androidx.core.view.isVisible
@@ -295,10 +294,10 @@ class PreviewerFragment :
295294
currentIndex: Int,
296295
): Intent {
297296
val arguments =
298-
bundleOf(
299-
CURRENT_INDEX_ARG to currentIndex,
300-
CARD_IDS_FILE_ARG to idsFile,
301-
)
297+
Bundle().apply {
298+
putInt(CURRENT_INDEX_ARG, currentIndex)
299+
putParcelable(CARD_IDS_FILE_ARG, idsFile)
300+
}
302301
return CardViewerActivity.getIntent(context, PreviewerFragment::class, arguments)
303302
}
304303
}

AnkiDroid/src/main/java/com/ichi2/anki/previewer/TemplatePreviewerFragment.kt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ package com.ichi2.anki.previewer
1717

1818
import android.os.Bundle
1919
import android.view.View
20-
import androidx.core.os.bundleOf
2120
import androidx.fragment.app.viewModels
2221
import androidx.lifecycle.lifecycleScope
2322
import com.ichi2.anki.R
@@ -102,7 +101,7 @@ class TemplatePreviewerFragment :
102101

103102
fun newInstance(arguments: TemplatePreviewerArguments): TemplatePreviewerFragment =
104103
TemplatePreviewerFragment().apply {
105-
val args = bundleOf(ARGS_KEY to arguments)
104+
val args = Bundle().apply { putParcelable(ARGS_KEY, arguments) }
106105
this.arguments = args
107106
}
108107
}

AnkiDroid/src/main/java/com/ichi2/anki/previewer/TemplatePreviewerPage.kt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ import android.content.Intent
2020
import android.os.Bundle
2121
import android.view.View
2222
import androidx.core.os.BundleCompat
23-
import androidx.core.os.bundleOf
2423
import androidx.fragment.app.Fragment
2524
import androidx.fragment.app.commitNow
2625
import androidx.lifecycle.lifecycleScope
@@ -88,7 +87,7 @@ class TemplatePreviewerPage : Fragment(R.layout.fragment_template_previewer_cont
8887
CardViewerActivity.getIntent(
8988
context,
9089
TemplatePreviewerPage::class,
91-
bundleOf(ARGS_KEY to arguments),
90+
Bundle().apply { putParcelable(ARGS_KEY, arguments) },
9291
)
9392
}
9493
}

0 commit comments

Comments
 (0)