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
DB - Elwyn Forest loot #1715
DB - Elwyn Forest loot #1715
Conversation
(6, 11111, 11111, 0.5, 0, 1, 0, 1, 1, NULL); -- Bags | ||
|
||
-- Kobold Worker | ||
DELETE FROM `creature_loot_template` WHERE `Entry`=257; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also for these where you use Entry
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the statements are fine as they are, easier to read and better documented if they are structured in blocks for each creature. I also do the same in my PRs.
INSERT INTO `version_db_world` (`sql_rev`) VALUES ('1555017745656363300'); | ||
|
||
-- Item should only drop from Riverpaw Mongrel (123) | ||
DELETE FROM `reference_loot_template` WHERE `Item`=821; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I mean for these, miss click in previous comment 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right, tomorrow i will be home for that
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the statements are fine as they are, easier to read and better documented. No need to merge them into one statement in my opinion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SQL script is fine in my opinion. Did not test it, though.
Did also check the old and new DB entries, seems to be ok. I think this is ready to be merged. |
Changes from trinitycore
CHANGES PROPOSED:
It includes two unique items:
Riverpaw Leather Vest (821) is a unique drop from Riverpaw Mongrel.
Hard Crawler Carapace (2087) is a unique drop from Sand Crawler.
It also includes a crafted item that should never drop anywhere:
Embossed Leather Vest (2300)
TESTS PERFORMED:
Didn't test yet
Target branch(es):
Master