Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BfA 8.00 #7

Closed
StanoRiga opened this issue Jul 19, 2018 · 7 comments
Closed

BfA 8.00 #7

StanoRiga opened this issue Jul 19, 2018 · 7 comments
Assignees

Comments

@StanoRiga
Copy link

Button does not appear in BfA.
I tried to fix it by my self, but seems that I have no clue about addon programming :(

@patrickjDE
Copy link
Owner

Some Lua errors popping up by any chance?

I don't have an active subscription right now, so sadly I cannot test and debug it myself...

@StanoRiga
Copy link
Author

Hallo Patrick,

ich bin so frei uns wechsle auf Deutsch.

Nein, keine LUA Fehlermeldungen. Hab mir extra nen Grabber Addon geholt.
Ich bin den Quellcode mal durchgegangen und hab ein bisschen herum gespielt, und dabei festgestellt dass der Button nicht mehr erscheint weil die if-Abfrage
tradeSkillName == enchantingTradeSkillName
fehlschlägt und daher die else Schleife greift und den Button ausblendet.
Wenn ich die komplette elso/if abfrage entferne, dann passts wieder (weiter keine lua fehler, button ist auch da, und verhält sich korrekt.

Info:
tradeSkillName = "Verzauberkunst der Verehrten Inseln"
enchantingTradeSkillName = "Verzauberkunst"

Meine Frage ist nun: Was war der Grund für diesen Vergleich?

Und nachdem ab nächster Woche vermutlich sehr viele neue Spells dazu kommen, kannst Du mir vielleicht auch erklären wo ich die "mapSpellToItem" Werte her bekommen kann?
(wenn ich ehrlich bin verstehe ich nicht was da gemappt wird. Eine Spell ID finde ich auf WowHead, aber das mapping?)

Vorweg:vielen Dank für Deine Hilfe.
Ich hab früher bissi PHP und C# programmiert, aber WoW Addons sind mir nicht so geläufig.
Und da ich im Monat gut und gerne 100 Rollen verkaufe, ist dieses Addon Gold wert :)

PS: ich war so frei und hab auf curse einen Quick Fix verlinkt für alle die das Addon in BfA PRE nutzen wollen.

@patrickjDE
Copy link
Owner

Der Abgleich verhindert, dass der Button auch für andere Berufe angezeigt wird, die nichts damit anfangen können.
mapSpellToItem macht eigentlich genau was der Name sagt: SpellID (der VZ) zu ItemID (der Rolle) mappen.

Könntest du das Update testen, ob damit alles wieder funktioniert? Dann kann ich die Version auch auf WoWI und Curse updaten.
https://github.com/patrickjDE/OneClickEnchantScroll/releases/download/8.0.1.0-beta/OneClickEnchantScroll-8.0.1.0-beta.zip

@nancikennedy
Copy link

I just tested the fix. The button shows up when you click on an enchant you can put on a scroll. It correctly created the scroll. After that, it posts two "scan recipes" comments to chat and reloads the tradeskill window, and the selection defaults back to the first item in the window.

@patrickjDE
Copy link
Owner

After that, it posts two "scan recipes" comments to chat and reloads the tradeskill window, and the selection defaults back to the first item in the window.

I'm pretty sure that's another AddOn, there's no code that would output such comments or reload the window.

@StanoRiga
Copy link
Author

I tested also. Deactivated all other addons.

  1. Opened a profession different to enchanting: no button -> OK
  2. Opened enchanting, selected a legion enchant: button is shown and recipe is created ->OK
  3. Opened enchanting, selected a dreanor enchant: button is shown and recipe is created ->OK
  4. Tested everything again with BugGrabber/BugSake activated to itentify lua errors: -> no errors
    From my point of view: everything is ok.

Note: I also got the "scan recipes" comment shown first, but I had activated all my addons. It turned out that this message is caused (in my case) by the DataStore addon

@patrickjDE
Copy link
Owner

Great news :)
I'll push that version as a release later today.

Thanks @StanoRiga & @nancikennedy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants