2.6.0-alpha05 NavController visibleEntries flow contains many wrong entries [288520638]

Fixed

Bug

Status Update

No update yet.

Description

th...@outlook.com

created issue #1

Jun 23, 2023 12:30PM

Component used: Navigation
Version used: 2.6.0-alpha05 and up (including 2.6.0 stable)
Devices/Android versions reproduced on: Any

In the sample project I try to determine if the home fragment is visible using NavController.visibleEntries. In library version alpha04 it always contains the correct entries, but in alpha05 the list is a hot mess because it keeps growing when navigating and even the host fragment and fragments that are not on the backstack are added to the list. This is not possible in the sample project.

The sample project logs the information for demonstration.

VisibleEntriesBugReport.zip

148 KB

Download

Comments

il...@google.com <il...@google.com> Jun 23, 2023 10:07PM

Assigned to cl...@google.com.

th...@outlook.com <th...@outlook.com> #2Aug 30, 2023 09:57AM

After creating the issue I've noticed that I mixed up a few terms, sorry about that. JobService should be JobScheduler. When referring to jobs in the description, those are classes extending Worker.

il...@google.com <il...@google.com> #3Aug 30, 2023 04:52PM

We have a large set of users seeing this same issue... I can confirm that it seems to only effect users using Android OS 4.4 - 5.1. We are also using WorkManager 2.0.0

ap...@google.com <ap...@google.com> #4Sep 7, 2023 05:19PM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 098c163581a360c32baa6e4c215227f0efd10889
Author: Rahul Ravikumar <rahulrav@google.com>
Date: Mon Mar 25 14:05:34 2019

Reset WorkConstraintsTracker when cleaning up in DelayMetCommandHandler.

* DelayMetCommandHandler would previously not cleanup correctly and WorkConstraintsTracker
would trigger callbacks without going through the SystemAlarmDispatcher lifecycle
of processing a command.

Fixes:

b/129226383

Test: Added a unit test.
Change-Id: Id2aaf0cce7c4a1a4526a25a211f3fe0551dbe557

M work/workmanager/src/androidTest/java/androidx/work/impl/background/systemalarm/SystemAlarmDispatcherTest.java
M work/workmanager/src/main/java/androidx/work/impl/background/systemalarm/DelayMetCommandHandler.java

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

https://goto.google.com/android-sha1/098c163581a360c32baa6e4c215227f0efd10889

cl...@google.com <cl...@google.com> #5Sep 7, 2023 05:25PM

Marked as fixed.

Great to see such a quick fix. Do you have any estimate when a new version that includes this change will be released?

na...@google.com <na...@google.com> #6Sep 20, 2023 07:16PM

It should be available in the next 2.x.x version.

th...@outlook.com <th...@outlook.com> #7Sep 21, 2023 08:48AM

ETA?

na...@google.com <na...@google.com> #8Jan 24, 2024 07:53PM

Hi, we have implemented a new big feature using WorkManager and got the same issue. We have been considering third party alternatives because we need to deliver this ASAP. But if you confirm that we will have that version within next few days we can wait for it.