vmangos / core

Progressive Vanilla Core aimed at all versions from 1.2 to 1.12
GNU General Public License v2.0
646 stars 529 forks source link

WIP Fix patch for several objects/NPCs. #2685

Open Daribon opened 1 week ago

Daribon commented 1 week ago

🍰 Pullrequest

This PR fixes the patch for several objects/NPCs that should not be spawned in earlier versions of the game. The PR also adds two objects that were previously missing from alliance/horde camp, used only pre patch 1.12.

Proof

From vmangos after PR applied: WoW_NxjF8de0cA WoW_ExvXibchLA

Old images from from Cenarion Hold pre 1.9: Wearing old t2 art, t2 was updated in 1.9 so this image is from 1.8 (Hallows End was added in 1.8 October 10 2005): 58965626301 Wearing old t2 art, t2 was updated in 1.9 so this image is from 1.8 (Hallows End was added in 1.8 October 10 2005): 58965626300 Unknown date, but from originally from same image folder as the rogue wearing old t2. (Hallows End was added in 1.8 October 10 2005): 58965626302 Wearing old t2 art, t2 was updated in 1.9 so this image is from 1.8 (Hallows End was added in 1.8 October 10 2005): 58965626308 Wearing old t2 art, t2 was updated in 1.9 so this image is from 1.8 (Hallows End was added in 1.8 October 10 2005): 58965626309 Unknown date, but from originally from same image folder as the rogue wearing old t2. 58965626319 Unknown date, but as the Cenarion Hold Infantry are mounted, I believe its from 1.8: ThebattlefieldofCenarionHold

Issues

How2Test

Todo / Checklist

Daribon commented 1 week ago

Added two objects that were previously missing from both alliance and horde camp in Silithus, in pre patch 1.12.

Daribon commented 1 week ago

Updated with removal of several Cenarion Hold guards pre 1.9. I think I've missed some NPCs, since I don't have images in all angles of Cenarion Hold. Some more dated images would be nice since the guard by the ladder has me confused why it would not be spawned post 1.9.

Needs more research.