Enhancement: Make Nopickup command optional (#819)

* Making Nopickup command optional

* Update bot.py

* Update config.py

* set default on, rework logic

Co-authored-by: mgleed <[email protected]>

config/params.ini

@@ -106,6 +106,8 @@ id_items=1
 sell_junk=0
 open_chests=1
 fill_shared_stash_first=0
+; helps reduce accidental pickups when enabled especially on walking characters
+enable_no_pickup=1
 ; to gamble, add any/all of the following: circlet, ring, coronet, talon, amulet
 gamble_items=
 
@@ -240,7 +242,7 @@ clay_golem=
 bone_wall=
 bone_spear=
 bone_spirit=
-; Damage scaling. One setting to adjust kill speed to account for better/worse gear. 
+; Damage scaling. One setting to adjust kill speed to account for better/worse gear.
 ; ex: 2 will cast all attack skills twice a long, .5 half as long. Currently only used by bone necro
 damage_scaling=1
 

src/bot.py

@@ -264,7 +264,7 @@ class Bot:
                 Logger.info(f"Please Enter the region ip and hot ip on config to use")
 
         # Run /nopickup command to avoid picking up stuff on accident
-        if not self._ran_no_pickup and not self._game_stats._nopickup_active:
+        if Config().char["enable_no_pickup"] and (not self._ran_no_pickup and not self._game_stats._nopickup_active):
             self._ran_no_pickup = True
             if view.enable_no_pickup():
                 self._game_stats._nopickup_active = True

src/config.py

@@ -300,6 +300,7 @@ class Config:
             "runs_per_repair": False if not self._select_val("char", "runs_per_repair") else int(self._select_val("char", "runs_per_repair")),
             "gamble_items": False if not self._select_val("char", "gamble_items") else self._select_val("char", "gamble_items").replace(" ","").split(","),
             "sell_junk": bool(int(self._select_val("char", "sell_junk"))),
+            "enable_no_pickup": bool(int(self._select_val("char", "enable_no_pickup"))),
         }
         # Sorc base config
         sorc_base_cfg = dict(self.configs["config"]["parser"]["sorceress"])