[Universal]

This would be more easier if you would have an array instead of a simple variable. For example:

pawn Код:

new pBagSlot[5];

In this case, it is possible to loop through all of the bag slots, this would probably improve some (or maybe all of your current) functions and would open more abbilities in future coding. Not to mention that a dynamic system like this should be coded following these basics. (Just pointing out if you are going to create something in near future)

pawn Код:

stock addItemToBag(playerid, itemid) {
   for (new i = 0; i < 5; i++) {
       if (pBagSlot[i] != 0) continue;

       pBagSlot[i] = itemid;
   }
}

Isnt this easier?

After all, its just a free advice.