physics_object_picking_first_only breaks mouse enter/exit/motion events

[mnemoli]

Tested versions

• Reproducible in 4.3-dev4
• Introduced by a407219

System information

Godot v4.3.dev4 - Windows 10.0.19045

Issue description

This PR introduced physics_object_picking_first_only: #75688

See test project attached.

If you wiggle the mouse around over two logos with sort+first_only off you get this:

With both settings on you get this, and clicking also fires spurious enter/exit events:

I think the problem is here:

godot/scene/main/viewport.cpp

Lines 844 to 846 in fe01776

	if (mm.is_valid() && mm->get_device() == InputEvent::DEVICE_ID_INTERNAL) {
	send_event = false;
	}

If send_event == false then the break at L860 is never called. Moving L859/860 out of the send_event check makes all the events work as expected.

Steps to reproduce

• Wiggle mouse over 2 logos with both physics settings off. 2 mouse_entered and 2 mouse_shape_entered events will be logged (one per logo).
• Click mouse over 2 logos with both settings off. 2 click events will be raised.
• Wiggle mouse (with brief pauses in motion) over 2 icons with both physics settings on. enter/exit events will be continuously raised.
• Click mouse over 2 logos with both settings on. 1 click event will be raised along with spurious enter/exit events.

Minimal reproduction project (MRP)

BugPickingZSort2.zip