delight-im / delight-im/Android-AdvancedWebView

[Improve] Use ActivityResultLauncher instead of onActivityResult, because old way has been deprecated.

Offen
#283 4 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

enhancement
Vorherrschende Sprache
Java
Sterne
2.4k
Forks
578
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

I know that onActivityResult deprecated but still used in next API versions. But I suggest to upgrade to consistent with newer API.

Old way (Kotlin)

override fun onActivityResult(requestCode: Int, resultCode: Int, data: Intent?) {
   @Suppress("DEPRECATION")
   super.onActivityResult(requestCode, resultCode, data)
   getViewBinding().advancedWebView.onActivityResult(requestCode, resultCode, data)
}

New way (Kotlin)

private val mOnWebViewActivityResult = registerForActivityResult(
   ActivityResultContracts.StartActivityForResult()
) { result ->
   getViewBinding().advancedWebView.handleActivityResultCallback(result)
}

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, das vorhandene onActivityResult-Override und den im Issue gezeigten Aufruf von advancedWebView.onActivityResult zu finden, und untersuche dann, wie Aktivitätsergebnisse in den umgebenden Android-Komponenten verarbeitet werden. Die Arbeit ist abgeschlossen, wenn der veraltete Callback-Pfad durch die Verarbeitung mit ActivityResultLauncher ersetzt ist und die WebView das Ergebnis über handleActivityResultCallback erhält.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
android, java, kotlin
Bereich
mobile-dev
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.