Muqsit / PlayerVaults

Per-player GUI-based vaults plugin for PocketMine-MP
https://poggit.pmmp.io/p/PlayerVaults
GNU Lesser General Public License v2.1
41 stars 33 forks source link

Fix #40 #41

Closed MasterOfTheRealm closed 6 years ago

MasterOfTheRealm commented 6 years ago

Even after commit https://github.com/Muqsit/PlayerVaults/commit/024ced359e0dbe2dc7b9e2b4adc99f30b77f56c7, I still kept getting the error in Issue #40. Turns out https://github.com/Muqsit/PlayerVaults/blob/master/src/PlayerVaults/Task/FetchInventoryTask.php#L102 was checking if an empty array was null.

Fix has been tested and it works.

Muqsit commented 6 years ago

This already seems impossible, I'm sure you're using a flat file. Can you send the data in the flat file? This line should already take care of that.

if(!empty($data)){
    $data = base64_decode($data);
}