Dont send item loot window if lockpicking for someone in trade Unsure…

… about the order change

(based on cmangos/mangos-wotlk@e86d51078)

Signed-off-by: Xfurry <xfurry.cmangos@outlook.com>

src/game/Spells/SpellEffects.cpp

@@ -5425,8 +5425,16 @@ void Spell::EffectOpenLock(SpellEffectEntry const* effect)
 
     // mark item as unlocked
     if (itemTarget)
+    {
         itemTarget->SetFlag(ITEM_FIELD_FLAGS, ITEM_DYNFLAG_UNLOCKED);
 
+        // only send loot if owner is player, else client sends release anyway
+        if (itemTarget->GetOwnerGuid() == m_caster->GetObjectGuid())
+            SendLoot(guid, LOOT_SKINNING, LockType(effect->EffectMiscValue));
+    }
+    else
+        SendLoot(guid, LOOT_SKINNING, LockType(effect->EffectMiscValue));
+
     // not allow use skill grow at item base open
     if (!m_CastItem && skillId != SKILL_NONE)
     {
@@ -5447,8 +5455,6 @@ void Spell::EffectOpenLock(SpellEffectEntry const* effect)
             }
         }
     }
-
-    SendLoot(guid, LOOT_SKINNING, LockType(effect->EffectMiscValue));
 }
 
 void Spell::EffectSummonChangeItem(SpellEffectEntry const* effect)