LEGION group.lua error #141


  • New
  • Defect
Open
Assigned to xuerian
  • transitbus created this issue May 16, 2016

    1x XLoot_Group\Group-Group.lua:186: hooksecurefunc(): AlertFrame_FixAnchors is not a function
    [C]: in function `hooksecurefunc'
    XLoot_Group\Group-Group.lua:186: in function <XLoot_Group\Group.lua:88>
    (tail call): ?
    [C]: ?
    [string "safecall Dispatcher[1]"]:9: in function <[string "safecall Dispatcher[1]"]:5>
    (tail call): ?
    ...aceMasque\Libs\AceAddon-3.0\AceAddon-3.0-12.lua:558: in function `EnableAddon'
    ...aceMasque\Libs\AceAddon-3.0\AceAddon-3.0-12.lua:571: in function `EnableAddon'
    ...aceMasque\Libs\AceAddon-3.0\AceAddon-3.0-12.lua:651: in function <...aceMasque\Libs\AceAddon-3.0\AceAddon-3.0.lua:636>
    [C]: in function `LoadAddOn'
    FrameXML\UIParent.lua:390: in function `UIParentLoadAddOn'
    FrameXML\UIParent.lua:413: in function `CombatLog_LoadUI'
    FrameXML\UIParent.lua:998: in function <FrameXML\UIParent.lua:894>

    Locals:
    nil

  • transitbus added the tags New Defect May 16, 2016
  • Xuerian posted a comment May 16, 2016

    I don't know what else has changed but this definitely isn't set up for legion

    try turning off the hooking options in group entirely

  • transitbus posted a comment May 17, 2016

    @Xuerian: Go

    For now I just commented that line out and it works for the most part. Just wanted to post the error you get in beta using xloot. I'll post anymore if they come up or if I find the replacement for AlertFrame_FixAnchors

  • transitbus posted a comment Jul 16, 2016

    @Xuerian: Go

    Looks like they just rolled all the fixanchor functions into one mixin

    function AlertFrameSimpleMixin:AdjustAnchors(relativeAlert)
    	if self.alertFrame:IsShown() then
    		self.alertFrame:SetPoint("BOTTOM", relativeAlert, "TOP", 0, 10);
    		return self.alertFrame;
    	end
    	return relativeAlert;
    end
    

    So I am thinking maybe you can just roll in the contents of that mixin into a function or add it to an existing function?


    Edited Jul 17, 2016
  • Xuerian posted a comment Jul 18, 2016

    @transitbus: Go

    Theoretically it should be easier to do what I need to do now, thanks for the tip.

    Realistically it's going to require me spending some time getting to understand their new alert system and where exactly it's best to do what.

    For now I'll just be disabling all alert/bonus modifying functionality as it won't work at all until I update it.

  • _ForgeUser698 posted a comment Jul 23, 2016

    Yeah, it seems like a lot of alerts (including dungeon complete and achievement) were rolled into this one alert frame. It's such an annoying frame to move and scale correctly. Having trouble getting MoveAnything to handle it right also.

    Since this frame handles more than just loot now, maybe there needs to be a separate addon entirely dedicated to customizing the frame? Moving, scaling, padding, and growth direction would be a good start, but skinning it might be tricky and better left out altogether.

  • Xuerian posted a comment Jul 23, 2016

    @Daemys: Go

    Yeah, it really should be a separate addon. Blizz has consolidated all of them into a slightly overcomplicated but reasonable system.

    Skinning them was always a hack. Granted, not as bad of a hack as actually moving them


To post a comment, please login or register a new account.