Download
(118Kb)
Download
Updated: 11-02-19 12:07 AM
Pictures
File Info
Compatibility:
Classic (1.13.2)
Updated:11-02-19 12:07 AM
Created:09-20-19 05:15 PM
Downloads:10,568
Favorites:16
MD5:
Categories:Classic - General, ToolTip

Theorycraft Classic [Quickfix]  Popular! (More than 5000 hits)

Version: 1.0.3
by: boothin [More]

Quick fix of theorycraft to work with WoW Classic. Some functionality from the original was taken out, but may be reimplemented later. Not everything will work as this is a work in progress, many things still need to be found and converted to work with the current API.

Use /tc to open the options window.

github repo at: https://github.com/Boothin/TheorycraftClassic if anyone would like to help fix and update this to fall in line with the modern API

1.0.3
Merged fix for missing argument, tooltips should now update on events like equipment changes

1.0.2
Fix missing calls

1.0.1b
Reupload of correct version

1.0.1
Additional removal of unnecessary bits and pieces.

1.0.0
Initial Release
Optional Files (0)


Post A Reply Comment Options
Unread 11-24-21, 01:47 AM  
endymon
A Kobold Labourer

Forum posts: 0
File comments: 13
Uploads: 0
I am currently working on a version for TBC.
There is a lot of work left to be done, and only one of me.
https://www.curseforge.com/wow/addon...ycraftclassic2
Report comment to moderator  
Reply With Quote
Unread 11-13-19, 10:24 AM  
redworm
A Defias Bandit

Forum posts: 2
File comments: 68
Uploads: 0
the crap aint working, you can open the tc menü and let show up hps or dps, but all +dmg or +heal and talents wont add up so its basicaly useless to use it in the first place.
Report comment to moderator  
Reply With Quote
Unread 10-21-19, 02:48 PM  
ptyyy
A Kobold Labourer

Forum posts: 0
File comments: 1
Uploads: 0
Originally Posted by Nihlo
Getting errors with latest version:

Code:
Message: ...AddOns\TheorycraftClassic\TheoryCraftClassicGear.lua:87: attempt to call a table value
Time: Sat Sep 21 14:45:33 2019
Count: 1
Stack: ...AddOns\TheorycraftClassic\TheoryCraftClassicGear.lua:87: attempt to call a table value

Locals:
Code:
Message: [string "TheoryCraft:OnMouseUp"]:2: attempt to call global 'TheoryCraft_UpdateDummyButtonText' (a nil value)
Time: Sat Sep 21 14:49:09 2019
Count: 2
Stack: [string "TheoryCraft:OnMouseUp"]:2: attempt to call global 'TheoryCraft_UpdateDummyButtonText' (a nil value)

Locals:
Code:
Message: ...ddOns\TheorycraftClassic\TheoryCraftClassicMessy.lua:88: 'string.gfind' was renamed to 'string.gmatch'
Time: Sat Sep 21 14:51:40 2019
Count: 1
Stack: ...ddOns\TheorycraftClassic\TheoryCraftClassicMessy.lua:88: 'string.gfind' was renamed to 'string.gmatch'

Locals:
For the first error (line 87 "attempt to call a table value"), to fix it you need to change
Code:
TheoryCraft_SetBonuses[setname]
to
Code:
pairs(TheoryCraft_SetBonuses[setname])
.
Report comment to moderator  
Reply With Quote
Unread 10-08-19, 12:20 AM  
Tubular1845
A Kobold Labourer

Forum posts: 0
File comments: 1
Uploads: 0
.....
Last edited by Tubular1845 : 02-25-21 at 03:18 PM.
Report comment to moderator  
Reply With Quote
Unread 10-07-19, 06:17 PM  
sunsmoon
A Defias Bandit
 
sunsmoon's Avatar

Forum posts: 3
File comments: 14
Uploads: 0
Originally Posted by andreuccio12
this is kinda awkward, but in the screenshot the casting time for Heal shown as 2.5s, while it should be 3.
The other parameters are correct, though, except HPS, which is basen on the wrong cast time.
Heal cast time is reduced to 2.5 sec through 5 points in Divine Fury (2nd tier Holy talent, left side).
Report comment to moderator  
Reply With Quote
Unread 10-02-19, 07:23 AM  
andreuccio12
A Kobold Labourer

Forum posts: 0
File comments: 1
Uploads: 0
this is kinda awkward, but in the screenshot the casting time for Heal shown as 2.5s, while it should be 3.
The other parameters are correct, though, except HPS, which is basen on the wrong cast time.
Report comment to moderator  
Reply With Quote
Unread 09-26-19, 06:57 PM  
Wyveryx
A Murloc Raider
 
Wyveryx's Avatar

Forum posts: 5
File comments: 8
Uploads: 0
I downloaded the Sarf version, yet my tooltips still show nothing either in the Spellbook or on the action bar tooltip. However, there is a weird tooltip issue that arises, that may be what was discussed in the pull on github.





Is there something I am missing to correct this?
Last edited by Wyveryx : 09-26-19 at 06:58 PM.
Report comment to moderator  
Reply With Quote
Unread 09-23-19, 07:48 AM  
Tydor
A Deviate Faerie Dragon
AddOn Author - Click to view AddOns

Forum posts: 13
File comments: 16
Uploads: 1
hey awesome work.
is there a way to have the spelldamage displayed directly on the spellicon ?
i think dr.damage did that long time ago.
Report comment to moderator  
Reply With Quote
Unread 09-22-19, 06:24 AM  
erispope
A Deviate Faerie Dragon

Forum posts: 10
File comments: 3
Uploads: 0
I found a fix for the tooltip but it has side effects (available as a pull request and on my fork). I think the display and calculation part should be split up further, which is what I'll do going forward.

Not entirely sure the whole equipment scanning part can be cut out, will have to check into what is available from Blizzard functions.

An alternative to the tooltip data collection currently done would be a static database. I think the best way is to make an optional add-on that collects data that is stored in the main add-on... But I prattle on.
Report comment to moderator  
Reply With Quote
Unread 09-22-19, 12:11 AM  
Freezy3
A Kobold Labourer

Forum posts: 0
File comments: 9
Uploads: 0
I've been looking for something like this ever since beta came up. Was suggested this for a long time but could never find the port. Trying it on classic and the tooltip will display on bars for a split second before disappearing to the standard tooltip. Even with disabling all addons and only having TC up it still won't display properly. It works perfectly for spellbooking though.
EDIT: I just realized the display issue is with macro spells.
Last edited by Freezy3 : 09-22-19 at 04:55 AM.
Report comment to moderator  
Reply With Quote
Unread 09-21-19, 03:52 PM  
Nihlo
A Murloc Raider

Forum posts: 7
File comments: 158
Uploads: 0
Originally Posted by boothin
Are you a rogue? My druid friend doesn't seem to have this issue with cat form but I'll see what i can figure out when I have a chance
Nope druid...and a new one:

Code:
Message: ...dOns\TheorycraftClassic\TheoryCraftClassicEngine.lua:1541: Usage: GetComboPoints("unit"[, "target"]
Time: Sat Sep 21 23:25:22 2019
Count: 1
Stack: ...dOns\TheorycraftClassic\TheoryCraftClassicEngine.lua:1541: Usage: GetComboPoints("unit"[, "target"]

Locals:
Report comment to moderator  
Reply With Quote
Unread 09-21-19, 11:45 AM  
boothin
A Kobold Labourer
AddOn Author - Click to view AddOns

Forum posts: 0
File comments: 5
Uploads: 1
Originally Posted by puda74
Originally Posted by Nihlo
You might need the old BonusScanner addon first to gather +heal I guess.

Can't remember what I did at the first error, either hovering a spell or something in the options.
The new API has calls for "bonusHeal = GetSpellBonusHealing();" and "spellDmg = GetSpellBonusDamage(spellTreeID);" spellTreeID being 1 for Physical 2 for Holy 3 for Fire 4 for Nature 5 for Frost 6 for Shadow 7 for Arcane.

Theres no need for a bonus scanner type addon.
Thanks for this, I'll work on refactoring the addon code to use this instead of what it was doing before
Report comment to moderator  
Reply With Quote
Unread 09-21-19, 11:44 AM  
boothin
A Kobold Labourer
AddOn Author - Click to view AddOns

Forum posts: 0
File comments: 5
Uploads: 1
Originally Posted by Nihlo
got this lua 87 again...after hearthstoning

Code:
Message: ...AddOns\TheorycraftClassic\TheoryCraftClassicGear.lua:87: attempt to call a table value
Time: Sat Sep 21 19:18:54 2019
Count: 1
Stack: ...AddOns\TheorycraftClassic\TheoryCraftClassicGear.lua:87: attempt to call a table value

Locals:
and a new one:

Code:
Message: ...dOns\TheorycraftClassic\TheoryCraftClassicEngine.lua:1541: Usage: GetComboPoints("unit"[, "target"]
Time: Sat Sep 21 19:29:25 2019
Count: 1
Stack: ...dOns\TheorycraftClassic\TheoryCraftClassicEngine.lua:1541: Usage: GetComboPoints("unit"[, "target"]

Locals:
actually doing nothing on the bars...like jumping down the dam.
Are you a rogue? My druid friend doesn't seem to have this issue with cat form but I'll see what i can figure out when I have a chance
Report comment to moderator  
Reply With Quote
Unread 09-21-19, 11:20 AM  
Nihlo
A Murloc Raider

Forum posts: 7
File comments: 158
Uploads: 0
got this lua 87 again...after hearthstoning

Code:
Message: ...AddOns\TheorycraftClassic\TheoryCraftClassicGear.lua:87: attempt to call a table value
Time: Sat Sep 21 19:18:54 2019
Count: 1
Stack: ...AddOns\TheorycraftClassic\TheoryCraftClassicGear.lua:87: attempt to call a table value

Locals:
and a new one:

Code:
Message: ...dOns\TheorycraftClassic\TheoryCraftClassicEngine.lua:1541: Usage: GetComboPoints("unit"[, "target"]
Time: Sat Sep 21 19:29:25 2019
Count: 1
Stack: ...dOns\TheorycraftClassic\TheoryCraftClassicEngine.lua:1541: Usage: GetComboPoints("unit"[, "target"]

Locals:
actually doing nothing on the bars...like jumping down the dam.


Code:
Message: ...dOns\TheorycraftClassic\TheoryCraftClassicEngine.lua:1541: Usage: GetComboPoints("unit"[, "target"]
Time: Sat Sep 21 23:25:22 2019
Count: 1
Stack: ...dOns\TheorycraftClassic\TheoryCraftClassicEngine.lua:1541: Usage: GetComboPoints("unit"[, "target"]

Locals:
Last edited by Nihlo : 09-21-19 at 03:25 PM.
Report comment to moderator  
Reply With Quote
Unread 09-21-19, 10:29 AM  
puda74
A Kobold Labourer
AddOn Author - Click to view AddOns

Forum posts: 0
File comments: 5
Uploads: 2
Originally Posted by Nihlo
You might need the old BonusScanner addon first to gather +heal I guess.

Can't remember what I did at the first error, either hovering a spell or something in the options.
The new API has calls for "bonusHeal = GetSpellBonusHealing();" and "spellDmg = GetSpellBonusDamage(spellTreeID);" spellTreeID being 1 for Physical 2 for Holy 3 for Fire 4 for Nature 5 for Frost 6 for Shadow 7 for Arcane.

Theres no need for a bonus scanner type addon.
Report comment to moderator  
Reply With Quote
Post A Reply



Category Jump: