Skip to content

Change PB_UTIL.is_paperclip to use a lookup table #326

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Yule42 wants to merge 3 commits into
base: beta
Choose a base branch
from
Open

Change PB_UTIL.is_paperclip to use a lookup table #326

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
1d96ecb
Change is_paperclip to use a lookup table
Yule42 Apr 3, 2026
666da6a
Merge remote-tracking branch 'upstream/beta' into beta
Yule42 May 19, 2026
c927bc4
Make paperclip keys loaded at runtime
Yule42 May 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions utilities/definitions/paperclips.lua
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ PB_UTIL.ENABLED_PAPERCLIPS = {
if PB_UTIL.config.paperclips_enabled then
-- Table to hold all paperclip keys regardless of mod of origin for easy reference
PB_UTIL.Paperclips = {}
PB_UTIL.Paperclips_keys = {}

PB_UTIL.Paperclip = SMODS.Sticker:extend {
prefix_config = { key = true },
Expand All @@ -31,6 +32,7 @@ if PB_UTIL.config.paperclips_enabled then

inject = function(self, i)
SMODS.Sticker.inject(self, i)
PB_UTIL.Paperclips_keys[self.key] = true
table.insert(PB_UTIL.Paperclips, self.key)
end,

Expand Down
7 changes: 1 addition & 6 deletions utilities/misc_functions.lua
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,7 @@ end
---@param str string
---@return boolean
function PB_UTIL.is_paperclip(str)
for _, v in ipairs(PB_UTIL.Paperclips or {}) do
if v == str then
return true
end
end
return false
return PB_UTIL.Paperclips_keys[str] == true
end

---Checks if a card has a paperclip. If found, the first value returned is the key.
Expand Down