feat(Docker): update to Ubuntu 20.04 + more improvements (#3277)

src/server/game/Entities/Unit/Unit.cpp

@@ -501,7 +501,7 @@ void Unit::UpdateSplineMovement(uint32 t_diff)
     // pussywizard: update always! not every 400ms, because movement generators need the actual position
     //m_movesplineTimer.Update(t_diff);
     //if (m_movesplineTimer.Passed() || arrived)
-        UpdateSplinePosition();
+    UpdateSplinePosition();
 }
 
 void Unit::UpdateSplinePosition()
@@ -15094,7 +15094,7 @@ void Unit::ProcDamageAndSpellFor(bool isVictim, Unit* target, uint32 procFlag, u
                     continue;
                 // Some spells must always trigger
                 //if (isAlwaysTriggeredAura[aurEff->GetAuraType()])
-                    triggerData.effMask |= 1<<i;
+                triggerData.effMask |= 1<<i;
             }
         }
         if (triggerData.effMask)

src/server/game/Handlers/QuestHandler.cpp

@@ -91,7 +91,7 @@ void WorldSession::HandleQuestgiverHelloOpcode(WorldPacket & recvData)
         GetPlayer()->RemoveAurasByType(SPELL_AURA_FEIGN_DEATH);
     // Stop the npc if moving
     //if (!creature->GetTransport()) // pussywizard: reverted with new spline (old: without this check, npc would stay in place and the transport would continue moving, so the npc falls off. NPCs on transports don't have waypoints, so stopmoving is not needed)
-        creature->StopMoving();
+    creature->StopMoving();
 
 #ifdef ELUNA
         if (sEluna->OnGossipHello(_player, creature))

src/server/game/Spells/SpellEffects.cpp

@@ -2517,7 +2517,7 @@ void Spell::EffectSummonType(SpellEffIndex effIndex)
             // xinef: i think this is wrong, found only 2 vehicles with faction override and one of them should inherit caster faction...
             //uint32 faction = properties->Faction;
             //if (!faction)
-                uint32 faction = m_originalCaster->getFaction();
+            uint32 faction = m_originalCaster->getFaction();
 
             summon->setFaction(faction);
             break;