WowAce

Whenever I try to save my toggle bindings for GatherMate2 in the Addon settings (GatherMate2 --> Minimap), I get the below error. The binding saves and works for the session until I exit the game and load it again. Then I have to rebind them, get the error, ignore it, and use it for that session. Anyway to correct that? This is for WoW Classic.

I'm trying to set CTRL+M to the keybind toggle for the main map, and CTRL+N to the keybind toggle for the minimap. I've already removed them from the main Key Bindings in the WoW client

Version: GatherMate2-1.45-classic

Below is the full report.

Date: 2019-09-07 22:53:52
ID: 1
Error occured in: Global
Count: 1
Message: ..\AddOns\GatherMate2\Config.lua line 249:
attempt to call global 'SaveBindings' (a nil value)
Debug:
(tail call): ?
GatherMate2\Config.lua:249:
GatherMate2\Config.lua:231
[C]: ?
...nfig-3.0\AceConfigDialog-3.0\AceConfigDialog-3.0.lua:51:
...nfig-3.0\AceConfigDialog-3.0\AceConfigDialog-3.0.lua:49
...nfig-3.0\AceConfigDialog-3.0\AceConfigDialog-3.0.lua:843:
...nfig-3.0\AceConfigDialog-3.0\AceConfigDialog-3.0.lua:664
[C]: ?
...assicCastbars_Options\Libs\AceGUI-3.0\AceGUI-3.0.lua:72:
...assicCastbars_Options\Libs\AceGUI-3.0\AceGUI-3.0.lua:70
...assicCastbars_Options\Libs\AceGUI-3.0\AceGUI-3.0.lua:287: Fire()
...\Libs\AceGUI-3.0\widgets\AceGUIWidget-Keybinding.lua:85:
...\Libs\AceGUI-3.0\widgets\AceGUIWidget-Keybinding.lua:58
Locals:
None
AddOns:
Swatter, v8.2.6377 (SwimmingSeadragon)
ACP, v3.5.6
AtlasLootClassic, vv1.1.9-beta
AucAdvanced, v8.2.6385 (SwimmingSeadragon)
AucFilterBasic, v8.2.6364 (SwimmingSeadragon)
AucStatHistogram, v8.2.6366 (SwimmingSeadragon)
AucStatiLevel, v8.2.6370 (SwimmingSeadragon)
AucStatPurchased, v8.2.6367 (SwimmingSeadragon)
AucStatSimple, v8.2.6368 (SwimmingSeadragon)
AucStatStdDev, v8.2.6369 (SwimmingSeadragon)
AucUtilFixAH, v8.2.6371 (SwimmingSeadragon)
BagBrother, v
BeanCounter, v8.2.6381 (SwimmingSeadragon)
ClassicCastbars, vv1.0.4
ClassicCastbarsOptions, v
ClassicLFG, v1.0.0
ClassicThreatMeter, v1.06
Combuctor, v8.2.6
CombuctorConfig, v
DBMCore, v1.13.6
DBMDefaultSkin, v
DBMStatusBarTimers, v
Enchantrix, v8.2.6392 (SwimmingSeadragon)
GatherMate2, v1.45
Informant, v8.2.6374 (SwimmingSeadragon)
LeatrixMaps, v1.13.28
LFGShout, v11302.6
Mapster, v1.8.4
Questie, v4.0.17 BETA
RealMobHealth, v2.5
Recount, vv1.13.2b
SlideBar, v8.2.6375 (SwimmingSeadragon)
Stubby, v8.2.6376 (SwimmingSeadragon)
TitanClassic, v1.0.5.11302
TitanClassicAmmo, v1.0.5.11302
TitanClassicBag, v1.0.5.11302
TitanClassicClock, v1.0.5.11302
TitanClassicGold, v1.0.5.11302
TitanClassicLocation, v1.0.5.11302
TitanClassicLootType, v1.0.5.11302
TitanClassicPerformance, v1.0.5.11302
TitanClassicRepair, v1.0.5.11302
TitanClassicVolume, v1.0.5.11302
TitanClassicXP, v1.0.5.11302
TomTom, vv11302-1.0.7
TradeSkillMaster, vv4.8.7
TradeSkillMasterAppHelper, vv4.0.4
WeakAuras, v2.14.6
WIM, v3.7.33
BlizRuntimeLib_enUS v1.13.2.11302 <none>
(ck=642)

FYI, I fixed it. The API call for classic changed. Instead of SaveBindings, it's now called "AttemptToSaveBindings".

In the <addonfolder>\Gathermate2\Config.lua, change "SaveBindings" to "AttemptToSaveBindings" on both lines 219 and 249. 

Line 219 Before:

SaveBindings(GetCurrentBindingSet())

 Line 219 After:

AttemptToSaveBindings(GetCurrentBindingSet())

Line 249 Before: 

SaveBindings(GetCurrentBindingSet())

 Line 249 After:

AttemptToSaveBindings(GetCurrentBindingSet())

The toggle is now successfully saving across sessions and not generating errors.