Yeah there's no difference between picking doors and picking chests; they even have the same levels of difficulty. Might make it hard to differentiate between the two if that's what you're detecting.

So far this has been useful for the treasure chest achievement. However, it also auto-marks locked doors as "chests" for some reason.

I reported that this addon will suffer from buggy pins(pins that are never removed on map) when used with AUI minimap.

Since you didn't fix this, I investigated it by myself.

In fact, this bug can be triggered even without AUI.

In your addon, you use local function 'pinCallback(pinManager)' as the 'pinTypeAddCallback' for 'LibMapPins'.
If you add a wrapper function for 'pinCallback' like:

and use it as the 'pinTypeAddCallback'.

You can see that the 'pinTypeAddCallback' function will be called twice when player open Map or Wayshrine. Then pins in current map will become buggy pins and never removed.

I think this problem must be due to some issues in 'LibMapPins'. And when used with AUI, this problem will arise without the 'zo_callLater' wrapper.

Inspired by the addon 'Skyshards', I have learnt a workaround.
The code changed is as follows:

The trick is that by introducing a control parameter 'updating' and delaying its switch from 'true' to 'false', we can block the subsequent second 'pinTypeAddCallback' function call. As a result, no pins will be added twice and no pins will become buggy.

This is what I found by debugging for 4 hours. Hope this will make sense.

It's outdated.
Will there be an Update?

Dear CrazyDutchGuy,
First thanks for updating this awesome addon.
Well, here comes the problem.
Known by a lot of players that since version 1.3.x AUI suffered from compatibility problems with other addons related to map-pins. Recently AUI welcomed the 1.4.7 patch, and most addon compatibility problems have been solved.
However, there is still a problem with "Chest Master 9000". The issue is triggered when you use wayshrine and zoom out to world scale. Then chest pins in current area will remain on map and never be removed.
I will appreciate it if you could take some time investigating this problem. Or if you have no plan to deal with this, please let me know.
Best regards!

For, me, I wanted an addon which does *not* save each material location, but only chests, so thank you for this. Is it correct, that this addon has no options?

And perhaps you could, to make it standout from harvestmap, also add heavy sack locations to this addon, just as an additional idea.

greetings, Almariel

You'll need to update:

• API Version in addon manifest
• LibMapPins-1.0 library
• convert pinLayout.color to pinLayout.tint (tint is ZO_ColorDef object):
  
  Lua Code:

  --here
  local color =
  {
      [1] = ZO_ColorDef:New(0.8, 0.5, 0.2, 0.8),
      [2] = ZO_ColorDef:New(0.7, 0.7, 0.7, 0.8),
      [3] = ZO_ColorDef:New(1.0, 0.8, 0.0, 0.8),
  }
   
  local pinLayout =
  {
      level = 40,
      texture = "/esoui/art/icons/mapkey/mapkey_bank.dds",
      tint = color[1],
      size = 16,
  }
   
  --and here
  local function pinCallback(pinManager)
      local chestData = ChestMaster9000.SV["data"][GetMapTileTexture()]
   
      if not chestData then return end
   
      for _,chest in ipairs(chestData) do
          if chest[3] then
              pinLayout.tint = color[chest[3]]
          else
              pinLayout.tint = color[1]
          end
          LMP:CreatePin( Addon.Name.."MapPin", chest, chest[1], chest[2],nil)
      end
  end

• Import function from HarvestMap because it uses AceSerializer library to store data. You can deserialize data using the Harvest.Deserialize(datastring), you don't need to include library in your addon.
  
  Lua Code:

  --changes to the import function:
      for _,item in ipairs(zonedata[6]) do --replace v with item
          local v = type(item) == "string" and Harvest.Deserialize(item) or item --add this line
          if not knownChestLocation(longZoneName, v[1], v[2]) then -- rest of the function is the same

We ever gonna get an update for this?