Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[3.3.5] Quest - The Ravaged Caravan; NPC Coords #16286

Closed
BahDumbBugs opened this issue Jan 16, 2016 · 13 comments
Closed

[3.3.5] Quest - The Ravaged Caravan; NPC Coords #16286

BahDumbBugs opened this issue Jan 16, 2016 · 13 comments

Comments

@BahDumbBugs
Copy link

Horde Quest : The Ravaged Caravan, the NPC Morin Cloudstalker who starts and ends the quest is possibly in the wrong spot. Not sure if he is meant to be patrolling the perimeter of Bloodhoof Village, but he is currently located at the junction East of Bloodhoof. The icon on the map for turn in is showing he is at the junction North of the village. Is that correct, or does something need to be moved?

TrinityCore rev. 6a47175 2016-1-14 01:33:37 +0100 (3.3.5 branch) (Unix, Release)

@ghost
Copy link

ghost commented Jan 16, 2016

Depends on if you are using extra addons or not to show the positions on the map. Even without addons, the map might show some discrepancy between DB spot and actual spot. Morin Cloudstalker should be pacing up and down a bit east of the crossing east of Bloodhoof Village, up to where the waggon trail starts off towards The Ravaged Caravan (near the 2 Prarie Dogs and the 2-way signpost).

Currently, Morin Cloudstalker seems to be standing in one spot, not moving (pathing) like he should be.

@Killyana
Copy link
Member

-- position for 4.x and 6.x without WP already updated
-- UPDATE `creature` SET `position_x`=-2364.23, `position_y`=-905.439, `position_z`=-9.0225, `orientation`=2.40855 WHERE `id`=2988;

-- waypoint for 3.3.5
POINT, position_x, position_y, position_z, delay,
1, -2369.16, -876.936, -9.42208, 21600
2, -2366.53, -856.691, -9.36085, 21600
3, -2357.15, -811.864, -9.42138, 21600
4, -2345.23, -748.967, -9.42454, 21600
5, -2341.13, -700.582, -9.29519, 21600
6, -2335.57, -673.555, -9.41523, 21600
7, -2323.8, -641.711, -9.34184, 21600
8, -2310.09, -613.217, -9.42431, 21600
9, -2295.38, -586.808, -9.4231, 21600
10, -2323.09, -595.375, -9.42242, 21600
11, -2312.27, -614.66, -9.42405, 21600
12, -2326.41, -645.536, -9.34993, 21600
13, -2336.51, -676.214, -9.40864, 21600
14, -2344.05, -723.843, -9.42353, 21600
15, -2346.75, -760.367, -9.42353, 21600
16, -2357.56, -811.652, -9.42353, 21600
17, -2366.12, -855.675, -9.32695, 21600
18, -2376.83, -896.428, -9.39643, 21600
19, -2321.09, -634.302, -9.32889, 21600
20, -2303.73, -602.971, -9.42421, 21600
21, -2300.78, -598.174, -9.42421, 21600
22, -2320, -632.325, -9.35565, 21600
23, -2333.9, -668.09, -9.41774, 21600
24, -2340.99, -697.219, -9.24656, 21600
25, -2344.31, -732.61, -9.42379, 21600
26, -2346.68, -771.154, -9.42379, 21600
27, -2354.38, -802.706, -9.42379, 21600
28, -2363.17, -834.134, -9.29097, 21600
29, -2369.71, -872.365, -9.42371, 21600

@ghost
Copy link

ghost commented Jan 16, 2016

it didn't copy the whole query. :S

@ghost
Copy link

ghost commented Jan 16, 2016

OK, maybe that is not a query, just the WP listing so we can fill in a query ourselves? I haven't worked on waypoints before, don't remember if it is waypoint_data or waypoints to be used. Maybe some more updates, like creature_template.MovementType set to 2 instead of 0 and so on...

@ghost
Copy link

ghost commented Jan 16, 2016

I am working on converting this to a usable query and testing.

@ghost
Copy link

ghost commented Jan 16, 2016

SELECT * FROM `waypoints` WHERE `entry`= 2988;
 entry  pointid  position_x  position_y  position_z  point_comment  
------  -------  ----------  ----------  ----------  ---------------

@ghost
Copy link

ghost commented Jan 16, 2016

He starts walking off the path after ID: 18. Not sure if that is intended, but we could delete everything after 18, and he would walk back to ID: 1 while staying on the road. IDs 1-18 complete a loop. Everything after that just seems to be duplicate data. (I also removed the delay because I am pretty sure a 21600 delay on every point is incorrect.)

SET @NPC := 26577;
SET @PATH := @NPC * 10;
UPDATE `creature` SET `spawndist`=0,`MovementType`=2,`position_x`=-2369.16,`position_y`=-876.936,`position_z`=-9.42208 WHERE `guid`=@NPC;
DELETE FROM `creature_addon` WHERE `guid`=@NPC;
INSERT INTO `creature_addon` (`guid`,`path_id`,`mount`,`bytes1`,`bytes2`,`emote`,`auras`) VALUES (@NPC,@PATH,0,0,1,0, '');
DELETE FROM `waypoint_data` WHERE `id`=@PATH;
INSERT INTO `waypoint_data` (`id`,`point`,`position_x`,`position_y`,`position_z`,`orientation`,`delay`,`move_type`,`action`,`action_chance`,`wpguid`) VALUES
(@PATH,1,-2369.16,-876.936,-9.42208,0,0,0,0,100,0),
(@PATH,2,-2366.53,-856.691,-9.36085,0,0,0,0,100,0),
(@PATH,3,-2357.15,-811.864,-9.42138,0,0,0,0,100,0),
(@PATH,4,-2345.23,-748.967,-9.42454,0,0,0,0,100,0),
(@PATH,5,-2341.13,-700.582,-9.29519,0,0,0,0,100,0),
(@PATH,6,-2335.57,-673.555,-9.41523,0,0,0,0,100,0),
(@PATH,7,-2323.8,-641.711,-9.34184,0,0,0,0,100,0),
(@PATH,8,-2310.09,-613.217,-9.42431,0,0,0,0,100,0),
(@PATH,9,-2295.38,-586.808,-9.4231,0,0,0,0,100,0),
(@PATH,10,-2323.09,-595.375,-9.42242,0,0,0,0,100,0),
(@PATH,11,-2312.27,-614.66,-9.42405,0,0,0,0,100,0),
(@PATH,12,-2326.41,-645.536,-9.34993,0,0,0,0,100,0),
(@PATH,13,-2336.51,-676.214,-9.40864,0,0,0,0,100,0),
(@PATH,14,-2344.05,-723.843,-9.42353,0,0,0,0,100,0),
(@PATH,15,-2346.75,-760.367,-9.42353,0,0,0,0,100,0),
(@PATH,16,-2357.56,-811.652,-9.42353,0,0,0,0,100,0),
(@PATH,17,-2366.12,-855.675,-9.32695,0,0,0,0,100,0),
(@PATH,18,-2376.83,-896.428,-9.39643,0,0,0,0,100,0),
(@PATH,19,-2321.09,-634.302,-9.32889,0,0,0,0,100,0),
(@PATH,20,-2303.73,-602.971,-9.42421,0,0,0,0,100,0),
(@PATH,21,-2300.78,-598.174,-9.42421,0,0,0,0,100,0),
(@PATH,22,-2320,-632.325,-9.35565,0,0,0,0,100,0),
(@PATH,23,-2333.9,-668.09,-9.41774,0,0,0,0,100,0),
(@PATH,24,-2340.99,-697.219,-9.24656,0,0,0,0,100,0),
(@PATH,25,-2344.31,-732.61,-9.42379,0,0,0,0,100,0),
(@PATH,26,-2346.68,-771.154,-9.42379,0,0,0,0,100,0),
(@PATH,27,-2354.38,-802.706,-9.42379,0,0,0,0,100,0),
(@PATH,28,-2363.17,-834.134,-9.29097,0,0,0,0,100,0),
(@PATH,29,-2369.71,-872.365,-9.42371,0,0,0,0,100,0);

Suggested fix. Added the 21600 delay to ID 18, and added an orientation so he faces the road. Removed IDs 19-29. Can you update this with a patch-fix tag, @Killyana?

SET @NPC := 26577;
SET @PATH := @NPC * 10;
UPDATE `creature` SET `spawndist`=0,`MovementType`=2,`position_x`=-2369.16,`position_y`=-876.936,`position_z`=-9.42208 WHERE `guid`=@NPC;
DELETE FROM `creature_addon` WHERE `guid`=@NPC;
INSERT INTO `creature_addon` (`guid`,`path_id`,`mount`,`bytes1`,`bytes2`,`emote`,`auras`) VALUES (@NPC,@PATH,0,0,1,0, '');
DELETE FROM `waypoint_data` WHERE `id`=@PATH;
INSERT INTO `waypoint_data` (`id`,`point`,`position_x`,`position_y`,`position_z`,`orientation`,`delay`,`move_type`,`action`,`action_chance`,`wpguid`) VALUES
(@PATH,1,-2369.16,-876.936,-9.42208,0,0,0,0,100,0),
(@PATH,2,-2366.53,-856.691,-9.36085,0,0,0,0,100,0),
(@PATH,3,-2357.15,-811.864,-9.42138,0,0,0,0,100,0),
(@PATH,4,-2345.23,-748.967,-9.42454,0,0,0,0,100,0),
(@PATH,5,-2341.13,-700.582,-9.29519,0,0,0,0,100,0),
(@PATH,6,-2335.57,-673.555,-9.41523,0,0,0,0,100,0),
(@PATH,7,-2323.8,-641.711,-9.34184,0,0,0,0,100,0),
(@PATH,8,-2310.09,-613.217,-9.42431,0,0,0,0,100,0),
(@PATH,9,-2295.38,-586.808,-9.4231,0,0,0,0,100,0),
(@PATH,10,-2323.09,-595.375,-9.42242,0,0,0,0,100,0),
(@PATH,11,-2312.27,-614.66,-9.42405,0,0,0,0,100,0),
(@PATH,12,-2326.41,-645.536,-9.34993,0,0,0,0,100,0),
(@PATH,13,-2336.51,-676.214,-9.40864,0,0,0,0,100,0),
(@PATH,14,-2344.05,-723.843,-9.42353,0,0,0,0,100,0),
(@PATH,15,-2346.75,-760.367,-9.42353,0,0,0,0,100,0),
(@PATH,16,-2357.56,-811.652,-9.42353,0,0,0,0,100,0),
(@PATH,17,-2366.12,-855.675,-9.32695,0,0,0,0,100,0),
(@PATH,18,-2376.83,-896.428,-9.39643,6.216497,21600,0,0,100,0);

@Killyana
Copy link
Member

Yes looks good, thanks.

@ghost
Copy link

ghost commented Jan 17, 2016

Note, he is stationary and spawned on the other side of the road in 6.x. (on retail)

@ghost
Copy link

ghost commented Jan 17, 2016

Hmm, what does http://www.wowhead.com/npc=26577/coldwind-witch have to do with Mulgore? I thought we were talking about http://www.wowhead.com/npc=2988/morin-cloudstalker ?

@ghost
Copy link

ghost commented Jan 17, 2016

waypoint data is by GUID, not entry

@ghost
Copy link

ghost commented Jan 17, 2016

right. sorry, my bad. I always read id (or other name) as the NPC's entry, as long as it's not guid.

@BahDumbBugs
Copy link
Author

Looks good. Thanks everyone for your input and work. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants