Fix scarpet's unmountable attribute

docs/scarpet/api/Entities.md

@@ -204,7 +204,7 @@ Entity that `e` rides.
 
 ### `query(e, 'unmountable')`
 
-Boolean, true if the entity cannot be mounted.
+Boolean, true if the entity cannot be unmounted.
 
 ### `(deprecated) query(e, 'tags')`
 
@@ -727,7 +727,7 @@ Mounts the entity to the `other`.
 
 ### `modify(e, 'unmountable', boolean)`
 
-Denies or allows an entity to be mounted.
+Denies or allows an entity to be unmounted.
 
 ### `modify(e, 'drop_passengers')`
 

src/main/java/carpet/mixins/Player_scarpetEventsMixin.java

@@ -18,6 +18,7 @@ import org.spongepowered.asm.mixin.injection.Inject;
 import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
 import org.spongepowered.asm.mixin.injection.callback.CallbackInfoReturnable;
 import org.spongepowered.asm.mixin.injection.callback.LocalCapture;
+import com.llamalad7.mixinextras.injector.ModifyReturnValue;
 
 import static carpet.script.CarpetEventServer.Event.PLAYER_ATTACKS_ENTITY;
 import static carpet.script.CarpetEventServer.Event.PLAYER_DEALS_DAMAGE;
@@ -86,4 +87,10 @@ public abstract class Player_scarpetEventsMixin extends LivingEntity
             }
         }
     }
+
+    @ModifyReturnValue(method = "wantsToStopRiding", at = @At("TAIL"))
+    private boolean dontUnmountFromIfPermanentVehicle(boolean original)
+    {
+        return original && !((EntityInterface) this.getVehicle()).isPermanentVehicle();
+    }
 }