Remove the log, was causing a crash.

[bmarty]

kotlin.reflect.jvm.internal.KotlinReflectionInternalError: Function 'handleEvents' (JVM signature: present$handleEvents(Landroidx/compose/runtime/MutableState;Lkotlin/jvm/internal/Ref$ObjectRef;Lio/element/android/libraries/permissions/api/PermissionsEvents;)V) not resolved in class kotlin.jvm.internal.Intrinsics$Kotlin: no members found
at kotlin.reflect.jvm.internal.KDeclarationContainerImpl.findFunctionDescriptor(KDeclarationContainerImpl.kt:131)
at kotlin.reflect.jvm.internal.KFunctionImpl$descriptor$2.invoke(KFunctionImpl.kt:56)
at kotlin.reflect.jvm.internal.KFunctionImpl$descriptor$2.invoke(KFunctionImpl.kt:55)
at kotlin.reflect.jvm.internal.ReflectProperties$LazySoftVal.invoke(ReflectProperties.java:93)
at kotlin.reflect.jvm.internal.ReflectProperties$Val.getValue(ReflectProperties.java:32)
at kotlin.reflect.jvm.internal.KFunctionImpl.getDescriptor(KFunctionImpl.kt:55)
at kotlin.reflect.jvm.internal.KFunctionImpl.toString(KFunctionImpl.kt:185)
at kotlin.jvm.internal.FunctionReference.toString(FunctionReference.java:130)
at java.lang.String.valueOf(String.java:4092)
at java.lang.StringBuilder.append(StringBuilder.java:179)
at io.element.android.libraries.permissions.api.PermissionsState.toString
at java.lang.String.valueOf(String.java:4092)
at java.lang.StringBuilder.append(StringBuilder.java:179)
at io.element.android.libraries.permissions.impl.DefaultPermissionsPresenter.present(DefaultPermissionsPresenter.kt:128)

Type of change

• Feature
• Bugfix
• Technical
• Other :

Content

Fix crash https://sentry.tools.element.io/organizations/element/issues/85090

Not sure how the crash is occurring only now, but no need to investigate more I guess.

Motivation and context

Stable app.

Screenshots / GIFs

Tests

• Step 1
• Step 2
• Step ...

Tested devices

• Physical
• Emulator
• OS version(s):

Checklist

• Changes have been tested on an Android device or Android emulator with API 23
• UI change has been tested on both light and dark themes
• Accessibility has been taken into account. See https://github.com/vector-im/element-x-android/blob/develop/CONTRIBUTING.md#accessibility
• Pull request is based on the develop branch
• Pull request includes a new file under ./changelog.d. See https://github.com/vector-im/element-x-android/blob/develop/CONTRIBUTING.md#changelog
• Pull request includes screenshots or videos if containing UI changes
• Pull request includes a sign off
• You've made a self review of your PR

[github-actions]

📱 Scan the QR code below to install the build (arm64 only) for this PR.

If you can't scan the QR code you can install the build via this link: https://i.diawi.com/8aW7Zp

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[jmartinesp]

I can approve if it fixes the issue, but in my case I've seen this error plenty of times and most of them there wasn't a log involved. For some reason, the *State data classes sometimes crash when the eventSink is a function reference instead of a eventSink = { handleEvents(it) }.

[jmartinesp]

LGTM, but I think we should try to investigate why this is happening.

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: -0.01% ⚠️

Comparison is base (a96baf9) 57.44% compared to head (d776000) 57.44%.
Report is 2 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #1206      +/-   ##
===========================================
- Coverage    57.44%   57.44%   -0.01%     
===========================================
  Files         1045     1045              
  Lines        27295    27292       -3     
  Branches      5617     5617              
===========================================
- Hits         15680    15677       -3     
  Misses        9185     9185              
  Partials      2430     2430              

Files Changed	Coverage Δ
...es/permissions/impl/DefaultPermissionsPresenter.kt	90.69% <ø> (-0.61%)	⬇️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[bmarty]

Yes, noted. Let's merge this since this log was quite useless anyway.