make Player:SetPData and Player:RemovePData actually return a boolean (#1394) * make Player:SetPData and Player:RemovePData actually return a boolean According to the wiki [SetPData](https://wiki.garrysmod.com/page/Player/SetPData) and [RemovePData](https://wiki.garrysmod.com/page/Player/RemovePData) should return a boolean on success/failure which at the moment they're not. Additionally it would be optimal if they would also throw an error (sql.LastError()) if it fails so people actually know why their stuff didn't get stored.