onActivityResult() deprecated for AppCompatActivity

Question

I am using in-app updates for android and as per the documentation, they are using onActivityResult to handle app behaviour incase the update is interrupted.

This is my function that is called from my fragment:

private fun startImmediateUpdate(appUpdateInfo: AppUpdateInfo) {
    appUpdateManager.startUpdateFlowForResult(
        appUpdateInfo,
        AppUpdateType.IMMEDIATE,
        requireActivity(),
        Constants.CODES.APP_UPDATE_REQUEST_CODE
    )

}

This is how i am handling results in parent activity

override fun onActivityResult(requestCode: Int, resultCode: Int, data: Intent?) {
    when (requestCode) {
        Constants.CODES.APP_UPDATE_REQUEST_CODE -> {
            if (resultCode != RESULT_OK || resultCode == RESULT_CANCELED || resultCode == ActivityResult.RESULT_IN_APP_UPDATE_FAILED) {
                //Do whatever i want to
            }
        }
    }
    super.onActivityResult(requestCode, resultCode, data)
}

Now super.onActivityResult(requestCode, resultCode, data) is deprecated. Things are working fine for now but i am worried the app will crash if its wiped out completely

What can i do to replace onActivityResult()? I have looked into registerForActivityResult() but could not find anything that suits my usecase.

[OnActivityResult method is deprecated, what is the alternative?](https://stackoverflow.com/q/62671106/3290339). You should go as per [Getting a result from an activity](https://developer.android.com/training/basics/intents/result) — Onik, Dec 22 '20 at 13:27
@Onik i am not starting any activity on my own. Its a built in function used by appUpdateManager, although works in the similar fashion as startActivityForResult() , but i just want to handle the results. Not launching anything on my own. — Hashir Ali, Dec 22 '20 at 13:31
Check this on google documentation: https://developer.android.com/training/basics/intents/result#separate. Maybe will help you — ziselos, Dec 22 '20 at 13:51

Anders Ullnæss · Answer 1 · 2021-03-04T12:56:55.607

4

I have the same issue and I see that one possibility is to rewrite to use the startUpdateFlow call instead.

Personally, I think this is not worth doing, so I will ignore this specific deprecation warning instead and wait/hope for Android/Google to make a method where we can pass an ActivityResultLauncher instead.

edited Mar 04 '21 at 12:56

answered Mar 04 '21 at 12:49

Anders Ullnæss

588
5
7

as of 22-DEC-21, no update from google yet they want us to target sdk 30 at min and use scopedstorage and what not – asim Dec 21 '21 at 20:05

score -1 · Answer 2 · answered May 07 '21 at 18:38

The New Way

    appUpdateInfoTask.addOnSuccessListener { appUpdateInfo ->
        if (appUpdateInfo.updateAvailability() == UpdateAvailability.UPDATE_AVAILABLE
            && (appUpdateInfo.clientVersionStalenessDays() ?: -1) >= 2
            && appUpdateInfo.updatePriority() >= updatePriority
            && appUpdateInfo.isUpdateTypeAllowed(AppUpdateType.IMMEDIATE)
        ) {
            appUpdateManager.startUpdateFlowForResult(
                appUpdateInfo,
                AppUpdateType.IMMEDIATE,
                this,
                updateCode
            )
            openActivityForResult()
        }
    }
}

private fun openActivityForResult(){
    resultUpdate.launch(Intent(this, MainActivity::class.java))
}

var resultUpdate = registerForActivityResult(ActivityResultContracts.StartActivityForResult()){ result->
    if (result.resultCode != Activity.RESULT_OK){
        Toast.makeText(
            this,
            "App Update failed, please try again on the next app launch.",
            Toast.LENGTH_SHORT
        ).show()
    }
}

Thanks for the suggestion. However I'm waiting for an official support of the new API with `ActivityResultLauncher` as Anders Ullnæss wrote above. — Sven Jacobs, May 10 '21 at 07:34

onActivityResult() deprecated for AppCompatActivity

2 Answers2