findClosest returning RoomPosition objects?

[avdg_]

I'm passing creep and structure objects to RoomPosition.prototype.findClosest and the recent update seems to return RoomPosition objects of these objects.

I'm pretty sure this is a mistake

[avdg_]

Code to reproduce:

var creeps = [];
for (var i in Game.creeps) {
    creeps.push(Game.creeps[i])
}
Game.spawns.Spawn1.pos.findClosest(creeps) instanceof Creep;

[avdg_]

Also try

Game.spawns.Spawn1.pos.findClosest(creeps) instanceof RoomPosition;

This shouldn't be true

[chris]

Same for me - my whole code breaks and returns ERR_INVALID_TARGET.

[pmaidens]

I am having the same issue.

[flyingbeast]

same here.

[GrandonBroseph]

Aw man, code broke 3h ago because of this. Are we expected to start using .lookForAt calls or something?

[avdg_]

@GrandonBroseph There can be many objects on the same place though...

[avdg_]

Also note that using findClosest with a FIND_ constant isn't affected by this bug

[pmaidens]

I don't think that will be the case. I think this is a bug that I am sure will be fixed shortly.

[artch]

Thanks for reporting, fixed.

[GrandonBroseph]

@avdg_: Yeah, I figured that out lol. I just used .lookAt if it returned a pos value, but as the bug is fixed it's no big deal.

[pmaidens]

@Artem, Thanks! Everything is working great now!