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

Pink petal count #2632

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from
Open

Pink petal count #2632

wants to merge 6 commits into from

Conversation

zozer
Copy link
Contributor

@zozer zozer commented Jun 25, 2024

Added count property to pink petals

mcmonkey4eva
mcmonkey4eva previously approved these changes Jun 25, 2024
@tal5
Copy link
Member

tal5 commented Aug 31, 2024

See message on Discord - not really able to test rn, but afaics it should work fine? having version helper methods isn't awful, but would be nicer to just do it normally if possible

mcmonkey4eva
mcmonkey4eva previously approved these changes Jan 23, 2025
@@ -132,6 +131,9 @@ else if (isRespawnAnchor()) {
else if (isCandle()) {
return getCandle().getCandles();
}
else if ((NMSHandler.getVersion().isAtLeast(NMSVersion.v1_19) && material.getModernData() instanceof PinkPetals pinkPetals)) {
Copy link
Member

Choose a reason for hiding this comment

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

Pretty sure these are redundant double brackets on all of these?

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.

3 participants