WhoLib Compatibility

akhtar · Posted: Sat Aug 08, 2009 9:14 am Post subject: WhoLib Compatibility

I'm creating this topic to discuss getting census+ to work with wholib.

Essentially, any addon that wants to run background /whos will work fine on its own - as long as its the only such addon a user is running. When there are more than one - the addons often interact very badly and in difficult to fix ways.

Example: Consider that you have 3 addons that each run a who request at the exact same time. The server sends back one response. Which /who was it to?

WhoLib is supposed to make life easy for addon developers, and it actually does - if you support it in your addon.

If you don't support wholib, well then wholib is just like any other addon which want to run who requests in the background - you will have conflicts.

There are 2 main strategies:

1) Use wholib as part of your addon
2) Use wholib if it is there and your original code if not.

I have modded the current census+ to support wholib if it is there, and work normally if not. I haven't done much testing, I'm hoping that folks will pitch in and help. I posted here almost a year ago about this - but we were all busy getting ready for 3.0 i guess, and noone followed up.

Here's a link to the modded version:

http://www.wowace.com/addons/census-wholib-version/

and the ticket on wowace.com

http://www.wowace.com/addons/wholib/tickets/10-census-compat/

Sylvanaar

Rollie · Posted: Tue Aug 11, 2009 1:35 pm Post subject:

Thank you for starting this and doing the leg work on getting it all rolling, it is much appreciated!

I am going to look at incorporating your modifications.

akhtar · Posted: Tue Aug 11, 2009 2:57 pm Post subject:

No prob - its been on my todo list for almost a year since I first posted over here about it.

The updated wholib combined with the modified census+ i linked have been running really well.

You should consider hosting the census+ source over on wowace.com or curseforge.com. If you have an account over there let me know - i'll add you onto the wholib project.
_________________
Jon S Akhtar

Rollie · Posted: Tue Aug 11, 2009 3:33 pm Post subject:

It does currently throw an error if LibWho is NOT present, working on a fix atm.

Rollie · Posted: Tue Aug 11, 2009 3:40 pm Post subject:

Changing

Rollie · Posted: Tue Aug 11, 2009 3:49 pm Post subject:

Okay, testing using WhoLib, and it is doing each query twice. Will need to dig into that.

akhtar · Posted: Fri Aug 14, 2009 11:46 am Post subject:

One thought may be that you don't need to retry queries. Wholib will do that for you.
_________________
Jon S Akhtar

akhtar · Posted: Mon Aug 31, 2009 8:44 am Post subject:

Still running really well with the version i modified - did you resolve the double query issue?
_________________
Jon S Akhtar

Isaura · Posted: Mon Aug 31, 2009 4:24 pm Post subject:

Will this version also prevent the popping up from the who window every second, when census is active.
It is pretty worse atm, probly cause of another addon that was updated lately.
Before, it stopped popping up by clicking the friendstab and close the social window.
That doesn't work anymore. Actually nothing works.
Well one thing does. Dragging the social window almost out of your screen so its not in the way anymore Laughing

So if this version can change this annoying behavior I will we very glad, and a lot of others with me I think. Smile

akhtar · Posted: Mon Sep 07, 2009 12:09 am Post subject:

I cleaned up the issue page at wowace.com, and rezipped the last version I had worked on. The link is updated in the OP, and here : http://static.wowace.com/content/attachments/1/296/CensusPlus-Wholib-003.zip
_________________
Jon S Akhtar

Rollie · Posted: Mon Sep 07, 2009 7:18 pm Post subject:

Yeah, I haven't looked into it again since the first time. It's up on my list though.

zybron · Posted: Tue Sep 08, 2009 10:05 am Post subject:

It looks like the section of code beginning on line 2020 of CensusPlus.lua is responsible for re-queuing a /who request:

Kosh · Posted: Wed Sep 09, 2009 1:41 am Post subject:

If the goal is to produce a version of CP that optionally uses WhoLib, only if it is present, then removing the section of code wouldn't be the way to go. Instead, if you are right and WhoLib negates the need to resend, then the block should be enclosed in an if() to only execute when WhoLib isn't being used. Without looking more closely at the code, it's possible that some of that may still need to execute (or alternate code might be needed if WhoLib never comes back for a query).

zybron · Posted: Wed Sep 09, 2009 5:31 am Post subject:

akhtar · Posted: Fri Sep 18, 2009 4:03 am Post subject:

Sounds like its pretty close - enough so - that we could put together an alpha/beta version for further testing.
_________________
Jon S Akhtar

Isaura · Posted: Sun Sep 20, 2009 10:02 am Post subject:

This afternoon I installed this version from wowace: CensusPlus-Wholib-003 (.zip)
I must say I didn't play very long, but yeah, hurrah, I didn't see any popups anymore.

Just to let you know Smile

akhtar · Posted: Thu Oct 01, 2009 10:26 pm Post subject:

I have created a project on wowace.com to store the source changes I have made, in case anyone wants to submit patches etc.

http://www.wowace.com/addons/census-wholib-version/
_________________
Jon S Akhtar

Ody · Posted: Tue Dec 08, 2009 1:24 pm Post subject:

I'm using your modified version for serveral months, waiting for a 3.3 release.

akhtar · Posted: Sun Nov 14, 2010 8:30 am Post subject:

Updated wholib version for 4.0

http://www.wowace.com/addons/census-wholib-version/

Sylvanaar
_________________
Jon S Akhtar

Ody · Posted: Mon Nov 15, 2010 3:38 pm Post subject:

Thanks Sylvanaar for the update Wink