IssueTracker

Artifact used: androidx.navigation:navigation-fragment-ktx:2.3.4
Version used: 2.3.4
Theme used: N/A
Devices/Android versions reproduced on: Any

Problem

We have the following navigation graph:

<?xml version="1.0" encoding="utf-8"?>
<navigation xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:id="@+id/profile_graph"
    app:startDestination="@id/profileFragment">

    <fragment
        android:id="@+id/profileFragment"
        android:name="com.myapp.profile.ProfileFragment"
        tools:layout="@layout/fragment_profile">

        <argument
            android:name="userId"
            app:argType="string" />

        <action
            android:id="@+id/toProfileDetails"
            app:destination="@id/profileDetailsFragment" />

        <deepLink
            android:id="@+id/deepLinkProfile"
            app:uri="myapp://profiles/{userId}/" />

    </fragment>

    <fragment
        android:id="@+id/profileDetailsFragment"
        android:name="com.myapp.profile.ProfileDetailsFragment"
        tools:layout="@layout/fragment_profile_details">

        <argument
            android:name="userId"
            app:argType="string" />

        <deepLink
            android:id="@+id/deepLinkProfileDetails"
            app:uri="myapp://profiles/{userId}/details/" />

    </fragment>
</navigation>

Here are declared two deep links patterns:

• myapp://profiles/{userId}/
• myapp://profiles/{userId}/details/

But the second deep link will never be handled because these deep links are "equal" for navigation when it tries to find the best matching deep link. In this case, NavGraph will always use first matching NavDestination (profileFragment).

For example link myapp://profiles/u2/details/ will be handled as myapp://profils/{userId}/ where userId = u2/details.

There is a workaround - remove ending slash in second pattern to make patterns tails not matching:

• myapp://profiles/{userId}/
• myapp://profiles/{userId}/details

It works but it is not obvious.

Purposed solution

Make arguments to not include / character.

In resulting regexp: ^\Qmyapp://profiles/\E(.+?)\Q/\E($|(\?(.)*))
Replace: (.+?)
With: ([^/]+?)

If this solution is right, I can create pull request on GitHub.