Compact code and remove unused use statements

Correct PhpDoc comments
All spacing is not the same
Version bump due to new feature and BC breaks
This commit is contained in:
jasonwynn10 2017-08-01 14:41:59 -04:00
parent 84868ea7c7
commit 42a28033ed
6 changed files with 136 additions and 237 deletions

View file

View file

@ -1,9 +1,11 @@
---
name: Elytra name: Elytra
author: Ad5001 author: Ad5001
version: 2.0 version: 2.1
api: [3.0.0-ALPHA1, 3.0.0, 2.0.0] api:
- 3.0.0
- 3.0.0-ALPHA7
main: Ad5001\Elytra\Main main: Ad5001\Elytra\Main
load: STARTUP
commands: commands:
opelytra: opelytra:
description: "Make your elytras your wear MAGIC !" description: "Make your elytras your wear MAGIC !"
@ -18,5 +20,4 @@ permissions:
elytra.getopelytra: elytra.getopelytra:
default: op default: op
elytra.boost: elytra.boost:
default: true default: true
...

View file

@ -1,5 +1,4 @@
# Welcome to Elytras 2.x.x config ! # Welcome to Elytra 2.x.x config !
# Here configure which block are considered "bouncable", enter their ids. # Here configure which block are considered "bouncable", enter their ids.
# Try going on those blocks by usini the elytra and the see the magic. # Try going on those blocks by using the elytra and the see the magic.
bouncable_blocks: [165, 88] bouncable_blocks: [165, 88]

View file

@ -1,24 +1,11 @@
<?php <?php
namespace Ad5001\Elytra; namespace Ad5001\Elytra;
use pocketmine\item\Armor; use pocketmine\item\Armor;
class Elytra extends Armor{ class Elytra extends Armor{
public function __construct($meta = 0, $count = 1){ public function __construct($meta = 0, $count = 1){
parent::__construct(444, $meta, $count, "Elytra Wings"); parent::__construct(444, $meta, $count, "Elytra Wings");
} }
}
public function getArmorType(){
return Armor::TYPE_CHESTPLATE;
}
public function getMaxDurability(){
return 431;
}
public function isChestplate(){
return true;
}
}

View file

@ -1,154 +1,128 @@
<?php <?php
namespace Ad5001\Elytra; namespace Ad5001\Elytra;
use pocketmine\command\CommandSender;
use pocketmine\command\Command; use pocketmine\command\Command;
use pocketmine\command\CommandSender;
use pocketmine\event\entity\EntityDamageEvent;
use pocketmine\event\Listener; use pocketmine\event\Listener;
use pocketmine\event\player\PlayerKickEvent;
use pocketmine\event\player\PlayerMoveEvent;
use pocketmine\plugin\PluginBase; use pocketmine\event\server\DataPacketReceiveEvent;
use pocketmine\Server;
use pocketmine\Player;
use pocketmine\item\Item; use pocketmine\item\Item;
use pocketmine\math\Vector3;
use pocketmine\network\mcpe\protocol\PlayerActionPacket;
use pocketmine\nbt\tag\CompoundTag; use pocketmine\plugin\PluginBase;
use pocketmine\nbt\tag\StringTag; use pocketmine\Player;
use pocketmine\utils\TextFormat;
use pocketmine\utils\BlockIterator;
use pocketmine\item\enchantment\Enchantment;
use Ad5001\Elytra\tasks\AdminGotoTask; use Ad5001\Elytra\tasks\AdminGotoTask;
class Main extends PluginBase implements Listener { class Main extends PluginBase implements Listener {
protected $ops; protected $ops = [];
/* public function onEnable() {
Called when the plugin enables $this->saveDefaultConfig();
*/ $this->getServer()->getPluginManager()->registerEvents($this, $this);
public function onEnable() { $this->getServer()->getScheduler()->scheduleRepeatingTask(new AdminGotoTask($this), 10);
$this->getServer()->getPluginManager()->registerEvents($this,$this);
$this->getServer()->getScheduler()->scheduleRepeatingTask(new AdminGotoTask($this), 10);
Item::$list[444] = Elytra::class; Item::$list[444] = Elytra::class;
Item::addCreativeItem(new Elytra()); Item::addCreativeItem(new Elytra());
$this->ops = []; }
}
/**
* @priority LOW
*
* @param EntityDamageEvent $event
*/
public function onEntityDamage(EntityDamageEvent $event) {
$damaged = $event->getEntity();
if($damaged instanceof Player) {
if($event->getCause() == 4 and $damaged->getInventory()->getChestplate()->getId() == 444) {
$event->setCancelled();
}
}
}
/* /**
Prevent when someone is falling * @priority LOW
@param $event \pocketmine\event\entity\EntityDamageEvent *
@return null * @param PlayerKickEvent $event
*/ */
public function onEntityDamage(\pocketmine\event\entity\EntityDamageEvent $event) { public function onPlayerKick(PlayerKickEvent $event) {
if($event->getCause() == 4 && $event->getEntity()->getInventory()->getChestplate()->getId() == 444) { if(strpos(strtolower($event->getReason()), "flying") !== false and $event->getPlayer()->getInventory()->getChestplate()->getId() == 444) {
$event->setCancelled(); $event->setCancelled();
} }
} }
/**
* @priority LOW
*
* @param PlayerMoveEvent $event
*/
public function onPlayerMove(PlayerMoveEvent $event) {
$player = $event->getPlayer();
if($player->getInventory()->getChestplate()->getId() == 444) {
$flyingUp = false;
// TODO: change Bounding Box of player depending on their angle of flight
for($i = 2; $i > 0; $i--) {
if($player->getLevel()->getBlock(new Vector3(round($player->x), round($player->y) - $i, round($player->z)))->getId() !== 0) {
$flyingUp = true;
}
}
if(isset($this->ops[$player->getName()]) and $flyingUp) {
$player->setMotion(new Vector3($player->getMotion()->x, 3, $player->getMotion()->z));
}
$flyingUp = false;
for($i = 4; $i > 0; $i--) {
$id = $player->getLevel()->getBlock(new Vector3 (round($player->x), round($player->y) - $i, round($player->z)))->getId();
if(in_array($id, $this->getConfig()->get("bouncable_blocks",[]))) {
$flyingUp = true;
}
}
if($flyingUp) {
$player->setMotion(new Vector3($player->getMotion()->x, 3, $player->getMotion()->z));
}
}
}
/* public function onDataPacket(DataPacketReceiveEvent $ev) {
Prevents the player from being kicked of flyign by using the elytras. $packet = $ev->getPacket();
@param $event \pocketmine\event\player\PlayerKickEvent if($packet instanceof PlayerActionPacket) {
*/ if($packet->action === PlayerActionPacket::ACTION_START_GLIDE) {
public function onPlayerKick(\pocketmine\event\player\PlayerKickEvent $event) { $ev->getPlayer()->setDataFlag(Player::DATA_FLAGS, Player::DATA_FLAG_GLIDING, true, Player::DATA_TYPE_BYTE);
if(strpos($event->getReason(), "Flying is not enabled on this server") !== false && $event->getPlayer()->getInventory()->getChestplate()->getId() == 444) { }elseif($packet->action === PlayerActionPacket::ACTION_STOP_GLIDE) {
$event->setCancelled(); $ev->getPlayer()->setDataFlag(Player::DATA_FLAGS, Player::DATA_FLAG_GLIDING, false, Player::DATA_TYPE_BYTE);
} }
} }
}
/*
When a player moves. To make it bounce with elytras.
@param $event \pocketmine\event\player\PlayerMoveEvent
*/
public function onPlayerMove(\pocketmine\event\player\PlayerMoveEvent $event) {
$player = $event->getPlayer();
if($player->getInventory()->getChestplate()->getId() == 444) {
$flyingup = false;
for($i = 2; $i > 0; $i--) {
if($player->getLevel()->getBlock(new \pocketmine\math\Vector3 (round($player->x), round($player->y) - $i, round($player->z)))->getId() !== 0) {
$flyingup = true;
}
}
if(isset($this->getAdminsModePlayers()[$player->getName()]) && $flyingup) {
$player->setMotion(new \pocketmine\math\Vector3($player->getMotion()->x, 3, $player->getMotion()->z));
}
$flyingup = false;
for($i = 4; $i > 0; $i--) {
$id = $player->getLevel()->getBlock(new \pocketmine\math\Vector3 (round($player->x), round($player->y) - $i, round($player->z)))->getId();
if(in_array($id, $this->getConfig()->get("bouncable_blocks"))) {
$flyingup = true;
}
}
if($flyingup) {
$player->setMotion(new \pocketmine\math\Vector3($player->getMotion()->x, 3, $player->getMotion()->z));
}
}
}
/*
Called when one of the defined commands of the plugin has been called
@param $sender \pocketmine\command\CommandSender
@param $cmd \pocketmine\command\Command
@param $label mixed
@param $args array
return bool
*/
public function onCommand(\pocketmine\command\CommandSender $sender, \pocketmine\command\Command $cmd,$label, array $args): bool {
switch($cmd->getName()) {
case "opelytra":
if($sender instanceof Player) {
if(isset($this->ops[$sender->getName()])) {
unset($this->ops[$sender->getName()]);
$sender->sendMessage("§aYou are back to te original elytra !");
} else {
$this->ops[$sender->getName()] = true;
$sender->sendMessage("§aYou are now in the admin elytra mode ! Go try out your powers !");
}
}
break;
case "boost":
if($sender instanceof Player && $sender->getInventory()->getChestplate()->getId() == 444) {
if(!isset($args[0])) $args[0] = 2;
$sender->setMotion(new \pocketmine\math\Vector3($sender->getMotion()->x, $args[0], $sender->getMotion()->z));
}
break;
}
return false;
}
/*
Returns players in ADMIN mode
@return array
*/
public function getAdminsModePlayers() : array {
return $this->ops;
}
/**
* @param CommandSender $sender
* @param Command $cmd
* @param string $label
* @param array $args
* @return bool
*/
public function onCommand(CommandSender $sender, Command $cmd, string $label, array $args) : bool {
switch($cmd->getName()) {
case "opelytra":
if($sender instanceof Player) {
if(isset($this->ops[$sender->getName()])) {
unset($this->ops[$sender->getName()]);
$sender->sendMessage(TextFormat::GREEN."You are back to the original elytra!");
} else {
$this->ops[$sender->getName()] = true;
$sender->sendMessage(TextFormat::GREEN."You are now in the admin elytra mode! Go try out your powers!");
}
}
break;
case "boost":
if($sender instanceof Player and $sender->getInventory()->getChestplate()->getId() == 444) {
if(!isset($args[0])) $args[0] = 2;
$sender->setMotion(new Vector3($sender->getMotion()->x, $args[0], $sender->getMotion()->z));
}
break;
}
return false;
}
} }

View file

@ -1,88 +1,26 @@
<?php <?php
namespace Ad5001\Elytra\tasks; namespace Ad5001\Elytra\tasks;
use pocketmine\Server;
use pocketmine\scheduler\PluginTask; use pocketmine\scheduler\PluginTask;
use pocketmine\utils\BlockIterator;
use pocketmine\Player;
use Ad5001\Elytra\Main; use Ad5001\Elytra\Main;
class AdminGotoTask extends PluginTask { class AdminGotoTask extends PluginTask {
public function __construct(Main $main) {
parent::__construct($main);
}
public function __construct(Main $main) { public function onRun(int $tick) {
foreach ($this->getOwner()->getServer()->getOnlinePlayers() as $player) {
//Part needed for players to fly upwards
parent::__construct($main); $ref = new \ReflectionClass("pocketmine\\Player");
$prop = $ref->getProperty("gravity");
$prop->setAccessible(true);
$this->main = $main; $prop->setValue($player, 0);
$prop->setAccessible(false);
if($player->getMotion()->y !== 0) {
$this->server = $main->getServer(); $this->getOwner()->getLogger()->debug("{$player->getName()}'s y motion is {$player->getMotion()->y}");
}
}
} }
public function onRun($tick) {
foreach ($this->server->getOnlinePlayers() as $player) {
//Part needed for player's good working
$ref = new \ReflectionClass("pocketmine\\Player");
$prop = $ref->getProperty("gravity");
$prop->setAccessible(true);
$prop->setValue($player, 0);
$prop->setAccessible(false);
if($player->getMotion()->y !== 0) {
// echo "{$player->getName()}:" . $player->getMotion()->y . "\n";
}
}
}
/*
Inverts a number
@param $num int
@return int
*/
public function invert(int $num) : int {
if($num < 0) {
echo $num . " +> " . abs($num) . "\n";
return abs($num);
} else {
echo $num . " -> " . -$num . "\n";
return -$num;
}
return 0;
}
} }