Placer upgrade stops when an area is designated and a block is present

[Mystara0001]

I noticed that if you select multiple areas for placing a block, if one is occupied, the4 placer stops placing blocks until it can place a block in that space.

Set up a placer.
Set a work area card for a ten block row.
Place a block somewhere in the row
Activate the placer

Expected.
Placer reaches the occupied space and skips it going to the next to place a block.

Actual.
Placer reaches occupied space and then stops until the space is no longer occupied.

[Mowmaster]

all upgrades that iterate through a list of spaces to enact work on act the exact same way. while in the case of a planter it might feel like a bug, if fixed it could cause things like the chopper/harvester/pump/quarry to skip spots and leave the whole area not processed.

[Mowmaster]

22e55f7

[Nagapito]

Hi, your fix does not work for the block placer....

On line 124, you only iterate to the next position if player action returns true...
Issue is, on line 183, you check if the space is empty and if it isnt, you return false, so it never iterates to next position on 124....

Also, if I read it correctly, on line 124 you are negating the value of having the workpositionskip. This means if the pedestal is upgraded, it sends the value false. Later on player action, even if it bypasses the empty block check, since it cant place an item and consume the one on pedestal, it will return the skip variable, which will have false value and prevent again to iterate to next position.

So, I think line 183 should return the skip variable instead of false value and on line 124, you should remove the negator on last parameter!