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

[fix] behavior of the Cauldrons #256

Merged

Conversation

lonefelidae16
Copy link
Contributor

@lonefelidae16 lonefelidae16 commented Apr 2, 2023

I apologize for any inconvenience caused by my lack of attention. Please wait a moment while I run the final check... Looks good.

fix #252 , resolve #176

Important: This is the Breaking Changes to the existing world.

This allows following:

  • Colored Water Cauldron
    • Stain the item 6 times
    • Mix already colored water with different dyes
  • Potion Cauldron
    • Create Tipped Arrows

Breaks

The maximum level of Potion Cauldron become 8, and Colored Water Cauldron become 6. If they exist in your world, the fluid level will be lower.

The way of retaining color information has changed. The Water Cauldron which is dyed may appear to be black water.

To avoid these errors, please empty the cauldron before installing the new version.


Known Issues

The Bedrock’s cauldron behaves as follows:

  1. Full-filled with Potion
  2. Create 48 Tipped Arrows
  3. Take out once in Glass Bottle
  4. Create 16 Tipped Arrows and cauldron become empty
  • Can wash the dye off 6 times of Leather Armors, Shulker Boxes, and top layer of Banners

  • When there is water fluid directly above the cauldron, it fills with water

These behavior still cannot be reproduced.


Changes

  • new file ColorBlenderHelper

    • the method of blendColors is inspired by DyeableItem#blendAndSetColor
  • update files

    • AbstractBECauldronBlock: now extends AbstractCauldronBlock
    • BedrockCauldronBehavior: add TIPPED_ARROW_WITH_POTION behavior
    • WaterCauldronBlockEntity: now can blend two or more colors
    • ColoredWaterCauldronBlock, PotionCauldronBlock: add more helper method
  • unregistered mixins

    • mixin.common.features.cauldron.AbstractCauldronBlockMixin
    • mixin.common.features.cauldron.LeveledCauldronBlockAccessor
  • several number of new block models without using LeveledCauldronBlock

**Breaking changes to your existing world!**

This allows to create Tipped Arrows with Potion Cauldron.

The maximum level of Potion Cauldron become 8. If it exists in your world, the fluid level will be lower.

To avoid this error, please take these out.

* update `BedrockCauldronBehavior`
  - add `TIPPED_ARROW_WITH_POTION` behavior

* update `AbstractBECauldronBlock`
  - now extends `AbstractCauldronBlock`

* unregistered mixins
  - `mixin.common.features.cauldron.AbstractCauldronBlockMixin`
  - `mixin.common.features.cauldron.LeveledCauldronBlockAccessor`

* new block models without using `LeveledCauldronBlock`
**Breaking changes to your existing world!**

This allows following:

* Stain the item 6 times
* Mix already colored water with different dyes

The maximum level of Colored Water Cauldron become 6. If it exists in your world, the fluid level will be lower.

The way of retaining color info has changed. Colored Water Cauldron may appear to be black water.

To avoid this error, please take these out.

* new file `ColorBlenderHelper`
  - the method of `blendColors` is inspired by `DyeableItem#blendAndSetColor`

* update `BedrockCauldronBehavior`
  - change the behavior

* update `ColoredWaterCauldronBlock`
  - maximum level become 6

* update `WaterCauldronBlockEntity`
  - blend the color

* new block models without using `LeveledCauldronBlock`
* update `BedrockIfyMixinPlugin`
  - cannot launch on DedicatedServer

* update `BedrockCauldronProperties`
  - property name `level` will conflict with vanilla's cauldron
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.

[Bug]Arrows can not be used on the cauldron [feature request] bedrock cauldrons
2 participants