Change keyword "bind" to "loot" to be in line with the user documentation. The keyword "bind" will continue to work for legacy compatibility, but it is deprecated.
KyrosKrane [07-21-16 - 18:25]
Change keyword "bind" to "loot" to be in line with the user documentation. The keyword "bind" will continue to work for legacy compatibility, but it is deprecated.
diff --git a/AnnoyingPopupRemover.lua b/AnnoyingPopupRemover.lua
index de4ec59..29724ee 100644
--- a/AnnoyingPopupRemover.lua
+++ b/AnnoyingPopupRemover.lua
@@ -207,7 +207,7 @@ function APR:HandleCommandLine(msg, editbox)
-- Validate parameters. Only 1 and 2 are checked; rest are ignored.
if Line[2] then
- if "bind" == Line[2] or "loot" == Line[2] or "roll" == Line[2] or "void" == Line[2] or "vendor" == Line[2] then
+ if "loot" == Line[2] or "loot" == Line[2] or "roll" == Line[2] or "void" == Line[2] or "vendor" == Line[2] then
if "show" == Line[1] or "hide" == Line[1] then
APR:TogglePopup(Line[2], Line[1])
return;
@@ -243,7 +243,7 @@ end -- APR:HandleCommandLine()
-- Print the instructions for the user.
function APR:PrintHelp()
APR:ChatPrint (L["Allowed commands for"] .. " " .. L["Annoying Pop-up Remover"] .. ":");
- APR:ChatPrint("/apr show |cffFFCC00OR|r hide bind |cffFFCC00OR|r roll |cffFFCC00OR|r void |cffFFCC00OR|r vendor"); -- not localized on purpose
+ APR:ChatPrint("/apr show |cffFFCC00OR|r hide loot |cffFFCC00OR|r roll |cffFFCC00OR|r void |cffFFCC00OR|r vendor"); -- not localized on purpose
APR:ChatPrint("/apr status"); -- not localized on purpose
APR:ChatPrint("/apr help"); -- not localized on purpose
end -- APR:PrintHelp()
@@ -252,7 +252,7 @@ end -- APR:PrintHelp()
-- Print the status for a given popup type, or for all if not specified.
-- popup is optional
function APR:PrintStatus(popup)
- if not popup or "bind" == popup then
+ if not popup or "loot" == popup then
if APR.DB.HideBind then
APR:ChatPrint (L["Confirmation pop-up when |cff00ff00looting|r bind-on-pickup items will be |cff00ff00hidden|r."]);
else
@@ -286,10 +286,10 @@ end -- APR:PrintStatus()
-- Dispatcher function to call the correct show or hide function for the appropriate popup window.
-- popup is required, state is optional
function APR:TogglePopup(popup, state)
- -- The status message highlighting highlights the word "looting" for looting bind on pickup items. So, the user might use the keyword "loot" instead of "bind". Handle that here.
- if "loot" == popup then popup = "bind" end;
+ -- Older versions of the addon used the keyword "bind" instead of "loot". Handle the case where a user tries to use the old keyword.
+ if "bind" == popup then popup = "loot" end;
- if "bind" == popup then
+ if "loot" == popup then
if state then
if "show" == state then
APR:ShowPopupBind(PRINT_CONFIRMATION);
@@ -408,7 +408,7 @@ function APR:ShowPopupBind(printconfirm)
-- else already shown, nothing to do.
end
- if printconfirm then APR:PrintStatus("bind") end;
+ if printconfirm then APR:PrintStatus("loot") end;
end -- APR:ShowPopupBind()
@@ -476,7 +476,7 @@ function APR:HidePopupBind(printconfirm, ForceHide)
-- else already hidden, nothing to do.
end
- if printconfirm then APR:PrintStatus("bind") end;
+ if printconfirm then APR:PrintStatus("loot") end;
end -- APR:HidePopupBind()