Commit message (Collapse) | Author | Age | Lines | |
---|---|---|---|---|
* | Improvements to BlockStates | Lukas Hennig | 2017-08-05 | -645/+524 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | * Actually capture all the data of TileEntities. This is done by creating a copy of the TileEntity. The methods of BlockState which currently directly access the TileEntity reference will modify the data of that TileEntity-snapshot instead. * With the call to BlockState.update, the captured TileEntity data gets applied to the current TileEntity in the world. * Methods which trigger block specific actions will use the current TileEntity from the world. * CraftBlockState does not hand out the wrapped or the snapshot TileEntity directly. Instead, it provides an applyTo method to copy the data to a given TileEntity and a method to directly get a copy of the TileEntity NBT data represented by the BlockState. CraftMetaBlockState was updated to make use of that. * Added #getSnapshotInventory() to bukkit which allows modifiying the captured inventory snapshots of containers. * Tried to clarify which methods only work if the BlockState is placed, which methods require the block in the world to still be of the same type (methods which trigger actions), and that .getInventory() directly modifies the inventory of the block in the world if the BlockState is placed and becomes invalid if the block type is changed. Backwards compatibility * If the BlockState acts as InventoryHolder, getInventory() will still return the inventory directly backed by the TileEntity in the world (like before), and not the snapshot inventory. This compromise should reduce the potential of these changes to break existing plugins, or craftbukkit's own use of BlockState. * The snapshot's inventory can be accessed by a new method getSnapshotInventory() * In case the BlockState is not placed (if it was retrieved from the MetaBlockState of an item), the getInventory() method will however return the snapshot inventory. So that when the BlockState gets applied back to the item, the inventory changes are properly included. * With the changes to CraftMetaBlockState it is no longer required to call the update method before passing a modified BlockState to the CraftMetaBlockState. For backwards compatibility the update method will simply return true for a non-placed BlockState, without actually doing anything. Impact on plugins * Restoring blocks now actually works as expected, properly restoring the TileEntity data, reglardless if the block changed its type in the meantime. * Plugins are now consistently required to call the update method in order to apply changes to blocks. Though, regarding the Javadoc they should have been required to do so anyways. * New feature: Plugins can take and modify inventory snapshots. * Breaking change: If a plugin gets the BlockState of a block in the world, modifies the inventory returned by .getInventory(), and then tries to use the same BlockState to apply the TileEntity data to an ItemStack block meta, the ItemStack will use the snapshot inventory, disregarding the changes made to the inventory returned by .getInventory(). This is the compromise of .getInventory() returning the inventory directly backed by the TileEntity in the world. Other fixes related to BlockState: * TileEntityContainer#getLocation() will run into a NPE if the TileEntity is non-placed (ex. when getting the BlockState from a CraftMetaBlockState). * Beacon.getEntitiesInRange() would previously throw a NPE if called for a non-placed BlockState. It was changed to now require to be placed and use the current TileEntity in the world. If the TileEntity in the world is no longer a beacon, it will return an empty list. * EndGateway now supports setting and getting the exit location even for non-placed EndGateways (inside BlockStateMeta) by using / returning a location with world being null. | |||
* | SPIGOT-3492: Ensure player functionality uses their specific scoreboard | md_5 | 2017-08-05 | -6/+12 |
| | ||||
* | Add build delay back | md_5 | 2017-08-04 | -1/+1 |
| | ||||
* | Update to Minecraft 1.12.1 | md_5 | 2017-08-03 | -254/+179 |
| | ||||
* | Remove outdated build delay. | md_5 | 2017-08-03 | -1/+1 |
| | ||||
* | SPIGOT-3486: Missing check in redstone placement | md_5 | 2017-08-03 | -0/+11 |
| | ||||
* | SPIGOT-3485: Launch shulker bullets as projectiles | md_5 | 2017-08-03 | -0/+7 |
| | ||||
* | SPIGOT-3487: Llama doesn't drop chest | md_5 | 2017-08-03 | -0/+19 |
| | ||||
* | SPIGOT-3482: ProjectileHitEvent for shulker bullet and llama spit | md_5 | 2017-08-02 | -0/+16 |
| | ||||
* | SPIGOT-3456: Expand calling of ServerCommandEvent | md_5 | 2017-07-28 | -4/+13 |
| | | | | Existing usage preserved by checking sender. | |||
* | SPIGOT-3403: Add an EntityPickupItemEvent | md_5 | 2017-07-28 | -25/+56 |
| | ||||
* | SPIGOT-3463: Spurious item drops from event | md_5 | 2017-07-28 | -69/+104 |
| | ||||
* | Implement BroadcastMessageEvent | BillyGalbreath | 2017-07-28 | -8/+18 |
| | ||||
* | SPIGOT-3461: Standardise plugin load timing | md_5 | 2017-07-26 | -11/+12 |
| | ||||
* | Correct scheduler behaviour with intervals of >= Integer.MAX_VALUE (circa 3 ↵ | md_5 | 2017-07-25 | -2/+2 |
| | | | | years). | |||
* | SPIGOT-3432: Add playNote support for new sounds | md_5 | 2017-07-24 | -17/+62 |
| | ||||
* | SPIGOT-3440: Non player entities should not make new portals | md_5 | 2017-07-22 | -7/+12 |
| | ||||
* | SPIGOT-3433: Improve AreaEffectCloud#getSource | md_5 | 2017-07-22 | -11/+12 |
| | ||||
* | SPIGOT-3428: Recover from invalid firework data | md_5 | 2017-07-19 | -1/+9 |
| | ||||
* | Update Eclipse compiler. | md_5 | 2017-07-17 | -6/+3 |
| | ||||
* | Fix memory leak in PlayerList | md_5 | 2017-07-17 | -35/+139 |
| | ||||
* | SPIGOT-3418: Packaging error causing wrong netty version to be used | md_5 | 2017-07-16 | -7/+7 |
| | ||||
* | SPIGOT-3425: Correct slot type calculation | md_5 | 2017-07-16 | -1/+1 |
| | ||||
* | SPIGOT-3423: Don't treat technical piston block as BlockStateMeta | md_5 | 2017-07-16 | -3/+2 |
| | ||||
* | SPIGOT-3417: Event for shearing snowman | md_5 | 2017-07-15 | -7/+22 |
| | ||||
* | SPIGOT-3416: Cap team prefix / suffix at correct value of 16 | md_5 | 2017-07-13 | -2/+2 |
| | ||||
* | SPIGOT-3409: Improve performance of registering stupid amounts of ↵ | md_5 | 2017-07-10 | -3/+4 |
| | | | | permissions in plugin.yml | |||
* | SPIGOT-3407: Set damager for evoker fangs | md_5 | 2017-07-08 | -0/+14 |
| | ||||
* | Update to Netty 4.1.13.Final. | md_5 | 2017-07-07 | -1/+1 |
| | ||||
* | Increase expiration time to two weeks. | md_5 | 2017-07-04 | -1/+1 |
| | ||||
* | SPIGOT-3387: Prevent null enchantments in ItemMeta | BlackHole | 2017-06-29 | -0/+4 |
| | ||||
* | SPIGOT-3382: Ensure players can pickup stuff | md_5 | 2017-06-26 | -1/+1 |
| | ||||
* | SPIGOT-3377: Fix subspace bubble advancement | md_5 | 2017-06-26 | -1/+1 |
| | ||||
* | SPIGOT-3381: Save older serialized enchants | md_5 | 2017-06-26 | -1/+6 |
| | ||||
* | SPIGOT-3379: Generalise canPickupItems to other entities. | md_5 | 2017-06-26 | -32/+40 |
| | ||||
* | SPIGOT-3376: Fix nested functions | md_5 | 2017-06-26 | -28/+42 |
| | ||||
* | SPIGOT-3373: Fix /execute in gameloop functions | md_5 | 2017-06-25 | -5/+23 |
| | ||||
* | SPIGOT-3324: Improve tameable damage handling | md_5 | 2017-06-24 | -28/+29 |
| | ||||
* | SPIGOT-3368: Allow anvil level cost to be overwritten by properties | md_5 | 2017-06-23 | -9/+14 |
| | ||||
* | SPIGOT-3336: HideFlags will now serialize as a List rather than a Set. | Jeremy Wood | 2017-06-23 | -2/+2 |
| | | | | | | HideFlags will be serialized as a list and deserialized as an Iterable. This will allow maximum flexibility on both the formats it can be serialized into and formats it can be deserialized from. | |||
* | SPIGOT-3364: Correct EntityResurrectEvent | md_5 | 2017-06-23 | -1/+10 |
| | ||||
* | Treat Vanilla commands as Bukkit Commands | md_5 | 2017-06-21 | -2/+2 |
| | ||||
* | Fix nether world border center not adjusting | md_5 | 2017-06-18 | -0/+31 |
| | ||||
* | SPIGOT-3351: Replace bundled netty with 4.1.12.Final | md_5 | 2017-06-18 | -0/+7 |
| | ||||
* | Increase expire time to 7 days | md_5 | 2017-06-17 | -1/+1 |
| | ||||
* | SPIGOT-3329: Allow per world loot tables | md_5 | 2017-06-15 | -6/+5 |
| | ||||
* | SPIGOT-3342: Fix invalid usage of null in InventoryWrapper | md_5 | 2017-06-15 | -6/+6 |
| | ||||
* | SPIGOT-3337: Use special data value to indicate multiple recipe choices | md_5 | 2017-06-15 | -2/+2 |
| | ||||
* | Fix detection of missing or invalid tile entities for End portals and gateways | Jacob Martin | 2017-06-12 | -2/+2 |
| | ||||
* | Add PistonMoveReaction getters for Entity | md_5 | 2017-06-12 | -0/+6 |
| |