Templates and combat removal (#85)

* feat: Retain template option, blocklist rename, combat/PvP updates

- Add Retain checkbox to copy buff settings when switching to fresh templates
- Inclusivity: Rename blacklist → blocklist (with migration)
- Default InCombat to off
- Add PvP prep-only message for Arena/BG
- Add play-sound tooltip.
- bump version to r39.130226

* updated what's new.

Author: ynazar1

.cursorrules

@@ -32,6 +32,10 @@
 - Follow the existing file organization (buffs in SmartBuff.buffs.lua, main logic in SmartBuff.lua)
 - Use existing constants and patterns where possible
 
+## What's New (localization.en.lua)
+- SMARTBUFF_WHATSNEW should include brief notes for the last two versions only
+- When adding a new version, prepend its changes and keep the previous version's notes; remove notes older than two versions
+
 ## Documentation
 - Provide brief but clear explanations for any new functions
 - Include user-friendly descriptions in function headers

SmartBuff.globals.lua

@@ -1,4 +1,4 @@
--------------------------------------------------------------------------------
+-------------------------------------------------------------------------------
 -- Globals
 -------------------------------------------------------------------------------
 

SmartBuff.lua

@@ -10,7 +10,7 @@
 -- and options frame on first load... could be annoying if done too often
 -- What's new is pulled from the SMARTBUFF_WHATSNEW string in localization.en.lua
 -- this is mostly optional, but good for internal housekeeping
-SMARTBUFF_DATE               = "120226"; -- EU Date: DDMMYY
+SMARTBUFF_DATE               = "130226"; -- EU Date: DDMMYY
 SMARTBUFF_VERSION            = "r39." .. SMARTBUFF_DATE;
 -- Update the NR below to force reload of SB_Buffs on first login
 -- This is now OPTIONAL for most changes - only needed for major logical reworks or large patch changes.
@@ -113,7 +113,7 @@ local cClassGroups           = {};
 local cBuffs                 = {};
 local cBuffIndex             = {};
 local cBuffTimer             = {};
-local cBlacklist             = {};
+local cBlocklist             = {};
 local cUnits                 = {};
 local cBuffsCombat           = {};
 
@@ -924,8 +924,8 @@ local arg1, arg2, arg3, arg4, arg5 = ...;
     currentUnit = arg1;
     SMARTBUFF_AddMsgD(string.format("Spell failed: %s", arg1));
     if (currentUnit and (string.find(currentUnit, "party") or string.find(currentUnit, "raid") or (currentUnit == "target" and O.Debug))) then
-      if (UnitName(currentUnit) ~= sPlayerName and O.BlacklistTimer > 0) then
-        cBlacklist[currentUnit] = GetTime();
+      if (UnitName(currentUnit) ~= sPlayerName and O.BlocklistTimer > 0) then
+        cBlocklist[currentUnit] = GetTime();
         if (currentUnit and UnitName(currentUnit)) then
         end
       end
@@ -1169,6 +1169,42 @@ for i, key in ipairs({"Solo", "Party", "LFR", "Raid", "MythicKeystone", "Horrifi
   Enum.SmartBuffGroup[key] = i
 end
 
+-- True if template has at least one buff with EnableS or EnableG.
+local function templateHasEnabledBuffs(templateName)
+  local t = B and B[CS()] and B[CS()][templateName];
+  if (not t or type(t) ~= "table") then return false; end
+  for k, v in pairs(t) do
+    if (type(v) == "table" and (v.EnableS or v.EnableG)) then return true; end
+  end
+  return false;
+end
+
+-- Copy buff settings from one template to another (for RetainTemplate on first switch).
+-- Only copies when: RetainTemplate is true, source has enabled buffs, and dest is "fresh" (no data or no buffs enabled).
+-- Does not copy blank templates, so users cannot accidentally overwrite a configured template with a blank one.
+local function MaybeCopyTemplateOnFirstSwitch(fromT, toT)
+  if (not B or not B[CS()] or not B[CS()][fromT]) then return; end
+  if (not O or not O.RetainTemplate) then return; end
+  if (not templateHasEnabledBuffs(fromT)) then return; end  -- do not copy blank over configured
+  -- Dest is "fresh" if nil/empty or has no enabled buffs
+  if (B[CS()][toT] and templateHasEnabledBuffs(toT)) then return; end  -- dest already has enabled buffs, not fresh
+
+  local src = B[CS()][fromT];
+  B[CS()][toT] = B[CS()][toT] or {};
+  local dst = B[CS()][toT];
+
+  for k, v in pairs(src) do
+    if (type(v) == "table") then
+      dst[k] = {};
+      for k2, v2 in pairs(v) do
+        dst[k][k2] = (type(v2) == "table") and (function(t) local r = {}; for a, b in pairs(t) do r[a] = b; end return r; end)(v2) or v2;
+      end
+    else
+      dst[k] = v;
+    end
+  end
+end
+
 -- Set the current template and create an array of units
 function SMARTBUFF_SetTemplate(force)
   -- Only block in combat (not when mounted) - setup should work when mounted
@@ -1276,14 +1312,18 @@ function SMARTBUFF_SetTemplate(force)
   end
 
   if currentTemplate ~= newTemplate then
+    MaybeCopyTemplateOnFirstSwitch(currentTemplate, newTemplate);
     SMARTBUFF_AddMsgD("Current tmpl: " .. currentTemplate or "nil" .. " - new tmpl: " .. newTemplate or "nil");
     local reason = switchReason or "instance"
     SMARTBUFF_AddMsg(SMARTBUFF_TITLE .. ": " .. SMARTBUFF_OFT_AUTOSWITCHTMP .. " (" .. reason .. ") " .. currentTemplate .. " -> " .. newTemplate);
+    if (reason == "arena" or reason == "battleground") then
+      SMARTBUFF_AddMsg(SMARTBUFF_TITLE .. ": " .. SMARTBUFF_MSG_PVP_PREP_ONLY, true);
+    end
   end
   currentTemplate = newTemplate;
 
   SMARTBUFF_SetBuffs();
-  wipe(cBlacklist);
+  wipe(cBlocklist);
   wipe(cBuffTimer);
   wipe(cUnits);
   wipe(cGroups);
@@ -2628,7 +2668,7 @@ function SMARTBUFF_Check(mode, force)
   local reagent;
   local nGlobal = 0;
 
-  SMARTBUFF_checkBlacklist();
+  SMARTBUFF_checkBlocklist();
 
   -- Skip when: (in combat and O.InCombat disabled) OR (in PvP and match active, matchState >= 3). Allow PvP prep and combat buffs when O.InCombat.
   local _, instanceType = GetInstanceInfo();
@@ -2763,7 +2803,7 @@ function SMARTBUFF_BuffUnit(unit, subgroup, mode, spell)
   --SMARTBUFF_AddMsgD("Checking " .. unit);
 
   if (UnitExists(unit) and UnitIsFriend("player", unit) and not UnitIsDeadOrGhost(unit) and not UnitIsCorpse(unit)
-        and UnitIsConnected(unit) and UnitIsVisible(unit) and not UnitOnTaxi(unit) and not cBlacklist[unit]
+        and UnitIsConnected(unit) and UnitIsVisible(unit) and not UnitOnTaxi(unit) and not cBlocklist[unit]
         and ((not UnitIsPVP(unit) and (not isPvP or O.BuffPvP)) or (UnitIsPVP(unit) and (isPvP or O.BuffPvP)))) then
     --and not SmartBuff_UnitIsIgnored(unit)
 
@@ -3481,17 +3521,17 @@ end
 -- END SMARTBUFF_CanApplyWeaponBuff
 
 
--- Check the unit blacklist
-function SMARTBUFF_checkBlacklist()
+-- Check the unit blocklist
+function SMARTBUFF_checkBlocklist()
   local t = GetTime();
-  for unit in pairs(cBlacklist) do
-    if (t > (cBlacklist[unit] + O.BlacklistTimer)) then
-      cBlacklist[unit] = nil;
+  for unit in pairs(cBlocklist) do
+    if (t > (cBlocklist[unit] + O.BlocklistTimer)) then
+      cBlocklist[unit] = nil;
     end
   end
 end
 
--- END SMARTBUFF_checkBlacklist
+-- END SMARTBUFF_checkBlocklist
 
 
 -- Casts a spell
@@ -4217,13 +4257,16 @@ function SMARTBUFF_Options_Init(self)
   if (type(SMARTBUFF_Options) ~= "table") then SMARTBUFF_Options = {}; end
   O = SMARTBUFF_Options;
 
+  -- Migrate blacklist -> blocklist (inclusive language)
+  if (O.BlacklistTimer ~= nil and O.BlocklistTimer == nil) then O.BlocklistTimer = O.BlacklistTimer; O.BlacklistTimer = nil; end
+
   SMARTBUFF_BROKER_SetIcon();
 
 
   if (O.Toggle == nil) then O.Toggle = true; end
   if (O.ToggleAuto == nil) then O.ToggleAuto = true; end
   if (O.AutoTimer == nil) then O.AutoTimer = 5; end
-  if (O.BlacklistTimer == nil) then O.BlacklistTimer = 5; end
+  if (O.BlocklistTimer == nil) then O.BlocklistTimer = 5; end
   if (O.ToggleAutoCombat == nil) then O.ToggleAutoCombat = false; end
   if (O.ToggleAutoChat == nil) then O.ToggleAutoChat = false; end
   if (O.ToggleAutoSplash == nil) then O.ToggleAutoSplash = true; end
@@ -4246,8 +4289,9 @@ function SMARTBUFF_Options_Init(self)
   if (O.ScrollWheelUp == nil) then O.ScrollWheelUp = true; end
   if (O.ScrollWheelDown == nil) then O.ScrollWheelDown = true; end
 
-  if (O.InCombat == nil) then O.InCombat = true; end
+  if (O.InCombat == nil) then O.InCombat = false; end
   if (O.IncludeToys == nil) then O.IncludeToys = false; end
+  if (O.RetainTemplate == nil) then O.RetainTemplate = false; end
   if (O.AutoSwitchTemplate == nil) then O.AutoSwitchTemplate = true; end
   if (O.AutoSwitchTemplateInst == nil) then O.AutoSwitchTemplateInst = true; end
   if (O.InShapeshift == nil) then O.InShapeshift = true; end
@@ -4594,7 +4638,7 @@ function SMARTBUFF_command(msg)
     SMARTBUFF_ShowBuffTimers();
   elseif (msg == "target") then
     if (SMARTBUFF_PreCheck(0)) then
-      SMARTBUFF_checkBlacklist();
+      SMARTBUFF_checkBlocklist();
       SMARTBUFF_BuffUnit("target", 0, 0);
     end
   elseif (msg == "debug") then
@@ -4769,6 +4813,10 @@ function SMARTBUFF_OHideSAButton()
   SMARTBUFF_ShowSAButton();
 end
 
+function SMARTBUFF_ORetainTemplate()
+  O.RetainTemplate = not O.RetainTemplate;
+end
+
 function SMARTBUFF_OSelfFirst()
   B[CS()][currentTemplate].SelfFirst = not B[CS()][currentTemplate].SelfFirst;
 end
@@ -5114,12 +5162,13 @@ function SMARTBUFF_Options_OnShow()
   SmartBuffOptionsFrame_cbMsgError:SetChecked(O.ToggleMsgError);
   SmartBuffOptionsFrame_cbHideMmButton:SetChecked(O.HideMmButton);
   SmartBuffOptionsFrame_cbHideSAButton:SetChecked(O.HideSAButton);
+  SmartBuffOptionsFrame_cbRetainTemplate:SetChecked(O.RetainTemplate);
 
   SmartBuffOptionsFrameRebuffTimer:SetValue(O.RebuffTimer);
   SmartBuff_SetSliderText(SmartBuffOptionsFrameRebuffTimer, SMARTBUFF_OFT_REBUFFTIMER, O.RebuffTimer,
     INT_SPELL_DURATION_SEC);
-  SmartBuffOptionsFrameBLDuration:SetValue(O.BlacklistTimer);
-  SmartBuff_SetSliderText(SmartBuffOptionsFrameBLDuration, SMARTBUFF_OFT_BLDURATION, O.BlacklistTimer,
+  SmartBuffOptionsFrameBLDuration:SetValue(O.BlocklistTimer);
+  SmartBuff_SetSliderText(SmartBuffOptionsFrameBLDuration, SMARTBUFF_OFT_BLDURATION, O.BlocklistTimer,
     INT_SPELL_DURATION_SEC);
 
   SMARTBUFF_SetCheckButtonBuffs(0);
@@ -5273,6 +5322,7 @@ function SMARTBUFF_DropDownTemplate_OnClick(self)
   UIDropDownMenu_SetSelectedValue(SmartBuffOptionsFrame_ddTemplates, i);
   tmp = SMARTBUFF_TEMPLATES[i];
   if (currentTemplate ~= tmp) then
+    MaybeCopyTemplateOnFirstSwitch(currentTemplate, tmp);
     SmartBuff_BuffSetup:Hide();
     iLastBuffSetup = -1;
     SmartBuff_PlayerSetup:Hide();

SmartBuff.xml

@@ -1196,9 +1196,9 @@
                     <OnValueChanged>
                         local o = SMARTBUFF_Options;
                         local n = self:GetValue();
-                        if (o and o.BlacklistTimer ~= n) then
+                        if (o and o.BlocklistTimer ~= n) then
                         SmartBuff_SetSliderText(self, SMARTBUFF_OFT_BLDURATION, n, INT_SPELL_DURATION_SEC);
-                        o.BlacklistTimer = n;
+                        o.BlocklistTimer = n;
                         end
                     </OnValueChanged>
                     <OnEnter>
@@ -1375,6 +1375,37 @@
                     </OnClick>
                 </Scripts>
             </Button>
+            <CheckButton name="SmartBuffOptionsFrame_cbRetainTemplate" inherits="UICheckButtonTemplate">
+                <Size>
+                    <AbsDimension x="20" y="20" />
+                </Size>
+                <Anchors>
+                    <Anchor point="LEFT" relativeTo="SmartBuffOptionsFrame_ddTemplates" relativePoint="RIGHT">
+                        <Offset>
+                            <AbsDimension x="8" y="0" />
+                        </Offset>
+                    </Anchor>
+                </Anchors>
+                <HitRectInsets>
+                    <AbsInset left="0" right="0" top="0" bottom="0" />
+                </HitRectInsets>
+                <Scripts>
+                    <OnLoad>
+                        getglobal(self:GetName().."Text"):SetText(SMARTBUFF_OFT_RETAINTEMPLATE);
+                        getglobal(self:GetName().."Text"):SetFontObject(GameFontNormalSmall);
+                    </OnLoad>
+                    <OnClick>
+                        SMARTBUFF_ORetainTemplate();
+                    </OnClick>
+                    <OnEnter>
+                        GameTooltip:SetOwner(self, "ANCHOR_RIGHT");
+                        GameTooltip:SetText(SMARTBUFF_OFTT_RETAINTEMPLATE, SMARTBUFF_TTC_R, SMARTBUFF_TTC_G, SMARTBUFF_TTC_B, SMARTBUFF_TTC_A);
+                    </OnEnter>
+                    <OnLeave>
+                        GameTooltip:Hide();
+                    </OnLeave>
+                </Scripts>
+            </CheckButton>
             <Button name="SmartBuffOptionsFrame_TutorialButton" parentKey="MainHelpButton" inherits="MainHelpPlateButton" toplevel="true">
                 <Anchors>
                     <Anchor x="-24" y="24" point="TOPLEFT" relativeTo="$parent" />
@@ -1635,6 +1666,13 @@
                     <OnClick>
                         SMARTBUFF_PlaySpashSound();
                     </OnClick>
+                    <OnEnter>
+                        GameTooltip:SetOwner(self, "ANCHOR_RIGHT");
+                        GameTooltip:SetText(SMARTBUFF_OFTT_PLAYSOUND, SMARTBUFF_TTC_R, SMARTBUFF_TTC_G, SMARTBUFF_TTC_B, SMARTBUFF_TTC_A);
+                    </OnEnter>
+                    <OnLeave>
+                        GameTooltip:Hide();
+                    </OnLeave>
                 </Scripts>
             </Button>
             <Slider name="SmartBuffOptionsFrame_sldSounds" inherits="OptionsSliderTemplate" minValue="1" maxValue="25" defaultValue="1">

localization.cn.lua

@@ -445,6 +445,7 @@ SMARTBUFF_MSG_LEFT           = "以后消失！";
 SMARTBUFF_MSG_CLASS          = "职业";
 SMARTBUFF_MSG_CHARGES        = "次数";
 SMARTBUFF_MSG_SOUNDS         = "飞溅声音选择: "
+SMARTBUFF_MSG_PVP_PREP_ONLY  = "由于API限制，增益仅在准备阶段有效，比赛开始后将停用。";
 
 -- Support
 SMARTBUFF_MINIMAP_TT         = "左键：选项菜单\n右键：开/关\nAlt+左键：SmartDebuff\n按Shift拖拽：移动按钮";

localization.de.lua

@@ -96,7 +96,7 @@ SMARTBUFF_OFT_CHECKCHARGES   = "Aufladungen";
 SMARTBUFF_OFT_RBT            = "R: Timers";
 SMARTBUFF_OFT_BUFFINCITIES   = "Bufft in Städten";
 SMARTBUFF_OFT_UISYNC         = "UI Sync";
-SMARTBUFF_OFT_BLDURATION     = "Blacklisted";
+SMARTBUFF_OFT_BLDURATION     = "Blockiert";
 SMARTBUFF_OFT_COMPMODE       = "Komp. Modus";
 SMARTBUFF_OFT_MINIGRP        = "Mini Gruppe";
 SMARTBUFF_OFT_ANTIDAZE       = "Anti-Daze";
@@ -154,7 +154,7 @@ SMARTBUFF_OFTT_AUTOSWITCHTMPINST = "Wechselt automatisch die Buff-Vorlage,\nwenn
 SMARTBUFF_OFTT_CHECKCHARGES  = "Erinnerung wenn die Aufladungen\neines Buffs bald aufgebraucht sind.\n0 = Deaktivert";
 SMARTBUFF_OFTT_BUFFINCITIES  = "Bufft auch in den Hauptstädten.\nWenn du PvP geflagged bist, bufft es immer.";
 SMARTBUFF_OFTT_UISYNC        = "Aktiviert die Synchronisation mit dem UI,\num die Buff-Zeiten der anderen Spieler zu erhalten.";
-SMARTBUFF_OFTT_BLDURATION    = "Wieviele Sekunden ein Spieler auf\ndie schwarze Liste gesetzt wird.\n0 = Deaktivert";
+SMARTBUFF_OFTT_BLDURATION    = "Wie viele Sekunden ein Spieler blockiert wird.\n0 = Deaktiviert";
 SMARTBUFF_OFTT_COMPMODE      = "Kompatibilitäts Modus\nWarnung!!!\nBenutzte diesen Modus nur, wenn Probleme auftreten\nBuffs auf sich selbst zu casten.";
 SMARTBUFF_OFTT_MINIGRP       = "Zeigt die Raid-Subgruppen Einstellungen in einem\neigenen verschiebbaren Mini-Fenster an.";
 SMARTBUFF_OFTT_ANTIDAZE      = "Bricht automatisch den\nAspekt des Geparden/Rudels ab,\nwenn jemand betäubt wird\n(Selbst oder Gruppe).";
@@ -221,6 +221,7 @@ SMARTBUFF_MSG_CLASS          = "Klasse";
 SMARTBUFF_MSG_CHARGES        = "Aufladungen";
 SMARTBUFF_MSG_SOUNDS         = "Splash-Sound-Auswahl: "
 SMARTBUFF_MSG_SPECCHANGED    = "Spec gewechselt (%s), lade Buff-Vorlagen...";
+SMARTBUFF_MSG_PVP_PREP_ONLY  = "Aufgrund von API-Einschränkungen funktioniert Buffen nur in der Vorbereitungsphase und wird deaktiviert, sobald das Match beginnt.";
 
 -- Support
 SMARTBUFF_MINIMAP_TT         = "Links Klick: Optionen Menü\nRechts Klick: An/Aus\nAlt-Links Klick: SmartDebuff\nShift-Ziehen: Knopf verschieben";

localization.en.lua

@@ -7,6 +7,13 @@ SMARTBUFF_WHATSNEW = "\n\n|cffffffff         Whats new:|r\n\n"
   .."          |cffffffffMidnight & Classic versions by Codermik, additional retail\n"
   .."          programming by MrWizard and Speedwaystar.\n"
   .."\n\n"
+  .."          Changes in r39.130226:\n\n"
+  .."             * Add Retain checkbox to copy buff settings when switching to fresh templates\n"
+  .."             * Inclusivity: Rename blacklist → blocklist (with migration)\n"
+  .."             * Default InCombat to off\n"
+  .."             * Add PvP prep-only message for Arena/BG\n"
+  .."             * Add play-sound tooltip\n"
+  .."\n\n"
   .."          Changes in r39.120226:\n\n"
   .."             * Template switching: BG/Arena, Solo fallback, single if/elseif chain, merged chat msg\n"
   .."             * Template split: GENERICS/INSTANCES/CUSTOM, assemble at load, enum lookup, SmartBuffGroup refactor\n"
@@ -106,9 +113,10 @@ SMARTBUFF_OFT_AUTOSWITCHTMPINST = "Instances";
 SMARTBUFF_OFT_CHECKCHARGES   = "Check charges";
 SMARTBUFF_OFT_RBT            = "R: Timers";
 SMARTBUFF_OFT_BUFFINCITIES   = "Buff in cities";
-SMARTBUFF_OFT_BLDURATION     = "Blacklisted";
+SMARTBUFF_OFT_BLDURATION     = "Blocked";
 SMARTBUFF_OFT_ANTIDAZE       = "Anti daze";
 SMARTBUFF_OFT_HIDESABUTTON   = "Hide action button";
+SMARTBUFF_OFT_RETAINTEMPLATE = "Retain";
 SMARTBUFF_OFT_INCOMBAT       = "in combat";
 SMARTBUFF_OFT_SMARTDEBUFF    = "SmartDebuff";
 SMARTBUFF_OFT_INSHAPESHIFT   = "Shapeshift";
@@ -163,17 +171,19 @@ SMARTBUFF_OFTT_AUTOSWITCHTMP = "Automatically switches the template,\nif the gro
 SMARTBUFF_OFTT_AUTOSWITCHTMPINST = "Automatically switches the template,\nif the instance changes.";
 SMARTBUFF_OFTT_CHECKCHARGES  = "Displays low amount of\ncharges on a buff.\n0 = Deactivated";
 SMARTBUFF_OFTT_BUFFINCITIES  = "Buffs also if you are in capital cities.\nIf you are PvP flagged, it buffs in any case.";
-SMARTBUFF_OFTT_BLDURATION    = "How many seconds, players will be blacklisted.\n0 = Deactivated";
+SMARTBUFF_OFTT_BLDURATION    = "How many seconds, players will be blocked.\n0 = Deactivated";
 SMARTBUFF_OFTT_ANTIDAZE      = "Automatically cancels the\naspect of the cheetah/pack\nif someone gets dazed\n(self or group).";
 SMARTBUFF_OFTT_SPLASHSTYLE   = "Changes the fontstyle of\nthe buff messages.";
 SMARTBUFF_OFTT_HIDESABUTTON  = "Hides the SmartBuff action button.";
+SMARTBUFF_OFTT_RETAINTEMPLATE = "When switching to a fresh template (never used, or no buffs enabled),\nthe current buff settings will be copied to it when this is checked.\nBlank templates are never copied over, so you cannot accidentally overwrite a configured template.";
 SMARTBUFF_OFTT_INCOMBAT      = "At the moment it only works on yourself.\nThe first buff you mark as in combat,\nwill set on the button before combat\nand you can use it in combat.\n!!! Warning !!!\nAll logic is disabled in combat!";
 SMARTBUFF_OFTT_SMARTDEBUFF   = "Shows the SmartDebuff frame.";
 SMARTBUFF_OFTT_SPLASHDURATION= "How many seconds the splash\nmessage will displayed,\nbefore it fades.";
 SMARTBUFF_OFTT_INSHAPESHIFT  = "Cast buffs also if you\nare shapeshifted.";
 SMARTBUFF_OFTT_LINKGRPBUFFCHECK  = "Checks if a buff of an other\nclass with similar effect\nis already active.";
 SMARTBUFF_OFTT_LINKSELFBUFFCHECK = "Checks if a self buff is active,\nwhose only one can be\nactive at a time.";
 SMARTBUFF_OFTT_SOUNDSELECT   = "Select the required splash sound.";
+SMARTBUFF_OFTT_PLAYSOUND     = "Controlled by Sound Effects volume.";
 
 -- Buffsetup Frame Text
 SMARTBUFF_BST_SELFONLY       = "Myself only";
@@ -228,6 +238,7 @@ SMARTBUFF_MSG_CLASS          = "Class";
 SMARTBUFF_MSG_CHARGES        = "charges";
 SMARTBUFF_MSG_SOUNDS         = "Splash Sound Selection: "
 SMARTBUFF_MSG_SPECCHANGED    = "Spec changed (%s), loading buff templates...";
+SMARTBUFF_MSG_PVP_PREP_ONLY  = "Due to API limitation, buffing only works during prep and is disabled once the match starts.";
 
 -- Support
 SMARTBUFF_MINIMAP_TT         = "Left click: options menu\nRight click: On/Off\nAlt-Left Click: SmartDebuff\nShift drag: Move button";