Entity Passengers get pushed wrongly (#3601)

* Fixed splicing issue that wrongly removes passengers

* Test: trying to fix CI tests

Author: SinanAkkoyun

lib/plugins/entities.js

@@ -808,9 +808,9 @@ function inject (bot) {
     const vehicle = packet.vehicleId === -1 ? null : fetchEntity(packet.vehicleId)
 
     const originalVehicle = passenger.vehicle
-    if (originalVehicle !== null) {
+    if (originalVehicle) {
       const index = originalVehicle.passengers.indexOf(passenger)
-      originalVehicle.passengers = originalVehicle.passengers.splice(index, 1)
+      originalVehicle.passengers.splice(index, 1)
     }
     passenger.vehicle = vehicle
     vehicle.passengers.push(passenger)
@@ -833,12 +833,12 @@ function inject (bot) {
 
     for (const passengerEntity of passengerEntities) {
       const originalVehicle = passengerEntity.vehicle
-      if (originalVehicle !== null) {
+      if (originalVehicle) {
         const index = originalVehicle.passengers.indexOf(passengerEntity)
-        originalVehicle.passengers = originalVehicle.passengers.splice(index, 1)
+        originalVehicle.passengers.splice(index, 1)
       }
       passengerEntity.vehicle = vehicle
-      if (vehicle !== null) {
+      if (vehicle) {
         vehicle.passengers.push(passengerEntity)
       }
     }
@@ -869,7 +869,7 @@ function inject (bot) {
     if (entity.vehicle) {
       const index = entity.vehicle.passengers.indexOf(entity)
       if (index !== -1) {
-        entity.vehicle.passengers = entity.vehicle.passengers.splice(index, 1)
+        entity.vehicle.passengers.splice(index, 1)
       }
     }
   })