ReportExecutionOrderAmbiguities doesn't understand query filters

[SUPERCILEX]

Bevy version

e543941 and 0.7.0

What you did

Index: examples/ecs/ecs_guide.rs
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/examples/ecs/ecs_guide.rs b/examples/ecs/ecs_guide.rs
--- a/examples/ecs/ecs_guide.rs	(revision 314e1b7ed6ceff35dd26f90c66b59d5f908e121f)
+++ b/examples/ecs/ecs_guide.rs	(date 1653960365357)
@@ -336,5 +336,17 @@
         .add_plugin(LogPlugin::default())
         .init_resource::<ReportExecutionOrderAmbiguities>()
         // This call to run() starts the app we just built!
+        .add_system(a)
+        .add_system(b)
         .run();
 }
+
+#[derive(Component)]
+struct A;
+
+#[derive(Component)]
+struct B;
+
+fn a(_q: Query<&mut Transform, (With<A>, Without<B>)>) {}
+
+fn b(_q: Query<&mut Transform, (With<B>, Without<A>)>) {}

What went wrong

A conflict is incorrect reported.

[DJMcNab]

master is a significantly outdated branch, which might only be available on really old forks - main should be used instead. Could you mention the exact commit you're testing with?

[SUPERCILEX]

Sorry, I meant main.

[SUPERCILEX]

Specifically e543941 but this is also a bug on 0.7.0

[alice-i-cecile]

I discovered this in #4299 as well. My feeling is that we need to do a cleanup pass on our Access code to unify this.

[rparrett]

Is this a duplicate of #4381?

[alice-i-cecile]

Yep; closing as duplicate.