Census+ and Spartan UI Artwork

Found a bug with the CensusPlus UI Mod? Post it here!
Post Reply
User avatar
Poisonenvy
Posts: 48
Joined: Wed Jun 25, 2008 4:10 pm
Location: UK
Contact:

Census+ and Spartan UI Artwork

Post by Poisonenvy »

I've only just realised that I've not been uploading any info for a while .. when i've investigated I've tracked it down to a specific part of a specific addon.

I disabled ALL addons other than Census+ and it worked okay.

When I re-enabled my other addons one at a time, I managed to narrow it down to Spartan UI .. but only when the "artwork" part of it was loaded. Which is a problem as that is the main point of Spartan UI.

With the artwork loaded, Census+ opens, shows the interface, but when you try to actually run it, it completes immediately saying it didn't find anyone.

It looks like they are putting out fairly regular updates to Spartan at the moment, so I'll keep an eye on it and see if a fix turns up.

The most recent alpha update at the moment allows census+ to work, but it is a bit broken (as the artwork part of it wasn't working!)

User avatar
bringoutyourdead
Forums Admin & general flunky
Posts: 1432
Joined: Fri Nov 07, 2008 1:11 pm
Location: Texas, USA

Post by bringoutyourdead »

Multiple problems seen here.

An already spotted problem in Wholib uses a variable that can be modified by other addons.
This was noticed and a suggested fix has been sent to the library manager.

The current SpartanUI is very addon unfriendly as it leaks a number of standardized words as global variable instead of making them localized to the addon.

The file that seems to be the our culprit is SpartanUI\scripts\Module-FontSettings.lua
Which leaks the common words 'flags' and 'size' as globals. The first is what is killing Wholib.

User avatar
Poisonenvy
Posts: 48
Joined: Wed Jun 25, 2008 4:10 pm
Location: UK
Contact:

Post by Poisonenvy »

Ah right, okay thanks. Is there anything I can do, other than either disable it when needed or waiting for an update?

User avatar
bringoutyourdead
Forums Admin & general flunky
Posts: 1432
Joined: Fri Nov 07, 2008 1:11 pm
Location: Texas, USA

Post by bringoutyourdead »

if you have any comfort with editing code.

You can go to http://www.wowace.com/addons/wholib/tic ... ter-error/

and make the change as suggested by Esiemiat.
But be aware this might (not likely) break other addons that use the Libwho-2.0 library.

I added this change to my personal copy of CensusPlus and found that I could do census runs with the SpartanUI suite enable.
But do be aware if you run any error catcher code.. SpartanUI generates a number of errors and warnings.
It needs serious fixing.

User avatar
Poisonenvy
Posts: 48
Joined: Wed Jun 25, 2008 4:10 pm
Location: UK
Contact:

Post by Poisonenvy »

Okay thank you for that. I may just run CensusPlus for a while on my banker character and make sure that one doesn't use Spartan, and hope they fix it properly. I know they've been doing a few changes to it recently, so fingers crossed it will get sorted.

nosredna2101

Post by nosredna2101 »

Have you submitted this to SpartanUI's Authors yet? I know he responds really quickly if you post something on http://SpartanUI.net/. The Curse comments are not all that reliable.

User avatar
Poisonenvy
Posts: 48
Joined: Wed Jun 25, 2008 4:10 pm
Location: UK
Contact:

Post by Poisonenvy »

nosredna2101 wrote:Have you submitted this to SpartanUI's Authors yet? I know he responds really quickly if you post something on http://SpartanUI.net/. The Curse comments are not all that reliable.
I've done so now.

http://spartanui.net/question/i-have-an ... t-to-fail/

Post Reply