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

Fixes #2489 #2741

Closed
wants to merge 1 commit into from
Closed

Fixes #2489 #2741

wants to merge 1 commit into from

Conversation

Jedzkie
Copy link
Contributor

@Jedzkie Jedzkie commented May 27, 2020

Pull Request Prelude

Changes Proposed

Issues addressed:
#2489

@Jedzkie Jedzkie added the component:core Affecting the Hercules core (i.e. not the game mechanics directly) label May 27, 2020
@Jedzkie Jedzkie changed the title Fixes https://github.com/HerculesWS/Hercules/issues/2489 Fixes #2489 May 27, 2020
@Jedzkie Jedzkie linked an issue May 27, 2020 that may be closed by this pull request
@Jedzkie Jedzkie changed the base branch from stable to master May 27, 2020 12:22
Copy link
Member

@Kenpachi2k13 Kenpachi2k13 left a comment

Choose a reason for hiding this comment

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

Please use a more descriptive title for your PR.

@Kenpachi2k13 Kenpachi2k13 added component:mechanics Affecting the game mechanics in general status:code-review Awaiting code review type:bug Issue is a bug or describes an incorrect behavior that should be fixed and removed component:core Affecting the Hercules core (i.e. not the game mechanics directly) labels May 27, 2020
@Kenpachi2k13 Kenpachi2k13 added this to the Release v2020.05.31 milestone May 27, 2020
@Emistry
Copy link
Member

Emistry commented May 27, 2020

remove cash food when the normal food end?
perhaps it should be do not add the status if player already has cash food status?

remove a cash food when normal food expired, doesn't sound like it should happen.
after all player are most likely paid to obtain cash food in official server, and a non-paid item terminate a paid item, doesn't seem good approach?

Copy link
Member

@Kenpachi2k13 Kenpachi2k13 left a comment

Choose a reason for hiding this comment

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

@Emistry
status_end_sc_before_start() ends - as the name says - a status before starting another one.
In this case it would end SC_FOOD_STR_CASH prior to starting SC_FOOD_STR for example.

But in spite of that you're right. I did a test on Aegis and the behaviour is different.
Using a cash food cancels the effect of normal food but not vice versa.
Also the the old effect will only be canceled if the new one is stronger. Using +1 STR cash food won't cancel +1 STR normal food.
The item is used regardless of successfully applying the effect or not. This applies for normal and cash food items.

Removed approval.

@Jedzkie Jedzkie removed their assignment Jul 3, 2020
@Jedzkie Jedzkie removed component:mechanics Affecting the game mechanics in general status:code-review Awaiting code review type:bug Issue is a bug or describes an incorrect behavior that should be fixed labels Jul 3, 2020
@Jedzkie
Copy link
Contributor Author

Jedzkie commented Jul 3, 2020

Closing this PR, replacing with new one.

@Jedzkie Jedzkie closed this Jul 3, 2020
@MishimaHaruna MishimaHaruna removed this from the Release v2020.07.26 milestone Jul 27, 2020
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.

4 participants