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

[Rules] Add adjustment for zone forage. #2330

Merged
merged 2 commits into from
Jul 30, 2022
Merged

[Rules] Add adjustment for zone forage. #2330

merged 2 commits into from
Jul 30, 2022

Conversation

fryguy503
Copy link
Contributor

  • Added rule to allow the adjustment of zone foraging %

- Added rule to allow the adjustment of zone foraging %
zone/forage.cpp Outdated
@@ -457,37 +456,38 @@ void Client::ForageItem(bool guarantee) {
return;
}

if(foragedfood == 13106)
if(foragedfood == 13106) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we make this a constant?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FOOD_FISHING_GRUBS or something.

@Kinglykrab
Copy link
Contributor

Kinglykrab commented Jul 29, 2022

if ( over if( maybe as well?

Copy link
Contributor

@Kinglykrab Kinglykrab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, minor comments.

@mackal
Copy link
Member

mackal commented Jul 29, 2022

Our forage and fishing tables are so fucking wrong.

/*0620*/ uint32 ForageLow; // Forage loot table ID?
there is this data in that struct ...

@Akkadius Akkadius merged commit 0f94270 into master Jul 30, 2022
@Akkadius Akkadius deleted the rules/forage_tweak branch July 30, 2022 18:39
mackal pushed a commit that referenced this pull request Sep 3, 2022
* [Rules] Add adjustment for zone forage.

- Added rule to allow the adjustment of zone foraging %

* Cleanup if formating
catapultam-habeo pushed a commit to catapultam-habeo/pyrelight-server that referenced this pull request Mar 27, 2023
* [Rules] Add adjustment for zone forage.

- Added rule to allow the adjustment of zone foraging %

* Cleanup if formating
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.

None yet

4 participants