Quote:
|
The fix is on github... so everyone having this issue download version from GitHub and test it.
|
I question the placement, as social is the addon with ON_FRIENDSLIST not Nxcom. It works because it's being run every update tick instead of when it needs to be.
Does the same fix work if you move the call to NxSocial, line 861 just above self.List:Update()? |
from what I now understand it is the right place
because it is the comunication for pals (social) in NxCom.lua that throws the error on not online Code:
local msg = self.PalsSendMsg ps and this error accurs in every addon I have seen in comunication issues |
even if that's the case, it's being called too late, the function is called asking blizz to update the friends list and then right after is doing a check against a variable built from friends list updates
if self.PosSendNext > #self.Friends It may not have time to update self.Friends before it is checking and then still try to do the self:Send ("W", msg, self.Friends[self.PosSendNext]) With the current placement, it calls the function which initates FRIENDLIST_UPDATE and Carbonite.lua throws that back to Com.OnFriendguild_update which then rebuilds self.Friends and it would need to do that all before the if statement gets done. And not only that it's modifying self.Friends so say bubba, joe and john are online and on your friends list. bubba goes offline while this is running, self.Friends goes down to 2 and john never gets get message. |
ok understand what you mean will check your idea, to check if its possible if my brother has time :-)
|
Thinking more on this... it may be better to instead of using ShowFriends() add a check before the Send
something like if UnitIsConnected(self.Friends[self.PosSendNext]) then self:Send ("W", msg, self.Friends[self.PosSendNext]) end Edit: Hrmm no, that won't work as we don't have the unitID only the name. |
Putting in this place ShowFriends() isnt much affecting performance in this case.
I was searching and debuging to find right place for this issue. The problem was with sending this Whisps to Pals to show them on Main map. As far as for position, Rythal is right it should be in other place, but it works for now. The only side effect will be that if this Whisp isnt send, the position of Pal will not be updated on the map, but it will be updated in next cycle. And the real issue is Blitzz bug as event "FRIENDLIST_UPDATE" isnt fired everytime it should! We could replace ShowFirends() with Nx.Com:OnFriendguild_update() so it will execute function, update what we need and then do rest of the code... is that good idea? |
Quote:
GetFriendInfo won't probably help either, because it would suffer from the same inconvenient inconsistency about not noticing when someone goes offline in time. |
Quote:
|
Quote:
|
Yesterday I put a bug report in at Blizz and they said its a add on the is causing this message. They said recount with (sync ) that is causing the No player message, So i disabled all my add on's and reactivate them one at a time till i find the one that is causing this message. I do not use recount.
U have just started doing this will post my resalts |
GitHub version has fix for this, ofcourse only for our addon ;)
|
Quote:
Found an error with the new fix: Code:
Carbonite GetPos Err:975: Carbonite\NxMap.lua:9129: in function `GetWorldPos' |
I had this issue for weeks and weeks, but then I disabled recount and I haven't seen it since. I don't know if it was just recount or a recount/carbonite issue. My friend also had the issue and told him to disable recount, also no issue since.
|
I don't know if this works, but someone posted on the WoW forums about it.
Quote:
Here's the page link: http://us.battle.net/wow/en/forum/topic/16016271802#5 |
I did explained that in previos posts.
Carbonite is fixed but becouse of Blizz bug other addons may be affected with de same issue. Closing. |
All times are GMT -6. The time now is 09:54 PM. |
vBulletin © 2024, Jelsoft Enterprises Ltd
© 2004 - 2022 MMOUI