Invalid [Register] for nested default interface method invokers #661

Activity

1. jpobst commented on Jun 11, 2020

   

   jpobst

   on Jun 11, 2020

   Contributor

   More actions

   Thanks for the report! We were just in the middle of trying to track this down. Turns out it's a generator bug, so moving this to Java.Interop repo.

   👍React with 👍1Reacted by Vladislav Antonyuk
2. jpobst

   transferred this issue fromdotnet/androidon Jun 11, 2020
3. jpobst

   self-assigned this

   on Jun 11, 2020
4. jpobst

   added

   bugComponent does not function as intendedComponent does not function as intended

   generatorIssues binding a Java library (generator, class-parse, etc.)Issues binding a Java library (generator, class-parse, etc.)

   on Jun 11, 2020
5. jpobst commented on Jun 11, 2020

   

   jpobst

   on Jun 11, 2020

   Contributor

   More actions

   The issue is that when we [Register] the invoker for a default method in a nested class, we are using . to denote a nested class, when it should be a /.

   For example, in:

   [Register ("onActivityPostCreated", "(Landroid/app/Activity;Landroid/os/Bundle;)V", "GetOnActivityPostCreated_Landroid_app_Activity_Landroid_os_Bundle_Handler:Android.App.Application.IActivityLifecycleCallbacks, Mono.Android, Version=0.0.0.0, Culture=neutral, PublicKeyToken=null", ApiSince = 29)]
   

   Android.App.Application.IActivityLifecycleCallbacks should be Android.App.Application/IActivityLifecycleCallbacks.

   This is causing the nested type to not be found.
6. jpobst

   changed the title [-]Android SDK 30 runtime issue[/-] [+]Invalid [Register] for nested default interface method invokers[/+] on Jun 11, 2020
7. jonpryor

   closed this as completedin #662on Jun 11, 2020
8. jonpryor

   added 2 commits that reference this issue on Jun 11, 2020

   [generator] Use proper syntax for nested classes for DIM invokers (#662)

   ...

   9a56465

   [generator] Use proper syntax for nested classes for DIM invokers (#662)

   ...

   1f3388a
9. jpobst commented on Jun 11, 2020

   

   jpobst

   on Jun 11, 2020

   Contributor

   More actions

   The fix for this should be in VS2019 16.7 Preview 4.

   👍React with 👍1Reacted by Vladislav Antonyuk
10. brendanzagaeski commented on Jul 14, 2020

    

    brendanzagaeski

    on Jul 14, 2020

    Contributor

    More actions

    Release status update

    A new Preview version of Xamarin.Android has now been published that includes the fix for this item. The fix is not yet included in a Release version. I will update this again when a Release version is available that includes the fix.

    Fix included in Xamarin.Android 11.0.0.3.

    Fix included on Windows in Visual Studio 2019 version 16.7 Preview 4. To try the Preview version that includes the fix, check for the latest updates in Visual Studio Preview.

    Fix included on macOS in Visual Studio 2019 for Mac version 8.7 Preview 4. To try the Preview version that includes the fix, check for the latest updates on the Preview updater channel.
11. brendanzagaeski commented on Aug 5, 2020

    

    brendanzagaeski

    on Aug 5, 2020

    Contributor

    More actions

    Release status update

    A new Release version of Xamarin.Android has now been published that includes the fix for this issue.

    Fix included in Xamarin.Android SDK version 11.0.0.3.

    Fix included on Windows in Visual Studio 2019 version 16.7. To get the new version that includes the fix, check for the latest updates or install the most recent release from https://visualstudio.microsoft.com/downloads/.

    Fix included on macOS in Visual Studio 2019 for Mac version 8.7. To get the new version that includes the fix, check for the latest updates on the Stable updater channel.
12. github-actions

    locked and limited conversation to collaborators on Apr 13, 2024
13. pull

    added a commit that references this issue on Jul 1, 2026

    [generator] Use proper syntax for nested classes for DIM invokers (#662)

    ...

    8f2a400