WoWInterface - Comments for Ammo's AddOn(s)

Re: Re: Error since 5.0.4 (Better Inbox)

Nyril — Wed, 28 Nov 2012 21:20:35 +0000

That may 'fix' it but it's not good enough for me! so if you're feeling brave you can edit the code. It's *almost* perfect, although the corners don't quite line up any more but I just can't get it right.

Open up BetterInbox.lua in the addons folder. Navigate to line number 638 and you'll see function BetterInbox:SetSendMailShowing( flag ). This is the function that loads the textures that make BI look so good and where the error is. Delete the 5 lines between if not flag then and the end that matches it and replace them with the following

local tl = InboxFrame:CreateTexture("MailFrame","MailFrameTopLeft")
tl:SetParent(InboxFrame)
tl:SetTexture("Interface\\AddOns\\BetterInbox\\images\\BetterInbox-TopLeft")
tl:SetWidth(254)
tl:SetHeight(254)
tl:SetPoint("TOPLEFT", "MailFrame", "TOPLEFT", -12, 10)
local tr = InboxFrame:CreateTexture(nil,"MailFrameTopRight")
tr:SetTexture("Interface\\AddOns\\BetterInbox\\images\\BetterInbox-TopRight")
tr:SetParent(InboxFrame)
tr:SetWidth(128)
tr:SetHeight(256)
tr:SetPoint("TOPLEFT", tl, "TOPRIGHT")
local bl = InboxFrame:CreateTexture(nil,"MailFrameBotLeft")
bl:SetTexture("Interface\\AddOns\\BetterInbox\\images\\BetterInbox-BotLeft")
bl:SetParent(InboxFrame)
bl:SetWidth(254)
bl:SetHeight(256)
bl:SetPoint("TOPLEFT", tl, "BOTTOMLEFT")
local br = InboxFrame:CreateTexture(nil,"MailFrameBotRight")
br:SetTexture("Interface\\AddOns\\BetterInbox\\images\\BetterInbox-BotRight")
br:SetParent(InboxFrame)
br:SetWidth(128)
br:SetHeight(256)
br:SetPoint("TOPLEFT", tl, "BOTTOMRIGHT")

Good luck and if anyone can get it perfect let me know! :D

Just to say this is the BEST inbox addon around. I tried several after it broke and couldn't stand any of them. Being a .NET developer I decided it really couldn't be beyond me to figure out how to fix it. took me several evenings work but finally came up with this solution and am truly in awe of those that come up with these addons as the language is totally different to anything I know!

Re: Error since 5.0.4 (Better Inbox)

Elloria — Thu, 20 Sep 2012 00:20:42 +0000

There is a new update on curse.com.

Edit: Also THANK YOU FOR UPDATING!!!!! So happy to have this addon back :banana:

Error since 5.0.4 (Better Inbox)

Dessembrae — Sat, 08 Sep 2012 13:55:37 +0000

Error while opening mailbox since 5.0.4
Date: 2012-09-08 15:52:29
ID: 3
Error occured in: Global
Count: 1
Message: ..\AddOns\BetterInbox\BetterInbox.lua line 640:
attempt to index global 'MailFrameTopLeft' (a nil value)
Debug:
(tail call): ?
BetterInbox\BetterInbox.lua:640: SetSendMailShowing()
BetterInbox\BetterInbox.lua:405: SetupGUI()
BetterInbox\BetterInbox.lua:120: MAIL_SHOW()
BetterInbox\BetterInbox.lua:82:
BetterInbox\BetterInbox.lua:71
(tail call): ?
: ?
"]:9:
"]:5
(tail call): ?
Ace3\AceAddon-3.0\AceAddon-3.0.lua:543: EnableAddon()
Ace3\AceAddon-3.0\AceAddon-3.0.lua:636:
Ace3\AceAddon-3.0\AceAddon-3.0.lua:621
: LoadAddOn()
AddonLoader\AddonLoader.lua:125: LoadAddOn()
AddonLoader\AddonLoader.lua:146:
AddonLoader\AddonLoader.lua:135
: TurnOrActionStop()
:4:
:1

Re: Re: Your inbox is full. (Better Inbox)

Undu — Sun, 13 May 2012 04:43:26 +0000

The message that informs you that you have more than 50 mails in your inbox and that it is full overlaps with the information provided by Better Inbox as far as how much money total is in the mails and the amount of items in the inbox.

Is it within the scope of this addon to perhaps move either of those? The information or the error message. Would be really nice.

I finally got sick of this issue after a LONG time of just dealing with it. The change is pretty simple:

BetterInbox.lua, line 573
Change:
summary.moneyText:SetPoint( "TOPRIGHT", InboxFrame, "TOPLEFT", 160, -50 )
To:
summary.moneyText:SetPoint( "TOPRIGHT", InboxFrame, "TOPLEFT", 160, -80 )

Then BetterInbox.lua, line 581
Change:
summary.money:SetPoint( "TOPLEFT", InboxFrame, "TOPLEFT", 170, -50 )
To:
summary.money:SetPoint( "TOPLEFT", InboxFrame, "TOPLEFT", 170, -80 )

Thanks a lot! :)

Re: Your inbox is full. (Better Inbox)

Bunson — Thu, 01 Dec 2011 03:24:08 +0000

Your inbox is full. (Better Inbox)

Undu — Thu, 03 Nov 2011 18:45:26 +0000

Fixed, they should point to wowpedi... (Addon Loader)

Cladhaire — Sun, 03 Jul 2011 16:54:00 +0000

Fixed, they should point to wowpedia instead. Wowwiki is dead.

the wowwiki links are dead. (Addon Loader)

tinyu — Sun, 03 Jul 2011 15:28:22 +0000

the wowwiki links are dead.

What if I only wanted to see the il... (TooltipItemInfo)

spiralofhope — Fri, 10 Jun 2011 04:17:25 +0000

What if I only wanted to see the ilevel and not the stack size?

You can also add;... (LockSmith)

brknsoul — Sat, 14 May 2011 04:50:47 +0000

You can also add;

= 425, -- Elementium Lockbox

And these talent checks do not work... (oUF Debuff Highlight)

ylarion — Wed, 02 Mar 2011 22:40:59 +0000

And these talent checks do not work currently for me (german client)

I would guess the first is correct.... (oUF Debuff Highlight)

memnoch — Wed, 02 Mar 2011 22:27:20 +0000

I would guess the first is correct. Each class doesn't list every ailment. The ones that have falses have talents associated with adding the ability to cure that particular ailment.

Originally posted by drakull
The "CanDispel" table is not updated to 4.0.
local CanDispel = {
PRIEST = { Magic = true, Disease = true, },
SHAMAN = { Poison = true, Disease = true, Curse = true, },
PALADIN = { Magic = true, Poison = true, Disease = true, },
MAGE = { Curse = true, },
DRUID = { Curse = true, Poison = true, }
}
I think this is the right one for 4.0:
local CanDispel = {
PRIEST = { Magic = true, Disease = true },
SHAMAN = { Magic = true, Curse = true },
PALADIN = { Magic = true, Poison = true, Disease = true },
MAGE = { Curse = true },
DRUID = { Magic = true, Curse = true, Poison = true }
}

Great addon btw.

Edit: took too long to post. Elv22 was faster. :)

Originally posted by Ammo... (Money Trail)

WaldoJeffers — Mon, 28 Feb 2011 13:06:20 +0000

Originally posted by Ammo
cargbags is a pain to support sadly.

-Ammo

Any chance you'd support the painfulness? :p moneytrail and cargbags nivaya were unstoppable together!!

On a german client i cannot get my... (oUF Debuff Highlight)

ylarion — Sun, 27 Feb 2011 10:37:57 +0000

On a german client i cannot get my resto shaman to highlight magic debuffs. If i change the value in the CanDispel to true it works, but somehow the Talentcheck doesnt seem to work.

New Baud Bag Support (Money Trail)

AquilaChill — Sun, 20 Feb 2011 17:45:56 +0000

Baud Bag has changed (added token tracking & search, a few other small things) in the v4.0.3 beta's.
would love to have this addons functionality back :)
thanks

Addon won't show up. (LockSmith)

Movarrin — Thu, 20 Jan 2011 20:14:31 +0000

Edit: I had to add in this line and it starting working.

= 400, -- Flame-Scarred Junkbox

For some reason this latest verion of locksmith just will not work for me. I pickpocket Flame-Scarred lockboxes and nothing happens. If I get a low level lockbox it works.

Any idea what's going on?

Just wondering if there was any cha... (Cast Donut)

Aydan — Sat, 08 Jan 2011 20:55:41 +0000

Just wondering if there was any chance to get a swing timer put in? Such as for auto-shot. Thanks!

Originally posted by monk... (oUF Debuff Highlight)

drakull — Wed, 15 Dec 2010 13:18:37 +0000

Originally posted by monkie
I currently have borders around my health and power frames. For some reason the backdrop/backdrop border doesn't get highlighted properly and sometimes don't show up at all. But if i get rid of the borders it works fine. Any help would be appreciated. Thx
Yeah, it used to work just fine but since cata went live I've seeing some weird behaviours. Sometimes the debuffhighlight texture shows up with wrong colors (red or white mostly) and sometimes it does not shows at all.

Here's the code for my layout's raid frame:
if IsAddOnLoaded("oUF_DebuffHighlight") then
local dbh = self.Health:CreateTexture(nil, "OVERLAY")
dbh:SetAllPoints(self.Health)
dbh:SetTexture(cfg.glowTexture)
dbh:SetBlendMode("ADD")
dbh:SetVertexColor(0,0,0,0) -- set alpha to 0 to hide the texture
self.DebuffHighlight = dbh
self.DebuffHighlightAlpha = 0.8
self.DebuffHighlightFilter = true
end

Quick question.. Does it matter... (oUF Debuff Highlight)

Xposed — Wed, 24 Nov 2010 04:27:04 +0000

Quick question..
Does it matter what unitframe your using? Also, i'm new at this but what do you do to make the health frame light up when a debuff is on you?

Originally posted by monk... (oUF Debuff Highlight)

Ammo — Wed, 17 Nov 2010 09:44:04 +0000