navigation-fragment-compose: Safe Args plugin cannot escape composable destination name [362791955]

Fixed

Bug

androidx-navigation-2.9.0-alpha06

androidx-navigation-2.8.7

Status Update

No update yet.

Description

je...@pinkfroot.com

created issue #1

Aug 29, 2024 07:39AM

Component used: Navigation (androidx.navigation:navigation-fragment-compose)

Version used: 2.8.0-rc01

Devices/Android versions reproduced on: N/A (build issue)

As soon as I add an action to a composable destination like so:

<composable
  android:id="@+id/home_screen"
  android:name="com.example.HomeScreenKt\$HomeScreen" >
  <action
    android:id="@+id/action_home_screen_to_other_fragment"
    app:destination="@id/other_fragment" />
</composable>

My project build fails on the :app:generateSafeArgs task with the following error:

Can't escape identifier `HomeScreenKt\$HomeScreenDirections` because it contains illegal characters: \

Comments

cl...@couchbits.com <cl...@couchbits.com> #2Sep 25, 2024 07:58AM

Hi. Thanks for reporting this. Fixed in this changelist. This should be available in alpha-04

362791955.log

19 KB

View

Download

we...@gmail.com <we...@gmail.com> #3Oct 15, 2024 10:14PM

Project: platform/frameworks/support
Branch: androidx-main

commit e782987543a9f8ccd485e970ddc74564b24378db
Author: Vighnesh Raut <vighnesh.raut13@gmail.com>
Date: Mon Jan 02 15:27:40 2023

fix: tab row crashes when only 1 tab is added

Bug:

b/264018028

Test: Added unit test

Change-Id: I6381dbac304fc1d69d3708c6655f8b595668e93f

M tv/tv-material/src/androidTest/java/androidx/tv/material/TabRowTest.kt
M tv/tv-material/src/main/java/androidx/tv/material/TabRow.kt

https://android-review.googlesource.com/2373449

se...@gmail.com <se...@gmail.com> #4Nov 21, 2024 09:55PM

deleted

jb...@google.com <jb...@google.com> Jan 14, 2025 08:02PM

Assigned to il...@google.com.

pr...@google.com <pr...@google.com> Jan 22, 2025 05:37PM

Reassigned to jb...@google.com.

aa...@gmail.com <aa...@gmail.com> #5Jan 27, 2025 06:35AM

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.tv:tv-material:1.0.0-alpha04

ap...@google.com <ap...@google.com> #6Feb 5, 2025 08:51PM

Project: platform/frameworks/support
Branch: androidx-main
Author: Jeremy Woods <jbwoods@google.com>
Link: https://android-review.googlesource.com/3478934

Provide safe args classes for composables in safe args

Expand for full commit details

Provide safe args classes for composables in safe args 
 
Adding support for Directions and Args classes in Navigation safe args. 
 
RelNote: "Navigation Safe Args now provides support for composable 
destinations." 
Test: added tests 
Bug: 362791955 
 
Change-Id: I353205e72fe2896cae7ca1f563bfe58798e59578

Files:

M navigation/navigation-safe-args-generator/src/main/kotlin/androidx/navigation/safe/args/generator/models/Destination.kt
M navigation/navigation-safe-args-generator/src/test/kotlin/androidx/navigation/safe/args/generator/NavGeneratorTest.kt
M navigation/navigation-safe-args-generator/src/test/kotlin/androidx/navigation/safe/args/generator/NavParserTest.kt
M navigation/navigation-safe-args-generator/src/test/test-data/naive_test.xml

Hash: 44dc2bd5961dfd6665e06246c9b38ec916ea9364
Date: Wed Feb 05 19:24:28 2025

jb...@google.com <jb...@google.com> #7Feb 5, 2025 09:01PM

Marked as fixed.

This has been fixed internally and will be available in the Navigation 2.9.0-alpha06 and 2.8.7 releases.

pr...@google.com <pr...@google.com> #8Feb 12, 2025 06:40PM