fix: set player connected flag to false when controller is destroyed

[akiver]

Bot controller entities can be destroyed:

96322 player_controller_destroy 12 Baroud 0
107866 player_controller_destroy 14 Bank 0

If we don't update the prop Connected, the bot will still be in the slice returned by Playing().
As a result, calling a player's function that relies on its pawn entity such as EquipmentValueCurrent() may result in a panic as the pawn entity doesn't exists anymore.

Noticed with this demo https://mega.nz/file/uBdF3TBT#aoGpmJ6PKaNIMqPdZsVxHyhiv7DbxO3M9jJNWAOIhb0 and the following code:

p.RegisterEventHandler(func(e events.RoundStart) {
  fmt.Println(p.GameState().TeamCounterTerrorists().CurrentEquipmentValue())
  fmt.Println(p.GameState().TeamTerrorists().CurrentEquipmentValue())
})

ref akiver/cs-demo-manager#727

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (d7f0779) 80.67% compared to head (713ae71) 80.67%.
Report is 1 commits behind head on master.

❗ Current head 713ae71 differs from pull request most recent head 30541c4. Consider uploading reports for the commit 30541c4 to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #497   +/-   ##
=======================================
  Coverage   80.67%   80.67%           
=======================================
  Files          47       47           
  Lines        6338     6339    +1     
=======================================
+ Hits         5113     5114    +1     
  Misses       1015     1015           
  Partials      210      210           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.