Support Activity Result API

[bubelov]

Support Activity Result API

[stefan-niedermann]

Are there any reasons for not supporting Activity Result API except for the lack of dev time?

In short: Nope. Maybe you want to start a Pull Request? I will happily review it 🙂

[bubelov]

@stefan-niedermann I don't have a NC instance at the moment, so it would be hard to test the code. Do you know any reliable hosted services? I tried a couple from the official website, one of them was freezing for hours and another one wasn't able to send me an email confirmation for some reason =)

[stefan-niedermann]

Do you know any reliable hosted services?

For testing around just use one of those free hosters.

So, I played a bit around with the idea of ActivityResultContracts. I think, we need three different ActivityResultContracts:

1. CHOOSE_ACCOUNT_SSO → ChooseAccountContract
2. REQUEST_AUTH_TOKEN_SSO → RequestAuthTokenContract
3. REQUEST_GET_ACCOUNTS_PERMISSION → RequestGetAccountsPermissionContract

Further thoughts:

1. Do 3rd party apps really need to control the import aspect this detailed? Wouldn't it be better to provide just one ImportSingleSignOnAccountContract which handles the rest of the stuff under the hood?
2. While implementing ActivityResultContract I stumbled upon the problem, that one needs to override createIntent and parseResult - both methods do not throw any exception according their signature. This leads to the problem, that we can't check and throw NextcloudFilesAppNotInstalledException for example without wrapping it into a RuntimeException or extending SSOException RuntimeException - I don't like any of those approaches. Another idea is to do the check within a constructor, but this again leads to an ugly static initialization from the 3rd party apps perspective:

private final ActivityResultLauncher<Void> launcher;

{
  try {
      launcher = registerForActivityResult(
              new ImportAccountContract(this),
              (ActivityResultCallback<SingleSignOnAccount>) this::onActivityResult
      );
  } catch (NextcloudFilesAppNotInstalledException e) {
      throw new RuntimeException(e);
  }
}