Help with 1.4 oUF - Page 2

yj589794 · 08-03-10, 02:54 AM

Originally Posted by Game92

tryed to change somthings didnt go that welll

to spawn the raid frames i guess

*cough* post a link to your modified code *cough*

Aftermathhqt · 08-03-10, 09:04 AM

Its the same as the pastbin

yj589794 · 08-03-10, 09:24 AM

No, it's not.

The original pastebin link you posted had 980 lines of code.
The updated pastebin link I posted had 985 lines of code.

The last error message you posted indicated that the error was from line 1001, which does not exist in either pastebin link.

Can you check again, please?

Aftermathhqt · 08-10-10, 06:55 AM

sorry, i've been abit on holiday, soo im gonna try to update this

so lets try again!
and i've cleaned up the code abit... so here is it http://pastebin.com/24kTGqAx and here is the error:

Code:

Message: Interface\AddOns\oUF_Aftermathh\oUF_Aftermathh.lua:782: Bad argument #2 to 'Spawn' (string expected, got nil
Time: 08/10/10 14:54:57
Count: 1
Stack: [C]: in function `error'
Interface\AddOns\oUF\ouf.lua:14: in function <Interface\AddOns\oUF\ouf.lua:5>
Interface\AddOns\oUF\ouf.lua:664: in function `Spawn'
Interface\AddOns\oUF_Aftermathh\oUF_Aftermathh.lua:782: in main chunk

Locals: (*temporary) = "Bad argument #2 to 'Spawn' (string expected, got nil"

i hope we can make it work!

Mischback · 08-10-10, 07:15 AM

Originally Posted by Mischback

With 1.4, this function is deprecated, haste documented the new Spawn() and SpawnHeader() here.

You simple incorporate the options from SetManyAttributes into a SpawnHeader().

You want to use "SpawnHeader" instead of "Spawn".

Rostok · 08-10-10, 07:15 AM

Replace Spawn with SpawnHeader line 782.

Aftermathhqt · 08-10-10, 07:35 AM

Okey, done now i got this error:

Code:

Message: Interface\AddOns\oUF\elements\cpoints.lua:22: attempt to index field '?' (a nil value)
Time: 08/10/10 15:35:12
Count: 1
Stack: Interface\AddOns\oUF\elements\cpoints.lua:22: in function <Interface\AddOns\oUF\elements\cpoints.lua:7>
(tail call): ?
[C]: in function `ClearTarget'
Interface\FrameXML\UIParent.lua:2896: in function `ToggleGameMenu'
[string "TOGGLEGAMEMENU"]:1: in function <[string "TOGGLEGAMEMENU"]:1>

Locals: self = oUF_AftermathhTarget {
 ignoreHealComm = true
 UNIT_PORTRAIT_UPDATE = <function> defined @Interface\AddOns\oUF\elements\portraits.lua:4
 Combat = <unnamed> {
 }
 __tags = <table> {
 }
 PLAYER_ENTERING_WORLD = <function> defined @Interface\AddOns\oUF\ouf.lua:315
 Border = <table> {
 }
 UNIT_SPELLCAST_CHANNEL_START = <function> defined @Interface\AddOns\oUF\elements\castbar.lua:166
 UNIT_SPELLCAST_INTERRUPTIBLE = <function> defined @Interface\AddOns\oUF\elements\castbar.lua:100
 __elements = <table> {
 }
 OverrideUpdateHealth = <function> defined @Interface\AddOns\oUF_Aftermathh\oUF_Aftermathh.lua:92
 Castbar = <unnamed> {
 }
 UNIT_SPELLCAST_NOT_INTERRUPTIBLE = <function> defined @Interface\AddOns\oUF\elements\castbar.lua:114
 DebuffHighlightFilter = true
 UNIT_COMBO_POINTS = <function> defined @Interface\AddOns\oUF\elements\cpoints.lua:7
 style = "Aftermathh"
 RAID_TARGET_UPDATE = <function> defined @Interface\AddOns\oUF_Aftermathh\oUF_Aftermathh.lua:82
 Debuffs = <unnamed> {
 }
 Portrait = <unnamed> {
 }
 PLAYER_TARGET_CHANGED = <function> defined @Interface\AddOns\oUF\elements\cpoints.lua:7
 PostCreateAuraIcon = <function> defined @Interface\AddOns\oUF_Aftermathh\oUF_Aftermathh.lua:137
 UNIT_SPELLCAST_START = <function> defined @Interface\AddOns\oUF\elements\castbar.lua:15
 unit = "target"
 UNIT_SPELLCAST_DELAYED = <function> defined @Interface\AddOns\oUF\elements\castbar.lua:128
 UNIT_SPELLCAST_CHANNEL_STOP = <function> defined @Interface\AddOns\oUF\elements\castbar.lua:235
 UNIT_SPELLCAST_CHANNEL_UPDATE = <function> defined @Interface\AddOns\oUF\elements\castbar.lua:212
 UNIT_SPELLCAST_INTERRUPTED = <function> defined @Interface\AddOns\oUF\elements\castbar.lua:82
 HasBorder = true
 PvP = <unnamed> {
 }
 UNIT_AURA = <function> defined @Interface\AddOns\oUF\elements\aura.lua:221
 UNIT_SPELLCAST_STOP = <function> defined @Interface\AddOns\oUF\elements\castbar.lua:148
 UNIT_SPELLCAST_FAILED = <function> defined @Interface\AddOns\oUF\elements\castbar.lua:64
 Health = <unnamed> {
 }
 0 = <userdata>
 menu = <function> defined @Interface\AddOns\oUF_Aftermathh\oUF_Aftermathh.lua:55
 disallowVehicleSwap = true
 CPoints = <unnamed> {
 }
 UNIT_FACTION = <function> defined @Interface\AddOns\oUF\elements\pvp.lua:4
 colors = <table> {
 }
 PostUpdatePower = <function> defined @Interface\AddOns\oUF_Aftermathh\oUF_Aftermathh.lua:125
 DebuffHighlightBackdrop = false
 Info = <unnamed> {
 }
 Power = <unnamed> {
 }
 RIcon = <unnamed> {
 }
 Buffs = <unnamed> {
 }
 CombatFeedbackText = <unnamed> {
 }
 UNIT_MODEL_CHANGED = <function> defined @Interface\AddOns\oUF\elements\portraits.lua:4
}
event = "PLAYER_TARGET_CHANGED"
unit = nil
cp = 0
cpoints = <unnamed> {
 0 = <userdata>
}
(for index) = 1
(for limit) = 5
(for step) = 1
i = 1
(*temporary) = nil
(*temporary) = nil
(*temporary) = "attempt to index field '?' (a nil value)"
GetComboPoints = <function> defined =[C]:-1
MAX_COMBO_POINTS = 5

Mischback · 08-10-10, 07:44 AM

oUF's ComboPoint-module is working graphically, not with a string.

You'll have to create an array with MAX_COMBO_POINTS (5) texture-elements. These textures will be shown/hidden on gain/loss.

If you want a pure text display, you'll have to do it yourself.

yj589794 · 08-10-10, 07:46 AM

Originally Posted by Mischback

If you want a pure text display, you'll have to do it yourself.

No you don't, there is a default tag for it now [cpoints]

Code:

	local cpoints = self:CreateFontString(nil, 'OVERLAY')
	cpoints:SetFont(font, 18, 'OUTLINE')
	cpoints:SetPoint('RIGHT', self, 'LEFT', -10, 0)
	cpoints:SetJustifyH('RIGHT')
	self:Tag(cpoints, '[cpoints]')

Mischback · 08-10-10, 07:48 AM

Ahhh, I'll really have to look into this tag-module... Seems so powerful!

Aftermathhqt · 08-10-10, 07:54 AM

okey, I just remove'd it for now... now only one problem.. my textures from buffs/debuffs is gone? and my health bars is only white?

here is a pic:

Aftermathhqt · 08-10-10, 08:00 AM

Originally Posted by yj589794

No you don't, there is a default tag for it now [cpoints]

Code:

	local cpoints = self:CreateFontString(nil, 'OVERLAY')
	cpoints:SetFont(font, 18, 'OUTLINE')
	cpoints:SetPoint('RIGHT', self, 'LEFT', -10, 0)
	cpoints:SetJustifyH('RIGHT')
	self:Tag(cpoints, '[cpoints]')

thanks got it to work again! =)

now is it only the healthbars

and buffs textures

yj589794 · 08-10-10, 08:00 AM

not related to your current issues, but at lines 117 and 119 you are assigning values to x, y and z. These variables are not declared as local so you are throwing them into the global namespace

Aftermathhqt · 08-10-10, 08:06 AM

Hehe okey i didnt really add that it was Jasje, it's hes base of the layout but i highly modifyed the hole layout how i wanted :P

this is the orgnial of Jasje's layout.. soo you can see how much i changed etc..

yj589794 · 08-10-10, 08:15 AM

For the aura textures for are assigning your PostCreateIcon function incorrectly.

You need to remove line 691:

Code:

self.PostCreateAuraIcon = auraIcon

where you add buffs (around 495) add in the following line:

Code:

buffs.PostCreateIcon = auraIcon

where you add debuffs (around 520) add in the following line:

Code:

debuffs.PostCreateIcon = auraIcon

** this will probably cause errors within the auraIcon function, but that's where we can fix some more stuff

Mischback · 08-10-10, 08:20 AM

Originally Posted by Game92

and my health bars is only white?

In the layout you're setting the status-bar-color in an OverrideUpdateHealth-function, but this hook is no longer present in oUF1.4

You could go with PostUpdate() or simply set the statusbar-color directly in the style-function

Aftermathhqt · 08-10-10, 08:24 AM

Originally Posted by yj589794

For the aura textures for are assigning your PostCreateIcon function incorrectly.

You need to remove line 691:

Code:

self.PostCreateAuraIcon = auraIcon

where you add buffs (around 495) add in the following line:

Code:

buffs.PostCreateIcon = auraIcon

where you add debuffs (around 520) add in the following line:

Code:

debuffs.PostCreateIcon = auraIcon

** this will probably cause errors within the auraIcon function, but that's where we can fix some more stuff

'

okey did that

and yeah, i got lots of error :P

yj589794 · 08-10-10, 08:37 AM

Originally Posted by Game92

'

okey did that

and yeah, i got lots of error :P

OK, any chance of a hint at what the errors were?

Originally Posted by Mischback

In the layout you're setting the status-bar-color in an OverrideUpdateHealth-function, but this hook is no longer present in oUF1.4

You could go with PostUpdate() or simply set the statusbar-color directly in the style-function

I'm not liking the look of the updateHealth function, not liking it one bit. It's going to take some significant changes to separate the health value updating from the health bar colouring

Aftermathhqt · 08-10-10, 08:44 AM

[quote=yj589794;201925]OK, any chance of a hint at what the errors were?

yes!

Code:

Message: Interface\AddOns\oUF_Aftermathh\oUF_Aftermathh.lua:141: attempt to index local 'icons' (a nil value)
Time: 08/10/10 16:43:55
Count: 1
Stack: Interface\AddOns\oUF_Aftermathh\oUF_Aftermathh.lua:141: in function `PostCreateIcon'
Interface\AddOns\oUF\elements\aura.lua:75: in function <Interface\AddOns\oUF\elements\aura.lua:28>
Interface\AddOns\oUF\elements\aura.lua:99: in function <Interface\AddOns\oUF\elements\aura.lua:94>
Interface\AddOns\oUF\elements\aura.lua:203: in function <Interface\AddOns\oUF\elements\aura.lua:198>
Interface\AddOns\oUF\elements\aura.lua:249: in function `func'
Interface\AddOns\oUF\ouf.lua:320: in function <Interface\AddOns\oUF\ouf.lua:315>
[C]: in function `Show'
Interface\FrameXML\SecureStateDriver.lua:72: in function <Interface\FrameXML\SecureStateDriver.lua:62>
Interface\FrameXML\SecureStateDriver.lua:111: in function <Interface\FrameXML\SecureStateDriver.lua:81>

Locals: self = <unnamed> {
 1 = <unnamed> {
 }
 spacing = 0
 PostCreateIcon = <function> defined @Interface\AddOns\oUF_Aftermathh\oUF_Aftermathh.lua:137
 num = 9
 initialAnchor = "TOPLEFT"
 0 = <userdata>
 growth-x = "DOWN"
 growth-y = "RIGHT"
 size = 28
}
button = <unnamed> {
 overlay = <unnamed> {
 }
 cd = <unnamed> {
 }
 UpdateTooltip = <function> defined @Interface\AddOns\oUF\elements\aura.lua:7
 count = <unnamed> {
 }
 0 = <userdata>
 parent = <unnamed> {
 }
 icon = <unnamed> {
 }
 stealable = <unnamed> {
 }
}
icons = nil
count = <unnamed> {
 0 = <userdata>
}
(*temporary) = <function> defined =[C]:-1
(*temporary) = <unnamed> {
 0 = <userdata>
}
(*temporary) = "BOTTOM"
(*temporary) = <unnamed> {
 overlay = <unnamed> {
 }
 cd = <unnamed> {
 }
 UpdateTooltip = <function> defined @Interface\AddOns\oUF\elements\aura.lua:7
 count = <unnamed> {
 }
 0 = <userdata>
 parent = <unnamed> {
 }
 icon = <unnamed> {
 }
 stealable = <unnamed> {
 }
}
(*temporary) = 1
(*temporary) = 7
(*temporary) = "attempt to index local 'icons' (a nil value)"
border = "Interface\AddOns\oUF_Aftermathh\media\border"

yj589794 · 08-10-10, 08:54 AM

change line 137 from:

Code:

local auraIcon = function(self, button, icons)

to:

Code:

local auraIcon = function(icons, button)