godotengine
/
godot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #62393 from godotengine/revert-62227-fix-rotation-mode-node3d 

Revert "Refactored `rotation_mode` in Node3D"
This commit is contained in:
Juan Linietsky 2022-06-25 10:22:18 +02:00 committed by GitHub
parent c87f9fe4ce 14f9fbfed8
commit c4f0062f15
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 20 additions and 49 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

68
scene/3d/node_3d.cpp
View File

@ -85,13 +85,10 @@ void Node3D::_notify_dirty() {
} }
void Node3D::_update_local_transform() const { void Node3D::_update_local_transform() const {
if (this->get_rotation_edit_mode() == ROTATION_EDIT_MODE_EULER) { if (this->get_rotation_edit_mode() != ROTATION_EDIT_MODE_BASIS) {
data.local_transform.basis.set_euler(data.rotation, data.rotation_order); data.local_transform = data.local_transform.orthogonalized();
data.local_transform.basis.scale_local(data.scale);
} else if (this->get_rotation_edit_mode() == ROTATION_EDIT_MODE_QUATERNION) {
data.local_transform.basis = Basis(data.quaternion);
data.local_transform.basis.scale_local(data.scale);
} }
data.local_transform.basis.set_euler_scale(data.rotation, data.scale);
data.dirty &= ~DIRTY_LOCAL; data.dirty &= ~DIRTY_LOCAL;
} }
@ -215,18 +212,7 @@ void Node3D::set_basis(const Basis &p_basis) {
set_transform(Transform3D(p_basis, data.local_transform.origin)); set_transform(Transform3D(p_basis, data.local_transform.origin));
} }
void Node3D::set_quaternion(const Quaternion &p_quaternion) { void Node3D::set_quaternion(const Quaternion &p_quaternion) {
if (data.dirty & DIRTY_VECTORS) { set_transform(Transform3D(Basis(p_quaternion), data.local_transform.origin));
data.rotation = get_transform().basis.get_euler_normalized(data.rotation_order);
data.scale = get_transform().basis.get_scale();
data.dirty &= ~DIRTY_VECTORS;
}
data.quaternion = p_quaternion;
data.dirty |= DIRTY_LOCAL;
_propagate_transform_changed(this);
if (data.notify_local_transform) {
notification(NOTIFICATION_LOCAL_TRANSFORM_CHANGED);
}
} }
void Node3D::set_transform(const Transform3D &p_transform) { void Node3D::set_transform(const Transform3D &p_transform) {
@ -242,14 +228,7 @@ Basis Node3D::get_basis() const {
return get_transform().basis; return get_transform().basis;
} }
Quaternion Node3D::get_quaternion() const { Quaternion Node3D::get_quaternion() const {
if (data.dirty & DIRTY_VECTORS) { return Quaternion(get_transform().basis);
data.quaternion = get_transform().basis.get_rotation_quaternion();
data.rotation = get_transform().basis.get_euler_normalized(data.rotation_order);
data.scale = get_transform().basis.get_scale();
data.dirty &= ~DIRTY_VECTORS;
}
return data.quaternion;
} }
void Node3D::set_global_transform(const Transform3D &p_transform) { void Node3D::set_global_transform(const Transform3D &p_transform) {
@ -276,9 +255,9 @@ Transform3D Node3D::get_global_transform() const {
} }
if (data.parent && !data.top_level_active) { if (data.parent && !data.top_level_active) {
data.global_transform = data.parent->get_global_transform() * get_transform(); data.global_transform = data.parent->get_global_transform() * data.local_transform;
} else { } else {
data.global_transform = get_transform(); data.global_transform = data.local_transform;
} }
if (data.disable_scale) { if (data.disable_scale) {
@ -335,13 +314,9 @@ void Node3D::set_rotation_edit_mode(RotationEditMode p_mode) {
if (data.rotation_edit_mode == p_mode) { if (data.rotation_edit_mode == p_mode) {
return; return;
} }
data.quaternion = get_transform().basis.get_rotation_quaternion();
data.rotation = get_transform().basis.get_euler_normalized(data.rotation_order);
data.scale = get_transform().basis.get_scale();
data.rotation_edit_mode = p_mode; data.rotation_edit_mode = p_mode;
// Shearing is not allowed except in ROTATION_EDIT_MODE_BASIS, so validate local_transform. // Shearing is not allowed except in ROTATION_EDIT_MODE_BASIS.
data.dirty |= DIRTY_LOCAL; data.dirty |= DIRTY_LOCAL;
_propagate_transform_changed(this); _propagate_transform_changed(this);
if (data.notify_local_transform) { if (data.notify_local_transform) {
@ -365,9 +340,8 @@ void Node3D::set_rotation_order(RotationOrder p_order) {
ERR_FAIL_INDEX(int32_t(order), 6); ERR_FAIL_INDEX(int32_t(order), 6);
if (data.dirty & DIRTY_VECTORS) { if (data.dirty & DIRTY_VECTORS) {
data.quaternion = get_transform().basis.get_rotation_quaternion(); data.rotation = data.local_transform.basis.get_euler_normalized(order);
data.rotation = get_transform().basis.get_euler_normalized(order); data.scale = data.local_transform.basis.get_scale();
data.scale = get_transform().basis.get_scale();
data.dirty &= ~DIRTY_VECTORS; data.dirty &= ~DIRTY_VECTORS;
} else { } else {
data.rotation = Basis::from_euler(data.rotation, data.rotation_order).get_euler_normalized(order); data.rotation = Basis::from_euler(data.rotation, data.rotation_order).get_euler_normalized(order);
@ -385,8 +359,7 @@ Node3D::RotationOrder Node3D::get_rotation_order() const {
void Node3D::set_rotation(const Vector3 &p_euler_rad) { void Node3D::set_rotation(const Vector3 &p_euler_rad) {
if (data.dirty & DIRTY_VECTORS) { if (data.dirty & DIRTY_VECTORS) {
data.quaternion = get_transform().basis.get_rotation_quaternion(); data.scale = data.local_transform.basis.get_scale();
data.scale = get_transform().basis.get_scale();
data.dirty &= ~DIRTY_VECTORS; data.dirty &= ~DIRTY_VECTORS;
} }
@ -400,8 +373,7 @@ void Node3D::set_rotation(const Vector3 &p_euler_rad) {
void Node3D::set_scale(const Vector3 &p_scale) { void Node3D::set_scale(const Vector3 &p_scale) {
if (data.dirty & DIRTY_VECTORS) { if (data.dirty & DIRTY_VECTORS) {
data.quaternion = get_transform().basis.get_rotation_quaternion(); data.rotation = data.local_transform.basis.get_euler_normalized(data.rotation_order);
data.rotation = get_transform().basis.get_euler_normalized(data.rotation_order);
data.dirty &= ~DIRTY_VECTORS; data.dirty &= ~DIRTY_VECTORS;
} }
@ -414,14 +386,14 @@ void Node3D::set_scale(const Vector3 &p_scale) {
} }
Vector3 Node3D::get_position() const { Vector3 Node3D::get_position() const {
return get_transform().origin; return data.local_transform.origin;
} }
Vector3 Node3D::get_rotation() const { Vector3 Node3D::get_rotation() const {
if (data.dirty & DIRTY_VECTORS) { if (data.dirty & DIRTY_VECTORS) {
data.quaternion = get_transform().basis.get_rotation_quaternion(); data.scale = data.local_transform.basis.get_scale();
data.rotation = get_transform().basis.get_euler_normalized(data.rotation_order); data.rotation = data.local_transform.basis.get_euler_normalized(data.rotation_order);
data.scale = get_transform().basis.get_scale();
data.dirty &= ~DIRTY_VECTORS; data.dirty &= ~DIRTY_VECTORS;
} }
@ -430,9 +402,9 @@ Vector3 Node3D::get_rotation() const {
Vector3 Node3D::get_scale() const { Vector3 Node3D::get_scale() const {
if (data.dirty & DIRTY_VECTORS) { if (data.dirty & DIRTY_VECTORS) {
data.quaternion = get_transform().basis.get_rotation_quaternion(); data.scale = data.local_transform.basis.get_scale();
data.rotation = get_transform().basis.get_euler_normalized(data.rotation_order); data.rotation = data.local_transform.basis.get_euler_normalized(data.rotation_order);
data.scale = get_transform().basis.get_scale();
data.dirty &= ~DIRTY_VECTORS; data.dirty &= ~DIRTY_VECTORS;
} }
@ -893,7 +865,7 @@ Variant Node3D::property_get_revert(const String &p_name) {
} else if (p_name == "quaternion") { } else if (p_name == "quaternion") {
Variant variant = PropertyUtils::get_property_default_value(this, "transform", &valid); Variant variant = PropertyUtils::get_property_default_value(this, "transform", &valid);
if (valid && variant.get_type() == Variant::Type::TRANSFORM3D) { if (valid && variant.get_type() == Variant::Type::TRANSFORM3D) {
r_ret = Transform3D(variant).get_basis().get_rotation_quaternion(); r_ret = Quaternion(Transform3D(variant).get_basis());
} else { } else {
return Quaternion(); return Quaternion();
} }

1
scene/3d/node_3d.h
View File

@ -81,7 +81,6 @@ private:
mutable Transform3D global_transform; mutable Transform3D global_transform;
mutable Transform3D local_transform; mutable Transform3D local_transform;
mutable Basis::EulerOrder rotation_order = Basis::EULER_ORDER_YXZ; mutable Basis::EulerOrder rotation_order = Basis::EULER_ORDER_YXZ;
mutable Quaternion quaternion;
mutable Vector3 rotation; mutable Vector3 rotation;
mutable Vector3 scale = Vector3(1, 1, 1); mutable Vector3 scale = Vector3(1, 1, 1);
mutable RotationEditMode rotation_edit_mode = ROTATION_EDIT_MODE_EULER; mutable RotationEditMode rotation_edit_mode = ROTATION_EDIT_MODE_EULER;