vs as a view for a NavHost

Question

When using androidx.fragment.app.FragmentContainerView as a navHost instead of a regular fragment app is not able to navigate to a destination after orientation change.

I get a following error: java.lang.IllegalStateException: no current navigation node

Is there a gotcha that I should know about to use it properly or is my way of using nav components is incorrect?

Simple activity xml with a view:


...
    <androidx.fragment.app.FragmentContainerView
        android:id="@+id/nav_host_fragment"
        android:name="androidx.navigation.fragment.NavHostFragment"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        app:defaultNavHost="true"
        app:layout_behavior="@string/appbar_scrolling_view_behavior"
        app:navGraph="@navigation/nav_simple" />
...

Navigation code:

<?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/nav_legislator.xml"
    app:startDestination="@id/initialFragment">

    <fragment
        android:id="@+id/initialFragment"
        android:name="com.example.fragmenttag.InitialFragment"
        android:label="Initial Fragment"
        tools:layout="@layout/initial_fragment">
        <action
            android:id="@+id/action_initialFragment_to_destinationFragment"
            app:destination="@id/destinationFragment" />
    </fragment>
    <fragment
        android:id="@+id/destinationFragment"
        android:name="com.example.fragmenttag.DestinationFragment"
        android:label="Destination Fragment"
        tools:layout="@layout/destination_fragment" />

</navigation>

Here is a github repo where you can easily reproduce a bug: https://github.com/dmytroKarataiev/navHostBug

You need to put the ` – EpicPandaForce Nov 07 '19 at 14:40 — EpicPandaForce, Nov 07 '19 at 14:40

ianhanniballake · Accepted Answer · 2019-11-06T22:42:14.393

30

The no current navigation node error occurs when there's no graph set and you attempt to call navigate(). If it only occurs when you're using FragmentContainerView and after a configuration change, then this would be related to this bug, which is fixed and scheduled for release with Navigation 2.2.0-rc03.

To work around this issue, you can either switch back to <fragment> or remove app:navGraph="@navigation/nav_simple" and instead call navController.setGraph(R.navigation.nav_simple).

edited Nov 06 '19 at 22:42

answered Nov 06 '19 at 21:08

ianhanniballake

155,909
19
351
362

1

I actually tried this approach and I got exactly the same error. Would you be able to check out the repo and try to make it work with this approach? – Dmytro Karataiev Nov 06 '19 at 22:07
Also, this crash happens only after orientation change. It works all the other times. – Dmytro Karataiev Nov 06 '19 at 22:11
I actually think this is not the same error, nav controller is not null in my question, but its back stack is empty and the graph is null. – Dmytro Karataiev Nov 06 '19 at 22:15
Ah, yeah, that's pretty important information, Sounds like what you're experiencing is this [other issue](https://issuetracker.google.com/issues/143752103). I've updated my answer to discuss what is going on in that issue. – ianhanniballake Nov 06 '19 at 22:47
Thank you, perfect. – Dmytro Karataiev Nov 07 '19 at 16:39
Just stumbled upon this as well. Thank you. – ror Nov 25 '19 at 08:59
I was getting an IllegalStateException You must call setGraph() before calling getGraph() and switching to solved my problem. Thank you! – G_comp Dec 04 '19 at 16:59
@G_comp - Navigation 2.2.0-rc03 is out now, so I'd certainly recommend trying it again and ensuring that it is fixed before the stable release – ianhanniballake Dec 04 '19 at 18:27
1

This should be explained somewhere in official codelab! Or am I miss it? – Robert Mar 13 '20 at 02:58

score 21 · Answer 2 · answered Jan 28 '20 at 12:16

I do not see any explanation why replace Fragment view with FragmentContainerView. This is why add this answer:

One of the common patterns to host fragments in an activity is to use a FrameLayout. The Android community have been doing this for years but this is going to change now. The androidx team introduced this new view called FragmentContainerView to do this for you.

All you have to do is replace your FrameLayout with FragmentContainerView and everything should work out of the box, you won’t need to change anything on the way you handle fragment transactions.

The benefits gained here is the improved handling of what’s called the z-ordering for fragments. Here’s the example they used, but basically this means , for example, that the exit and enter transitions between two fragments will not overlap each other. Instead this FragmentContainerView is going to start the exit animation first then the enter animation.

If you are asking can this replace tag? then the answer is yes. All you have to do is to add android:name="your_class_name" when defining FragmentContainerView and it will use a FragmentTransaction under the hood to build and show your fragment. This mean you can use fragment transactions later on to replace it.

Source

Android Docs

I don't believe this answers the question the author was asking. — AlgoRyan, Feb 20 '20 at 23:55
Thanks! See also https://stackoverflow.com/questions/60009473/classnotfoundexception-when-using-androidx-fragment-app-fragmentcontainerview/64225308 for adding rules in `proguard-rules.pro`. — CoolMind, Oct 06 '20 at 12:03
Thanks a lot for this answer. This helps with the issue I face now. It doesn't solve it. But I know more about the `FrameLayout` vs `FragmentContainerView` hell. — iam thadiyan, Jan 16 '21 at 15:15
Regarding android:name="your_class_name" what class should "your_class_name" be? the hosting Activity or a Fragment? — ProjectDelta, Jan 17 '21 at 02:31

score 0 · Answer 3 · answered Jun 22 '20 at 08:14

override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?, savedInstanceState: Bundle?): View? {
    return inflater.inflate(R.layout.frag_splash, container, false)
}

Maybe you forgot the 3rd "false" parameter above. Only this three parameters onCreateView fun can be accept.

android developer website

pic from : https://developer.android.com/reference/androidx/fragment/app/FragmentContainerView

score 0 · Answer 4 · edited Mar 11 '21 at 09:12

0

// drawer layout that I pass is the id of the layout in the activity from which I move to fragment//

SacnFragment sacnFragment = new SacnFragment();
    FragmentTransaction transaction = getSupportFragmentManager().beginTransaction();
    transaction.replace(R.id.drawer_layout, sacnFragment);
    transaction.addToBackStack(null);
    transaction.commit();

edited Mar 11 '21 at 09:12

Suraj Rao

28,186
10
88
94

answered Mar 11 '21 at 09:03

mazhar iqbal

71
3

vs as a view for a NavHost

4 Answers4

Linked