Skip batching for phase items from other pipelines

[james-j-obrien]

Objective

• Fix Improper batching of non-mesh phase items. #14295

Solution

• Early out when GFBD::get_index_and_compare_data returns None.

Testing

• Tested on a selection of examples including many_foxes and 3d_shapes.
• Resolved the original issue in bevy_vector_shapes.

[Hexorg]

Can you turn the original bevy vector shapes issue into a unit-test to use as regression testing please?

[james-j-obrien]

Can you turn the original bevy vector shapes issue into a unit-test to use as regression testing please?

I considered this but it's a pretty non-trivial amount of setup, definitely not much of a unit test, would require a example setting up pipelines with custom phase items and batching.

It can still be done but wanted to put up this fix so we can try and get it in for 0.14.1

There's also equivalent if statements in the other batching methods, so there's a question of to what fidelity are we going to test every piece of the rendering logic.

[alice-i-cecile]

Yeah, I wouldn't worry about automated testing here for now. We need a more cohesive design to this across rendering.

[TrialDragon]

LGTM when testing it.