ins's Portal : Bug --Setup Colours? for 0.6 - ESOUI

5

103,985

289

My AddOns

View Bug Report

--Setup Colours? for 0.6
Bug #:	45
File:	ins:JunkYard
Date:	04/02/14 05:35 PM
By:	graywind
Status:	Fixed
I kept getting errors with the concatenating nils, and using globals for the colours removed a lot of errors for me. I'm just messing around with your addon right now on the US server. If you ever log there I'm @Uberraschungen :) Ex: c1 = "\|cFFFFFF" -- white c2 = "\|cFFFF00" --yellow c3 = "\|c00FF00" --green c4 = "\|c00FFFF" -- teal

Notes

By: ins - 04/02/14 05:39 PM

Cheers.

Any chance you could take a screenshot or a cut and paste of the error message you get?

I'm not sure what you mean with concatenating nils, so an image/text copy would be nice to have to see the error message.

Glad to hear it works with globals, they used to be globals but saw someone mention it being bad in another thread

Now I'm gonna sleep while they struggle to patch EU

By: graywind - 04/02/14 05:48 PM

Thats kinda odd that globals would be bad for something that is static all the time and only ever set once. Can other addons stomp on them? That doesn't make much sense though. I'm assuming you could have unexpected behavior for variables that are in constant flux but I don't see how it could go wrong here. Link me to the thread if you could!

Concatenation is longwinded way of saying adding many strings into one. The problem here is when you call the colors that are locally set, they aren't found in your other functions (and are therfore nil, or null in other languages) You cant take a nil value and .. it to another string. I can undo my changes and show you in a screenshot though. Have a good rest!

By: ins - 04/02/14 06:00 PM

I (THINK I) found out why it works with globals, and why I do not get the error message.

Another addon I made uses the same colour variables, but there they are globals.

I'll rename the variables to something more 'unique' and put them up as globals.

Thanks for the heads up.

Now I really should go to bed.

By: graywind - 04/02/14 06:05 PM

It looks like there is no way to attach an image so here you go http://imgur.com/WdTYlyW

By: ins - 04/02/14 06:11 PM

I updated the addon now, 0.6.b, should be available shortly. Hopefully it is fixed now and I haven't introduced something new

EU still down so I cant test it.

By: ins - 04/02/14 06:11 PM

Thanks for the screenshot

By: ins - 04/02/14 06:54 PM