Merge branch 'main' of https://forge.chapril.org/ayte/DeadGodHelper

README.md

@@ -1,16 +1,23 @@
 # Dead God Helper
 
-A small mod for The Binding Of Isaac: Repentance!
+Spot missing items!
 
-It signals items that were not previously picked up on this savefile when you encounter them.Very useful when you're trying to get a second or third Dead God.
+Easily spot items that were not previously picked up on this savefile. Very useful when you're trying to get a second or third Dead God.
+
+Now supports modded items, and integration with Repentogon!
 
 I made it mainly for myself, because it was annoying to go back to the menu every time I encountered an item I was unsure about.
 
 There's an integration with External Item Description.
 It is also compatible with Mod Config Menu, if you want to toggle things (such as displaying info when on Curse Of The Blind)
 
-There's however a limitation: since (to my knowledge) there's no way to get information about seen items from the savefile, this mod takes it upon itself to track which item you picked up. (This means that when you install the mod for the first time, every item will be tagged as Unseen)
+Mods can't access the savefile directly, so the mod tracks itself witch items are taken. This means that the first time you install it, everything will be marked as new.
+
+If you want to import data from your savefile, you can either use the extractor I wrote (adapting EID's one for Linux), or install Repentogon.
+The extractor is available here : https://forge.chapril.org/ayte/DeadGodHelper/src/branch/main/SaveExtractor
+
+This mod is still relevant in itself if you do not want to use external tools, or you want to track modded items ^^
 
 Anyway, I hope that this mod will be useful to someone :)
 
-(Source code is available at https://forge.chapril.org/ayte/DeadGodHelper)
+Source code is available at https://forge.chapril.org/ayte/DeadGodHelper

main.lua

@@ -1,3 +1,4 @@
+-- Formatted using lua-format
 local modname = "DeadGod Helper"
 local mod = RegisterMod(modname, 1)
 local math = require("math")
@@ -7,34 +8,44 @@ local savegen = include("save")
 local save = savegen:new()
 local json = include("json")
 local DEBUG = false
+local getgamedata = Isaac.GetPersistentGameData -- Repentogon exclusive
+local isrepentogon = getgamedata ~= nil
 
 mod.font = Font()
 mod.font:Load("font/terminus.fnt")
 
-function mod:isCurseBlind()
+function mod:isCurseBlind(pickupEntity)
+    if isrepentogon then if pickupEntity:IsBlind() then return true end end
     return mod.game:GetLevel():GetCurses() & LevelCurse.CURSE_OF_BLIND > 0
 end
 
 function mod.isIdNotTakenAlready(id)
-	if id>=1 and id<=save.settings.maxid then
+    if id > 0 then
-		return not save.seen[id]
+        -- Always returns seen if on a challenge
-	end
+        if Isaac.GetChallenge() ~= 0 then return false end
-	if id>save.settings.maxid and save.settings.showonmodded then
+
+        if id > save.settings.maxid then
+            if save.settings.showonmodded then
                 local itemconfig = Isaac.GetItemConfig():GetCollectible(id)
                 if itemconfig then
                     local name = itemconfig.Name
                     return not save.seenmodded[name]
                 end
             end
-
+        else
-	return false
+            return not save.seen[id]
+        end
+    else
+        return false -- Don't show on empty pedestals
+    end
 end
 
 function mod.registerTouched(id)
-	if id>=1 and id<=save.settings.maxid then
+    -- Only register when not on a challenge
-		save:touchid(id)
+    if Isaac.GetChallenge() == 0 then
-	end
+        local shouldsave = mod.isIdNotTakenAlready(id)
-	if id >=save.settings.maxid then
+        if id >= 1 and id <= save.settings.maxid then save:touchid(id) end
+        if id >= save.settings.maxid then
             -- Modded item, save them by name
             -- (as IDs are choosen manually by the game, and may vary when changing mods)
             local itemconfig = Isaac.GetItemConfig():GetCollectible(id)
@@ -45,7 +56,10 @@ function mod.registerTouched(id)
         end
 
         -- Save
+        if shouldsave then -- Only save if there's a change
             mod:save()
+        end
+    end
 end
 
 -- Used to check if EID's description should be modified
@@ -64,31 +78,34 @@ end
 function mod.modifierCallback(descObj)
     if save.settings.eid then
         local str = "#{{Collectible}} Not picked up yet!#"
-		if descObj.ObjSubType>save.settings.maxid then
+        if descObj.ObjSubType > save.settings.maxid then
             str = "#{{Trinket}} Not picked up yet!#"
         end
-		descObj.Description = str..descObj.Description
+        descObj.Description = str .. descObj.Description
     end
     return descObj
 end
 
 -- Used to detect when an item is picked up
 function mod:update()
-	for i=0, mod.game:GetNumPlayers()-1 do
+    for i = 0, mod.game:GetNumPlayers() - 1 do
         local player = mod.game:GetPlayer(i)
         local data = player:GetData()
         local item = player.QueuedItem.Item
-		if data._deadgodlastid and data._deadgodlastcount and data._deadgodlastid>0 then
+        if data._deadgodlastid and data._deadgodlastcount and
-			if item==nil then -- Finished animation
+            data._deadgodlastid > 0 then
-				if player:GetCollectibleNum(data._deadgodlastid)>data._deadgodlastcount then -- Have it on me
+            if item == nil then -- Finished animation
+                if player:GetCollectibleNum(data._deadgodlastid) >
+                    data._deadgodlastcount then -- Have it on me
                     -- Then I just picked it up, yay
                     mod.registerTouched(data._deadgodlastid)
                     if DEBUG then
-						mod.print("> "..tostring(data._deadgodlastid).."\n")
+                        mod.print("> " .. tostring(data._deadgodlastid) .. "\n")
                     end
                     data._deadgodlastid = 0
                     data._deadgodlastcount = 0
-				end end
+                end
+            end
         end
         if item and item:IsCollectible() then
             data._deadgodlastid = item.ID
@@ -99,16 +116,20 @@ end
 
 -- Used to draw the ! sign
 function mod:PickupDrawCallback(pickupEntity, _)
-	if save.settings.visual and (not mod:isCurseBlind() or save.settings.showonblind)then
+    if save.settings.visual and
-		if pickupEntity.Variant==PickupVariant.PICKUP_COLLECTIBLE then
+        (not (mod:isCurseBlind(pickupEntity)) or save.settings.showonblind) then
+        if pickupEntity.Variant == PickupVariant.PICKUP_COLLECTIBLE then
             if mod.isIdNotTakenAlready(pickupEntity.SubType) then
-				local v = Isaac.WorldToScreen(pickupEntity.Position + pickupEntity.SpriteOffset)
+                local v = Isaac.WorldToScreen(
-				local color = KColor(0.98,0.93,0.55,1)
+                              pickupEntity.Position + pickupEntity.SpriteOffset)
+                local color = KColor(0.98, 0.93, 0.55, 1)
                 if pickupEntity.SubType > save.settings.maxid then
                     -- Modded item
-					color = KColor(0.68,0.88,1,1)
+                    color = KColor(0.68, 0.88, 1, 1)
                 end
-				mod.font:DrawString("!", v.X-1, v.Y-52+math.sin(mod.game:GetFrameCount()/10)*2, color , 2, true)
+                mod.font:DrawString("!", v.X - 1, v.Y - 52 +
+                                        math.sin(mod.game:GetFrameCount() / 10) *
+                                        2, color, 2, true)
             end
         end
     end
@@ -117,122 +138,112 @@ end
 function mod:newrun()
     if mod:HasData() then
         local data = mod:LoadData()
-		if DEBUG then
+        if DEBUG then mod.print(data .. "\n") end
-			mod.print(data.."\n")
-		end
         save = savegen:new()
         save:load(json, data)
     else
         save = savegen:new()
     end
-	local c = 0
+    -- Load info from repentogon if available
-	for _,v in ipairs(save.seen) do
+    if isrepentogon then
-		if v then c = c+1 end
+        for i = 1, save.settings.maxid do
+            if getgamedata():IsItemInCollection(i) then
+                mod.registerTouched(i)
             end
-	mod.print(modname.." loaded, "..c.." vanilla items seen")
+        end
+    end
+    local c = 0
+    for _, v in ipairs(save.seen) do if v then c = c + 1 end end
+    mod.print(modname .. " loaded, " .. c .. " vanilla items seen")
     if save.settings.showonmodded then
         local c = 0
-		for _,v in pairs(save.seenmodded) do
+        for _, v in pairs(save.seenmodded) do if v then c = c + 1 end end
-			if v then
+        mod.print(", " .. tostring(c) .. " modded")
-				c = c + 1
-			end
-		end
-		mod.print(", "..tostring(c).." modded")
     end
     mod.print(".\n")
 end
 
-function mod:save()
+function mod:save() mod:SaveData(save:dump(json)) end
-	mod:SaveData(save:dump(json))
-end
 
 mod:AddCallback(ModCallbacks.MC_POST_GAME_END, mod.save)
 mod:AddCallback(ModCallbacks.MC_POST_GAME_STARTED, mod.newrun)
 
-mod:AddCallback(ModCallbacks.MC_POST_UPDATE, mod.update)
+if not isrepentogon then
+    -- Used to check for new items
+    mod:AddCallback(ModCallbacks.MC_POST_UPDATE, mod.update)
+else
+    function mod.addCollectibleCallaback(_, id, _, _, _, _)
+        mod.registerTouched(id)
+    end
+    -- This is not a vanilla callback, it is from repentogon
+    mod:AddCallback(ModCallbacks.MC_POST_ADD_COLLECTIBLE,
+                    mod.addCollectibleCallaback)
+end
+
+-- Used to draw indicators
 mod:AddCallback(ModCallbacks.MC_POST_PICKUP_RENDER, mod.PickupDrawCallback)
 
 if EID then
-	EID:addDescriptionModifier(
+    EID:addDescriptionModifier(modname, mod.shouldBeModified,
-		modname,
+                               mod.modifierCallback)
-		mod.shouldBeModified,
-		mod.modifierCallback
-	)
 end
 
 if ModConfigMenu then
-	ModConfigMenu.AddSetting(
+    ModConfigMenu.AddSetting(modname, -- This should be unique for your mod
-		modname, -- This should be unique for your mod
+    nil, {
-		nil,
-		{
         Type = ModConfigMenu.OptionType.BOOLEAN,
-			CurrentSetting = function()
+        CurrentSetting = function() return save.settings.eid end,
-				return save.settings.eid
-			end,
         Display = function()
             return "Show in EID: " .. (save.settings.eid and "on" or "off")
         end,
         OnChange = function(b)
             save.settings.eid = b
             mod:save()
-			end,
+        end
-		}
+    })
-	)
+    ModConfigMenu.AddSetting(modname, -- This should be unique for your mod
-	ModConfigMenu.AddSetting(
+    nil, {
-		modname, -- This should be unique for your mod
-		nil,
-		{
         Type = ModConfigMenu.OptionType.BOOLEAN,
-			CurrentSetting = function()
+        CurrentSetting = function() return save.settings.visual end,
-				return save.settings.visual
-			end,
         Display = function()
-				return "Show on pedestals: " .. (save.settings.visual and "on" or "off")
+            return "Show on pedestals: " ..
+                       (save.settings.visual and "on" or "off")
         end,
         OnChange = function(b)
             save.settings.visual = b
             mod:save()
-			end,
+        end
-		}
+    })
-	)
 
-	ModConfigMenu.AddSetting(
+    ModConfigMenu.AddSetting(modname, -- This should be unique for your mod
-		modname, -- This should be unique for your mod
+    nil, {
-		nil,
-		{
         Type = ModConfigMenu.OptionType.BOOLEAN,
-			CurrentSetting = function()
+        CurrentSetting = function() return save.settings.showonmodded end,
-				return save.settings.showonmodded
-			end,
         Display = function()
-				return "Also track modded items: " .. (save.settings.showonmodded and "on" or "off")
+            return "Also track modded items: " ..
+                       (save.settings.showonmodded and "on" or "off")
         end,
         OnChange = function(b)
             save.settings.showonmodded = b
-				if b==false then
+            if b == false then
                 -- Reset modded items
                 save.seenmodded = {}
             end
             mod:save()
-			end,
+        end
-		}
+    })
-	)
 
-	ModConfigMenu.AddSetting(
+    ModConfigMenu.AddSetting(modname, -- This should be unique for your mod
-		modname, -- This should be unique for your mod
+    nil, {
-		nil,
-		{
         Type = ModConfigMenu.OptionType.BOOLEAN,
-			CurrentSetting = function()
+        CurrentSetting = function() return save.settings.showonblind end,
-				return save.settings.showonblind
-			end,
         Display = function()
-				return "Show while having Curse Of The Blind: " .. (save.settings.showonblind and "on" or "off")
+            return "Show while having Curse Of The Blind: " ..
+                       (save.settings.showonblind and "on" or "off")
         end,
         OnChange = function(b)
             save.settings.showonblind = b
             mod:save()
-			end,
+        end
-		}
+    })
-	)
 end

metadata.xml

@@ -3,31 +3,29 @@
     <name>DeadGod Helper</name>
     <directory>deadgod helper</directory>
     <id>2985754961</id>
-    <description>A small mod for The Binding Of Isaac: Repentance!
+    <description>Spot missing items!
 
 Easily spot items that were not previously picked up on this savefile. Very useful when you're trying to get a second or third Dead God.
 
-Now supports modded items!
+Now supports modded items, and integration with Repentogon!
 
 I made it mainly for myself, because it was annoying to go back to the menu every time I encountered an item I was unsure about.
 
-There's an integration with External Item Description.
 It is also compatible with Mod Config Menu, if you want to toggle things (such as displaying info when on Curse Of The Blind)
 
-Since mods can't directly access the savefile, this mod takes it upon itself to track wich items are picked up. This means that the first time you install this mod, everything will be marked as unseen.
+Mods can't access the savefile directly, so the mod tracks itself witch items are taken. This means that the first time you install it, everything will be marked as new.
 
-Turns out EID already has a some support for this, using a save extractor. If you want to go down that route, here's the link to the extractor : https://github.com/wofsauge/External-Item-Descriptions/tree/master/scripts
+If you want to import data from your savefile, you can either use the extractor I wrote (adapting EID's one for Linux), or install Repentogon.
-
+The extractor is available here : https://forge.chapril.org/ayte/DeadGodHelper/src/branch/main/SaveExtractor
-However, EID's save extractor doesn't really work on Linux, so I made a port of it in python, in order to be more easily ran. This port is available here : https://forge.chapril.org/ayte/DeadGodHelper/src/branch/main/SaveExtractor
-This port can also extract save file's info to be used for Dead God Helper.
 
 This mod is still relevant in itself if you do not want to use external tools, or you want to track modded items ^^
 
 Anyway, I hope that this mod will be useful to someone :)
 
 Source code is available at https://forge.chapril.org/ayte/DeadGodHelper</description>
-    <version>1.11</version>
+    <version>1.13</version>
     <visibility>Public</visibility>
     <tag id="Items"/>
+    <tag id="Tweaks"/>
     <tag id="Graphics"/>
 </metadata>