Skip to content

Stored Bee API#1339

Closed
EOT3000 wants to merge 1 commit into
PurpurMC:ver/1.19.4from
EOT3000:ver/1.19.4
Closed

Stored Bee API#1339
EOT3000 wants to merge 1 commit into
PurpurMC:ver/1.19.4from
EOT3000:ver/1.19.4

Conversation

@EOT3000

@EOT3000 EOT3000 commented Jun 10, 2023

Copy link
Copy Markdown
Contributor

Adds a way to modify some properties of individual stored entities (bees in beehive), as well as release them individually, without having to release them all.

@katekerllenevich

Copy link
Copy Markdown
Member

I like this, unfortunately your timings is just off. You'll have to rebase this onto 1.20, I'm too tired to review this tonight but I'll take a look at it later.

@katekerllenevich katekerllenevich self-requested a review June 10, 2023 03:23

@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.

Did a quick skim of your PR. For future reference it's also a good idea to create a new branch for every PR you create. Do what Omega said as well.

Comment on lines +15 to +16
+import org.jetbrains.annotations.Nullable;
+import org.purpurmc.purpur.entity.StoredEntity;

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.

inline the fqn

Comment on lines +65 to +67
- private static class BeeData {
+ // Purpur - change from private to public
+ public static class BeeData {

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.

inline the comment

import java.util.List;
import net.minecraft.core.BlockPos;
import net.minecraft.world.entity.Entity;
+import net.minecraft.world.level.Level;

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.

doesn't look like this is being used anywhere?

Comment on lines +90 to +93
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
+import org.purpurmc.purpur.entity.PurpurStoredBee;
+import org.purpurmc.purpur.entity.StoredEntity;

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.

inline the fqn

@EOT3000

EOT3000 commented Jun 10, 2023

Copy link
Copy Markdown
Contributor Author

I don't understand how github works too well so I'll make another PR with all the requested changes.

@EOT3000 EOT3000 closed this Jun 10, 2023
@EOT3000 EOT3000 deleted the ver/1.19.4 branch June 11, 2023 22:06
@EOT3000 EOT3000 restored the ver/1.19.4 branch June 11, 2023 22:06
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