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

Removed all ambient gossip that didn't/shouldn't exist. #194

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Removed all ambient gossip that didn't/shouldn't exist. #194

wants to merge 3 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Aug 24, 2012

Redundancies are in place to prevent potential undesired loss of data.

Signed-off-by: sycantha [email protected]

@Kretol
Copy link

Kretol commented Aug 25, 2012

After doing this, we'll probably also want to include a query to remove creature_ai_scripts entries pointing to these no-longer-existing gossips.

DELETE FROM creature_ai_scripts WHERE event_type=1 AND action1_type=1 AND action1_param1 NOT IN (SELECT entry FROM creature_ai_texts)

This is to mostly remove all the console errors about the ai_texts not existing when the scripts try to run. However, this also brings up other console errors about NPCs not having script entries. I'd suggest we then run:

ALTER TABLE  `creature_ai_scripts` ADD INDEX ( `creature_id` )
UPDATE `creature_template` SET AIName='' WHERE AIName = 'EventAI' AND entry NOT IN (SELECT creature_id FROM creature_ai_scripts)
ALTER TABLE  `creature_ai_scripts` DROP INDEX `creature_id`

@ghost
Copy link
Author

ghost commented Aug 25, 2012

agreed.

@ghost
Copy link
Author

ghost commented Aug 25, 2012

http://pastebin.com/h88siTfu

Paste bin for prettiernesseress.

@SkyFire
Copy link
Contributor

SkyFire commented Aug 25, 2012

Will add this to the next DB update coming soon.

@ghost
Copy link
Author

ghost commented Aug 26, 2012

: D

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

Successfully merging this pull request may close these issues.

2 participants