Setup a forced max item cap, so people can't accidentally cause server to lag out by giving to many items

Author: kharidiron

plugins/core/admin_commands_plugin/admin_command_plugin.py

@@ -281,14 +281,14 @@ def item(self, data):
                         item_count = item[1]
                     else:
                         item_count = 1
-                    give_item_to_player(target_protocol, item_name, item_count)
+                    given = give_item_to_player(target_protocol, item_name, item_count)
                     target_protocol.send_chat_message(
                         "%s^green; has given you: ^yellow;%s^green; (count: ^cyan;%s^green;)" % (
-                            self.protocol.player.colored_name(self.config.colors), item_name, item_count))
+                            self.protocol.player.colored_name(self.config.colors), item_name, given))
                     self.protocol.send_chat_message("Sent ^yellow;%s^green; (count: ^cyan;%s^green;) to %s" % (
-                        item_name, item_count, target_player.colored_name(self.config.colors)))
+                        item_name, given, target_player.colored_name(self.config.colors)))
                     self.logger.info("%s gave %s %s (count: %s)", self.protocol.player.name, name, item_name,
-                                     item_count)
+                                     given)
                 else:
                     self.protocol.send_chat_message("You have to give an item name.")
             else:

plugins/fuelgiver/fuelgiver_plugin.py

@@ -28,9 +28,9 @@ def fuel(self, data):
             return
         if not 'last_given_fuel' in my_storage or float(my_storage['last_given_fuel']) <= float(time()) - 86400:
             my_storage['last_given_fuel'] = str(time())
-            give_item_to_player(self.protocol, "fillerup", 1)
+            given = give_item_to_player(self.protocol, "fillerup", 1)
             self.protocol.player.storage = my_storage
             self.protocol.send_chat_message("You were given a daily fuel supply! Now go explore ;)")
             self.logger.info("Gave fuel to %s.", self.protocol.player.name)
         else:
-            self.protocol.send_chat_message("^red;No... -.- Go mining!")
+            self.protocol.send_chat_message("^red;No... -.- Go mining!")

plugins/new_player_greeter_plugin/new_player_greeter_plugin.py

@@ -25,7 +25,7 @@ def after_world_start(self, data):
 
     def give_items(self):
         for item in self.config.plugin_config["items"]:
-            give_item_to_player(self.protocol, item[0], item[1])
+            given = give_item_to_player(self.protocol, item[0], item[1])
 
     def send_greetings(self):
-        self.protocol.send_chat_message(self.config.plugin_config["message"])
+        self.protocol.send_chat_message(self.config.plugin_config["message"])

plugins/starteritems/starteritems_plugin.py

@@ -34,7 +34,7 @@ def starteritems(self, data):
 
     def give_items(self):
         for item in self.config.plugin_config["items"]:
-            give_item_to_player(self.protocol, item[0], item[1])
+            given = give_item_to_player(self.protocol, item[0], item[1])
 
     def send_greetings(self):
-        self.protocol.send_chat_message(self.config.plugin_config["message"])
+        self.protocol.send_chat_message(self.config.plugin_config["message"])

utility_functions.py

@@ -13,16 +13,23 @@
 
 
 def give_item_to_player(player_protocol, item, count=1):
-    logger.debug("Giving item %s (count: %s) to %s", item, count, player_protocol.player.name)
+    logger.debug("Attempting to give item %s (count: %s) to %s", item, count, player_protocol.player.name)
     item_count = int(count)
+    hard_max = 90000
+    if item_count > hard_max:
+        logger.warn("Attempted to give more items than the max allowed (%s). Capping amount.", hard_max)
+        item_count = hard_max
     maximum = 1000
+    given = 0
     while item_count > 0:
         x = item_count
         if x > maximum:
             x = maximum
         item_packet = build_packet(packets.Packets.GIVE_ITEM, packets.give_item_write(item, x + 1))
         player_protocol.transport.write(item_packet)
         item_count -= x
+        given += x
+    return given
 
 
 def recursive_dictionary_update(d, u):