Skip to content
This repository has been archived by the owner on May 24, 2018. It is now read-only.

Fixed to prevent an error #83

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Lesmiscore
Copy link

This can prevent these errors:

2015-09-10 [22:21:15] [Server thread/CRITICAL]: "Could not pass event 'pocketmine\event\player\PlayerJoinEvent' to 'SimpleAuth v1.7.1': Illegal offset type on SimpleAuth\EventListener
2015-09-10 [22:21:15] [Server thread/WARNING]: RuntimeException: "Illegal offset type" (E_WARNING) in "/plugins/SimpleAuth_v1.7.1.phar/src/SimpleAuth/task/ShowMessageTask" at line 42

2015-09-10 [22:21:29] [Server thread/CRITICAL]: Unhandled exception executing command 'register [HIDDEN]' in register: Illegal offset type in unset
2015-09-10 [22:21:29] [Server thread/WARNING]: RuntimeException: "Illegal offset type in unset" (E_WARNING) in "/plugins/SimpleAuth_v1.7.1.phar/src/SimpleAuth/task/ShowMessageTask" at line 46
2015-09-10 [22:21:39] [Server thread/CRITICAL]: "Could not pass event 'pocketmine\event\player\PlayerQuitEvent' to 'SimpleAuth v1.7.1': Illegal offset type in unset on SimpleAuth\EventListener
2015-09-10 [22:21:39] [Server thread/WARNING]: RuntimeException: "Illegal offset type in unset" (E_WARNING) in "/plugins/SimpleAuth_v1.7.1.phar/src/SimpleAuth/task/ShowMessageTask" at line 46

This patch is very easy, but it wastes memory a little.

@@ -53,8 +53,10 @@ public function onRun($currentTick){
}

foreach($this->playerList as $player){
if($player==null){
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please follow the code style.

@cwren
Copy link

cwren commented Dec 4, 2015

please accept this patch and then fix the whitespace. The plugin is broken (at least on 13) without the patch.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants