Is calling BagSlot.Handover() through all quest items risky or bad?

[Neverdyne]

Wondering if doing something like this would be bad:

foreach (var s in InventoryManager.GetBagByInventoryBagId(InventoryBagId.KeyItems).FilledSlots) 
            {
                s.Handover();
                await Coroutine.Sleep(250);
                if (Request.HandOverButtonClickable) { break; }
            }

            if (Request.HandOverButtonClickable) { Request.HandOver(); }

I've noticed only the correct quest item gets placed on the NPC's request box so it seems to work, but don't know if it's something Square can notice or if it can cause problems in some way.

 

[mastahg]

Gonna have a bad time. I wouldn't do it.

Also handover checks if the button is clickable now, no need to doublcheck it.