More Bug Fixing

Author: bleonheart

gamemode/core/libraries/admin.lua

@@ -267,9 +267,11 @@ function lia.administrator.hasAccess(ply, privilege)
     end
 
     if not lia.administrator.privileges[privilege] then
-        lia.information(L("privilegeNotExist", privilege))
-        if IsValid(ply) then ply:notifyLocalized("privilegeNotExist", privilege) end
-        return getGroupLevel(grp) >= (lia.administrator.DefaultGroups.superadmin or 3)
+        if SERVER then
+            local playerInfo = IsValid(ply) and ply:Nick() .. " (" .. ply:SteamID() .. ")" or "Unknown"
+            lia.log.add(ply, "missingPrivilege", privilege, playerInfo, grp)
+        end
+        return getGroupLevel(grp) >= (lia.administrator.DefaultGroups.admin or 3)
     end
 
     if getGroupLevel(grp) >= (lia.administrator.DefaultGroups.superadmin or 3) then return true end

gamemode/core/libraries/logger.lua

@@ -741,6 +741,16 @@ lia.log.types = {
         func = function(client, charID) return L("logCharUnbanOffline", client:Name(), tostring(charID)) end,
         category = L("admin")
     },
+    ["missingPrivilege"] = {
+        func = function(client, privilege, playerInfo, groupInfo) 
+            if client then
+                return L("logMissingPrivilege", client:Name(), privilege, playerInfo or "Unknown", groupInfo or "Unknown")
+            else
+                return L("logMissingPrivilegeNoClient", privilege, playerInfo or "Unknown", groupInfo or "Unknown")
+            end
+        end,
+        category = L("admin")
+    },
 }
 
 function lia.log.addType(logType, func, category)

gamemode/core/libraries/vendor.lua

@@ -128,7 +128,18 @@ end
 function lia.vendor.addPreset(name, items)
     assert(isstring(name), L("vendorPresetNameString"))
     assert(istable(items), L("vendorPresetItemsTable"))
-    lia.vendor.presets[string.lower(name)] = items
+    
+    -- Filter out items that don't exist
+    local validItems = {}
+    for itemType, itemData in pairs(items) do
+        if lia.item.list[itemType] then
+            validItems[itemType] = itemData
+        else
+            print("[Vendor] Warning: Item '" .. itemType .. "' in preset '" .. name .. "' does not exist and will be skipped.")
+        end
+    end
+    
+    lia.vendor.presets[string.lower(name)] = validItems
 end
 
 function lia.vendor.getPreset(name)

gamemode/languages/english.lua

@@ -606,7 +606,7 @@ LANGUAGE = {
     categoryServer = "Server",
     mustSuperAdminStopSound = "You need the '%s' privilege to run stopsound globally.",
     commandConsoleOnly = "This command can only be run from the server console.",
-    privilegeNotExist = "The privilege '%s' does not exist and has been restricted to super administrators. Please notify an administrator.",
+
     resetInv = "You have cleared %s's inventory!",
     searchingChar = "Searching for character...",
     noAvailableFlags = "No available flags to give.",
@@ -2268,4 +2268,6 @@ Reload: Drop]],
     adminStickCategoryUtilityDesc = "General utility commands and tools",
     adminStickCategoryAdministrationDesc = "Advanced administration tools",
     configReset = "All configuration values have been reset to defaults.",
+    logMissingPrivilege = "%s requested missing privilege '%s' (Player: %s, Group: %s)",
+    logMissingPrivilegeNoClient = "Missing privilege '%s' requested (Player: %s, Group: %s)",
 }

gamemode/modules/inventory/submodules/vendor/config.lua

@@ -1,11 +1,10 @@
-lia.vendor.addPreset("utility_vendor", {
+lia.vendor.addPreset("Sample Preset 1", {
     manhack_welder = {
-        mode = VENDOR_SELLANDBUY
+        mode = VENDOR_SELLANDBUY,
+        price = 50,
     },
-    item_suit = {
-        mode = VENDOR_SELLANDBUY
-    },
-    universalammo3 = {
-        mode = VENDOR_SELLANDBUY
+    weapon_flechettegun = {
+        mode = VENDOR_SELLANDBUY,
+        price = 0,
     },
 })

gamemode/modules/inventory/submodules/vendor/entities/entities/lia_vendor/init.lua

@@ -1,4 +1,4 @@
-local function safeSendToReceivers(entity, netName)
+local function safeSendToReceivers(entity)
     if entity.receivers and #entity.receivers > 0 then net.Send(entity.receivers) end
 end
 
@@ -49,6 +49,11 @@ function ENT:setWelcomeMessage(value)
 end
 
 function ENT:setStock(itemType, value)
+    if not lia.item.list[itemType] then
+        print("[Vendor] Warning: Cannot set stock for non-existent item '" .. itemType .. "'")
+        return
+    end
+
     self.items[itemType] = self.items[itemType] or {}
     if not self.items[itemType][VENDOR_MAXSTOCK] then self:setMaxStock(itemType, value) end
     self.items[itemType][VENDOR_STOCK] = math.Clamp(value, 0, self.items[itemType][VENDOR_MAXSTOCK])
@@ -59,12 +64,22 @@ function ENT:setStock(itemType, value)
 end
 
 function ENT:addStock(itemType, value)
+    if not lia.item.list[itemType] then
+        print("[Vendor] Warning: Cannot add stock for non-existent item '" .. itemType .. "'")
+        return
+    end
+
     local current = self:getStock(itemType)
     if not current then return end
     self:setStock(itemType, self:getStock(itemType) + (value or 1))
 end
 
 function ENT:takeStock(itemType, value)
+    if not lia.item.list[itemType] then
+        print("[Vendor] Warning: Cannot take stock for non-existent item '" .. itemType .. "'")
+        return
+    end
+
     if not self.items[itemType] or not self.items[itemType][VENDOR_MAXSTOCK] then return end
     self:addStock(itemType, -(value or 1))
 end
@@ -133,6 +148,11 @@ function ENT:setName(name)
 end
 
 function ENT:setTradeMode(itemType, mode)
+    if not lia.item.list[itemType] then
+        print("[Vendor] Warning: Cannot set trade mode for non-existent item '" .. itemType .. "'")
+        return
+    end
+
     if not ALLOWED_MODES[mode] then mode = nil end
     self.items[itemType] = self.items[itemType] or {}
     self.items[itemType][VENDOR_MODE] = mode
@@ -143,6 +163,11 @@ function ENT:setTradeMode(itemType, mode)
 end
 
 function ENT:setItemPrice(itemType, value)
+    if not lia.item.list[itemType] then
+        print("[Vendor] Warning: Cannot set price for non-existent item '" .. itemType .. "'")
+        return
+    end
+
     if not isnumber(value) or value < 0 then value = nil end
     self.items[itemType] = self.items[itemType] or {}
     self.items[itemType][VENDOR_PRICE] = value
@@ -153,6 +178,11 @@ function ENT:setItemPrice(itemType, value)
 end
 
 function ENT:setItemStock(itemType, value)
+    if not lia.item.list[itemType] then
+        print("[Vendor] Warning: Cannot set stock for non-existent item '" .. itemType .. "'")
+        return
+    end
+
     if not isnumber(value) or value < 0 then value = nil end
     self.items[itemType] = self.items[itemType] or {}
     self.items[itemType][VENDOR_STOCK] = value
@@ -163,6 +193,11 @@ function ENT:setItemStock(itemType, value)
 end
 
 function ENT:setItemMaxStock(itemType, value)
+    if not lia.item.list[itemType] then
+        print("[Vendor] Warning: Cannot set max stock for non-existent item '" .. itemType .. "'")
+        return
+    end
+
     if not isnumber(value) or value < 0 then value = nil end
     self.items[itemType] = self.items[itemType] or {}
     self.items[itemType][VENDOR_MAXSTOCK] = value
@@ -235,10 +270,14 @@ function ENT:applyPreset(name)
     if not preset then return end
     self.items = {}
     for itemType, data in pairs(preset) do
-        if data.mode ~= nil then self:setTradeMode(itemType, data.mode) end
-        if data.price ~= nil then self:setItemPrice(itemType, data.price) end
-        if data.maxStock ~= nil then self:setMaxStock(itemType, data.maxStock) end
-        if data.stock ~= nil then self:setStock(itemType, data.stock) end
+        if lia.item.list[itemType] then
+            if data.mode ~= nil then self:setTradeMode(itemType, data.mode) end
+            if data.price ~= nil then self:setItemPrice(itemType, data.price) end
+            if data.maxStock ~= nil then self:setMaxStock(itemType, data.maxStock) end
+            if data.stock ~= nil then self:setStock(itemType, data.stock) end
+        else
+            print("[Vendor] Warning: Skipping invalid item '" .. itemType .. "' when applying preset '" .. name .. "' to vendor.")
+        end
     end
 
     if self.receivers then