Remove Shares from admin packet format

For OpenTTD 14.0 support - maybe we should add support for detecting game version and decoding as such? This isn't backwards compatible.

Signed-off-by: duck. <[email protected]>

pkg/admin/events.go

@@ -122,10 +122,11 @@ type CompanyUpdate struct { // Type 115
 	Colour             uint8  // Main company colour.
 	Password           bool   // Company is password protected.
 	BankruptcyQuarters uint8  // Quarters of Bankruptcy.
-	Share1             uint8  // Owner of Share 1.
-	Share2             uint8  // Owner of Share 2.
-	Share3             uint8  // Owner of Share 3.
-	Share4             uint8  // Owner of Share 4.
+	// Removed in OpenTTD 14.0
+	// Share1             uint8  // Owner of Share 1.
+	// Share2             uint8  // Owner of Share 2.
+	// Share3             uint8  // Owner of Share 3.
+	// Share4             uint8  // Owner of Share 4.
 }
 
 // CompanyRemove fires when a company is removed from the game.

pkg/admin/packets/packets_server.go

@@ -272,10 +272,10 @@ type ServerCompanyUpdate struct { // Type 115
 	Colour             uint8  // Main company colour.
 	Password           bool   // Company is password protected.
 	BankruptcyQuarters uint8  // Quarters of Bankruptcy.
-	Share1             uint8  // Owner of Share 1.
-	Share2             uint8  // Owner of Share 2.
-	Share3             uint8  // Owner of Share 3.
-	Share4             uint8  // Owner of Share 4.
+	// Share1             uint8  // Owner of Share 1.
+	// Share2             uint8  // Owner of Share 2.
+	// Share3             uint8  // Owner of Share 3.
+	// Share4             uint8  // Owner of Share 4.
 }
 
 func (p ServerCompanyUpdate) String() string {

pkg/admin/state.go

@@ -306,10 +306,12 @@ func (s *State) onCompanyUpdate(se *Session, r *CompanyUpdate) (err error) {
 	com.Colour = helpers.OpenttdColour(r.Colour)
 	com.Passworded = r.Password
 	com.Bankruptcy = r.BankruptcyQuarters
-	com.Share1 = r.Share1
-	com.Share2 = r.Share2
-	com.Share3 = r.Share3
-	com.Share4 = r.Share4
+
+	// Shares removed in OpenTTD 14.0
+	// com.Share1 = r.Share1
+	// com.Share2 = r.Share2
+	// com.Share3 = r.Share3
+	// com.Share4 = r.Share4
 
 	s.Companies[r.ID] = com
 

pkg/admin/structs.go

@@ -106,15 +106,15 @@ type Company struct {
 	// The number of quarters this company has been in a bankruptcy state (this may mean they're about to be dissolved!)
 	Bankruptcy uint8 `json:"bankruptcy"`
 
-	// Share data
-	// Share 1 is owned by...
-	Share1 uint8 `json:"share1"`
-	// Share 2 is owned by...
-	Share2 uint8 `json:"share2"`
-	// Share 3 is owned by...
-	Share3 uint8 `json:"share3"`
-	// Share 4 is owned by...
-	Share4 uint8 `json:"share4"`
+	// Share data - removed in OpenTTD 14
+	// // Share 1 is owned by...
+	// Share1 uint8 `json:"share1"`
+	// // Share 2 is owned by...
+	// Share2 uint8 `json:"share2"`
+	// // Share 3 is owned by...
+	// Share3 uint8 `json:"share3"`
+	// // Share 4 is owned by...
+	// Share4 uint8 `json:"share4"`
 
 	// Cargo delivered in the current quarter.
 	CargoThisQuarter uint16 `json:"cargo_current"`