Fixes#14384
Author: @Gultask
<!-- First of all, THANK YOU for your contribution. -->
## Issues Addressed:
<!-- If your fix has a relating issue, link it below -->
- Closes#14384
## Tests Performed:
<!-- Does it build without errors? Did you test in-game? What did you
test? On which OS did you test? Describe any other tests performed -->
- Tested ingame.
## How to Test the Changes:
<!-- Describe in a detailed step-by-step order how to test the changes
-->
`.go c id 20363`
<!-- If you intend to contribute repeatedly to our project, it is a good
idea to join our discord channel. We set ranks for our contributors and
give them access to special resources or knowledge:
https://discord.com/invite/DasJqPba)
Do not remove the instructions below about testing, they will help users
to test your PR -->
## How to Test AzerothCore PRs
When a PR is ready to be tested, it will be marked as **[WAITING TO BE
TESTED]**.
You can help by testing PRs and writing your feedback here on the PR's
page on GitHub. Follow the instructions here:
http://www.azerothcore.org/wiki/How-to-test-a-PR
**REMEMBER**: when testing a PR that changes something **generic** (i.e.
a part of code that handles more than one specific thing), the tester
should not only check that the PR does its job (e.g. fixing spell XXX)
but **especially** check that the PR does not cause any regression (i.e.
introducing new bugs).
**For example**: if a PR fixes spell X by changing a part of code that
handles spells X, Y, and Z, we should not only test X, but **we should
test Y and Z as well**.
Co-authored-by: Gultask <gultask@gmail.com>
WIP: Need to find more guids
<!-- First of all, THANK YOU for your contribution. -->
## Changes Proposed:
- Sniffs critters for Old Hillsbrad, corrects wander distance and spawn
timer
## Issues Addressed:
<!-- If your fix has a relating issue, link it below -->
- Closes
## SOURCE:
<!-- If you can, include a source that can strengthen your claim -->
Retail sniffs
## Tests Performed:
<!-- Does it build without errors? Did you test in-game? What did you
test? On which OS did you test? Describe any other tests performed -->
- Tested ingame
-
## How to Test the Changes:
<!-- Describe in a detailed step-by-step order how to test the changes
-->
1. `.tele OldHillsbrad`
2.
3.
## Known Issues and TODO List:
<!-- Is there anything else left to do after this PR? -->
- [ ]
- [ ]
<!-- If you intend to contribute repeatedly to our project, it is a good
idea to join our discord channel. We set ranks for our contributors and
give them access to special resources or knowledge:
https://discord.com/invite/DasJqPba)
Do not remove the instructions below about testing, they will help users
to test your PR -->
## How to Test AzerothCore PRs
When a PR is ready to be tested, it will be marked as **[WAITING TO BE
TESTED]**.
You can help by testing PRs and writing your feedback here on the PR's
page on GitHub. Follow the instructions here:
http://www.azerothcore.org/wiki/How-to-test-a-PR
**REMEMBER**: when testing a PR that changes something **generic** (i.e.
a part of code that handles more than one specific thing), the tester
should not only check that the PR does its job (e.g. fixing spell XXX)
but **especially** check that the PR does not cause any regression (i.e.
introducing new bugs).
**For example**: if a PR fixes spell X by changing a part of code that
handles spells X, Y, and Z, we should not only test X, but **we should
test Y and Z as well**.
## Changes Proposed:
- This properly moves things from `DOCKER_ETC_FOLDER` to `confDir`
- originally broken with
888a718cf3
## Issues Addressed:
- Should hopefully close
https://github.com/azerothcore/acore-docker/issues/12
## SOURCE:
## Tests Performed:
- I'm not exactly aware of the testing procedure for this, as I don't
use acore-docker
- That said, the previous behavior has been replicated:
2420d19e54/apps/compiler/includes/functions.sh (L148-L150)
- Open to suggestions
## How to Test the Changes:
1. Open to Suggestions
## Known Issues and TODO List:
- Test properly
## How to Test AzerothCore PRs
When a PR is ready to be tested, it will be marked as **[WAITING TO BE
TESTED]**.
You can help by testing PRs and writing your feedback here on the PR's
page on GitHub. Follow the instructions here:
http://www.azerothcore.org/wiki/How-to-test-a-PR
**REMEMBER**: when testing a PR that changes something **generic** (i.e.
a part of code that handles more than one specific thing), the tester
should not only check that the PR does its job (e.g. fixing spell XXX)
but **especially** check that the PR does not cause any regression (i.e.
introducing new bugs).
**For example**: if a PR fixes spell X by changing a part of code that
handles spells X, Y, and Z, we should not only test X, but **we should
test Y and Z as well**.
<!-- First of all, THANK YOU for your contribution. -->
## Changes Proposed:
- Implement Tarren Mill Fishermen
- Remove wrong spawn point for Phil (should be one of the pentagram
children instead)
- Fix waypoints and add emotes for Jay Lemieux & Julie Honeywell
- Rebuild guard spawns, remove custom guards and custom waypoints
- Implement the Tarren Mill Horsehand (Bessy and Young Blanchy event)
- Corrects waypoints for Tarren Mill Peasants in the orchard.
- Fix equip template for Tarren Mill Peasant (18644)
- Adds spawns of Tarren Mill Horse and Tarren Mill Horsehand
- Corrects Young Blanchy position
- Fix unit_flags for Tarren Mill Horse and Tarren Mill Peasant (sniffed)
- Implement the Orchard events (Peasants will periodically gather apples
and drop off sacks of apples at a nearby cart)
## Issues Addressed:
<!-- If your fix has a relating issue, link it below -->
- Closes
## SOURCE:
<!-- If you can, include a source that can strengthen your claim -->
Retail sniffs mostly
Classic sniffs for Spell IDs
## Tests Performed:
<!-- Does it build without errors? Did you test in-game? What did you
test? On which OS did you test? Describe any other tests performed -->
- Tested ingame
-
## How to Test the Changes:
<!-- Describe in a detailed step-by-step order how to test the changes
-->
1. `.go c 83518`
2. observe all 👁️
## Known Issues and TODO List:
<!-- Is there anything else left to do after this PR? -->
- [ ] The Tarren Mill Horsehand phases through the wall due to his
waypoints. He does not pathfind properly to the next waypoint so he
phases instead. This is sniffed data and possibly a core issue.
- [ ] The spell cast by Tarren Mill Peasants is **not** sniffed.
<!-- If you intend to contribute repeatedly to our project, it is a good
idea to join our discord channel. We set ranks for our contributors and
give them access to special resources or knowledge:
https://discord.com/invite/DasJqPba)
Do not remove the instructions below about testing, they will help users
to test your PR -->
## How to Test AzerothCore PRs
When a PR is ready to be tested, it will be marked as **[WAITING TO BE
TESTED]**.
You can help by testing PRs and writing your feedback here on the PR's
page on GitHub. Follow the instructions here:
http://www.azerothcore.org/wiki/How-to-test-a-PR
**REMEMBER**: when testing a PR that changes something **generic** (i.e.
a part of code that handles more than one specific thing), the tester
should not only check that the PR does its job (e.g. fixing spell XXX)
but **especially** check that the PR does not cause any regression (i.e.
introducing new bugs).
**For example**: if a PR fixes spell X by changing a part of code that
handles spells X, Y, and Z, we should not only test X, but **we should
test Y and Z as well**.
Process of translation: only original sources of deDE texts by
researching multiple sources, reverse translation by searching for
related quest items/NPCs and using these names to reconstruct a proper
translation.
This fixes the terms
Scrimshank (Kieselschliff), Banehollow (Schattenfluch), Trollbane
(Trollbann), Stonesplitter (Splitterstein), Stonemaul (Steinbrecher),
Captain (Hauptmann), Kromcrush (Krombruch), Earthbinder (Erdbinder),
Razorfen (Klingenhauer), Razorflank (Klingenflanke), Stonard (Steinard)
for quest progress/request text entries for the deDE localisation with
proper casus/declension (these are not proper translated names of
locations/NPCs that have been left over by Blizzard since their language
localisations in TBC in 2006 and onward).
Fixed a self-made translation for quest 334.
Added missing deDE text entries for 4481, 4482, 4483, 5306, 5863, 7723,
9436.
<!-- First of all, THANK YOU for your contribution. -->
## Changes Proposed:
- Fixing missing or wrong entries in the quest_request_items_locale
table.
## Issues Addressed:
<!-- If your fix has a relating issue, link it below -->
Fixes some of the tasks listed in the issues below:
- https://github.com/azerothcore/azerothcore-wotlk/issues/14244
- https://github.com/azerothcore/azerothcore-wotlk/issues/14618
- From CC Github:
https://github.com/chromiecraft/chromiecraft/issues/4697
## SOURCE:
<!-- If you can, include a source that can strengthen your claim -->
Read the text on top.
## Tests Performed:
<!-- Does it build without errors? Did you test in-game? What did you
test? On which OS did you test? Describe any other tests performed -->
- Not tested.
## How to Test the Changes:
<!-- Describe in a detailed step-by-step order how to test the changes
-->
All of the changes are to request texts of quests, can be tested by
accepting quests or simply reviewing the changed file.
## Known Issues and TODO List:
<!-- Is there anything else left to do after this PR? -->
- [ ] I still might want to check which of the tasks in above mentioned
issues are resolved by this.
<!-- If you intend to contribute repeatedly to our project, it is a good
idea to join our discord channel. We set ranks for our contributors and
give them access to special resources or knowledge:
https://discord.com/invite/DasJqPba)
Do not remove the instructions below about testing, they will help users
to test your PR -->
## How to Test AzerothCore PRs
When a PR is ready to be tested, it will be marked as **[WAITING TO BE
TESTED]**.
You can help by testing PRs and writing your feedback here on the PR's
page on GitHub. Follow the instructions here:
http://www.azerothcore.org/wiki/How-to-test-a-PR
**REMEMBER**: when testing a PR that changes something **generic** (i.e.
a part of code that handles more than one specific thing), the tester
should not only check that the PR does its job (e.g. fixing spell XXX)
but **especially** check that the PR does not cause any regression (i.e.
introducing new bugs).
**For example**: if a PR fixes spell X by changing a part of code that
handles spells X, Y, and Z, we should not only test X, but **we should
test Y and Z as well**.
Co-authored-by: Gultask <100873791+Gultask@users.noreply.github.com>
