Skip to content

feat: Cure effects food property#1146

Closed
RealRONiN wants to merge 7 commits into
PurpurMC:ver/1.19.2from
RealRONiN:ver/1.19.2
Closed

feat: Cure effects food property#1146
RealRONiN wants to merge 7 commits into
PurpurMC:ver/1.19.2from
RealRONiN:ver/1.19.2

Conversation

@RealRONiN

Copy link
Copy Markdown

Honey bottles can cure any effect same as milk bukkets, it can also heal a player and add additional potion effects if any specified.
IRL Honey has numerous benefits, it'd make sense if it serves more benefits than just the saturation effect.

@RealRONiN

Copy link
Copy Markdown
Author

I'm not sure what to do with this,

user.removeEffect(MobEffects.POISON, org.bukkit.event.entity.EntityPotionEffectEvent.Cause.FOOD); // Paper

@BillyGalbreath BillyGalbreath left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I just woke up, so I hope I got it all in one go ^_^

Comment thread patches/server/0309-Honey-bottle-cures-and-heals.patch Outdated
Comment thread patches/server/0309-Honey-bottle-cures-and-heals.patch Outdated
Comment thread patches/server/0309-Honey-bottle-cures-and-heals.patch Outdated
Comment thread patches/server/0309-Honey-bottle-cures-and-heals.patch Outdated
Comment thread patches/server/0309-Honey-bottle-cures-and-heals.patch Outdated
Removed previous honey bottle patches to include cure effect property in food properties so any food item can use it.
@RealRONiN RealRONiN changed the title feat: honey bottle cures and heals feat: Cure effects food property Oct 8, 2022
@RealRONiN

Copy link
Copy Markdown
Author

Technically, milk bukket should be a food item, should I add it to Foods class and move MilkBukketItem class's properties to it?

@RealRONiN

Copy link
Copy Markdown
Author

from 21608 characters to 9759; over 54% of characters were removed.

from 304 lines long to 160; 47% of lines were removed.

@granny granny left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

PR is ready to merge. Waiting on Billy in case he wants to make any changes.

@BillyGalbreath BillyGalbreath left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is far from ready. I will rewrite it myself when I get back online.

@encode42 encode42 added the needs-docs A pull request to PurpurDocs is required before merge label Dec 27, 2022
@granny

granny commented Jan 20, 2023

Copy link
Copy Markdown
Member

Closing as it doesn't seem like there will be any progress done anytime soon.

@granny granny closed this Jan 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest-accepted needs-docs A pull request to PurpurDocs is required before merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants