Solved Sign Format

Hi,

I am thinking of how to change the Sign's Text? by lines.

thanks

https://github.com/EvolSoft/ServerKits/blob/master/src/ServerKits/EventListener.php#L144

PHP:

$signTile->setText("", "", "", "");

Oh Thanks now i understand

But it's Not working :/ this is the function:

PHP:

public function onSignCreate(SignChangeEvent $event){
$player = $event->getPlayer();
$sign = $event->getPlayer()->getLevel()->getTile($event->getBlock());
if($event->getBlock()->getID() == 323 || $event->getBlock()->getID() == 63 || $event->getBlock()->getID() == 68){
if($sign instanceof Sign){
$line_0 = $event->getLine(0);
$line_1 = $event->getLine(1);
$lvl = $player->getServer()->getLevelByName($line_1);
if($line_0 =="[SG]"){
if(empty($line_1) !== true){
if(!$player->getServer()->isLevelGenerated($line_1)){
$player->sendMessage("Faild, Not found.");
return false;
}//Level
if($player->getServer()->isLevelGenerated($line_1)){
$player->sendMessage("Created!.");
if($lvl instanceof Level){
foreach($lvl->getPlayers() as $p){
$sign->setText("§c[§l§6SG§r§c]", $line_1, "[" . $lvl->getPlayers() . '/10]', "§l§aTap To Join");
}
}
}
}//empty!
}//sg
}//sign
}//block
}//Event


I get no errors in console i get the success message but the sign won't change :/
Please help!

WTF 323 is not a block ID. Did you copy code, or did you hear about 323 somewhere?

Idk... But anyway there is an ||

Anyways learn how to indent your code, it hurts our eyes. *rubs eyes*

But.. What is wrong???

Lemme tell you what is id 323:
It is bad practice and I hate it

@PEMapModder will scream his head off at you for using 323 as a block ID. And use indents.

* @PEMapModder screams his head off at @Taha_The_Hacker for using 323 as a block ID*

Here:

PHP:

use pocketmine\level\Level;
use pocketmine\event\block\SignChangeEvent;

    public function onSignCreate(SignChangeEvent $event){
        $player = $event->getPlayer(); //Get the player
        $line_0 = $event->getLine(0); //The sign's line 1 (Despite the number)
        $line_1 = $event->getLine(1); //The sign's line 2
        if(strtolower($line_0) === "[sg]" && !empty($line_1)){ //Check if the line 1 is equal to [sg] (Since strtolower) and if line 1 is not empty
            $level = $this->getServer()->getLevelByName($line_1); //Get the level object with the name of $line_1
            if(!$this->getServer()->isLevelGenerated($line_1)){ //Check if level is generated
                $player->sendMessage("Failed, Not found."); //LOL
                return false; //Return a boolean value of false
            }
            $player->sendMessage("Created!"); //AGAIN LOL
            if($level instanceof Level){ //No need this but I am Bored
                $event->setLine("§c[§l§6SG§r§c]", $line_1, "[" . count($level->getPlayers()) . "/10]", "§l§aTap To Join"); //Set the text. BTW, better use TextFormat::**
            }
        }
    }

Correct me @Gamecrafter... I am not sure.

No need to return anything.

She/He (Whatever) had it inside, so just leave it

BTW, if it is a SignChangeEvent why would you need to check if it is a sign or not again? You messed up PlayerInteractEvent and SignChangeEvent.

Some people just blindly copy code, without understanding what it does. In fact I could give him a script that downloaded viruses and he could've blindly added it to his plugin.

Dang.

Does it work?

Nope.

Errors?PM me the whole code?