6bf8685628
Drop all support for animation data from Blender versions 2.49 and older. - The `IPO` DNA struct is deleted, as is the `IDType_ID_IP` type definition. - Versioning code has been removed in its entirety. - Loading a pre-2.50 blend file will issue a warning, stating that any animation data that was there is now lost, and that the file should be loaded & re-saved with Blender 4.5 to properly port the data. Note that versioning of Action assignments (as in, picking a slot) still uses the tagging + updating all tagged assignments as a post-processing step. This is necessary because picking the right slot is only possible after all Actions (also those from libraries) have been versioned. We might be able to address this as well, by upgrading legacy → slotted Actions "on the fly" versioning these Action assignments. If we do that, I think that would be better in a separate PR though, as it could introduce issues by itself. Ref: #134219 Pull Request: https://projects.blender.org/blender/blender/pulls/145188
83 lines
2.6 KiB
C++
83 lines
2.6 KiB
C++
/* SPDX-FileCopyrightText: 2024 Blender Authors
|
|
*
|
|
* SPDX-License-Identifier: GPL-2.0-or-later */
|
|
|
|
/* The tests in this file need to be able to test deprecated data as well. */
|
|
#define DNA_DEPRECATED_ALLOW
|
|
|
|
#include "ANIM_versioning.hh"
|
|
|
|
#include "DNA_action_types.h"
|
|
|
|
#include "BLI_listbase.h"
|
|
|
|
#include "testing/testing.h"
|
|
|
|
namespace blender::animrig::versioning::tests {
|
|
|
|
TEST(animrig_versioning, action_is_layered)
|
|
{
|
|
/* This unit test doesn't put valid data in the action under test. Since action_is_layered()
|
|
* only looks at the length of lists, and not their contents, that should be fine. */
|
|
|
|
{ /* Animato Action only fcurves / Blender version [2.5, 4.4) */
|
|
bAction action = {};
|
|
Link /* FCurve */ fake_fcurve = {};
|
|
|
|
BLI_addtail(&action.curves, &fake_fcurve);
|
|
EXPECT_FALSE(action_is_layered(action))
|
|
<< "Animato Actions should NOT be considered 'layered'";
|
|
}
|
|
|
|
{ /* Animato Action with fcurves + groups / Blender version [2.5, 4.4) */
|
|
bAction action = {};
|
|
Link /* FCurve */ fake_fcurve = {};
|
|
Link /* bActionGroup */ fake_group = {};
|
|
|
|
BLI_addtail(&action.curves, &fake_fcurve);
|
|
BLI_addtail(&action.groups, &fake_group);
|
|
EXPECT_FALSE(action_is_layered(action))
|
|
<< "Animato Actions should NOT be considered 'layered'";
|
|
}
|
|
|
|
{ /* Animato Action with only groups / Blender version [2.5, 4.4) */
|
|
bAction action = {};
|
|
Link /* bActionGroup */ fake_group = {};
|
|
|
|
BLI_addtail(&action.groups, &fake_group);
|
|
EXPECT_FALSE(action_is_layered(action))
|
|
<< "Animato Actions should NOT be considered 'layered'";
|
|
}
|
|
|
|
{ /* Layered Action with only layers / Blender version 4.4 and newer. */
|
|
bAction action = {};
|
|
action.layer_array_num = 1;
|
|
|
|
EXPECT_TRUE(action_is_layered(action)) << "Layered Actions should be considered 'layered'";
|
|
}
|
|
|
|
{ /* Layered Action with only slots / Blender version 4.4 and newer. */
|
|
bAction action = {};
|
|
action.slot_array_num = 1;
|
|
|
|
EXPECT_TRUE(action_is_layered(action)) << "Layered Actions should be considered 'layered'";
|
|
}
|
|
|
|
{ /* Layered Action as it exists on disk, with forward-compatible info in there. */
|
|
bAction action = {};
|
|
Link /* FCurve */ fake_fcurve = {};
|
|
action.layer_array_num = 1;
|
|
|
|
BLI_addtail(&action.curves, &fake_fcurve);
|
|
EXPECT_TRUE(action_is_layered(action))
|
|
<< "Layered Actions with forward-compat data should be considered 'layered'";
|
|
}
|
|
|
|
{ /* Completely zeroed out Action. */
|
|
bAction action = {};
|
|
EXPECT_TRUE(action_is_layered(action)) << "Zero'ed-out Actions should be considered 'layered'";
|
|
}
|
|
}
|
|
|
|
} // namespace blender::animrig::versioning::tests
|