refactor: Remove redundant code in SOP.UpdatePrimFlags()

1 files changed, 21 insertions, 27 deletions

diff --git a/OpenSim/Region/Framework/Scenes/SceneObjectPart.cs b/OpenSim/Region/Framework/Scenes/SceneObjectPart.cs
index 822d54f..2659a69 100644
--- a/OpenSim/Region/Framework/Scenes/SceneObjectPart.cs
+++ b/OpenSim/Region/Framework/Scenes/SceneObjectPart.cs
@@ -4411,9 +4411,7 @@ namespace OpenSim.Region.Framework.Scenes
                 if (ParentGroup.Scene == null)
                     return;
 
-                PhysicsActor pa = PhysActor;
-
-                if (pa == null)
+                if (PhysActor == null)
                 {
                     // It's not phantom anymore. So make sure the physics engine get's knowledge of it
                     PhysActor = m_parentGroup.Scene.PhysicsScene.AddPrimShape(
@@ -4425,38 +4423,34 @@ namespace OpenSim.Region.Framework.Scenes
                         UsePhysics,
                         m_localId);
 
-                    pa = PhysActor;
-                    if (pa != null)
-                    {
-                        PhysActor.SetMaterial(Material);
-                        DoPhysicsPropertyUpdate(UsePhysics, true);
+                    PhysActor.SetMaterial(Material);
+                    DoPhysicsPropertyUpdate(UsePhysics, true);
 
-                        if (!m_parentGroup.IsDeleted)
+                    if (!m_parentGroup.IsDeleted)
+                    {
+                        if (LocalId == m_parentGroup.RootPart.LocalId)
                         {
-                            if (LocalId == m_parentGroup.RootPart.LocalId)
-                            {
-                                m_parentGroup.CheckSculptAndLoad();
-                            }
+                            m_parentGroup.CheckSculptAndLoad();
                         }
+                    }
 
-                        if (
-                            ((AggregateScriptEvents & scriptEvents.collision) != 0) ||
-                            ((AggregateScriptEvents & scriptEvents.collision_end) != 0) ||
-                            ((AggregateScriptEvents & scriptEvents.collision_start) != 0) ||
-                            ((AggregateScriptEvents & scriptEvents.land_collision_start) != 0) ||
-                            ((AggregateScriptEvents & scriptEvents.land_collision) != 0) ||
-                            ((AggregateScriptEvents & scriptEvents.land_collision_end) != 0) ||
-                            (CollisionSound != UUID.Zero)
-                            )
-                        {
-                            PhysActor.OnCollisionUpdate += PhysicsCollision;
-                            PhysActor.SubscribeEvents(1000);
-                        }
+                    if (
+                        ((AggregateScriptEvents & scriptEvents.collision) != 0) ||
+                        ((AggregateScriptEvents & scriptEvents.collision_end) != 0) ||
+                        ((AggregateScriptEvents & scriptEvents.collision_start) != 0) ||
+                        ((AggregateScriptEvents & scriptEvents.land_collision_start) != 0) ||
+                        ((AggregateScriptEvents & scriptEvents.land_collision) != 0) ||
+                        ((AggregateScriptEvents & scriptEvents.land_collision_end) != 0) ||
+                        (CollisionSound != UUID.Zero)
+                        )
+                    {
+                        PhysActor.OnCollisionUpdate += PhysicsCollision;
+                        PhysActor.SubscribeEvents(1000);
                     }
                 }
                 else // it already has a physical representation
                 {
-                    pa.IsPhysical = UsePhysics;
+                    PhysActor.IsPhysical = UsePhysics;
 
                     DoPhysicsPropertyUpdate(UsePhysics, false); // Update physical status. If it's phantom this will remove the prim