Very... interesting method. This doesn't really make sense in OO languages. If it were a static method, yes but not like this. What you're doing is calling the move method on an entity to move a different entity. Doesn't make much sense.

Instead it should only take a Position as a parameter and then move the current instance of the entity. i.e. mPos = newPosition. (this.mPos = newPosition)

Don't use a setter because setters are normally used to signal that you're only changing 1 value. Whilst the only difference is the name, move is more correct in this case because it doesn't just change the position of the entity but also modifies the map.

Also, change name to just move.