Skip to content

Added block data to FakeBlockAction and ShowGen, so BlockStates are saved and loaded. #1

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.

Sign up for GitHub

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

Open
KyGuy2002 wants to merge 9 commits into
base: staging
Choose a base branch
from
Open

Added block data to FakeBlockAction and ShowGen, so BlockStates are saved and loaded. #1

Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
e9d278d
Added fake block data
KyGuy2002 Apr 19, 2022
ffd6701
Running show fully working, showgen untested but implemented
KyGuy2002 Apr 19, 2022
2c9d378
Added multiple faces to glass and fence, showgen untested
KyGuy2002 Apr 19, 2022
55e049e
Showgen finished, fixed updater message on join not registered
KyGuy2002 Apr 19, 2022
a31e33e
updated version number and removed unused import
KyGuy2002 Apr 19, 2022
196ce79
deleted more imports
KyGuy2002 Apr 19, 2022
153a1b5
added slabs
KyGuy2002 Apr 19, 2022
e1d7348
Fence and glasspane faces easier syntax, blockstate own package, othe…
KyGuy2002 Apr 22, 2022
9af1bfe
changed default bottom from false to true. This seems to fix all issu…
KyGuy2002 Apr 22, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 10 additions & 4 deletions dependency-reduced-pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
<modelVersion>4.0.0</modelVersion>
<groupId>network.palace</groupId>
<artifactId>Show</artifactId>
<version>1.6.0</version>
<version>1.6.1</version>
<build>
<finalName>${project.artifactId}</finalName>
<plugins>
Expand Down Expand Up @@ -82,7 +82,7 @@
<dependency>
<groupId>org.projectlombok</groupId>
<artifactId>lombok</artifactId>
<version>1.18.20</version>
<version>1.18.22</version>
<scope>compile</scope>
</dependency>
<dependency>
Expand All @@ -96,11 +96,17 @@
<artifactId>worldedit-bukkit</artifactId>
<version>7.2.0-SNAPSHOT</version>
<scope>provided</scope>
<exclusions>
<exclusion>
<artifactId>bstats-bukkit</artifactId>
<groupId>org.bstats</groupId>
</exclusion>
</exclusions>
</dependency>
<dependency>
<groupId>io.netty</groupId>
<artifactId>netty-buffer</artifactId>
<version>4.1.51.Final</version>
<version>4.1.75.Final</version>
<scope>compile</scope>
</dependency>
<dependency>
Expand All @@ -112,7 +118,7 @@
<dependency>
<groupId>com.craftmend.openaudiomc</groupId>
<artifactId>OpenAudioMc</artifactId>
<version>6.5.4</version>
<version>6.6.3</version>
<scope>provided</scope>
</dependency>
</dependencies>
Expand Down
16 changes: 11 additions & 5 deletions pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

<groupId>network.palace</groupId>
<artifactId>Show</artifactId>
<version>1.6.0</version>
<version>1.6.1</version>
<repositories>
<repository>
<id>enginehub-maven</id>
Expand All @@ -32,7 +32,7 @@
<dependency>
<groupId>org.projectlombok</groupId>
<artifactId>lombok</artifactId>
<version>1.18.20</version>
<version>1.18.22</version>
<type>jar</type>
</dependency>
<dependency>
Expand All @@ -45,6 +45,12 @@
<artifactId>worldedit-bukkit</artifactId>
<version>7.2.0-SNAPSHOT</version>
<scope>provided</scope>
<exclusions>
<exclusion>
<groupId>org.bstats</groupId>
<artifactId>bstats-bukkit</artifactId>
</exclusion>
</exclusions>
</dependency>
<dependency>
<groupId>com.goebl</groupId>
Expand All @@ -61,7 +67,7 @@
<dependency>
<groupId>io.netty</groupId>
<artifactId>netty-buffer</artifactId>
<version>4.1.51.Final</version>
<version>4.1.75.Final</version>
</dependency>
<dependency>
<groupId>org.spigotmc</groupId>
Expand All @@ -71,13 +77,13 @@
<dependency>
<groupId>com.craftmend.openaudiomc</groupId>
<artifactId>OpenAudioMc</artifactId>
<version>6.5.4</version>
<version>6.6.3</version>
<scope>provided</scope>
</dependency>
<dependency>
<groupId>org.bstats</groupId>
<artifactId>bstats-bukkit</artifactId>
<version>2.2.1</version>
<version>3.0.0</version>
<scope>compile</scope>
</dependency>
</dependencies>
Expand Down
10 changes: 6 additions & 4 deletions src/main/java/network/palace/show/ShowPlugin.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
import org.bukkit.configuration.file.FileConfiguration;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.plugin.PluginLoadOrder;
import org.bukkit.plugin.java.JavaPlugin;
Expand All @@ -37,7 +38,7 @@
* Created by Marc on 12/6/16.
* Updated to be Core free by Tom 07/10/2021
*/
@Plugin(name = "Show", version = "1.6.0")
@Plugin(name = "Show", version = "1.6.1")
@Description(value = "Create Shows in Minecraft with easy to use files!")
@LoadOrder(value = PluginLoadOrder.POSTWORLD)
@Author(value = "Legobuilder0813")
Expand All @@ -52,7 +53,7 @@
@Command(name = "show", desc = "Main show command", permission = "show.main", permissionMessage = "You do not have permission!", usage = "/show [list|start|stop]")
@Command(name = "showdebug", desc = "Showdebug command", permission = "show.debug", permissionMessage = "You do not have permission!", usage = "/showdebug")
@Command(name = "showgen", desc = "Showgen commands", permission = "show.showgen", permissionMessage = "You do not have permission!", usage = "/showgen")
public class ShowPlugin extends JavaPlugin {
public class ShowPlugin extends JavaPlugin implements Listener {
@Getter private ArmorStandManager armorStandManager;
@Getter private FountainManager fountainManager;
@Getter private static ShowGenerator showGenerator;
Expand Down Expand Up @@ -103,13 +104,14 @@ public void onEnable() {
}


//has to be loaded after github token
// Has to be loaded after github token
showGenerator = new ShowGenerator();


this.getServer().getPluginManager().registerEvents(fountainManager, this);
this.getServer().getPluginManager().registerEvents(new PlayerInteract(), this);
this.getServer().getPluginManager().registerEvents(new ChunkListener(), this);
this.getServer().getPluginManager().registerEvents(this, this);

// Show Ticker
taskid = Bukkit.getScheduler().runTaskTimer(this, () -> {
Expand All @@ -128,7 +130,7 @@ public void onEnable() {

new UpdateUtil(this, 94141).getVersion(v -> {
if (!this.getDescription().getVersion().equalsIgnoreCase(v)) {
Bukkit.getConsoleSender().sendMessage(ChatColor.RED + "[Show] A New update is available for Show! It is always recommended that you upgrade! Link: https://www.spigotmc.org/resources/show-make-huge-spectaculars-in-minecraft.94141/");
Bukkit.getConsoleSender().sendMessage(ChatColor.AQUA + "[Show] " + ChatColor.RED + "A New update is available for Show! It is always recommended that you upgrade! Link: https://www.spigotmc.org/resources/show-make-huge-spectaculars-in-minecraft.94141/");
}
});

Expand Down
2 changes: 0 additions & 2 deletions src/main/java/network/palace/show/actions/BlockAction.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,6 @@

import network.palace.show.Show;
import network.palace.show.exceptions.ShowParseException;
import network.palace.show.handlers.BlockData;
import network.palace.show.utils.ShowUtil;
import network.palace.show.utils.WorldUtil;
import org.bukkit.Location;
import org.bukkit.Material;
Expand Down
24 changes: 8 additions & 16 deletions src/main/java/network/palace/show/actions/FakeBlockAction.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,20 +1,15 @@
package network.palace.show.actions;

import com.comphenix.protocol.wrappers.BlockPosition;
import com.comphenix.protocol.wrappers.WrappedBlockData;
import lombok.Getter;
import lombok.Setter;
import network.palace.show.packets.server.block.WrapperPlayServerBlockChange;
import network.palace.show.Show;
import network.palace.show.exceptions.ShowParseException;
import network.palace.show.handlers.BlockData;
import network.palace.show.utils.MiscUtil;
import network.palace.show.utils.ShowUtil;
import network.palace.show.utils.WorldUtil;
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.block.data.BlockData;
import org.bukkit.entity.Player;

/**
Expand All @@ -25,30 +20,27 @@
@SuppressWarnings("deprecation")
public class FakeBlockAction extends ShowAction {
private Location loc;
private Material mat;
private BlockData data;

public FakeBlockAction(Show show, long time) {
super(show, time);
}

public FakeBlockAction(Show show, long time, Location loc, Material mat) {
public FakeBlockAction(Show show, long time, Location loc, BlockData data) {
super(show, time);
this.loc = loc;
this.mat = mat;
this.data = data;
}

@Override
public void play(Player[] nearPlayers) {
try {
WrapperPlayServerBlockChange p = new WrapperPlayServerBlockChange();
p.setLocation(new BlockPosition(loc.getBlockX(), loc.getBlockY(), loc.getBlockZ()));
p.setBlockData(WrappedBlockData.createData(mat));
for (Player tp : nearPlayers) {
if (tp != null) MiscUtil.sendPacket(p, tp);
if (tp != null) tp.sendBlockChange(loc, data);
}
} catch (Exception e) {
Bukkit.getLogger().severe("FakeBlockAction -" + ChatColor.RED + "Error sending FakeBlockAction for type (" +
mat.toString() + ") at location " + loc.getX() + "," + loc.getY() + "," + loc.getZ() + " at time " +
data + ") at location " + loc.getX() + "," + loc.getY() + "," + loc.getZ() + " at time " +
time + " for show " + show.getName());
e.printStackTrace();
}
Expand All @@ -62,7 +54,7 @@ public ShowAction load(String line, String... args) throws ShowParseException {
}
try {
this.loc = loc;
this.mat = Material.valueOf(args[2].toUpperCase());
this.data = ShowUtil.getBlockData(line);
} catch (IllegalArgumentException e) {
throw new ShowParseException(e.getMessage());
}
Expand All @@ -71,6 +63,6 @@ public ShowAction load(String line, String... args) throws ShowParseException {

@Override
protected ShowAction copy(Show show, long time) throws ShowParseException {
return new FakeBlockAction(show, time, loc, mat);
return new FakeBlockAction(show, time, loc, data);
}
}
2 changes: 0 additions & 2 deletions src/main/java/network/palace/show/actions/FountainAction.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,7 @@
import network.palace.show.Show;
import network.palace.show.ShowPlugin;
import network.palace.show.exceptions.ShowParseException;
import network.palace.show.handlers.BlockData;
import network.palace.show.handlers.Fountain;
import network.palace.show.utils.ShowUtil;
import network.palace.show.utils.WorldUtil;
import org.bukkit.Location;
import org.bukkit.Material;
Expand Down
11 changes: 6 additions & 5 deletions src/main/java/network/palace/show/commands/showgen/GenerateCommand.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -109,25 +109,26 @@ public void handle(CommandSender sender, String[] args) {
changeInTime = delayPerLayer;
}


// y is startingY, and is incremented by yChange every loop until y is equal to endingY.
for (int y = startingY; compare(y, endingY, !bottom); y += yChange) {
for (int x = 0; x < initialScene.getXLength(); x++) {
for (int z = 0; z < initialScene.getZLength(); z++) {
Block oldBlock = initialScene.getBlock(x, y, z);
Block newBlock = finalScene.getBlock(x, y, z);
if (newBlock.getType().equals(oldBlock.getType()) && newBlock.getData() == oldBlock.getData()) {
if (newBlock.getType().equals(oldBlock.getType()) && newBlock.getBlockData().matches(oldBlock.getBlockData())) {
continue;
}
Material material = newBlock.getType();
byte data = newBlock.getData();
FakeBlockAction act = new FakeBlockAction(null, (long) (localTime * 1000));
act.setMat(material);
act.setData(newBlock.getBlockData());
act.setLoc(new Location(corner.getWorld(), corner.getBlockX() + x, corner.getBlockY() + y, corner.getBlockZ() + z));
actions.add(act);
}
}
localTime += changeInTime;
}


if (actions.isEmpty()) {
player.sendMessage(ChatColor.RED + "There aren't any differences between the two selected regions!");
return;
Expand Down Expand Up @@ -157,6 +158,6 @@ public void handle(CommandSender sender, String[] args) {
}

private boolean compare(int x, int y, boolean inverted) {
return inverted == (x >= y);
return inverted == (x <= y);
}
}
Loading