Blender Git Loki

Tests: Updated test message for Directory Not Found

Splines: Add a method for reversing a Spline

This moved the spline reversing logic out of the Curve Reverse geometry
node and into the spline class. This allows a spline to reverse itself
with a call to `my_spline.reverse()`

The base class will reverse position, radii & tilt, while specialized
versions are created for Bezier and Nurbs splines to reverse the
additional data that these classes encapsulate.

Differential Revision: https://developer.blender.org/D12501

Fix: Division by zero in curve spiral primitive node

Geometry Nodes: Hide values for selection inputs

Toggling the selection off in the node is the same as muting it,
so exposing it there doesn't help, and makes it less clear that it's
meant to be used as a field.

Cleanup: Remove duplicate warning from subdivision surface node

Cleanup: Use function to mark mesh normals dirty

GPencil: Fix dash modifier reading error.

The reference for parent modifier in segment data is not assigned. Now fixed.

Fix bisect gizmo offset while dragging

Fix memory leak if an error occurred assigning id-property sequence

Fix security popup re-displaying after undo

Don't reset these flags when loading a file (or undoing)
rely on BPY_python_reset to reset the flags.

Fix ID-property UI versioning skipping nested meta-strips

IDType: Cleanup: Remove useless `IDTYPE_FLAGS_NO_MAKELOCAL`.

This flag became a full duplicate of `IDTYPE_FLAGS_NO_LIBLINKING`, which
is a good thing (don't think we ever want to be able to link some data,
without being able to make it local...).

So we can now remove it and use `IDTYPE_FLAGS_NO_LIBLINKING` instead.

IDType: Add `BKE_idtype_idcode_is_localizable`.

This is the same as `BKE_idtype_idcode_is_linkable` currently, used only
in one place in UI code of IDtemplate.

IDManagement: refactor: Remove 'test' part from `BKE_lib_id_make_local`.

Mixing testing and actual action in a single function is just not a good
way to do things, and the 'testing' feature is not used anywhere
anymore, time to get rid of it.

Fix/refactor `BKE_id_clear_newpoin` and `BKE_main_id_newptr_and_tag_clear`.

Those were not clearing embdeed IDs flags and `newid` pointers at all...

Cleanup: Rename `BKE_id_clear_newpoin` to `BKE_id_newptr_and_tag_clear`.

Cleanup: Add comment about 'make_local' callback of Brush doing bad thing.

Callbacks in IDTypeInfo should never affect other IDs if they are not
embedded.

We break this rule in some cases, at least each of those should be
clearly commented about and get some security checks to try to avoid
unexpected issues as much as possible.

Cleanup: Comment about shpaekey being treated as embedded IDs in `BKE_id_newptr_and_tag_clear`.

Cleanup: no need to clear new flags and pointers from whole Main when making a single ID local.

Cleanup/Fix outliner 'make local' code.

While likely harmless, this code was doing extremely bad thing,
by-passing the whole lower-level `BKE_lib_id_make_local` call in case it
would fail and deciding by itself to forcefully make the given ID local.
Bad. Very, very, very bad.