falkirks / RankUp

A comically full-featured prison rankup plugin for PocketMine-MP
MIT License
19 stars 49 forks source link

RankUp doesn't check if perm managers are still linked #5

Closed accelerate0 closed 9 years ago

accelerate0 commented 9 years ago

ocketMine-MP Crash Dump Sun Jun 28 15:23:02 ACT 2015

Error: Call to a member function getGroup() on null File: /RankUp_v0.3.0.phar/src/rankup/rank/RankStore Line: 39 Type: E_ERROR

THIS CRASH WAS CAUSED BY A PLUGIN BAD PLUGIN: RankUp v0.3.0

Code: [30] } [31] /* [32] * @return \rankup\RankUp [33] / [34] public function getMain(){ [35] return $this->main; [36] } [37] public function getNextRank(Player $player){ [38] //TODO check if perm is linked [39] $group = $this->getMain()->getPermManager()->getGroup($player); [40] if($group !== false){ [41] $rank = $this->getRankByName($group); [42] if($rank !== false){ [43] if($rank->getOrder() == count($this->ranks)-1){ [44] return false; [45] } [46] else{ [47] return $this->ranks[$rank->getOrder()+1]; [48] } [49] }

Backtrace:

0 (): pocketmine\Server->crashDump()

PocketMine-MP version: 1.5dev #1254 [Protocol 27; API 1.12.0] Git commit: 0000000000000000000000000000000000000000 uname -a: Linux Apocalypse-pe.com 2.6.32-042stab093.5 #1 SMP Wed Sep 10 17:39:49 MSK 2014 x86_64 PHP Version: 5.6.2 Zend version: 2.6.0 OS : Linux, linux

Loaded plugins: FactionsPro 1.3.2 by Tethered_ for API(s) 1.12.0 MagicTelePortal 1.3.2 by aliuly for API(s) 1.10.0 DevTools 1.10.0 by PocketMine Team for API(s) 1.3.1 BamBam 1 by Array for API(s) 1.10.0 EconomyAPI 2.0.7 by onebone for API(s) 1.0.0, 1.1.0, 1.2.0, 1.2.1, 1.3.0, 1.3.1, 1.4.0, 1.4.1 ImportMap 2.2-1.5upd1 by aliuly for API(s) 1.10.0 AutoUpdater 1.1 by EvolSoft for API(s) 1.8.0 ManyWorlds 2.0.0 by aliuly for API(s) 1.10.0 SlappEr 1.0.0 by jojoe77777 for API(s) 1.10.0 RankUp 0.3.0 by Falkirks for API(s) 1.4.0 BattleKits 1.0.4 by Falkirks for API(s) 1.0.0 CommandSpy 1.1 by EnderBrine101 for API(s) 1.10.0 SmartPVP 1.1 by TheDiamondYT7 for API(s) 1.10.0 ApocalypsePE-Suite 1.1 by TheDiamondYT7 for API(s) 1.10.0

----------------------REPORT THE DATA BELOW THIS LINE---------------

ghost commented 9 years ago

@goerick @Falkirks https://github.com/Falkirks/RankUp/pull/7

falkirks commented 9 years ago

@64FF00 I think this still may be an issue

ghost commented 9 years ago

@Falkirks Still?

falkirks commented 9 years ago

I haven't been able to reproduce this, if anyone else if able to, update this issue.