Fix provider hooks

This was ripped out of handynotes-travelguide, thanks Belazor @ WoWUIDev

provider.lua

@@ -40,6 +40,17 @@ function provider:ShouldShowQuest(questInfo)
 	return addon:IsChildMap(mapID, questInfo.mapID)
 end
 
+-- remove the default provider
+for dp in next, WorldMapFrame.dataProviders do
+	if not dp.GetPinTemplates and type(dp.GetPinTemplate) == 'function' then
+		if dp:GetPinTemplate() == 'WorldMap_WorldQuestPinTemplate' then
+			WorldMapFrame:RemoveDataProvider(dp)
+			break
+		end
+	end
+end
+
+-- add our own
 WorldMapFrame:AddDataProvider(provider)
 
 -- hook into changes
@@ -61,13 +72,6 @@ addon:RegisterOptionCallback('zoomFactor', updateVisuals)
 addon:RegisterOptionCallback('showAzeroth', updateVisuals)
 addon:RegisterOptionCallback('showEvents', updateVisuals)
 
--- remove the default provider
-for dp in next, WorldMapFrame.dataProviders do
-	if dp.GetPinTemplate and dp.GetPinTemplate() == 'WorldMap_WorldQuestPinTemplate' then
-		WorldMapFrame:RemoveDataProvider(dp)
-	end
-end
-
 -- change visibility
 local modifier
 local function toggleVisibility()

storms.lua

@@ -26,8 +26,11 @@ local function updatePOIs(self)
 	end
 end
 
-for provider in next, WorldMapFrame.dataProviders do
-	if provider.GetPinTemplate and provider:GetPinTemplate() == 'AreaPOIPinTemplate' then
-		hooksecurefunc(provider, 'RefreshAllData', updatePOIs)
+for dp in next, WorldMapFrame.dataProviders do
+	if not dp.GetPinTemplates and type(dp.GetPinTemplate) == 'function' then
+		if dp:GetPinTemplate() == 'AreaPOIPinTemplate' then
+			hooksecurefunc(dp, 'RefreshAllData', updatePOIs)
+			break
+		end
 	end
 end