Download
(2 Kb)
Download
Updated: 09/24/19 10:32 PM
Pictures
File Info
Compatibility:
Scalebreaker (5.1.5)
Updated:09/24/19 10:32 PM
Created:07/14/19 10:12 PM
Monthly downloads:79
Total downloads:7,157
Favorites:3
MD5:
Maarselok Helmet Proc cooldown tracker  Popular! (More than 5000 hits)
Version: 2.0.0
by: dowinterfor6 [More]
A simple addon to track the cooldown of the new helmet set.

(My first addon, was an exploration of LUA)

Finally updated to flesh out the bugs that have been bothering me for a while!

If you're looking for a more general addon that covers other sets, I'd recommend Cooldowns (https://www.esoui.com/downloads/info2164-Cooldowns.html), as suggested by a reply in this addon's comment thread. My addon should be much lighterweight and simplistic as it only serves one purpose.

Please let me know if there are any other issues I've happened to have missed!
v1.0.1 Updated file structure, implemented changes suggested by Baertram
v1.0.2 Updated for Scalebreaker live (proc changed from burst to dot)
v1.0.2b Reupload to update file structure
v1.0.2c Fix timing issue with counter
v1.0.2d Revert back to 1.0.2b timing (not quite sure what's causing the bug)
v2.0.0 Major update to fix timing desync issue, and conflict with other player sourced procs (will double check tomorrow if it's completely fixed)
Optional Files (0)


Archived Files (6)
File Name
Version
Size
Uploader
Date
1.0.2d
2kB
dowinterfor6
09/11/19 08:58 PM
1.0.2c
2kB
dowinterfor6
09/11/19 07:11 PM
1.0.2b
2kB
dowinterfor6
08/20/19 11:18 AM
1.0.2
2kB
dowinterfor6
08/19/19 08:07 PM
1.0.1
2kB
dowinterfor6
07/15/19 12:09 PM
1.0.0
2kB
07/14/19 10:12 PM


Post A Reply Comment Options
Unread 11/13/19, 06:56 PM  
dowinterfor6
AddOn Author - Click to view AddOns

Forum posts: 0
File comments: 4
Uploads: 1
Originally Posted by BruceLeeRoy
So wait is this thing working on the live server? I've tried installing it but nothing appears on the screen or in the addons list. any ideas?
Hey! Sorry for the slow reply, I've mostly left this addon untouched. I've confirmed it worked at the end of last patch, and got a friend to test that it works this patch (Dragonhold). Have you tried a clean install of the addon (in case you were running an old version)? Also, nothing shows up until the first time you proc Maarselok. Hope that helps!
Report comment to moderator  
Reply With Quote
Unread 11/07/19, 04:07 PM  
BruceLeeRoy

Forum posts: 0
File comments: 16
Uploads: 0
So wait is this thing working on the live server? I've tried installing it but nothing appears on the screen or in the addons list. any ideas?
Report comment to moderator  
Reply With Quote
Unread 08/20/19, 11:23 AM  
dowinterfor6
AddOn Author - Click to view AddOns

Forum posts: 0
File comments: 4
Uploads: 1
Thanks for the heads up, I noticed that right before I went to bed yesterday. I've uploaded the fix, and I'll be available to double check if it works tonight!
Originally Posted by RealReeth
Nice idea for the addon, but Minion doesn't find it for installing
and I try to avoid installing any addon manually because of updates.
Report comment to moderator  
Reply With Quote
Unread 08/20/19, 10:33 AM  
RealReeth

Forum posts: 1
File comments: 30
Uploads: 0
Nice idea for the addon, but Minion doesn't find it for installing
and I try to avoid installing any addon manually because of updates.
Report comment to moderator  
Reply With Quote
Unread 08/19/19, 08:08 PM  
dowinterfor6
AddOn Author - Click to view AddOns

Forum posts: 0
File comments: 4
Uploads: 1
Hey! Sorry for the late reply, I just pushed an update to fix this issue (forgot they changed the proc from burst to DoT from the PTS)

Originally Posted by BruceLeeRoy
I can't get this addon to work at all, it there some slash command I need to be using? any advice would be greatly appreciated :-)
Report comment to moderator  
Reply With Quote
Unread 08/19/19, 11:45 AM  
BruceLeeRoy

Forum posts: 0
File comments: 16
Uploads: 0
The addon called cooldowns works really well for this purpose.
Report comment to moderator  
Reply With Quote
Unread 08/16/19, 05:03 PM  
BruceLeeRoy

Forum posts: 0
File comments: 16
Uploads: 0
I can't get this addon to work at all, it there some slash command I need to be using? any advice would be greatly appreciated :-)
Report comment to moderator  
Reply With Quote
Unread 07/15/19, 12:11 PM  
dowinterfor6
AddOn Author - Click to view AddOns

Forum posts: 0
File comments: 4
Uploads: 1
Hey Baertram! Thanks for the quick response, I've updated the addon with your suggestions, and I'll double check it myself when I get back to my desktop again later today. Thank you for the suggestions and improvements!

Originally Posted by Baertram
...
Report comment to moderator  
Reply With Quote
Unread 07/15/19, 04:06 AM  
Baertram
Super Moderator
 
Baertram's Avatar
ESOUI Super Moderator
AddOn Author - Click to view AddOns

Forum posts: 4903
File comments: 5974
Uploads: 78
Hi, welcome to the addon community and thanks for the addon.
As you said it's your first addon I had a quick check if you are able to improve it somehow.

1st thing I recognized - ZP archive is missing the foldername -> Addon won't work this way!
The zip archive is packacked wrong!
You need the foldername of the addon (must be the same as your txt filename) on top level and then the files lua txt xml etc. below!
Or the game won't recognize this addon and Minion is not able to provide it as download neither.

Code:
MaarselokTracker.zip:
|-MaarselokTracker (folder)
   |-MaarselokTracker.lua
   |-MaarselokTracker.txt
   |-MaarselokTracker.xml
2nd thing - EVENT_ADD_ON_LOADED should be unregisteerd if not needed after addon was loaded:
Your event EVENT_ADD_ON_LOADED should be unregistered once your addon got loaded or it will be called for every other addon again. EVENT_ADD_ON_LOADED of each registred addon will be called each time an addon or ZOs code or a library is loaded again, that#s why the is addonName = myaddon.name check is in there.
If you do not need to recognize other addons (like you want to check if event's addonName variable is == "OtherTrackerName" to be sure it was loaded as well -> should be in your txt file's ##OptionalDependsOn: OtherTrackerName then to be sure it is loaded BEFORE your addon got loaded) you should unregister your's after it got loaded like this:
Lua Code:
  1. function MaarselokTracker.OnAddOnLoaded(event, addonName)
  2.   -- EVENT_ADD_ON_LOADED fires for each addon, check for own
  3.   if addonName ~= MaarselokTracker.NAME then return end
  4.   --We only get here if it's our addon, so initialize it and unregister the event
  5.   EventManager:UnregisterForEvent(MaarselokTracker.NAME, EVENT_ADD_ON_LOADED)
  6.   MaarselokTracker:Initialize()
  7. end

3rd - Event filtering for your own unitTag "player" only:
Well done with teh event filtering. It will only fire as your Maarselok abilityId is recognized but it will also fire if other players got it. Not sure if the game provides this information but to be sure you should also add an event_filter for your own player, which makes the game check less on lua side and more on c code side, making it work a bit faster. As the eventfilter and eventregister name bust be th same (and unique) for each eventName and filterName pair you can just reuse the same eventName MaarselokTracker.NAME as the filterName is another one (REGISTER_FILTER_UNIT_TAG):
Lua Code:
  1. EVENT_MANAGER:AddFilterForEvent(
  2.     MaarselokTracker.NAME,
  3.     EVENT_COMBAT_EVENT,
  4.     REGISTER_FILTER_UNIT_TAG,
  5.     "player"
  6.   )

4th proposal is only a minor thing, unregister update timer before registering it new:
Before using a RegisterForUpdate function you should unregister the old one with the same name just to be sure it is invalidated. If you are using the same name for them the unregister will happen automatically (I assume) as it will be overwritten with the new one.
But to be sure you could call
EVENT_MANAGER:UnregisterForUpdate(MaarselokTracker.NAME)
just before using EVENT_MANAGER:RegisterForUpdate(MaarselokTracker.NAME, ...).
If nothing was registered with this name it will silently fail and that's all.

Thanks again for the addon, I'm planign to play with the set and I think this is a nice helper.
Last edited by Baertram : 07/15/19 at 04:31 AM.
Report comment to moderator  
Reply With Quote
Post A Reply



Category Jump: