Solved Problem with my personal plugin.

Hi guys, I'm doing a plugin that send messages when players join in my server but I have got this crash error in the console when the players join:

Spoiler: Crash Dump

PocketMine-MP Crash Dump Mon Jul 13 16:48:45 UTC 2015
Error: Call to a member function sendMessage() on string
File: /ItalianMOTD_v1.0.0.phar/src/matcracker/Main
Line: 21
Type: E_ERROR
THIS CRASH WAS CAUSED BY A PLUGIN
BAD PLUGIN: ItalianMOTD v1.0.0
Code:
Backtrace:
#0 /src/pocketmine/Server__32bit(0): pocketmine\Server->crashDump()
#1 /src/pocketmine/plugin/MethodEventExecutor(36): matcracker\Main->onPlayerJoin()
#2 /src/pocketmine/plugin/RegisteredListener(96): pocketmine\plugin\MethodEventExecutor->execute()
#3 /src/pocketmine/plugin/PluginManager(673): pocketmine\plugin\RegisteredListener->callEvent()
#4 /src/pocketmine/Player__32bit(757): pocketmine\plugin\PluginManager->callEvent()
#5 /src/pocketmine/Player__32bit(717): pocketmine\Player->doFirstSpawn()
#6 /src/pocketmine/Player__32bit(1561): pocketmine\Player->sendNextChunk()
#7 /src/pocketmine/Server__32bit(2506): pocketmine\Player->checkNetwork()
#8 /src/pocketmine/Server__32bit(2306): pocketmine\Server->tick()
#9 /src/pocketmine/Server__32bit(2184): pocketmine\Server->tickProcessor()
#10 /src/pocketmine/Server__32bit(1784): pocketmine\Server->start()
#11 /src/pocketmine/PocketMine(464): pocketmine\Server->__construct()
#12 (1): ()
#13 (0): {main}()

Spoiler: Console Error

Fatal error: Call to a member function sendMessage() on string in phar://C:/Users/Matt/Desktop/PocketMine-MultiServer/PocketMine-Servers_1.5/PocketMine-MP(Lobby)_1.5/plugins/NamePlugin_v1.0.0.phar/src/matcracker/Main.php on line 21
Call Stack:
0.0171 412104 1. {main}() C:\Users\Matt\Desktop\PocketMine-MultiServer\PocketMine-Servers_1.5\PocketMine-MP(Lobby)_1.5\PocketMine-MP.phar:0
0.0200 407456 2. require('phar://C:/Users/Matt/Desktop/PocketMine-MultiServer/PocketMine-Servers_1.5/PocketMine-MP(Lobby)_1.5/PocketMine-MP.phar/src/pocketmine/PocketMine.php') C:\Users\Matteo\Desktop\PocketMine-MultiServer\PocketMine-Servers_1.5\PocketMine-MP(Lobby)_1.5\PocketMine-MP.phar:1
0.0491 549016 3. pocketmine\Server->__construct() phar://C:/Users/Matt/Desktop/PocketMine-MultiServer/PocketMine-Servers_1.5/PocketMine-MP(Lobby)_1.5/PocketMine-MP.phar/src/pocketmine/PocketMine.php:464
7.2571 18704200 4. pocketmine\Server->start() phar://C:/Users/Matt/Desktop/PocketMine-MultiServer/PocketMine-Servers_1.5/PocketMine-MP(Lobby)_1.5/PocketMine-MP.phar/src/pocketmine/Server__32bit.php:1784
7.2605 18715688 5. pocketmine\Server->tickProcessor() phar://C:/Users/Matt/Desktop/PocketMine-MultiServer/PocketMine-Servers_1.5/PocketMine-MP(Lobby)_1.5/PocketMine-MP.phar/src/pocketmine/Server__32bit.php:2184
9.0557 28483624 6. pocketmine\Server->tick() phar://C:/Users/Matt/Desktop/PocketMine-MultiServer/PocketMine-Servers_1.5/PocketMine-MP(Lobby)_1.5/PocketMine-MP.phar/src/pocketmine/Server__32bit.php:2306
9.0638 28986136 7. pocketmine\Player->checkNetwork() phar://C:/Users/Matt/Desktop/PocketMine-MultiServer/PocketMine-Servers_1.5/PocketMine-MP(Lobby)_1.5/PocketMine-MP.phar/src/pocketmine/Server__32bit.php:2506
9.0638 28986136 8. pocketmine\Player->sendNextChunk() phar://C:/Users/Matt/Desktop/PocketMine-MultiServer/PocketMine-Servers_1.5/PocketMine-MP(Lobby)_1.5/PocketMine-MP.phar/src/pocketmine/Player__32bit.php:1561
9.0638 28986696 9. pocketmine\Player->doFirstSpawn() phar://C:/Users/Matt/Desktop/PocketMine-MultiServer/PocketMine-Servers_1.5/PocketMine-MP(Lobby)_1.5/PocketMine-MP.phar/src/pocketmine/Player__32bit.php:717
9.0696 28997952 10. pocketmine\plugin\PluginManager->callEvent() phar://C:/Users/Matt/Desktop/PocketMine-MultiServer/PocketMine-Servers_1.5/PocketMine-MP(Lobby)_1.5/PocketMine-MP.phar/src/pocketmine/Player__32bit.php:757
9.5017 29256472 11. pocketmine\plugin\RegisteredListener->callEvent() phar://C:/Users/Matt/Desktop/PocketMine-MultiServer/PocketMine-Servers_1.5/PocketMine-MP(Lobby)_1.5/PocketMine-MP.phar/src/pocketmine/plugin/PluginManager.php:673
9.5017 29256472 12. pocketmine\plugin\MethodEventExecutor->execute() phar://C:/Users/Matt/Desktop/PocketMine-MultiServer/PocketMine-Servers_1.5/PocketMine-MP(Lobby)_1.5/PocketMine-MP.phar/src/pocketmine/plugin/RegisteredListener.php:96
9.5017 29256560 13. matcracker\Main->onPlayerJoin() phar://C:/Users/Matt/Desktop/PocketMine-MultiServer/PocketMine-Servers_1.5/PocketMine-MP(Lobby)_1.5/PocketMine-MP.phar/src/pocketmine/plugin/MethodEventExecutor.php:36

My code is:

PHP:

<?php
namespace matcracker;
use pocketmine\Player;
use pocketmine\event\Listener;
use pocketmine\event\player\PlayerJoinEvent;
use pocketmine\plugin\PluginBase;
use pocketmine\utils\TextFormat;
use pocketmine\Server;
class Main extends PluginBase implements Listener{
public function onEnable(){
$this->getServer()->getPluginManager()->registerEvents($this, $this);
$this->getLogger()->info(TextFormat::GREEN."PluginName on!");
 
}
public function onPlayerJoin(PlayerJoinEvent $event){
$player = $event->getPlayer()->getName();
$player->sendMessage(TextFormat::BLUE."Hi, ".$player.TextFormat::BLUE."!");
$player->sendMessage(TextFormat::BLUE."Welcome in".TextFormat::GOLD." Server Name");
$player->sendMessage(TextFormat::BLUE."Official Server TeamSpeak: ".TextFormat::GOLD."Ip TeamSpeak");
$player->sendMessage(TextFormat::BLUE."Warps: ".TextFormat::GOLD."/warp");
$player->sendMessage(TextFormat::BLUE."Digit ".TextFormat::GOLD."/help ".TextFormat::BLUE."for the commands list.");
$player->sendMessage(TextFormat::BLUE."Players online: ");
}
public function onDisable(){
$this->getLogger()->info(TextFormat::RED."PluginName off!");
}
}


Another question, if possible. How can I add a list of players online in the servers?

PHP:

$player->sendMessage(TextFormat::BLUE."Players online: ".player1.", " .player2", etc... );


Thanks for helping me

$player = $event->getPlayer()->getName();
$p = $event->getPlayer();
$p->sendMessage(TextFormat::BLUE."Hi, ".$player.TextFormat::BLUE."!");

Code:

$player = $event->getPlayer()->getName();

Just remove ->getName()

Thanks to all! I do this:

PHP:

$player = $event->getPlayer()
$player->sendMessage(TextFormat::BLUE."Hi, ".$player->getName().TextFormat::BLUE."!");


And for the "Players Online", how can I do?

foreach($this->getServer()->getOnlinePlayers() as $player)

Are you trying to get the number of online players or list every name of every player?

The second

Im not 100% sure if this is the best method(probably isnt) but If it was me I would create a new file and as players join Id add their names to the file.
Then Id send a message to the Joining player the players in that list. I looked through the Api and thats all I got. Give it a shot! If you need help with code let me know. But that would be my method

I understand your way but I think there is a best way for do this.

The proper way is what @Xenturio said.

Therefore... If I understand well:

PHP:

$online = "";
foreach($this->getServer()->getOnlinePlayers() as $player){
  $online .= $player->getDisplayName() . ", ";
}
$player-> sendMessage ("Players Online: ".$online = substr($online, 0, -2));



I would prefer this. Looks a bit nicer to me, although not much difference.

PHP:

$online = implode(", ", array_map(function($player){
  return $player->getDisplayName();
}, $this->getServer()->getOnlinePlayers()));