search

brittyazel / Neuron

A full-featured World of Warcraft action bar and interface replacement addon
https://www.curseforge.com/wow/addons/neuron
MIT License
31 stars 21 forks source link

I believe this is a PEBKAC issue, but...Hunter traps flyout #413

Open Sifuedition opened 3 years ago

Sifuedition commented 3 years ago

/flyout ,spell:Freezing Trap(),Tar Trap():LINEAR:TOP:BOTTOM:2:CLICK

I thought I had this correct, but, it's throwing a bugcheck.

8x Neuron\Objects\ACTIONBUTTON_Flyout.lua:485: Usage: IsSpellKnown(spellID[, isPet]) string "=[C]": in function IsSpellKnown' [string "@Neuron\Objects\ACTIONBUTTON_Flyout.lua"]:485: in functionfilter_spell' [string "@Neuron\Objects\ACTIONBUTTON_Flyout.lua"]:758: in function GetDataList' [string "@Neuron\Objects\ACTIONBUTTON_Flyout.lua"]:799: in functionFlyout_UpdateData' [string "@Neuron\Objects\ACTIONBUTTON_Flyout.lua"]:1072: in function UpdateFlyout' [string "@Neuron\Neuron-GUI.lua"]:2682: in functionmacroText_OnEditFocusLost' [string "@Neuron\Neuron-GUI.lua"]:3102: in function <Neuron\Neuron-GUI.lua:3102> string "=[C]": in function `?' [string "@FrameXML\UIParent.lua"]:2388: in function <FrameXML\UIParent.lua:1269>

Locals: (temporary) = nil (temporary) = true

Any help is appreciated.

Sifuedition commented 3 years ago

Ugh. Just noticed. There are a list of error messages.

Turned off all addons except Neuron, Masque, due to an optional dependency, buggrabber and bugsack. Reset the profile. Dropped in my attempt at this flyout syntax.

16x Neuron\Objects\ACTIONBUTTON_Flyout.lua:485: Usage: IsSpellKnown(spellID[, isPet]) string "=[C]": in function IsSpellKnown' [string "@Neuron\Objects\ACTIONBUTTON_Flyout.lua"]:485: in functionfilter_spell' [string "@Neuron\Objects\ACTIONBUTTON_Flyout.lua"]:758: in function GetDataList' [string "@Neuron\Objects\ACTIONBUTTON_Flyout.lua"]:799: in functionFlyout_UpdateData' [string "@Neuron\Objects\ACTIONBUTTON_Flyout.lua"]:1072: in function UpdateFlyout' [string "@Neuron\Neuron-GUI.lua"]:2682: in functionmacroText_OnEditFocusLost' [string "@Neuron\Neuron-GUI.lua"]:3102: in function <Neuron\Neuron-GUI.lua:3102> string "=[C]": in function `?' [string "@FrameXML\UIParent.lua"]:2388: in function <FrameXML\UIParent.lua:1269>

Locals: (temporary) = nil (temporary) = true

4x Neuron\Objects\ACTIONBUTTON.lua:547: attempt to index field 'binder' (a nil value) [string "@Neuron\Objects\ACTIONBUTTON.lua"]:547: in function `?' [string "@Ace3\CallbackHandler-1.0\CallbackHandler-1.0-7.lua"]:119: in function <...Ons\Ace3\CallbackHandler-1.0\CallbackHandler-1.0.lua:119>

[string "@Ace3\CallbackHandler-1.0\CallbackHandler-1.0-7.lua"]:29: in function <...Ons\Ace3\CallbackHandler-1.0\CallbackHandler-1.0.lua:25> [string "@Ace3\CallbackHandler-1.0\CallbackHandler-1.0-7.lua"]:64: in function `Fire' [string "@Ace3\AceEvent-3.0\AceEvent-3.0-4.lua"]:120: in function <Ace3\AceEvent-3.0\AceEvent-3.0.lua:119>

jpmeyers commented 3 years ago

/flyout ,spell:Freezing Trap(),Tar Trap():LINEAR:TOP:BOTTOM:2:CLICK

Is that comma in front of the word "spell" in your actual macro, and if yes, could that be what is throwing the error ?

On Fri, Jan 15, 2021 at 6:48 PM Sifuedition notifications@github.com wrote:

Ugh. Just noticed. There are a list of error messages.

Turned off all addons except Neuron, Masque, due to an optional dependency, buggrabber and bugsack. Reset the profile. Dropped in my attempt at this flyout syntax.

16x Neuron\Objects\ACTIONBUTTON_Flyout.lua:485: Usage: IsSpellKnown(spellID[, isPet]) string "=[C]": in function IsSpellKnown' [string "@Neuron\Objects\ACTIONBUTTON_Flyout.lua"]:485: in function filter_spell'

in function GetDataList' [string "@Neuron\Objects\ACTIONBUTTON_Flyout.lua"]:799: in function Flyout_UpdateData'

in function UpdateFlyout' [string "@Neuron\Neuron-GUI.lua"]:2682: in function macroText_OnEditFocusLost' [string "@neuron https://github.com/neuron\Neuron-GUI.lua"]:3102: in function <Neuron\Neuron-GUI.lua:3102> string "=[C]": in function `?' [string "@FrameXML\UIParent.lua"]:2388: in function <FrameXML\UIParent.lua:1269>

Locals: (temporary) = nil (temporary) = true

4x Neuron\Objects\ACTIONBUTTON.lua:547: attempt to index field 'binder' (a nil value)

in function ?' [string "@Ace3\CallbackHandler-1.0\CallbackHandler-1.0-7.lua"]:119: in function <...Ons\Ace3\CallbackHandler-1.0\CallbackHandler-1.0.lua:119> [string "=[C]"]: ? [string "@Ace3\CallbackHandler-1.0\CallbackHandler-1.0-7.lua"]:29: in function <...Ons\Ace3\CallbackHandler-1.0\CallbackHandler-1.0.lua:25> [string "@Ace3\CallbackHandler-1.0\CallbackHandler-1.0-7.lua"]:64: in function Fire'

in function <Ace3\AceEvent-3.0\AceEvent-3.0.lua:119>

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/brittyazel/Neuron/issues/413#issuecomment-761277008, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOU37AHPE2Q5TJ4B2LXKTLS2DO6FANCNFSM4WEZZUGQ .

Sifuedition commented 3 years ago

Copied from an autowrite flyout that works. Should I remove it completely, or replace it with something else?

jpmeyers commented 3 years ago

I tend to write flyouts a little oddly, but they work. I'd try:

/flyout s:~freezing trap,~tar trap:l:t:b:2:c (or in a readable way: /flyout spell:~freezing trap,~tar trap:linear:top:bottom:2:click)

the tilde ~ is a tooltip scan switch (I think) and for some reason I can't get most spell or item flyouts that I write to work without them, but it feels like a minor inconvenience if it works.

On Sun, Jan 17, 2021 at 10:35 AM Sifuedition notifications@github.com wrote:

Copied from an autowrite flyout that works. Should I remove it completely, or replace it with something else?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/brittyazel/Neuron/issues/413#issuecomment-761840443, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOU37E2SXJSSIGCSCSSHZ3S2MGUDANCNFSM4WEZZUGQ .

Sifuedition commented 3 years ago

The tilde is a "like" operator. I had forgotten about that one. In a readable way, that would be flyout spell like freezing trap, like tar trap, etc That has resolved quite a few flyouts I've used before. Unfortunately, it is not working here. I also found this is affecting windfury weapon and flametongue weapon in a flyout on my shaman. Thank you for the suggestion and I was sure that would fix it, but unfortunately it didn't.

Sifuedition commented 3 years ago

I can't get self-written flyouts to work for anything. I have also tried some I used in the past: /flyout item:hearth, town:linear:right:left:4:mouse /flyout mount:Favorite:linear:right:left:10:mouse /flyout profession:Any, !fish:linear:right:left:10:mouse /flyout type:quest:linear:right:left:10:mouse /flyout type:bandage:linear:right:left:10:mouse

This all started with the last updates. I'm starting to wonder if another addon changed a cvar that neuron needs and disabling the addon is not changing the cvar back to the previous value, or something to that effect.

Sifuedition commented 3 years ago

Ok...I have shaman weapon embues working now, but, I don't know the real problem. I have AdiBags installed. I also installed the plugins AdiBags By Expansion and AdiBags Shadowlands. Both attempted to create a category named "Shadowlands" for sorting the bags. AdiBags still worked completely normally, and, this would appear to be completely unrelated. In looking at the Neuron errors, I noticed this error in the list as well. I removed all of the duplicate filters for AdiBags....and suddenly flyouts are working again.

Hunter traps are still not working.