Use pickUpAll with 1 tile bounding box instead of findItems & pickUpgo_attack_pickup_stand_on_ignored_item

This is not a 1:1 replacement, as pickUpAll will loop until the end of
actors and findItems returned as soon as it finds a match.

But packetLimiter will kill off subsequent pickups until proper
refactoring.

1 files changed, 3 insertions, 7 deletions

diff --git a/src/being/localplayer.cpp b/src/being/localplayer.cpp
index 1e2f16bea..6dcf3e715 100644
--- a/src/being/localplayer.cpp
+++ b/src/being/localplayer.cpp
@@ -1617,10 +1617,7 @@ bool LocalPlayer::pickUpItems(int pickUpType)
     int y = mY;
 
     // first pick up item on player position
-    FloorItem *item =
-        actorManager->findItem(x, y);
-    if (item != nullptr)
-        status = pickUp(item);
+    status = actorManager->pickUpAll(x, y, x, y, false);
 
     if (pickUpType == 0)
         pickUpType = settings.pickUpType;
@@ -1643,9 +1640,8 @@ bool LocalPlayer::pickUpItems(int pickUpType)
                 case BeingDirection::RIGHT: ++x; break;
                 default: break;
             }
-            item = actorManager->findItem(x, y);
-            if (item != nullptr)
-                status = pickUp(item);
+            if (actorManager->pickUpAll(x, y, x, y, false))
+                status = true;
             break;
         case 2:
             switch (mDirection)