Skip to content

Commit

Permalink
fix release sign => it is not dependant on a craft existing!
Browse files Browse the repository at this point in the history
  • Loading branch information
DerToaster98 committed Aug 29, 2024
1 parent 68c62a1 commit 92141d4
Showing 1 changed file with 13 additions and 17 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -8,20 +8,10 @@
import org.bukkit.event.block.SignChangeEvent;
import org.jetbrains.annotations.Nullable;

public class ReleaseSign extends AbstractCraftSign {
public class ReleaseSign extends AbstractMovecraftSign {

public ReleaseSign() {
super(true);
}

@Override
protected void onCraftIsBusy(Player player, Craft craft) {

}

@Override
protected void onCraftNotFound(Player player, AbstractSignListener.SignWrapper sign) {

super(null);
}

@Override
Expand All @@ -38,13 +28,19 @@ protected boolean isSignValid(Action clickType, AbstractSignListener.SignWrapper
}

@Override
public boolean processSignChange(SignChangeEvent event, AbstractSignListener.SignWrapper sign) {
return false;
protected boolean internalProcessSign(Action clickType, AbstractSignListener.SignWrapper sign, Player player, @Nullable Craft craft) {
if (craft == null) {
craft = CraftManager.getInstance().getCraftByPlayer(player);
}
if (craft != null) {
CraftManager.getInstance().release(craft, CraftReleaseEvent.Reason.PLAYER, false);
}
return true;
}

@Override
protected boolean internalProcessSignWithCraft(Action clickType, AbstractSignListener.SignWrapper sign, Craft craft, Player player) {
CraftManager.getInstance().release(craft, CraftReleaseEvent.Reason.PLAYER, false);
return true;
public boolean processSignChange(SignChangeEvent event, AbstractSignListener.SignWrapper sign) {
return false;
}

}

0 comments on commit 92141d4

Please sign in to comment.