brittyazel/devtool

A multipurpose tool to assist with World of Warcraft addon development

devtool's Issues

Please add support for CLEU payload

Sorry I don't really know coding/github so not going to make a pull/push request or whatever it's called.

In BFA they removed the payload from CLEU, and you need to instead use CombatLogGetCurrentEventInfo to access the payload. Previously with VDT you could just click on the CLEU event and see the args, but now it does not work.

A very helpful users in the WeakAuras discord suggested I add this:

if event == "COMBAT_LOG_EVENT_UNFILTERED" then
    args = { event, CombatLogGetCurrentEventInfo() }
end

snippet to line 81 of the old github. I instead decided to do so at line 85 of your github and it appears to function well in my game in my limited (2 mins) testing.

When you are free could you please add this to the addon, thank you and I really appreciate you picking up the addon in the original author's absence!

How to inspect Functions?

Heyo @brittyazel i wanted to ask how to use Devtool to inspect Functions better?

Can i see which parameters a Function expects?

[BUG] Added events can not be monitored

Issue description:

Going to Events tab and adding any Event and starting monitoring, will result in no monitoring despite a message being printed that says "Start event monitoring: EventName"

Technical info:

• Addon Version:* 1.0.4
• Client: Retail
• Client Patch Version: 10.1.7

[BUG] bad argument #1 to 'insert'

3x DevTool/DevTool.lua:229: bad argument #1 to 'insert' (table expected, got nil)
[string "=[C]"]: in function `insert'
[string "@DevTool/DevTool.lua"]:229: in function `AddData'
[string "@sArena_Updated2_by_sammers/sArena.lua"]:273: in function <sArena_Updated2_by_sammers/sArena.lua:270>

Locals:
(*temporary) = nil
(*temporary) = <table> {
 value = "arena1"
 name = "My local var"
 indentation = 0
}
(*temporary) = "table expected, got nil"

Steps to reproduce:

1. Put DevTool:AddData(unit, "My local var") to the line #273 of https://github.com/Sammers21/sArena_Updated2_by_sammers/blob/master/sArena.lua#L273
   
2. Save & reload the interface. Type /sarena and press test button. The bug grabber fill show the error

Error on Addon load if an non existant event was added

How to reproduce:

1. Open VDT
2. Go into the Event Tab
3. Enter "ElvUI" in the bottom bar to add this "event"
4. Nothing will happen, because that event does not exist
5. Reload
6. VDT will crash while initializing with this error:

ViragDevToolListenerFrame:RegisterEvent(): ViragDevToolListenerFrame:RegisterEvent(): Attempt to register unknown event "ElvUI"
[string "=[C]"]: in function `RegisterEvent'
[string "@Interface/AddOns/ViragDevTool/Modules/ViragDevToolEvents.lua"]:30: in function `StartMonitorEvent'
[string "@Interface/AddOns/ViragDevTool/ViragDevTool.lua"]:159: in function `cmd'
[string "@Interface/AddOns/ViragDevTool/ViragDevTool.lua"]:428: in function `ExecuteCMD'
[string "@Interface/AddOns/ViragDevTool/ViragDevTool.lua"]:885: in function `SubmitEditBoxSidebar'
[string "*ViragDevTool.xml:662_OnEnterPressed"]:1: in function <[string "*ViragDevTool.xml:662_OnEnterPressed"]:1>

Locals:
(*temporary) = ViragDevToolListenerFrame {
 0 = <userdata>
}
(*temporary) = "ElvUI"

[Feature Request] Preserve shown state

Hello and thank you for continuing VDT!

VDT had the default behavior where it would pop up after a reload if it was also shown before that reload but DevTool does not have the same behavior, one has to open up the AddOn manually after every reload.
With how often reloads occur during AddOn development i really liked the behavior and would love to see it in DevTool. Maybe an option to enable it could be good?

Cheers!

Create release Asset of fork

With the change to WowUp, you can now have it install addons directly from a URL (Like GitHub!). https://wowup.io/guide/get-addons/overview

The only issue is that it is looking for a release asset in the branch (Im guessing it doesnt just like Tags).

I saw on the parent repo you were possibly trying to publish so this could be an easy way to add your fork for now.

Bug is gone

The bug I saw is now gone, just thought I'd let you know.

[BUG]

Issue description:

I tried to expand CommunitiesAddDialog and got the above error.

Lua error log:

1x Interface/AddOns/DevTool/DevTool.lua:292: wrong number of arguments to 'insert'
[string "=[C]"]: in function `insert'
[string "@Interface/AddOns/DevTool/DevTool.lua"]:292: in function `ExpandCell'
[string "@Interface/AddOns/DevTool/DevTool.lua"]:672: in function <Interface/AddOns/DevTool/DevTool.lua:668>
[string "@Interface/AddOns/DevTool/DevTool.lua"]:687: in function <Interface/AddOns/DevTool/DevTool.lua:681>

Locals:
(*temporary) = <table> {
 1 = <table> {
 }
 2 = <table> {
 }
 3 = <table> {
 }
 4 = <table> {
 }
 5 = <table> {
 }
 6 = <table> {
 }
 7 = <table> {
 }
 8 = <table> {
 }
 9 = <table> {
 }
 10 = <table> {
 }
 11 = <table> {
 }
 12 = <table> {
 }
 13 = <table> {
 }
 14 = <table> {
 }
 15 = <table> {
 }
 16 = <table> {
 }
 17 = <table> {
 }
}