Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Additions for improved BlockType-system (adds hay, ender-portal-frame, improved lever-facing) #60

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

Conversation

Tonodus
Copy link

@Tonodus Tonodus commented Nov 27, 2014

This changes are needed for this PR: GlowstoneMC/Glowstone#566 (improved Block/ItemType-System).

Adds:

  • EnderPortalFrame-MaterialData
  • HayBlock-MaterialData

Modifies:

  • Lever-MaterialData: setFacingDirection takes now notice of BlockFace.UP/DOWN. Old behaviour was to ignore this values silenty.

Edited by turt2live

@@ -102,47 +102,76 @@ public void setFacingDirection(BlockFace face) {
byte data = (byte) (getData() & 0x8);
BlockFace attach = getAttachedFace();

if (attach == BlockFace.DOWN) {

Choose a reason for hiding this comment

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

What is going on here? There appears to be several changes that do not need to be made

@turt2live
Copy link

@Tonodus Your PR has several formatting concerns. Please see the repository's contributing guidelines.

@turt2live turt2live self-assigned this Nov 27, 2014
@Tonodus Tonodus force-pushed the BBT branch 3 times, most recently from e9470b8 to 236c873 Compare November 29, 2014 13:15
@Tonodus
Copy link
Author

Tonodus commented Nov 29, 2014

@turt2live I hope I resolved the formatting concerns.

import org.bukkit.block.BlockFace;
import org.bukkit.Material;

public class EnderPortalFrame extends MaterialData implements Directional {

Choose a reason for hiding this comment

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

Missing javadocs

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants